Merge pull request #28 from Trunkrs/fix/25/nullability

Fix: Strip unnecessary null values from request body

VERSION

@@ -1 +1 @@
-2.0.1
+2.0.2

lib/RequestHandler.php

@@ -5,6 +5,7 @@
 use Trunkrs\SDK\Exception\GeneralApiException;
 use Trunkrs\SDK\Exception\NotAuthorizedException;
 use Trunkrs\SDK\Exception\ServerValidationException;
+use Trunkrs\SDK\Util\NullStripper;
 
 class RequestHandler {
     private static $_httpClient;
@@ -103,7 +104,7 @@ public static function post(string $resource, array $body) {
             "POST",
             self::createUrl($resource),
             self::createHeaders(!empty($body)),
-            $body
+            NullStripper::strip($body)
         );
 
         if (!self::isSuccessful($response)) {
@@ -127,7 +128,7 @@ public static function put(string $resource, array $body = []) {
             "PUT",
             self::createUrl($resource),
             self::createHeaders(!empty($body)),
-            $body
+            NullStripper::strip($body)
         );
 
         if (!self::isSuccessful($response)) {

lib/Settings.php

@@ -38,7 +38,7 @@ class Settings {
     /**
      * @var string The current version of the SDK.
      */
-    public static $sdkVersion = '2.0.1';
+    public static $sdkVersion = '2.0.2';
 
     /**
      * Sets the client credentials that will be used in subsequent requests.

lib/Util/NullStripper.php

@@ -0,0 +1,20 @@
+<?php
+
+
+namespace Trunkrs\SDK\Util;
+
+
+class NullStripper
+{
+    public static function strip(array $array): array {
+        foreach ($array as $key => $value) {
+            if (is_array($value)) {
+                $array[$key] = NullStripper::strip($value);
+            } else if (is_null($value)) {
+                unset($key);
+            }
+        }
+
+        return $array;
+    }
+}