Merge pull request #125 from renderforest/fix-optional-parameter-issue

fix-optional-parameter-issue

Author: narekhovhannisyan

composer.json

@@ -1,7 +1,7 @@
 {
   "name": "renderforest/sdk-php",
   "description": "Renderforest SDK for PHP",
-  "version": "0.3.10",
+  "version": "0.3.11",
   "homepage": "https://github.com/renderforest/renderforest-sdk-php",
   "authors": [
     {

src/Params.php

@@ -52,19 +52,16 @@ public function destructURLParam($payload, $param)
 
     /**
      * Destructs optional URL param from the given payload.
-     * @param $payload - Thr array to destruct.
-     * @param $param - The param to destruct from array.
+     * @param array $payload - Thr array to destruct.
+     * @param string $param - The param to destruct from array.
      * @return string
      * @throws RenderforestError
      */
     public function destructOptionalURLParam($payload, $param)
     {
-        if (!isset($payload) || sizeof($payload)) {
-            throw new RenderforestError("Missing optional parameter: ${param}.");
-        }
-
-        if (!isset($payload[$param])) {
+        if (!isset($payload) || !sizeof($payload) || !isset($payload[$param])) {
             return '';
+
         }
 
         return $payload[$param];

src/Resource/Projects.php

@@ -152,7 +152,7 @@ public function deleteProjectVideos($payload)
 
         $options = [
             'method' => 'DELETE',
-            'endpoints' => "{$this->CONFIG['API_PREFIX']}/projects/$projectId/videos/$quality"
+            'endpoint' => "{$this->CONFIG['API_PREFIX']}/projects/$projectId/videos/$quality"
         ];
         return $this->ApiRequest->authorizedRequest($options);
     }