Apply fixes from StyleCI (#96)

Co-authored-by: StyleCI Bot <bot@styleci.io>

Author: lesstif

src/Attachment/AttachmentService.php

@@ -30,7 +30,7 @@ class AttachmentService extends \JiraCloud\JiraClient
      *
      * @return Attachment
      */
-    public function get(int|string $id, string $outDir = null, bool $overwrite = false, int $mode = 0777, bool $recursive = true): Attachment
+    public function get(int|string $id, ?string $outDir = null, bool $overwrite = false, int $mode = 0777, bool $recursive = true): Attachment
     {
         $ret = $this->exec($this->uri.$id, null);
 

src/Auth/AuthService.php

@@ -80,7 +80,7 @@ public function getSessionCookieValue()
      * @throws \Exception
      * @throws \JiraCloud\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/Epic/EpicService.php

@@ -11,7 +11,7 @@ class EpicService extends \JiraCloud\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

@@ -9,7 +9,7 @@ class AgileIssueService extends \JiraCloud\JiraClient
 {
     private $uri = '/rest/agile/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);
     }

src/Issue/IssueField.php

@@ -61,7 +61,7 @@ class IssueField implements \JsonSerializable
     public ?Reporter $assignee = null;
 
     /** @var \JiraCloud\Issue\Version[]|null */
-    public array|null $versions = null;
+    public ?array $versions = null;
 
     /** @var \JiraCloud\Issue\Attachment[] */
     public $attachment;

src/Issue/IssueService.php

@@ -39,7 +39,7 @@ public function getIssueFromJSON($json): Issue
      *
      * @return Issue class
      */
-    public function get(int|string $issueIdOrKey, array $paramArray = [], Issue $issueObject = null): Issue
+    public function get(int|string $issueIdOrKey, array $paramArray = [], ?Issue $issueObject = null): Issue
     {
         $issueObject = ($issueObject) ? $issueObject : new Issue();
 

src/Issue/Reporter.php

@@ -14,7 +14,7 @@ class Reporter implements \JsonSerializable
 
     public ?string $emailAddress = null;
 
-    public array|null $avatarUrls;
+    public ?array $avatarUrls;
 
     public string $displayName;
 

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')) {
@@ -186,7 +186,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);
 
@@ -236,8 +236,8 @@ public function exec(string $context, array|string $post_data = null, string $cu
             curl_setopt($ch, CURLOPT_FOLLOWLOCATION, true);
         }
 
-       // See https://github.com/php/php-src/issues/14184
-       // curl_setopt($ch, CURLOPT_ENCODING, '');
+        // See https://github.com/php/php-src/issues/14184
+        // curl_setopt($ch, CURLOPT_ENCODING, '');
 
         curl_setopt(
             $ch,
@@ -405,7 +405,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()) {
@@ -472,7 +472,7 @@ public function toHttpQueryParameter(array $paramArray): string
     /**
      * 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

@@ -17,7 +17,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 \JiraCloud\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();