Uploaded image for project: 'Thrift'
  1. Thrift
  2. THRIFT-1942

Binary accelerated cpp extension does not use Thrift namespaces for Exceptions

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 0.9
    • Fix Version/s: 0.9.1
    • Labels:
      None
    • Patch Info:
      Patch Available

      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.

        Attachments

        1. THRIFT-1942-v1.txt
          1 kB
          Tom Hobbs

          Activity

            People

            • Assignee:
              thobbs Tom Hobbs
              Reporter:
              jeremy.rayner Jeremy Rayner
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: