Merge branch 'hotfix/1'

Close #1

Author: Xerkus

CHANGELOG.md

@@ -22,7 +22,8 @@ All notable changes to this project will be documented in this file, in reverse
 
 ### Fixed
 
-- Nothing.
+- [#1](https://github.com/laminas/laminas-json-server/pull/1) Allows optional `data` field in Error object to be  omitted
+
 
 ## 3.2.0 - 2019-10-17
 

src/Response.php

@@ -64,7 +64,8 @@ public function setOptions(array $options)
         // re-produce error state
         if (isset($options['error']) && is_array($options['error'])) {
             $error = $options['error'];
-            $options['error'] = new Error($error['message'], $error['code'], $error['data']);
+            $errorData = isset($error['data']) ? $error['data'] : null;
+            $options['error'] = new Error($error['message'], $error['code'], $errorData);
         }
 
         $methods = get_class_methods($this);

test/ResponseTest.php

@@ -16,6 +16,7 @@
 
 class ResponseTest extends TestCase
 {
+    private $response;
     /**
      * Sets up the fixture, for example, open a network connection.
      * This method is called before a test is executed.
@@ -212,4 +213,22 @@ public function testValueOfZeroForOptionsKeyShouldNotBeInterpretedAsVersionKey()
         ]);
         $this->assertNull($this->response->getVersion());
     }
+
+    /**
+     * Assert that error data can be omitted
+     * @see https://www.jsonrpc.org/specification#response_object
+     */
+    public function testSetOptionsAcceptsErrorWithEmptyDate()
+    {
+        $this->response->setOptions([
+            'error' => [
+                'code' => 0,
+                'message' => 'Lorem Ipsum',
+            ],
+        ]);
+        $this->assertInstanceOf(Error::class, $this->response->getError());
+        $this->assertEquals(-32000, $this->response->getError()->getCode());
+        $this->assertEquals('Lorem Ipsum', $this->response->getError()->getMessage());
+        $this->assertEquals(null, $this->response->getError()->getData());
+    }
 }