Apply fixes from StyleCI

src/Auth/AuthService.php

@@ -80,7 +80,7 @@ public function getSessionCookieValue()
      * @throws \Exception
      * @throws \JiraRestApi\JiraException
      */
-    public function __construct(ConfigurationInterface $configuration = null, LoggerInterface $logger = null, $path = './')
+    public function __construct(?ConfigurationInterface $configuration = null, ?LoggerInterface $logger = null, $path = './')
     {
         parent::__construct($configuration, $logger, $path);
         $this->setAPIUri($this->auth_api_uri);

src/Board/BoardService.php

@@ -14,7 +14,7 @@ class BoardService extends \JiraRestApi\JiraClient
 
     private $agileVersion = '1.0';
 
-    public function __construct(ConfigurationInterface $configuration = null, LoggerInterface $logger = null, $path = './')
+    public function __construct(?ConfigurationInterface $configuration = null, ?LoggerInterface $logger = null, $path = './')
     {
         parent::__construct($configuration, $logger, $path);
         $this->setAPIUri('/rest/agile/'.$this->agileVersion);
@@ -51,20 +51,22 @@ public function getBoardList($paramArray = []): ?\ArrayObject
      *
      * @param array $paramArray
      *
-     * @return PaginatedResult|null array of Board class
-     *@throws \JiraRestApi\JiraException
+     * @throws \JiraRestApi\JiraException
      *
+     * @return PaginatedResult|null array of Board class
      */
     public function getBoards($paramArray = []): ?PaginatedResult
     {
         $json = $this->exec($this->uri.$this->toHttpQueryParameter($paramArray), null);
+
         try {
             return $this->json_mapper->map(
                 json_decode($json, false, 512, $this->getJsonOptions()),
                 new PaginatedResult()
             );
         } catch (\JsonException $exception) {
             $this->log->error("Response cannot be decoded from json\nException: {$exception->getMessage()}");
+
             return null;
         }
     }
@@ -165,6 +167,7 @@ public function getSprintsForBoard($boardId, $paramArray = []): ?PaginatedResult
             );
         } catch (\JsonException $exception) {
             $this->log->error("Response cannot be decoded from json\nException: {$exception->getMessage()}");
+
             return null;
         }
     }

src/Board/PaginatedResult.php

@@ -3,7 +3,7 @@
 namespace JiraRestApi\Board;
 
 /**
- * Paginated Result object for BoardService
+ * Paginated Result object for BoardService.
  */
 class PaginatedResult
 {

src/Configuration/ArrayConfiguration.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Created by PhpStorm.
  * User: keanor

src/Configuration/ConfigurationInterface.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Created by PhpStorm.
  * User: keanor

src/Epic/EpicService.php

@@ -11,7 +11,7 @@ class EpicService extends \JiraRestApi\JiraClient
     private $uri = '/epic';
     private $version = '1.0';
 
-    public function __construct(ConfigurationInterface $configuration = null, LoggerInterface $logger = null, $path = './')
+    public function __construct(?ConfigurationInterface $configuration = null, ?LoggerInterface $logger = null, $path = './')
     {
         parent::__construct($configuration, $logger, $path);
         $this->setAPIUri('/rest/agile/'.$this->version);

src/Issue/AgileIssueService.php

@@ -11,7 +11,7 @@ class AgileIssueService extends \JiraRestApi\JiraClient
 
     private $agileVersion = '1.0';
 
-    public function __construct(ConfigurationInterface $configuration = null, LoggerInterface $logger = null, $path = './')
+    public function __construct(?ConfigurationInterface $configuration = null, ?LoggerInterface $logger = null, $path = './')
     {
         parent::__construct($configuration, $logger, $path);
         $this->setAPIUri('/rest/agile/'.$this->agileVersion);

src/Issue/IssueSearchResult.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Created by PhpStorm.
  * User: keanor

src/Issue/TimeTracking.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Created by PhpStorm.
  * User: keanor

src/JiraClient.php

@@ -59,7 +59,7 @@ class JiraClient
      *
      * @throws JiraException
      */
-    public function __construct(ConfigurationInterface $configuration = null, LoggerInterface $logger = null, string $path = './')
+    public function __construct(?ConfigurationInterface $configuration = null, ?LoggerInterface $logger = null, string $path = './')
     {
         if ($configuration === null) {
             if (!file_exists($path.'.env')) {
@@ -188,7 +188,7 @@ protected function filterNullVariable(array $haystack): array
      *
      * @return string|bool
      */
-    public function exec(string $context, array|string $post_data = null, string $custom_request = null, string $cookieFile = null): string|bool
+    public function exec(string $context, array|string|null $post_data = null, ?string $custom_request = null, ?string $cookieFile = null): string|bool
     {
         $url = $this->createUrlByContext($context);
 
@@ -441,7 +441,7 @@ protected function createUrlByContext(string $context): string
     /**
      * Add authorize to curl request.
      */
-    protected function authorization(\CurlHandle $ch, array &$curl_http_headers, string $cookieFile = null): void
+    protected function authorization(\CurlHandle $ch, array &$curl_http_headers, ?string $cookieFile = null): void
     {
         // use cookie
         if ($this->getConfiguration()->isCookieAuthorizationEnabled()) {
@@ -516,7 +516,7 @@ public function toHttpQueryParameter(array $paramArray, bool $dropNullKey = true
     /**
      * download and save into outDir.
      */
-    public function download(string $url, string $outDir, string $file, string $cookieFile = null): mixed
+    public function download(string $url, string $outDir, string $file, ?string $cookieFile = null): mixed
     {
         $curl_http_header = [
             'Accept: */*',

src/JiraException.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Created by PhpStorm.
  * User: keanor
@@ -23,7 +24,7 @@ class JiraException extends \Exception
     /**
      * Create a new Jira exception instance.
      */
-    public function __construct(?string $message = null, int $code = 0, \Throwable $previous = null, string $response = null)
+    public function __construct(?string $message = null, int $code = 0, ?\Throwable $previous = null, ?string $response = null)
     {
         parent::__construct($message, $code, $previous);
 

src/RapidCharts/ScopeChangeBurnDownChartService.php

@@ -12,7 +12,7 @@ class ScopeChangeBurnDownChartService extends \JiraRestApi\JiraClient
 
     private $uri = '/rapid/charts/scopechangeburndownchart';
 
-    public function __construct(ConfigurationInterface $configuration = null, LoggerInterface $logger = null, $path = './')
+    public function __construct(?ConfigurationInterface $configuration = null, ?LoggerInterface $logger = null, $path = './')
     {
         parent::__construct($configuration, $logger, $path);
         $this->setupAPIUri();

src/Request/RequestService.php

@@ -23,7 +23,7 @@ class RequestService extends \JiraRestApi\JiraClient
      * @throws JiraException
      * @throws \Exception
      */
-    public function __construct(ConfigurationInterface $configuration = null, LoggerInterface $logger = null, $path = './')
+    public function __construct(?ConfigurationInterface $configuration = null, ?LoggerInterface $logger = null, $path = './')
     {
         parent::__construct($configuration, $logger, $path);
         $this->setupAPIUri();

src/ServiceDesk/Request/RequestService.php

@@ -65,7 +65,7 @@ public function getRequestFromJSON(object $jsonData): Request
      *
      * @see https://docs.atlassian.com/jira-servicedesk/REST/3.6.2/#servicedeskapi/request-getCustomerRequestByIdOrKey
      */
-    public function get(string $issueId, array $expandParameters = [], Request $request = null): Request
+    public function get(string $issueId, array $expandParameters = [], ?Request $request = null): Request
     {
         $request = ($request) ?: new Request();
 
@@ -88,7 +88,7 @@ public function get(string $issueId, array $expandParameters = [], Request $requ
      *
      * @see https://docs.atlassian.com/jira-servicedesk/REST/3.6.2/#servicedeskapi/request-getMyCustomerRequests
      */
-    public function getRequestsByCustomer(Customer $customer, array $searchParameters, int $serviceDeskId = null): array
+    public function getRequestsByCustomer(Customer $customer, array $searchParameters, ?int $serviceDeskId = null): array
     {
         $defaultSearchParameters = [
             'requestOwnership' => 'OWNED_REQUESTS',

src/ServiceDesk/ServiceDeskClient.php

@@ -12,8 +12,8 @@
 class ServiceDeskClient extends JiraClient
 {
     public function __construct(
-        ConfigurationInterface $configuration = null,
-        LoggerInterface $logger = null,
+        ?ConfigurationInterface $configuration = null,
+        ?LoggerInterface $logger = null,
         string $path = './'
     ) {
         parent::__construct($configuration, $logger, $path);

src/Sprint/SprintSearchResult.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Created by PhpStorm.
  * User: keanor

src/Sprint/SprintService.php

@@ -14,7 +14,7 @@ class SprintService extends JiraClient
 {
     private $uri = '/sprint';
 
-    public function __construct(ConfigurationInterface $configuration = null, LoggerInterface $logger = null, $path = './')
+    public function __construct(?ConfigurationInterface $configuration = null, ?LoggerInterface $logger = null, $path = './')
     {
         parent::__construct($configuration, $logger, $path);
         $this->setAPIUri('/rest/agile/1.0');