Webklex / php-imap

src/Header.php

Braces +6 added lines, -2 removed lines

@@ -285,7 +285,9 @@ 
         }
 
         foreach ($headers as $key => $values) {
-            if (isset($imap_headers[$key])) continue;
+            if (isset($imap_headers[$key])) {
+                continue;
+            }
             $value = null;
             switch ($key) {
                 case 'from':
@@ -506,7 +508,9 @@ 
      * Try to extract the priority from a given raw header string
      */
     private function findPriority() {
-        if (($priority = $this->get("x_priority")) === null) return;
+        if (($priority = $this->get("x_priority")) === null) {
+            return;
+        }
         switch ((int)"$priority") {
             case IMAP::MESSAGE_PRIORITY_HIGHEST;
                 $priority = IMAP::MESSAGE_PRIORITY_HIGHEST;

src/ClientManager.php

Braces +15 added lines, -7 removed lines

@@ -80,13 +80,13 @@ 
             if($value === null) {
                 if(isset(self::$config[$part])) {
                     $value = self::$config[$part];
-                }else{
+                } else{
                     break;
                 }
-            }else{
+            } else{
                 if(isset($value[$part])) {
                     $value = $value[$part];
-                }else{
+                } else{
                     break;
                 }
             }
@@ -230,15 +230,21 @@ 
 
         // From https://stackoverflow.com/a/173479
         $isAssoc = function(array $arr) {
-            if (array() === $arr) return false;
+            if (array() === $arr) {
+                return false;
+            }
             return array_keys($arr) !== range(0, count($arr) - 1);
         };
 
-        if(!is_array($base)) $base = empty($base) ? array() : array($base);
+        if(!is_array($base)) {
+            $base = empty($base) ? array() : array($base);
+        }
 
         foreach($arrays as $append) {
 
-            if(!is_array($append)) $append = array($append);
+            if(!is_array($append)) {
+                $append = array($append);
+            }
 
             foreach($append as $key => $value) {
 
@@ -262,7 +268,9 @@ 
                     // results in $resultConfig['dispositions'] = ['attachment', 'inline']
                     $base[$key] = $this->array_merge_recursive_distinct($base[$key], $value);
                 } else if(is_numeric($key)) {
-                    if(!in_array($value, $base)) $base[] = $value;
+                    if(!in_array($value, $base)) {
+                        $base[] = $value;
+                    }
                 } else {
                     $base[$key] = $value;
                 }

src/Attribute.php

Braces +5 added lines, -3 removed lines

@@ -78,7 +78,9 @@ 
      */
     public function toDate(): Carbon {
         $date = $this->first();
-        if ($date instanceof Carbon) return $date;
+        if ($date instanceof Carbon) {
+            return $date;
+        }
 
         return Carbon::parse($date);
     }
@@ -141,7 +143,7 @@ 
     public function add($value, bool $strict = false): Attribute {
         if (is_array($value)) {
             return $this->merge($value, $strict);
-        }elseif ($value !== null) {
+        } elseif ($value !== null) {
             $this->attach($value, $strict);
         }
 
@@ -188,7 +190,7 @@ 
             if ($this->contains($value) === false) {
                 $this->values[] = $value;
             }
-        }else{
+        } else{
             $this->values[] = $value;
         }
     }

src/Query/WhereQuery.php

Braces +7 added lines, -3 removed lines

@@ -137,7 +137,7 @@ 
             foreach ($criteria as $key => $value) {
                 if (is_numeric($key)) {
                     $this->where($value);
-                }else{
+                } else{
                     $this->where($key, $value);
                 }
             }
@@ -173,7 +173,9 @@ 
      */
     public function orWhere(Closure $closure = null): WhereQuery {
         $this->query->push(['OR']);
-        if ($closure !== null) $closure($this);
+        if ($closure !== null) {
+            $closure($this);
+        }
 
         return $this;
     }
@@ -185,7 +187,9 @@ 
      */
     public function andWhere(Closure $closure = null): WhereQuery {
         $this->query->push(['AND']);
-        if ($closure !== null) $closure($this);
+        if ($closure !== null) {
+            $closure($this);
+        }
 
         return $this;
     }

src/Query/Query.php

Braces +9 added lines, -3 removed lines

@@ -91,7 +91,9 @@ 
         $this->setClient($client);
 
         $this->sequence = ClientManager::get('options.sequence', IMAP::ST_MSGN);
-        if (ClientManager::get('options.fetch') === IMAP::FT_PEEK) $this->leaveUnread();
+        if (ClientManager::get('options.fetch') === IMAP::FT_PEEK) {
+            $this->leaveUnread();
+        }
 
         if (ClientManager::get('options.fetch_order') === 'desc') {
             $this->fetch_order = 'desc';
@@ -137,7 +139,9 @@ 
      * @throws MessageSearchValidationException
      */
     protected function parse_date($date): Carbon {
-        if ($date instanceof Carbon) return $date;
+        if ($date instanceof Carbon) {
+            return $date;
+        }
 
         try {
             $date = Carbon::parse($date);
@@ -644,7 +648,9 @@ 
      * @return $this
      */
     public function limit(int $limit, int $page = 1): Query {
-        if ($page >= 1) $this->page = $page;
+        if ($page >= 1) {
+            $this->page = $page;
+        }
         $this->limit = $limit;
 
         return $this;

src/Message.php

Braces +17 added lines, -11 removed lines

@@ -299,7 +299,7 @@ 
         if(strtolower(substr($method, 0, 3)) === 'get') {
             $name = Str::snake(substr($method, 3));
             return $this->get($name);
-        }elseif (strtolower(substr($method, 0, 3)) === 'set') {
+        } elseif (strtolower(substr($method, 0, 3)) === 'set') {
             $name = Str::snake(substr($method, 3));
 
             if(in_array($name, array_keys($this->attributes))) {
@@ -505,7 +505,7 @@ 
             if ($this->getFlags()->get("seen") == null) {
                 $this->unsetFlag("Seen");
             }
-        }elseif ($this->getFlags()->get("seen") != null) {
+        } elseif ($this->getFlags()->get("seen") != null) {
             $this->setFlag("Seen");
         }
     }
@@ -549,7 +549,7 @@ 
     private function fetchPart(Part $part) {
         if ($part->isAttachment()) {
             $this->fetchAttachment($part);
-        }else{
+        } else{
             $encoding = $this->getEncoding($part);
 
             $content = $this->decodeString($part->content, $part->encoding);
@@ -574,7 +574,7 @@ 
 
             if (isset($this->bodies[$subtype])) {
                 $this->bodies[$subtype] .= "\n".$content;
-            }else{
+            } else{
                 $this->bodies[$subtype] = $content;
             }
         }
@@ -745,9 +745,9 @@ 
                     return EncodingAliases::get($parameter->value, "ISO-8859-2");
                 }
             }
-        }elseif (property_exists($structure, 'charset')){
+        } elseif (property_exists($structure, 'charset')){
             return EncodingAliases::get($structure->charset, "ISO-8859-2");
-        }elseif (is_string($structure) === true){
+        } elseif (is_string($structure) === true){
             return mb_detect_encoding($structure);
         }
 
@@ -936,13 +936,15 @@ 
      * @throws MessageHeaderFetchingException
      */
     protected function fetchNewMail(Folder $folder, int $next_uid, string $event, bool $expunge): Message {
-        if($expunge) $this->client->expunge();
+        if($expunge) {
+            $this->client->expunge();
+        }
 
         $this->client->openFolder($folder->path);
 
         if ($this->sequence === IMAP::ST_UID) {
             $sequence_id = $next_uid;
-        }else{
+        } else{
             $sequence_id = $this->client->getConnection()->getMessageNumber($next_uid);
         }
 
@@ -976,7 +978,9 @@ 
             $trash_path = $trash_path === null ? $this->config["common_folders"]["trash"]: $trash_path;
             $status = $this->move($trash_path);
         }
-        if($expunge) $this->client->expunge();
+        if($expunge) {
+            $this->client->expunge();
+        }
 
         $event = $this->getEvent("message", "deleted");
         $event::dispatch($this);
@@ -996,7 +1000,9 @@ 
      */
     public function restore(bool $expunge = true): bool {
         $status = $this->unsetFlag("Deleted");
-        if($expunge) $this->client->expunge();
+        if($expunge) {
+            $this->client->expunge();
+        }
 
         $event = $this->getEvent("message", "restored");
         $event::dispatch($this);
@@ -1424,7 +1430,7 @@ 
         if ($this->getSequence() === IMAP::ST_UID) {
             $this->setUid($uid);
             $this->setMsglist($msglist);
-        }else{
+        } else{
             $this->setMsgn($uid, $msglist);
         }
     }

src/Folder.php

Braces +7 added lines, -3 removed lines

@@ -240,7 +240,9 @@ 
     public function move(string $new_name, bool $expunge = true): bool {
         $this->client->checkConnection();
         $status = $this->client->getConnection()->renameFolder($this->full_name, $new_name);
-        if($expunge) $this->client->expunge();
+        if($expunge) {
+            $this->client->expunge();
+        }
 
         $folder = $this->client->getFolder($new_name);
         $event = $this->getEvent("folder", "moved");
@@ -316,7 +318,9 @@ 
      */
     public function delete(bool $expunge = true): bool {
         $status = $this->client->getConnection()->deleteFolder($this->path);
-        if($expunge) $this->client->expunge();
+        if($expunge) {
+            $this->client->expunge();
+        }
 
         $event = $this->getEvent("folder", "deleted");
         $event::dispatch($this);
@@ -395,7 +399,7 @@ 
                     $connection->done();
                     $connection->idle();
                 }
-            }catch (Exceptions\RuntimeException $e) {
+            } catch (Exceptions\RuntimeException $e) {
                 if(strpos($e->getMessage(), "empty response") >= 0 && $connection->connected()) {
                     $connection->done();
                     $connection->idle();

src/Connection/Protocols/ImapProtocol.php

Braces +13 added lines, -6 removed lines

@@ -111,7 +111,9 @@ 
         if ($line === "") {
             throw new RuntimeException('empty response');
         }
-        if ($this->debug) echo "<< ".$line."\n";
+        if ($this->debug) {
+            echo "<< ".$line."\n";
+        }
         return $line . "\n";
     }
 
@@ -326,7 +328,9 @@ 
      * @throws RuntimeException
      */
     public function write(string $data) {
-        if ($this->debug) echo ">> ".$data ."\n";
+        if ($this->debug) {
+            echo ">> ".$data ."\n";
+        }
 
         if (fwrite($this->stream, $data . "\r\n") === false) {
             throw new RuntimeException('failed to write - connection closed?');
@@ -478,7 +482,9 @@ 
     public function getCapabilities(): array {
         $response = $this->requestAndResponse('CAPABILITY');
 
-        if (!$response) return [];
+        if (!$response) {
+            return [];
+        }
 
         $capabilities = [];
         foreach ($response as $line) {
@@ -638,13 +644,14 @@ 
             // if we want only one message we can ignore everything else and just return
             if ($to === null && !is_array($from) && ($uid ? $tokens[2][$uidKey] == $from : $tokens[0] == $from)) {
                 // we still need to read all lines
-                while (!$this->readLine($tokens, $tag))
-
+                while (!$this->readLine($tokens, $tag)) {
+                
                     return $data;
+                }
             }
             if ($uid) {
                 $result[$tokens[2][$uidKey]] = $data;
-            }else{
+            } else{
                 $result[$tokens[0]] = $data;
             }
         }