anorgan / onfleet-client

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/Task.php

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,

Doc Comments +3 added lines, -3 removed lines

@@ -141,7 +141,7 @@ 
     }
 
     /**
-     * @param mixed $merchant
+     * @param string|null $merchant
      */
     public function setMerchant($merchant)
     {
@@ -157,7 +157,7 @@ 
     }
 
     /**
-     * @param mixed $executor
+     * @param string|null $executor
      */
     public function setExecutor($executor)
     {
@@ -365,7 +365,7 @@ 
     }
 
     /**
-     * @param mixed $destination
+     * @param string|null $destination
      */
     public function setDestination($destination)
     {

src/Client.php

Doc Comments +4 added lines, -4 removed lines

@@ -35,10 +35,10 @@ 
     }
 
     /**
-     * @param null $url
+     * @param string $url
      * @param array $options
      * @throws \Exception
-     * @return \GuzzleHttp\Message\FutureResponse|\GuzzleHttp\Message\ResponseInterface|\GuzzleHttp\Ring\Future\FutureInterface|null
+     * @return Client
      */
     public function post($url = null, array $options = [])
     {
@@ -59,9 +59,9 @@ 
     }
 
     /**
-     * @param null $url
+     * @param string $url
      * @param array $options
-     * @return \GuzzleHttp\Message\FutureResponse|\GuzzleHttp\Message\ResponseInterface|\GuzzleHttp\Ring\Future\FutureInterface|null
+     * @return \GuzzleHttp\Message\ResponseInterface|null
      */
     public function get($url = null, $options = [])
     {

Spacing +13 added lines, -13 removed lines

@@ -49,12 +49,12 @@ 
             $message = $error['message']['message'];
             if (isset($error['message']['cause'])) {
                 if (is_array($error['message']['cause'])) {
-                    $message .= ' '. implode(', ', $error['message']['cause']);
+                    $message .= ' '.implode(', ', $error['message']['cause']);
                 } else {
-                    $message .= ' '. $error['message']['cause'];
+                    $message .= ' '.$error['message']['cause'];
                 }
             }
-            throw new \Exception('Error while calling post on '. $url .': '. $message);
+            throw new \Exception('Error while calling post on '.$url.': '.$message);
         }
     }
 
@@ -91,7 +91,7 @@ 
      */
     public function getOrganization($id): Organization
     {
-        $response = $this->get('organizations/'. $id);
+        $response = $this->get('organizations/'.$id);
         return Organization::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -183,7 +183,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',
@@ -245,7 +245,7 @@ 
      */
     public function getTeam($id): Team
     {
-        $response = $this->get('teams/'. $id);
+        $response = $this->get('teams/'.$id);
         return Team::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -287,7 +287,7 @@ 
      */
     public function getDestination($id): Destination
     {
-        $response = $this->get('destinations/'. $id);
+        $response = $this->get('destinations/'.$id);
         return Destination::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -318,7 +318,7 @@ 
      */
     public function getRecipient($id): Recipient
     {
-        $response = $this->get('recipients/'. $id);
+        $response = $this->get('recipients/'.$id);
         return Recipient::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -329,7 +329,7 @@ 
     public function getRecipientByName($name)
     {
         $name     = str_replace('+', '%20', urlencode(strtolower($name)));
-        $response = $this->get('recipients/name/'. $name);
+        $response = $this->get('recipients/name/'.$name);
 
         if (null === $response) {
             return null;
@@ -344,7 +344,7 @@ 
      */
     public function getRecipientByPhone($phone)
     {
-        $response = $this->get('recipients/phone/'. $phone);
+        $response = $this->get('recipients/phone/'.$phone);
 
         if (null === $response) {
             return null;
@@ -405,7 +405,7 @@ 
      */
     public function getTask($id): Task
     {
-        $response = $this->get('tasks/'. $id);
+        $response = $this->get('tasks/'.$id);
         return Task::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -415,7 +415,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);
     }
 
@@ -527,7 +527,7 @@ 
             array_unshift($taskIds, $position);
         }
 
-        $this->put('containers/'. $containerEndpoint .'/'. $targetId, [
+        $this->put('containers/'.$containerEndpoint.'/'.$targetId, [
             'json' => [
                 'tasks' => $taskIds
             ]