Disable BigInteger for i8 type if machine is 64-bit

Author: Justin Van Dort

CHANGELOG.md

@@ -10,7 +10,7 @@ All notable changes to this project will be documented in this file, in reverse
 
 ### Changed
 
-- Nothing.
+Disables use of BigInteger for XMLRPC i8 type if host machine is 64-bit.
 
 ### Deprecated
 

docs/book/client.md

@@ -133,8 +133,8 @@ XML-RPC Type     | `Zend\XmlRpc\AbstractValue` Constant               | `Zend\Xm
 ---------------- | -------------------------------------------------- | --------------------------
 int              | `Zend\XmlRpc\AbstractValue::XMLRPC_TYPE_INTEGER`   | `Zend\XmlRpc\Value\Integer`
 i4               | `Zend\XmlRpc\AbstractValue::XMLRPC_TYPE_I4`        | `Zend\XmlRpc\Value\Integer`
-i8               | `Zend\XmlRpc\AbstractValue::XMLRPC_TYPE_I8`        | `Zend\XmlRpc\Value\BigInteger`
-ex:i8            | `Zend\XmlRpc\AbstractValue::XMLRPC_TYPE_APACHEI8`  | `Zend\XmlRpc\Value\BigInteger`
+i8               | `Zend\XmlRpc\AbstractValue::XMLRPC_TYPE_I8`        | `Zend\XmlRpc\Value\BigInteger` or `Zend\XmlRpc\Value\Integer` if machine is 64-bit
+ex:i8            | `Zend\XmlRpc\AbstractValue::XMLRPC_TYPE_APACHEI8`  | `Zend\XmlRpc\Value\BigInteger` or `Zend\XmlRpc\Value\Integer` if machine is 64-bit
 double           | `Zend\XmlRpc\AbstractValue::XMLRPC_TYPE_DOUBLE`    | `Zend\XmlRpc\ValueDouble`
 boolean          | `Zend\XmlRpc\AbstractValue::XMLRPC_TYPE_BOOLEAN`   | `Zend\XmlRpc\Value\Boolean`
 string           | `Zend\XmlRpc\AbstractValue::XMLRPC_TYPE_STRING`    | `Zend\XmlRpc\Value\Text`

src/AbstractValue.php

@@ -48,6 +48,11 @@ abstract class AbstractValue
      */
     protected static $generator;
 
+    /**
+     * True if BigInteger should be used for XMLRPC i8 types
+     */
+    public static $USE_BIGINT_FOR_I8 = PHP_INT_SIZE < 8;
+
     /**
      * Specify that the XML-RPC native type will be auto detected from a PHP variable type
      */
@@ -193,7 +198,7 @@ public static function getXmlRpcValue($value, $type = self::AUTO_DETECT_TYPE)
             case self::XMLRPC_TYPE_I8:
                 // fall through to the next case
             case self::XMLRPC_TYPE_APACHEI8:
-                return new Value\BigInteger($value);
+                return self::$USE_BIGINT_FOR_I8 ? new Value\BigInteger($value) : new Value\Integer($value);
 
             case self::XMLRPC_TYPE_DOUBLE:
                 return new Value\Double($value);
@@ -337,7 +342,7 @@ protected static function xmlStringToNativeXmlRpc($xml)
             case self::XMLRPC_TYPE_APACHEI8:
                 // Fall through to the next case
             case self::XMLRPC_TYPE_I8:
-                $xmlrpcValue = new Value\BigInteger($value);
+                $xmlrpcValue = self::$USE_BIGINT_FOR_I8 ? new Value\BigInteger($value) : new Value\Integer($value);
                 break;
             case self::XMLRPC_TYPE_DOUBLE:
                 $xmlrpcValue = new Value\Double($value);

test/BigIntegerValueTest.php

@@ -21,6 +21,7 @@ class BigIntegerValueTest extends TestCase
 {
     public function setUp()
     {
+        AbstractValue::$USE_BIGINT_FOR_I8 = true;
         if (extension_loaded('gmp')) {
             $this->markTestSkipped('gmp causes test failure');
         }