Merge branch 'feature/5933' into develop

Close zendframework/zendframework#5933

src/Server/Error.php

@@ -18,24 +18,11 @@ class Error
     const ERROR_INTERNAL        = -32603;
     const ERROR_OTHER           = -32000;
 
-    /**
-     * Allowed error codes
-     * @var array
-     */
-    protected $allowedCodes = array(
-        self::ERROR_PARSE,
-        self::ERROR_INVALID_REQUEST,
-        self::ERROR_INVALID_METHOD,
-        self::ERROR_INVALID_PARAMS,
-        self::ERROR_INTERNAL,
-        self::ERROR_OTHER,
-    );
-
     /**
      * Current code
      * @var int
      */
-    protected $code = -32000;
+    protected $code = self::ERROR_OTHER;
 
     /**
      * Error data
@@ -56,29 +43,36 @@ class Error
      * @param  int $code
      * @param  mixed $data
      */
-    public function __construct($message = null, $code = -32000, $data = null)
+    public function __construct($message = null, $code = self::ERROR_OTHER, $data = null)
     {
         $this->setMessage($message)
              ->setCode($code)
              ->setData($data);
     }
 
     /**
-     * Set error code
+     * Set error code.
+     *
+     * If the error code is 0, it will be set to -32000 (ERROR_OTHER).
      *
      * @param  int $code
      * @return \Zend\Json\Server\Error
      */
     public function setCode($code)
     {
-        if (!is_scalar($code)) {
+        if (!is_scalar($code) || is_bool($code) || is_float($code)) {
+            return $this;
+        }
+
+        if (is_string($code) && !is_numeric($code)) {
             return $this;
         }
 
         $code = (int) $code;
-        if (in_array($code, $this->allowedCodes)) {
-            $this->code = $code;
-        } elseif (in_array($code, range(-32099, -32000))) {
+
+        if (0 === $code) {
+            $this->code = self::ERROR_OTHER;
+        } else {
             $this->code = $code;
         }
 

test/Server/ErrorTest.php

@@ -54,7 +54,7 @@ public function testSetCodeShouldCastToInteger()
 
     public function testCodeShouldBeLimitedToStandardIntegers()
     {
-        foreach (array(true, 'foo', array(), new \stdClass, 2.0, 25) as $code) {
+        foreach (array(null, true, 'foo', array(), new \stdClass, 2.0) as $code) {
             $this->error->setCode($code);
             $this->assertEquals(Server\Error::ERROR_OTHER, $this->error->getCode());
         }
@@ -68,6 +68,24 @@ public function testCodeShouldAllowArbitraryAppErrorCodesInXmlRpcErrorCodeRange(
         }
     }
 
+    public function arbitraryErrorCodes()
+    {
+        return array(
+            '1000'  => array(1000),
+            '404'   => array(404),
+            '-3000' => array(-3000),
+        );
+    }
+
+    /**
+     * @dataProvider arbitraryErrorCodes
+     */
+    public function testCodeShouldAllowArbitraryErrorCode($code)
+    {
+        $this->error->setCode($code);
+        $this->assertEquals($code, $this->error->getCode());
+    }
+
     public function testMessageShouldBeNullByDefault()
     {
         $this->assertNull($this->error->getMessage());