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)
     {

tests/ApiTestCase.php

Spacing +3 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
         $request = $this->history->getLastRequest();
         $this->assertEquals('GET', $request->getMethod());
 
-        $this->assertEquals($this->baseUrl . $path, $request->getUrl());
+        $this->assertEquals($this->baseUrl.$path, $request->getUrl());
     }
 
     /**
@@ -67,7 +67,7 @@ 
         $this->assertEquals('POST', $request->getMethod());
         $this->assertEquals('application/json', $request->getHeader('Content-type'));
 
-        $this->assertEquals($this->baseUrl . $path, $request->getUrl());
+        $this->assertEquals($this->baseUrl.$path, $request->getUrl());
 
         if (null !== $data) {
             $payload = $request->getBody()->__toString();
@@ -87,7 +87,7 @@ 
         $this->assertEquals('PUT', $request->getMethod());
         $this->assertEquals('application/json', $request->getHeader('Content-type'));
 
-        $this->assertEquals($this->baseUrl . $path, $request->getUrl());
+        $this->assertEquals($this->baseUrl.$path, $request->getUrl());
 
         if (null !== $data) {
             $payload = $request->getBody()->__toString();

tests/bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -1,3 +1,3 @@
 <?php
 
-require_once __DIR__ . '/../vendor/autoload.php';
+require_once __DIR__.'/../vendor/autoload.php';

src/Client.php

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);
         }
     }
 
@@ -92,7 +92,7 @@ 
      */
     public function getOrganization($id): Organization
     {
-        $response = $this->get('organizations/'. $id);
+        $response = $this->get('organizations/'.$id);
         return Organization::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -187,7 +187,7 @@ 
             'filter'    => $filter,
             'analytics' => $analytics ? 'true' : 'false',
         ]);
-        $response = $this->get('workers/'. $id, compact('query'));
+        $response = $this->get('workers/'.$id, compact('query'));
 
         return Worker::fromJson($response->json(['object' => true]), $this);
     }
@@ -244,7 +244,7 @@ 
      */
     public function getTeam($id): Team
     {
-        $response = $this->get('teams/'. $id);
+        $response = $this->get('teams/'.$id);
         return Team::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -286,7 +286,7 @@ 
      */
     public function getDestination($id): Destination
     {
-        $response = $this->get('destinations/'. $id);
+        $response = $this->get('destinations/'.$id);
         return Destination::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -317,7 +317,7 @@ 
      */
     public function getRecipient($id): Recipient
     {
-        $response = $this->get('recipients/'. $id);
+        $response = $this->get('recipients/'.$id);
         return Recipient::fromJson($response->json(['object' => true]), $this);
     }
 
@@ -328,7 +328,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)
     {
         $phone = preg_replace('/[^\d]/', '', $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
             ]