[THRIFT-1942] Binary accelerated cpp extension does not use Thrift namespaces for Exceptions

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.9
• Fix Version/s: 0.9.1
• Component/s: PHP - Compiler, PHP - Library
• Patch Info: Patch Available
• Assignee: Tom Hobbs
• Reporter: Jeremy Rayner
• Created: 23/Apr/13 03:46
• Updated: 08/Jun/13 03:15
• Resolved: 08/Jun/13 03:15

Description

When an Exception is required to be thrown by the binary accelerated library, an Class Not Found error will occur:

PHP Fatal error: Class 'TProtocolException' not found in XXXX

Line 426 needs to change from
createObject("TProtocolException", ex, 2, zwhat, zerrorcode);
to
createObject("\\Thrift\\Exception
TProtocolException", ex, 2, zwhat, zerrorcode);

and line 1104 needs to change from
createObject("TApplicationException", ex);
to
createObject("\\Thrift\\Exception
TApplicationException", ex);

Basically, there is no accounting for the fact that the exception classes are now namespaced.