Merge branch 'trunk'

README.md

@@ -15,3 +15,6 @@ For more info read [API reference](http://public.innomdc.com/inno-helper/)
 
 ### [0.0.7] - 2017-06-13
 - Added delay parameter for the Scheduler API
+
+### [0.0.8] - 2017-06-26
+- Added validation in methods for the Scheduler API

composer.json

@@ -3,7 +3,7 @@
     "type": "library",
     "description": "Innometrics helper",
     "homepage": "http://www.innometrics.com",
-    "version": "0.0.7",
+    "version": "0.0.8",
     "require": {
         "php": ">=5.3.2",
         "illuminate/container": "v5.0.33",

lib/Innometrics/Helper.php

@@ -167,7 +167,14 @@ public function getTasks () {
      * @return bool
      */
     public function addTask ($params) {
-        if (isset($params['timestamp']) && isset($params['delay'])) {
+        $timestampExists = isset($params['timestamp']);
+        $delayExists = isset($params['delay']);
+
+        if (!$timestampExists && !$delayExists) {
+            throw new \ErrorException('Either use timestamp or delay');
+        }
+
+        if ($timestampExists && $delayExists) {
             throw new \ErrorException('You should use only one field: timestamp or delay');
         }
 
@@ -194,6 +201,10 @@ public function addTask ($params) {
      * @return bool
      */
     public function deleteTask ($params) {
+        if (!isset($params['taskId'])) {
+            throw new \ErrorException('Parameter "taskId" required');
+        }
+
         $url = $this->getSchedulerApiUrl($params);
         $response = $this->request(array(
             'url'  => $url,