[THRIFT-5318] PHP 8 compatible version of binary protocol

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.15.0
• Component/s: PHP - Library
• Patch Info: Patch Available
• Assignee: Tyler Christensen
• Reporter: Tyler Christensen
• Created: 02/Dec/20 01:33
• Updated: 11/Sep/21 12:31
• Resolved: 14/Feb/21 10:36

Description

PHP 8 has been released and necessitates some changes to PHP extensions. In the case of thrift_protocol these changes are minor:

• Update callsites of zend_call_method, zend_read_property, and zend_update_property
• Add arginfo as requested by PHP 8 migration guide (supports reflection features)

I believe it's possible to do these whilst retaining backwards compatibility with PHP 7; I'll open a PR on Github momentarily.

Issue Links

• is related to: THRIFT-5440 Add php8 to composer.json
• links to: GitHub Pull Request #2288