Dragonqos / php-jira-rest-client

src/JiraClient.php

Doc Comments +2 added lines, -2 removed lines

@@ -243,7 +243,7 @@ 
     /**
      * @param $rawResponse
      *
-     * @return mixed
+     * @return JiraClientResponse
      */
     public function parseResponse(ResponseInterface $rawResponse)
     {
@@ -252,7 +252,7 @@ 
 
     /**
      * @param          $result
-     * @param array    $responseCodes
+     * @param integer[]    $responseCodes
      * @param \Closure $callback
      *
      * @return mixed

Spacing +8 added lines, -8 removed lines

@@ -63,7 +63,7 @@ 
         $this->json_mapper = new \JsonMapper();
         $this->json_mapper->bEnforceMapType = false;
         $this->json_mapper->setLogger($log);
-        $this->json_mapper->undefinedPropertyHandler = function ($obj, $val) {
+        $this->json_mapper->undefinedPropertyHandler = function($obj, $val) {
             $this->log->debug('Handle undefined property', [$val, $obj]);
         };
 
@@ -105,11 +105,11 @@ 
         try {
             $this->log->info('JiraRestApi request: ', [$httpMethod, $url, $options]);
             $response = $this->transport->request($httpMethod, $url, $options);
-            $this->log->info('JiraRestApi response: ', [$response->getHeaders(), (string)$response->getBody()]);
+            $this->log->info('JiraRestApi response: ', [$response->getHeaders(), (string) $response->getBody()]);
         } catch (ConnectException $e) {
             $this->log->critical('JiraRestApi connection exception: ', [$e->getMessage()]);
         } catch (RequestException $e) {
-            $this->log->error('JiraRestApi response fail with code : ' . $e->getCode(), [(string)$e->getRequest()->getBody(), $e->getRequest()->getHeaders(), (string)$e->getResponse()->getBody()]);
+            $this->log->error('JiraRestApi response fail with code : ' . $e->getCode(), [(string) $e->getRequest()->getBody(), $e->getRequest()->getHeaders(), (string) $e->getResponse()->getBody()]);
             $response = $e->getResponse();
         }
 
@@ -140,7 +140,7 @@ 
 
         $promises = [];
 
-        if(!empty($filePathArray)) {
+        if (!empty($filePathArray)) {
 
             foreach ($filePathArray as $filename => $filePath) {
                 // load each files separately
@@ -162,11 +162,11 @@ 
                 $this->log->info('JiraRestApi requestAsync: ', [Request::METHOD_POST, $url, $options]);
                 $promises[] = $this->transport
                     ->requestAsync(Request::METHOD_POST, $url, $options)
-                    ->then(function (ResponseInterface $response) {
+                    ->then(function(ResponseInterface $response) {
                         $this->log->info('JiraRestApi responseAsync: ', [$response->getHeaders(), (string) $response->getBody()]);
                         return $response;
-                    }, function (RequestException $e) {
-                        if($e instanceof ConnectException) {
+                    }, function(RequestException $e) {
+                        if ($e instanceof ConnectException) {
                             $this->log->critical('JiraRestApi connection exception: ', [$e->getMessage()]);
                             return false;
                         } else {
@@ -230,7 +230,7 @@ 
     {
         $array = json_decode(json_encode($array), true); // toArray
 
-        $array = is_callable($callback) ? array_filter($array, $callback) : array_filter((array)$array);
+        $array = is_callable($callback) ? array_filter($array, $callback) : array_filter((array) $array);
         foreach ($array as &$value) {
             if (is_array($value)) {
                 $value = call_user_func([$this, 'filterNullVariable'], $value, $callback);

src/JiraClientResponse.php

Doc Comments +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
     }
 
     /**
-     * @return bool
+     * @return JiraClientResponse
      */
     public function parse()
     {
@@ -93,7 +93,7 @@ 
     }
 
     /**
-     * @return ResponseInterface
+     * @return integer
      */
     public function getCode()
     {
@@ -109,7 +109,7 @@ 
     }
 
     /**
-     * @return ResponseInterface
+     * @return boolean
      */
     public function hasErrors()
     {

Spacing +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
      */
     public function setError($message)
     {
-        if(is_array($this->error) && isset($this->error['errorMessages'])) {
+        if (is_array($this->error) && isset($this->error['errorMessages'])) {
             $this->error['errorMessages'][] = $message;
         } else {
             $this->error = ['errorMessages' => [$message]];

src/Webhook/WebhookService.php

Spacing +4 added lines, -4 removed lines

@@ -22,7 +22,7 @@ 
     {
         $result = $this->exec($this->uri);
 
-        return $this->extractErrors($result, [200], function () use ($result) {
+        return $this->extractErrors($result, [200], function() use ($result) {
             return $this->json_mapper->mapArray(
                 $result->getRawData(), new \ArrayObject(), '\JiraRestApi\Webhook\Webhook'
             );
@@ -39,7 +39,7 @@ 
     {
         $result = $this->exec($this->uri . '/' . $webhookId);
 
-        return $this->extractErrors($result, [200], function () use ($result) {
+        return $this->extractErrors($result, [200], function() use ($result) {
             return $this->json_mapper->map(
                 $result->getRawData(), new Webhook()
             );
@@ -57,7 +57,7 @@ 
 
         $result = $this->exec($this->uri, $data, Request::METHOD_POST);
 
-        return $this->extractErrors($result, [201], function () use ($result) {
+        return $this->extractErrors($result, [201], function() use ($result) {
             return $this->json_mapper->map(
                 $result->getRawData(), new Webhook()
             );
@@ -73,7 +73,7 @@ 
     {
         $result = $this->exec($this->uri . '/' . $webhookId, null, Request::METHOD_DELETE);
 
-        return $this->extractErrors($result, [204], function () use ($result) {
+        return $this->extractErrors($result, [204], function() use ($result) {
             return $result;
         });
     }

src/Webhook/Webhook.php

Spacing +6 added lines, -6 removed lines

@@ -73,7 +73,7 @@ 
     {
         if (!is_null($this->self)) {
             $parts = explode("/", $this->self);
-            return (int)end($parts);
+            return (int) end($parts);
         }
 
         return null;
@@ -106,16 +106,16 @@ 
             self::JIRA_USER_DELETED,
         ];
 
-        if(is_array($events)) {
-            foreach($events as $event) {
-                if(in_array($event, $allowed_hooks)) {
+        if (is_array($events)) {
+            foreach ($events as $event) {
+                if (in_array($event, $allowed_hooks)) {
                     $this->events[] = $event;
                 }
             }
         }
 
-        if(is_string($events)) {
-            if(in_array($events, $allowed_hooks)) {
+        if (is_string($events)) {
+            if (in_array($events, $allowed_hooks)) {
                 $this->events[] = $events;
             }
         }

src/Provider/Silex2/JiraRestApiProvider.php

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
     {
         $app['jira.config'] = [];
 
-        $app['jira.rest.transport'] = function () use ($app) {
+        $app['jira.rest.transport'] = function() use ($app) {
             $cfg = $app['jira.rest.configuration'];
 
             return new Client([
@@ -32,11 +32,11 @@ 
         };
 
         $app['jira.rest.service.builder'] = $app->protect(function($serviceName) use ($app) {
-            if(class_exists($serviceName)) {
+            if (class_exists($serviceName)) {
                 return new $serviceName($app['jira.rest.configuration'], $app['jira.rest.transport'], $app['logger']);
             }
 
-            throw new \Exception('Service ' . $serviceName .' not found');
+            throw new \Exception('Service ' . $serviceName . ' not found');
         });
 
         $app['jira.rest.issue'] = function() use ($app) {

src/Provider/Silex1/JiraRestApiProvider.php

Spacing +3 added lines, -3 removed lines

@@ -17,7 +17,7 @@ 
     {
         $app['jira.config'] = [];
 
-        $app['jira.rest.transport'] = $app->share(function () use ($app) {
+        $app['jira.rest.transport'] = $app->share(function() use ($app) {
             $cfg = $app['jira.rest.configuration'];
 
             return new Client([
@@ -31,11 +31,11 @@ 
         });
 
         $app['jira.rest.service.builder'] = $app->protect(function($serviceName) use ($app) {
-            if(class_exists($serviceName)) {
+            if (class_exists($serviceName)) {
                 return new $serviceName($app['jira.rest.configuration'], $app['jira.rest.transport'], $app['logger']);
             }
 
-            throw new \Exception('Service ' . $serviceName .' not found');
+            throw new \Exception('Service ' . $serviceName . ' not found');
         });
 
         $app['jira.rest.issue'] = $app->share(function() use ($app) {

src/Issue/IssueService.php

Spacing +14 added lines, -14 removed lines

@@ -23,7 +23,7 @@ 
     {
         $result = $this->exec($this->uri . '/' . $issueIdOrKey);
 
-        return $this->extractErrors($result, [200], function () use ($result) {
+        return $this->extractErrors($result, [200], function() use ($result) {
             return $this->json_mapper->map(
                 $result->getRawData(), new Issue()
             );
@@ -52,7 +52,7 @@ 
         /** @var JiraClientResponse $result */
         $result = $this->exec($this->searchUri, $data, Request::METHOD_POST);
 
-        return $this->extractErrors($result, [200], function () use ($result) {
+        return $this->extractErrors($result, [200], function() use ($result) {
             return $this->json_mapper->map(
                 $result->getRawData(), new IssueSearchResult()
             );
@@ -74,7 +74,7 @@ 
 
         $result = $this->exec($this->uri, $data, Request::METHOD_POST);
 
-        return $this->extractErrors($result, [201], function () use ($result) {
+        return $this->extractErrors($result, [201], function() use ($result) {
             return $this->json_mapper->map(
                 $result->getRawData(), new Issue()
             );
@@ -99,7 +99,7 @@ 
 
         $result = $this->exec($this->uri . '/' . $issueIdOrKey, $data, Request::METHOD_PUT);
 
-        return $this->extractErrors($result, [204], function () use ($result) {
+        return $this->extractErrors($result, [204], function() use ($result) {
             return $result;
         });
     }
@@ -114,11 +114,11 @@ 
      */
     public function addAttachments($issueIdOrKey, array $filePathArray)
     {
-        $results = $this->upload($this->uri . '/' . $issueIdOrKey .'/attachments', $filePathArray);
+        $results = $this->upload($this->uri . '/' . $issueIdOrKey . '/attachments', $filePathArray);
 
         $resArr = [];
         foreach ($results as $result) {
-            $extracted = $this->extractErrors($result, [200], function () use ($result) {
+            $extracted = $this->extractErrors($result, [200], function() use ($result) {
                 return $this->json_mapper->mapArray(
                     $result->getRawData(), new \ArrayObject(), '\JiraRestApi\Issue\Attachment'
                 );
@@ -154,7 +154,7 @@ 
         $data = $this->filterNullVariable($comment);
         $result = $this->exec($this->uri . '/' . $issueIdOrKey . '/comment', $data, Request::METHOD_POST);
 
-        return $this->extractErrors($result, [201], function () use ($result) {
+        return $this->extractErrors($result, [201], function() use ($result) {
             return $this->json_mapper->map(
                 $result->getRawData(), new Comment()
             );
@@ -183,7 +183,7 @@ 
 
         $result = $this->exec($this->uri . '/' . $issueIdOrKey . '/comment/' . $commentId, $data, Request::METHOD_PUT);
 
-        return $this->extractErrors($result, [200], function () use ($result) {
+        return $this->extractErrors($result, [200], function() use ($result) {
             return $this->json_mapper->map(
                 $result->getRawData(), new Comment()
             );
@@ -203,7 +203,7 @@ 
     {
         $result = $this->exec($this->uri . '/' . $issueIdOrKey . '/transitions');
 
-        return $this->extractErrors($result, [200], function () use ($result) {
+        return $this->extractErrors($result, [200], function() use ($result) {
             $data = $result->getRawData();
             return $this->json_mapper->mapArray(
                 $data['transitions'], new \ArrayObject(), '\JiraRestApi\Issue\Transition'
@@ -255,7 +255,7 @@ 
 
         $result = $this->exec($this->uri . '/' . $issueIdOrKey . '/transitions', $data, Request::METHOD_POST);
 
-        return $this->extractErrors($result, [204], function () use ($result) {
+        return $this->extractErrors($result, [204], function() use ($result) {
             return $result;
         });
     }
@@ -293,7 +293,7 @@ 
 
         $result = $this->exec($this->uri . '/' . $issueIdOrKey, $data, Request::METHOD_PUT);
 
-        return $this->extractErrors($result, [204], function () use ($result) {
+        return $this->extractErrors($result, [204], function() use ($result) {
             return $result;
         });
     }
@@ -309,7 +309,7 @@ 
     {
         $result = $this->exec($this->uri . '/' . $issueIdOrKey . '/worklog');
 
-        return $this->extractErrors($result, [200], function () use ($result) {
+        return $this->extractErrors($result, [200], function() use ($result) {
             return $this->json_mapper->map(
                 $result->getRawData(), new Worklog()
             );
@@ -329,7 +329,7 @@ 
         $data['update']['labels'][]['set'] = $labels;
 
         $result = $this->exec($this->uri . '/' . $issueIdOrKey, $data, Request::METHOD_PUT);
-        return $this->extractErrors($result, [204], function () use ($result) {
+        return $this->extractErrors($result, [204], function() use ($result) {
             return $result;
         });
     }
@@ -347,7 +347,7 @@ 
         $data['update']['labels'][]['remove'] = $labels;
 
         $result = $this->exec($this->uri . '/' . $issueIdOrKey, $data, Request::METHOD_PUT);
-        return $this->extractErrors($result, [204], function () use ($result) {
+        return $this->extractErrors($result, [204], function() use ($result) {
             return $result;
         });
     }

src/Issue/IssueTypeService.php

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
     {
         $result = $this->exec($this->uri);
 
-        return $this->extractErrors($result, [200], function () use ($result) {
+        return $this->extractErrors($result, [200], function() use ($result) {
             return $this->json_mapper->mapArray(
                 $result->getRawData(), new \ArrayObject(), '\JiraRestApi\Issue\IssueType'
             );
@@ -31,7 +31,7 @@ 
     {
         $result = $this->exec($this->uri . '/' . $issuetypeId);
 
-        return $this->extractErrors($result, [200], function () use ($result) {
+        return $this->extractErrors($result, [200], function() use ($result) {
             return $this->json_mapper->map(
                 $result->getRawData(), new \JiraRestApi\Issue\IssueType
             );

src/Issue/IssueField.php

Spacing +2 added lines, -2 removed lines

@@ -230,7 +230,7 @@ 
 
     public function setProjectKey($key)
     {
-        if(is_null($this->project)) {
+        if (is_null($this->project)) {
             $this->project = new Project();
         }
 
@@ -241,7 +241,7 @@ 
 
     public function setProjectId($id)
     {
-        if(is_null($this->project)) {
+        if (is_null($this->project)) {
             $this->project = new Project();
         }