Merge pull request #661 from barbushin/Fix-php-cs-fixer-issues

Fix php-cs-fixer formatting issue

Author: Sebbo94BY

src/PhpImap/Exceptions/ConnectionException.php

@@ -13,12 +13,12 @@
  */
 class ConnectionException extends Exception
 {
-    public function __construct($message, $code = 0, Exception $previous = null)
+    public function __construct(array $message, int $code = 0, Exception $previous = null)
     {
         parent::__construct(json_encode($message), $code, $previous);
     }
 
-    public function getErrors($select = 'first')
+    public function getErrors(string $select = 'first')
     {
         $message = $this->getMessage();
 

src/PhpImap/Mailbox.php

@@ -837,21 +837,20 @@ public function markMailsAsImportant(array $mailId): void
     /**
      * Check, if the specified flag for the mail is set or not.
      *
-     * @param int    $mailsId A single mail ID
-     * @param string $flag    Which you can get are \Seen, \Answered, \Flagged, \Deleted, and \Draft as defined by RFC2060
+     * @param int    $mailId A single mail ID
+     * @param string $flag   Which you can get are \Seen, \Answered, \Flagged, \Deleted, and \Draft as defined by RFC2060
      *
      * @return bool True, when the flag is set, false when not
      *
-     * @psalm-param list<int> $mailId
+     * @psalm-param int $mailId
      */
     public function flagIsSet(int $mailId, string $flag): bool
     {
         $flag = str_replace('\\', '', strtolower($flag));
 
         $overview = Imap::fetch_overview($this->getImapStream(), $mailId, ST_UID);
 
-        if ($overview[0]->$flag == 1)
-        {
+        if ($overview[0]->$flag == 1) {
             return true;
         }
 
@@ -1677,7 +1676,7 @@ protected function initImapStreamWithRetry()
                 return $this->initImapStream();
             } catch (ConnectionException $exception) {
             }
-        } while (--$retry > 0 && (!$this->connectionRetryDelay || !\usleep($this->connectionRetryDelay * 1000)));
+        } while (--$retry > 0 && (!$this->connectionRetryDelay || !\usleep((int) $this->connectionRetryDelay * 1000)));
 
         throw $exception;
     }
@@ -1899,8 +1898,11 @@ protected function getCombinedPath(string $folder, bool $absolute = false): stri
     protected function possiblyGetEmailAndNameFromRecipient(object $recipient): ?array
     {
         if (isset($recipient->mailbox, $recipient->host)) {
+            /** @var string */
             $recipientMailbox = $recipient->mailbox;
+            /** @var string */
             $recipientHost = $recipient->host;
+            /** @var string|null */
             $recipientPersonal = $recipient->personal ?? null;
 
             if (!\is_string($recipientMailbox)) {

tests/unit/AbstractLiveMailboxTest.php

@@ -45,8 +45,6 @@ abstract class AbstractLiveMailboxTest extends TestCase
 
     /**
      * @psalm-return Generator<empty, empty, mixed, void>
-     *
-     * @return Generator
      */
     public function ComposeProvider(): Generator
     {

tests/unit/LiveMailboxIssue250Test.php

@@ -38,8 +38,6 @@ class LiveMailboxIssue250Test extends AbstractLiveMailboxTest
 {
     /**
      * @psalm-return Generator<int, array{0: array{subject: string}, 1: array{0: array{type: 0, 'contents.data': 'test'}}, 2: string}, mixed, void>
-     *
-     * @return Generator
      */
     public function ComposeProvider(): Generator
     {

tests/unit/LiveMailboxTest.php

@@ -144,8 +144,6 @@ public function testGetImapStream(HiddenString $imapPath, HiddenString $login, H
 
     /**
      * @psalm-return Generator<int, array{0: array{subject: string}, 1: array{0: array{type: 0|1|3, 'contents.data'?: string, encoding?: 3, subtype?: 'octet-stream', description?: '.gitignore'|'gitignore.', 'disposition.type'?: 'attachment', disposition?: array{filename: '.gitignore'|'gitignore.'}, 'type.parameters'?: array{name: '.gitignore'|'gitignore.'}}, 1?: array{type: 0, 'contents.data': 'test'}, 2?: array{type: 3, encoding: 3, subtype: 'octet-stream', description: 'foo.bin', 'disposition.type': 'attachment', disposition: array{filename: 'foo.bin'}, 'type.parameters': array{name: 'foo.bin'}, 'contents.data': string}, 3?: array{type: 3, encoding: 3, subtype: 'octet-stream', description: 'foo.bin', 'disposition.type': 'attachment', disposition: array{filename: 'foo.bin'}, 'type.parameters': array{name: 'foo.bin'}, 'contents.data': string}}, 2: string}, mixed, void>
-     *
-     * @return Generator
      */
     public function ComposeProvider(): Generator
     {