anorgan / onfleet-client

src/Task.php

Doc Comments +5 added lines, -5 removed lines

@@ -141,7 +141,7 @@ 
     }
 
     /**
-     * @param mixed $merchant
+     * @param null|Organization $merchant
      */
     public function setMerchant($merchant)
     {
@@ -157,7 +157,7 @@ 
     }
 
     /**
-     * @param mixed $executor
+     * @param null|Organization $executor
      */
     public function setExecutor($executor)
     {
@@ -221,7 +221,7 @@ 
     }
 
     /**
-     * @param mixed $completeAfter
+     * @param integer|null $completeAfter
      */
     public function setCompleteAfter($completeAfter)
     {
@@ -237,7 +237,7 @@ 
     }
 
     /**
-     * @param mixed $completeBefore
+     * @param integer|null $completeBefore
      */
     public function setCompleteBefore($completeBefore)
     {
@@ -365,7 +365,7 @@ 
     }
 
     /**
-     * @param mixed $destination
+     * @param Destination $destination
      */
     public function setDestination($destination)
     {

Spacing +1 added lines, -1 removed lines

@@ -433,7 +433,7 @@
      */
     public function complete($success = true, $notes = null)
     {
-        $this->client->post($this->endpoint .'/'. $this->id .'/complete', [
+        $this->client->post($this->endpoint.'/'.$this->id.'/complete', [
             'json' => [
                 'completionDetails' => [
                     'success' => $success,

src/Entity.php

Spacing +8 added lines, -8 removed lines

@@ -88,9 +88,9 @@ 
                     }
 
                     $subtype = gettype($value[0]);
-                    array_walk($value, function ($item) use ($subtype, $property) {
+                    array_walk($value, function($item) use ($subtype, $property) {
                         if (gettype($item) !== $subtype) {
-                            throw new \InvalidArgumentException('All array items have to be of the same type for metadata property '. $property);
+                            throw new \InvalidArgumentException('All array items have to be of the same type for metadata property '.$property);
                         }
                     });
 
@@ -99,13 +99,13 @@ 
                     }
                     $allowedSubtypes = ['boolean', 'number', 'string', 'object'];
                     if (!in_array($subtype, $allowedSubtypes)) {
-                        throw new \InvalidArgumentException('Unallowed type of '. $subtype .' for array item of metadata property '. $property);
+                        throw new \InvalidArgumentException('Unallowed type of '.$subtype.' for array item of metadata property '.$property);
                     }
 
                     break;
 
                 default:
-                    throw new \InvalidArgumentException('Unallowed type of '. $type .' for metadata property '. $property);
+                    throw new \InvalidArgumentException('Unallowed type of '.$type.' for metadata property '.$property);
                     break;
             }
 
@@ -131,7 +131,7 @@ 
      */
     public function update()
     {
-        $response = $this->client->put($this->endpoint .'/'. $this->id, [
+        $response = $this->client->put($this->endpoint.'/'.$this->id, [
             'json' => $this->toArray()
         ]);
 
@@ -146,14 +146,14 @@ 
     public function delete()
     {
         try {
-            $this->client->delete($this->endpoint .'/'. $this->id);
+            $this->client->delete($this->endpoint.'/'.$this->id);
         } catch (ClientException $e) {
             $error   = $e->getResponse()->json();
             $message = $error['message']['message'];
             if (isset($error['message']['cause'])) {
-                $message .= ' '. $error['message']['cause'];
+                $message .= ' '.$error['message']['cause'];
             }
-            throw new \Exception('Unable to delete entity: '. $message);
+            throw new \Exception('Unable to delete entity: '.$message);
         }
 
         $this->id = null;

src/Webhook.php

Spacing +11 added lines, -11 removed lines

@@ -11,57 +11,57 @@
     /**
      * Task started by worker. Request includes: taskId, data.task
      */
-    const TRIGGER_TASK_STARTED          = 0;
+    const TRIGGER_TASK_STARTED = 0;
 
     /**
      * Worker ETA less than or equal to notification threshold. Request includes: taskId, data.task
      */
-    const TRIGGER_TASK_ETA              = 1;
+    const TRIGGER_TASK_ETA = 1;
 
     /**
      * Worker arriving, 150 meters away or closer. Request includes: taskId, data.task
      */
-    const TRIGGER_TASK_ARRIVAL          = 2;
+    const TRIGGER_TASK_ARRIVAL = 2;
 
     /**
      * Task completed by worker. Request includes: taskId, data.task
      */
-    const TRIGGER_TASK_COMPLETED        = 3;
+    const TRIGGER_TASK_COMPLETED = 3;
 
     /**
      * Task failed. Request includes: taskId, data.task
      */
-    const TRIGGER_TASK_FAILED           = 4;
+    const TRIGGER_TASK_FAILED = 4;
 
     /**
      * New task created. Request includes: taskId, data.task
      */
-    const TRIGGER_TASK_CREATED          = 6;
+    const TRIGGER_TASK_CREATED = 6;
 
     /**
      * Task updated, including assignment changes. Request includes: taskId, data.task
      */
-    const TRIGGER_TASK_UPDATED          = 7;
+    const TRIGGER_TASK_UPDATED = 7;
 
     /**
      * Task deleted. Request includes: taskId
      */
-    const TRIGGER_TASK_DELETED          = 8;
+    const TRIGGER_TASK_DELETED = 8;
 
     /**
      * Task assigned to worker. Request includes: taskId, data.task
      */
-    const TRIGGER_TASK_ASSIGNED         = 9;
+    const TRIGGER_TASK_ASSIGNED = 9;
 
     /**
      * Task unassigned from worker. Request includes: taskId, data.task
      */
-    const TRIGGER_TASK_UNASSIGNED       = 10;
+    const TRIGGER_TASK_UNASSIGNED = 10;
 
     /**
      * Worker status changed (0 for off-duty, 1 for on-duty). Request includes: workerId, status, data.worker
      */
-    const TRIGGER_WORKER_DUTY           = 5;
+    const TRIGGER_WORKER_DUTY = 5;
 
     protected $url;
     protected $trigger;

src/Client.php

Spacing +7 added lines, -7 removed lines

@@ -130,7 +130,7 @@ 
      */
     public function getWorker($id, $filter = null, $analytics = false): Worker
     {
-        $response = $this->get('workers/'. $id, [
+        $response = $this->get('workers/'.$id, [
             'query' => [
                 'filter'    => $filter,
                 'analytics' => $analytics ? 'true' : 'false',
@@ -192,7 +192,7 @@ 
      */
     public function getTeam($id): Team
     {
-        $response = $this->get('teams/'. $id);
+        $response = $this->get('teams/'.$id);
         return Team::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -234,7 +234,7 @@ 
      */
     public function getDestination($id): Destination
     {
-        $response = $this->get('destinations/'. $id);
+        $response = $this->get('destinations/'.$id);
         return Destination::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -265,7 +265,7 @@ 
      */
     public function getRecipient($id): Recipient
     {
-        $response = $this->get('recipients/'. $id);
+        $response = $this->get('recipients/'.$id);
         return Recipient::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -334,7 +334,7 @@ 
      */
     public function getTask($id): Task
     {
-        $response = $this->get('tasks/'. $id);
+        $response = $this->get('tasks/'.$id);
         return Task::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -344,7 +344,7 @@ 
      */
     public function getTaskByShortId($shortId): Task
     {
-        $response = $this->get('tasks/shortId/'. $shortId);
+        $response = $this->get('tasks/shortId/'.$shortId);
         return Task::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -456,7 +456,7 @@ 
             array_unshift($taskIds, $position);
         }
 
-        $this->put('containers/'. $containerEndpoint .'/'. $targetId, [
+        $this->put('containers/'.$containerEndpoint.'/'.$targetId, [
             'json' => [
                 'tasks' => $taskIds
             ]