Merge pull request #6 from artyuum/patch-2

Fixes #5

src/JsendResponse/JSendResponse.php

@@ -52,13 +52,8 @@ public function __construct(string $status, $data = null, string $message = null
 
         $jsend['status'] = $status;
 
-        // the "data" key is required for this status
-        if ($status === self::STATUS_SUCCESS) {
-            $jsend['data'] = $data;
-        }
-
-        // the "data" key is optional for these statuses so we only add it if set
-        if (isset($data) && ($status === self::STATUS_FAIL || $status === self::STATUS_ERROR)) {
+        // the "data" key is required for these statuses
+        if ($status === self::STATUS_SUCCESS || $status === self::STATUS_FAIL) {
             $jsend['data'] = $data;
         }
 
@@ -68,6 +63,11 @@ public function __construct(string $status, $data = null, string $message = null
                 throw new JSendSpecificationViolation('The "message" key is required');
             }
 
+            // the "data" key is optional for this status so we only add it if set
+            if (isset($data)) {
+                $jsend['data'] = $data;
+            }
+
             $jsend['message'] = $message;
 
             // adds the "code" key only if it's set (because it's optional)