[thrift] Fix extension deserialization call with no classname
Summary: The extension expects a classname as a string and does not allow null;
use the empty string instead of null in the absence of a classname to pass.
Reviewed By: mcslee
Test Plan: php -l
Revert: svn
git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@665347 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/lib/php/src/Thrift.php b/lib/php/src/Thrift.php
index fdec97f..10728a1 100644
--- a/lib/php/src/Thrift.php
+++ b/lib/php/src/Thrift.php
@@ -220,7 +220,7 @@
if ($ftype == $fspec['type']) {
$xfer = 0;
if ($fast_binary) {
- $class = isset($fspec['class']) ? $fspec['class'] : null;
+ $class = isset($fspec['class']) ? $fspec['class'] : '';
$this->$var = thrift_protocol_binary_deserialize($ftype, $input, $class);
} else {
if (isset(TBase::$tmethod[$ftype])) {
@@ -567,7 +567,7 @@
if ($ftype == $fspec['type']) {
$xfer = 0;
if ($fast_binary) {
- $class = isset($fspec['class']) ? $fspec['class'] : null;
+ $class = isset($fspec['class']) ? $fspec['class'] : '';
$this->$var = thrift_protocol_binary_deserialize($ftype, $input, $class);
} else {
if (isset(TBase::$tmethod[$ftype])) {