Fix lint errors

Add composer license
Add Errors response class

Author: nohponex

composer.json

@@ -1,6 +1,7 @@
 {
   "name": "phramework/jsonapi-client",
   "homepage": "https://phramework.github.io/",
+  "license": "Apache-2.0",
   "authors": [
     {
       "name": "Xenofon Spafaridis",
@@ -39,6 +40,6 @@
       "find tests/ -name '*.php' -print0 | xargs -0 -L 1 php -l",
       "phpunit"
     ],
-    "lint": "php ./vendor/bin/phpcs -p -s --standard=RSR2 ./src ./public ./queued"
+    "lint": "php ./vendor/bin/phpcs -p -s --standard=PSR2 ./src ./tests/src"
   }
 }

src/Client.php

@@ -310,8 +310,8 @@ public static function post(
     public static function patch(
         string $id,
         \stdClass $attributes = null,
-        \stdClass $relationships = null)
-    {
+        \stdClass $relationships = null
+    ) {
 
     }
 
@@ -328,11 +328,10 @@ public static function request(
         string $url,
         string $method = self::METHOD_GET,
         \stdClass $headers = null,
-        $data = NULL,
+        $data = null,
         $flags = self::REQUEST_EMPTY_FLAG,
         //$accept = 'application/json',
-        $encoding = NULL
-
+        $encoding = null
     ) {
         //Extract flags
         //Is the request binary
@@ -366,7 +365,7 @@ public static function request(
 
         curl_setopt($handle, CURLOPT_URL, $url);
         curl_setopt($handle, CURLOPT_HTTPHEADER, $headersArray);
-        curl_setopt($handle, CURLOPT_RETURNTRANSFER, TRUE);
+        curl_setopt($handle, CURLOPT_RETURNTRANSFER, true);
         curl_setopt($handle, CURLOPT_HEADER, true);
 
         //Set timeout values ( in seconds )
@@ -375,11 +374,11 @@ public static function request(
         curl_setopt($handle, CURLOPT_NOSIGNAL, 1);
         //Security options
         curl_setopt($handle, CURLOPT_SSL_VERIFYHOST, 2);
-        curl_setopt($handle, CURLOPT_SSL_VERIFYPEER, FALSE);
+        curl_setopt($handle, CURLOPT_SSL_VERIFYPEER, false);
 
         /*//On binary transfers
         if ($binary) {
-            curl_setopt($handle, CURLOPT_BINARYTRANSFER, TRUE);
+            curl_setopt($handle, CURLOPT_BINARYTRANSFER, true);
         }*/
 
         //Switch on HTTP Request method
@@ -391,15 +390,27 @@ public static function request(
                 curl_setopt($handle, CURLOPT_POST, true);
                 if ($data && $form_encoded) { //Encode fields if required ( URL ENCODED )
                     curl_setopt(
-                        $handle, CURLOPT_POSTFIELDS, http_build_query($data));
-                } else if ($data) {
-                    curl_setopt($handle, CURLOPT_POSTFIELDS, $data);
+                        $handle,
+                        CURLOPT_POSTFIELDS,
+                        http_build_query($data)
+                    );
+                } elseif ($data) {
+                    curl_setopt(
+                        $handle,
+                        CURLOPT_POSTFIELDS,
+                        $data
+                    );
                 }
                 break;
             case self::METHOD_PUT: //On METHOD_PUT
                 curl_setopt($handle, CURLOPT_CUSTOMREQUEST, self::METHOD_PUT);
+                //todo only if json
                 if ($data) {
-                    curl_setopt($handle, CURLOPT_POSTFIELDS, json_encode($data));
+                    curl_setopt(
+                        $handle,
+                        CURLOPT_POSTFIELDS,
+                        json_encode($data)
+                    );
                 }
                 break;
             case self::METHOD_DELETE: //On METHOD_DELETE
@@ -432,10 +443,10 @@ public static function request(
         curl_close($handle);
 
         return [
-           $responseStatusCode,
-           $responseHeaders,
-           json_decode($responseBody)
-       ];
+            $responseStatusCode,
+            $responseHeaders,
+            json_decode($responseBody)
+        ];
 
        /* if (!$response) {
             throw new Exception('Error: ' . curl_error($handle));

src/Filter.php

@@ -15,6 +15,7 @@
  * limitations under the License.
  */
 namespace Phramework\JSONAPI\Client;
+
 use Phramework\JSONAPI\FilterAttribute;
 use Phramework\JSONAPI\FilterJSONAttribute;
 

src/ResourceObject.php

@@ -41,4 +41,4 @@ class ResourceObject
      * @var \stdClass
      */
     public $relationships;
-}
+}

src/Response/Collection.php

@@ -33,4 +33,4 @@ class Collection extends Response
      * @var ResourceObject[]
      */
     public $included;
-}
+}

src/Response/Errors.php

@@ -0,0 +1,30 @@
+<?php
+/**
+ * Copyright 2016 Xenofon Spafaridis
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+namespace Phramework\JSONAPI\Client\Response;
+
+/**
+ * @author Xenofon Spafaridis <[email protected]>
+ * @since 0.0.0
+ */
+class Errors extends Response
+{
+    /**
+     * @var \stdClass[]
+     * @todo add errors object
+     */
+    public $errors;
+}

src/Response/Resource.php

@@ -34,4 +34,4 @@ class Resource extends Response
      * @var ResourceObject[]
      */
     public $included;
-}
+}

src/Response/Response.php

@@ -20,7 +20,7 @@
  * @author Xenofon Spafaridis <[email protected]>
  * @since 0.0.0
  */
-class Response
+abstract class Response
 {
     /**
      * @var \stdClass
@@ -43,17 +43,13 @@ class Response
     protected $headers;
 
     /**
-     * Response constructor.
-     */
-    public function __construct(
-    ) {
-    }
-
-    /**
+     * Parse response object, will copy any top members available at this
+     * Response instance
      * @param \stdClass $response
      * @return $this
      */
-    public function parse(\stdClass $response) {
+    public function parse(\stdClass $response)
+    {
         $members = array_keys(get_object_vars($this));
 
         foreach ($members as $member) {