Mistralys / mailcode

src/Mailcode/Parser/PreParser.php

Spacing +17 added lines, -17 removed lines

@@ -116,7 +116,7 @@ 
      */
     public static function getContent(int $id) : string
     {
-        if(isset(self::$contents[$id]))
+        if (isset(self::$contents[$id]))
         {
             return self::$contents[$id];
         }
@@ -139,7 +139,7 @@ 
      */
     public static function clearContent(int $id) : void
     {
-        if(isset(self::$contents[$id]))
+        if (isset(self::$contents[$id]))
         {
             unset(self::$contents[$id]);
         }
@@ -156,7 +156,7 @@ 
      */
     public function parse() : self
     {
-        if($this->parsed)
+        if ($this->parsed)
         {
             return $this;
         }
@@ -166,7 +166,7 @@ 
 
         $this->detectCommands();
 
-        foreach($this->commands as $commandDef)
+        foreach ($this->commands as $commandDef)
         {
             $this->processCommand($commandDef);
         }
@@ -187,7 +187,7 @@ 
         $commands = Mailcode::create()->getCommands()->getContentCommands();
         $result = array();
 
-        foreach($commands as $command)
+        foreach ($commands as $command)
         {
             $result[] = $command->getName();
         }
@@ -260,9 +260,9 @@ 
 
     private function validateCommandContents() : void
     {
-        foreach($this->commands as $command)
+        foreach ($this->commands as $command)
         {
-            if(!$command->validateContent($this, $this->collection))
+            if (!$command->validateContent($this, $this->collection))
             {
                 return;
             }
@@ -274,12 +274,12 @@ 
         $openingCommands = $this->detectOpeningCommands();
         $closingCommands = $this->detectClosingCommands();
 
-        if(!$this->validateCommandsList($openingCommands, $closingCommands))
+        if (!$this->validateCommandsList($openingCommands, $closingCommands))
         {
             return;
         }
 
-        foreach($openingCommands as $idx => $def)
+        foreach ($openingCommands as $idx => $def)
         {
             $this->commands[] = new CommandDef(
                 $def['name'],
@@ -300,20 +300,20 @@ 
         $opening = count($openingCommands);
         $closing = count($closingCommands);
         $max = $opening;
-        if($closing > $max) {
+        if ($closing > $max) {
             $max = $closing;
         }
 
-        for($i=0; $i < $max; $i++)
+        for ($i = 0; $i < $max; $i++)
         {
             // command closed that was never opened
-            if(!isset($openingCommands[$i]))
+            if (!isset($openingCommands[$i]))
             {
                 return $this->addErrorClosedNeverOpened($closingCommands[$i]['matchedText']);
             }
 
             // command opened that was never closed
-            if(!isset($closingCommands[$i]))
+            if (!isset($closingCommands[$i]))
             {
                 return $this->addErrorNeverClosed(
                     $openingCommands[$i]['matchedText'],
@@ -322,7 +322,7 @@ 
             }
 
             // command closed does not match opening
-            if($openingCommands[$i]['name'] !== $closingCommands[$i]['name'])
+            if ($openingCommands[$i]['name'] !== $closingCommands[$i]['name'])
             {
                 return $this->addErrorClosingMismatch(
                     $openingCommands[$i]['name'],
@@ -377,7 +377,7 @@ 
 
         $result = array();
 
-        foreach($matches[0] as $idx => $matchedText)
+        foreach ($matches[0] as $idx => $matchedText)
         {
             $result[] = array(
                 'name' => strtolower(trim((string)$matches[1][$idx])),
@@ -461,7 +461,7 @@ 
             $matchedText,
             t(
                 'The command is never closed with a matching %1$s command.',
-                sb()->code('{' . $name . '}')
+                sb()->code('{'.$name.'}')
             ),
             Mailcode_Commands_CommonConstants::VALIDATION_MISSING_CONTENT_CLOSING_TAG
         );
@@ -485,7 +485,7 @@ 
                 )
                 ->t(
                     'It must be closed with a matching %1$s command.',
-                    sb()->code('{' . $name . '}')
+                    sb()->code('{'.$name.'}')
                 ),
             Mailcode_Commands_CommonConstants::VALIDATION_CONTENT_CLOSING_MISMATCHED_TAG
         );

src/Mailcode/Parser/PreParser/Debugger.php

Spacing +10 added lines, -10 removed lines

@@ -30,22 +30,22 @@ 
      */
     public function debugOpeningCommands(array $matches) : void
     {
-        if(!Mailcode::isDebugEnabled())
+        if (!Mailcode::isDebugEnabled())
         {
             return;
         }
 
         Mailcode::debug('Opening command matches:');
 
-        if(empty($matches))
+        if (empty($matches))
         {
             Mailcode::debug('...None found.');
             return;
         }
 
-        foreach($matches[0] as $idx => $matchedText)
+        foreach ($matches[0] as $idx => $matchedText)
         {
-            $number = $idx+1;
+            $number = $idx + 1;
 
             Mailcode::debug(sprintf('...#%02d matched: [%s]', $number, $matchedText));
             Mailcode::debug(sprintf('...#%02d name...: [%s]', $number, $matches[1][$idx]));
@@ -59,22 +59,22 @@ 
      */
     public function debugClosingCommands(array $commands) : void
     {
-        if(!Mailcode::isDebugEnabled())
+        if (!Mailcode::isDebugEnabled())
         {
             return;
         }
 
         Mailcode::debug('Closing command matches:', $commands);
 
-        if(empty($commands))
+        if (empty($commands))
         {
             Mailcode::debug('...None found.');
             return;
         }
 
-        foreach($commands as $idx => $command)
+        foreach ($commands as $idx => $command)
         {
-            $number = $idx+1;
+            $number = $idx + 1;
 
             Mailcode::debug(sprintf('...#%02d matched: [%s]', $number, $command['matchedText']));
             Mailcode::debug(sprintf('...#%02d name...: [%s]', $number, $command['name']));
@@ -83,14 +83,14 @@ 
 
     public function debugCommandDef(CommandDef $commandDef) : void
     {
-        if(!Mailcode::isDebugEnabled())
+        if (!Mailcode::isDebugEnabled())
         {
             return;
         }
 
         Mailcode::debug('Command definition:', $commandDef->toArray());
 
-        if(defined('TESTS_ROOT'))
+        if (defined('TESTS_ROOT'))
         {
             print_r($commandDef->toArray());
         }

src/Mailcode/Collection.php

Spacing +20 added lines, -20 removed lines

@@ -45,7 +45,7 @@ 
      */
     public function addCommand(Mailcode_Commands_Command $command) : Mailcode_Collection
     {
-        if($this->finalized)
+        if ($this->finalized)
         {
             throw new Mailcode_Exception(
                 'Cannot add commands to a finalized collection',
@@ -87,9 +87,9 @@ 
     {
         $keep = array();
 
-        foreach($this->commands as $existing)
+        foreach ($this->commands as $existing)
         {
-            if($existing !== $command)
+            if ($existing !== $command)
             {
                 $keep[] = $existing;
             }
@@ -126,11 +126,11 @@ 
 
         $hashes = array();
         
-        foreach($this->commands as $command)
+        foreach ($this->commands as $command)
         {
             $hash = $command->getHash();
             
-            if(!isset($hashes[$hash]))
+            if (!isset($hashes[$hash]))
             {
                 $hashes[$hash] = $command;
             }
@@ -148,7 +148,7 @@ 
      */
     public function addCommands(array $commands) : Mailcode_Collection
     {
-        foreach($commands as $command)
+        foreach ($commands as $command)
         {
             $this->addCommand($command);
         }
@@ -171,7 +171,7 @@ 
 
         $collection = new Mailcode_Variables_Collection_Regular();
         
-        foreach($this->commands as $command)
+        foreach ($this->commands as $command)
         {
             $collection->mergeWith($command->getVariables());
         }
@@ -211,7 +211,7 @@ 
     {
         $this->validate();
 
-        if(isset($this->validationResult))
+        if (isset($this->validationResult))
         {
             return $this->validationResult;
         }
@@ -225,7 +225,7 @@ 
 
     private function validate() : void
     {
-        if(isset($this->validationResult) || $this->validating)
+        if (isset($this->validationResult) || $this->validating)
         {
             return;
         }
@@ -245,9 +245,9 @@ 
     {
         $errors = $this->getErrors();
         
-        foreach($errors as $error)
+        foreach ($errors as $error)
         {
-            if($error->getCode() === $code)
+            if ($error->getCode() === $code)
             {
                 return true;
             }
@@ -282,7 +282,7 @@ 
 
         $errors = $this->errors;
 
-        if(!$result->isValid())
+        if (!$result->isValid())
         {
             $errors[] = new Mailcode_Collection_Error_Message(
                 '',
@@ -298,7 +298,7 @@ 
     {
         $errors = $this->getErrors();
 
-        if(!empty($errors))
+        if (!empty($errors))
         {
             return array_shift($errors);
         }
@@ -319,11 +319,11 @@ 
 
     private function validateNesting() : void
     {
-        foreach($this->commands as $command)
+        foreach ($this->commands as $command)
         {
             $command->validateNesting();
 
-            if(!$command->isValid()) {
+            if (!$command->isValid()) {
                 $this->addInvalidCommand($command);
             }
         }
@@ -400,7 +400,7 @@ 
     {
         $commands = $this->getCommands();
         
-        if(!empty($commands))
+        if (!empty($commands))
         {
             return array_shift($commands);
         }
@@ -432,16 +432,16 @@ 
      */
     private function pruneProtectedContentCommands() : void
     {
-        if(!$this->isValid())
+        if (!$this->isValid())
         {
             return;
         }
 
         $keep = array();
 
-        foreach($this->commands as $command)
+        foreach ($this->commands as $command)
         {
-            if(!$command->hasContentParent())
+            if (!$command->hasContentParent())
             {
                 $keep[] = $command;
             }
@@ -458,7 +458,7 @@ 
         $errors = $this->getErrors();
         $result = array();
 
-        foreach($errors as $error)
+        foreach ($errors as $error)
         {
             $result[] = $error->getCode();
         }

src/Mailcode/Parser/Safeguard/URLAnalyzer.php

Spacing +5 added lines, -5 removed lines

@@ -58,24 +58,24 @@ 
     private function analyzeURL(string $url) : void
     {
         // Ignore phone URLs
-        if(stripos($url, 'tel:') !== false)
+        if (stripos($url, 'tel:') !== false)
         {
             return;
         }
 
         $placeholders = $this->safeguard->getPlaceholdersCollection()->getAll();
 
-        foreach($placeholders as $placeholder)
+        foreach ($placeholders as $placeholder)
         {
             $command = $placeholder->getCommand();
 
             // The URL is not found in the replacement text
-            if(strpos($url, $placeholder->getReplacementText()) === false)
+            if (strpos($url, $placeholder->getReplacementText()) === false)
             {
                 continue;
             }
 
-            if($command instanceof URLEncodingInterface)
+            if ($command instanceof URLEncodingInterface)
             {
                 $this->applyEncoding($command);
             }
@@ -84,7 +84,7 @@ 
 
     private function applyEncoding(URLEncodingInterface $command) : void
     {
-        if(!$command->isURLDecoded())
+        if (!$command->isURLDecoded())
         {
             $command->setURLEncoding();
         }

src/Mailcode/Parser/Statement/Tokenizer/SpecialChars.php

Spacing +1 added lines, -1 removed lines

@@ -101,7 +101,7 @@
     {
         $replaces = array();
 
-        foreach(self::$charsEncoded as $char => $placeholder)
+        foreach (self::$charsEncoded as $char => $placeholder)
         {
             $escaped = self::$charsEscaped[$char];
 

src/Mailcode/Parser/Statement/Tokenizer/Token/Variable.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      */
     public function getVariable() : Mailcode_Variables_Variable
     {
-        if($this->subject instanceof Mailcode_Variables_Variable)
+        if ($this->subject instanceof Mailcode_Variables_Variable)
         {
             return $this->subject;
         }

src/Mailcode/Parser/Statement/Info/Keywords.php

Spacing +8 added lines, -8 removed lines

@@ -43,7 +43,7 @@ 
     {
         $token = $this->info->getTokenByIndex($index);
 
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Keyword)
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Keyword)
         {
             return $token;
         }
@@ -62,9 +62,9 @@ 
     {
         $tokens = $this->getAll();
 
-        foreach($tokens as $token)
+        foreach ($tokens as $token)
         {
-            if($token->getKeyword() === $name)
+            if ($token->getKeyword() === $name)
             {
                 return $token;
             }
@@ -81,9 +81,9 @@ 
         $result = array();
         $tokens = $this->info->getTokens();
 
-        foreach($tokens as $token)
+        foreach ($tokens as $token)
         {
-            if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Keyword)
+            if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Keyword)
             {
                 $result[] = $token;
             }
@@ -102,7 +102,7 @@ 
      */
     public function setEnabled(string $keyword, bool $enabled) : Mailcode_Parser_Statement_Info_Keywords
     {
-        if($enabled)
+        if ($enabled)
         {
             return $this->add($keyword);
         }
@@ -121,7 +121,7 @@ 
     {
         $keyword = rtrim($keyword, ':').':';
 
-        if(!$this->hasKeyword($keyword))
+        if (!$this->hasKeyword($keyword))
         {
             $this->tokenizer->appendKeyword($keyword);
         }
@@ -166,7 +166,7 @@ 
 
         foreach ($keywords as $kw)
         {
-            if($kw->getKeyword() === $keyword)
+            if ($kw->getKeyword() === $keyword)
             {
                 return true;
             }

src/Mailcode/Parser/StringPreProcessor.php

Spacing +2 added lines, -2 removed lines

@@ -79,7 +79,7 @@ 
      */
     private function stripStyleTags() : void
     {
-        if(!ConvertHelper::isStringHTML($this->subject))
+        if (!ConvertHelper::isStringHTML($this->subject))
         {
             return;
         }
@@ -93,7 +93,7 @@ 
 
     private function escapeRegexBracketMatch(string $subject, string $needle) : string
     {
-        $replacement =  str_replace(
+        $replacement = str_replace(
             array('{', '}'),
             array('\{', '\}'),
             $needle

src/Mailcode/Factory/Instantiator.php

Spacing +23 added lines, -23 removed lines

@@ -22,11 +22,11 @@ 
  */
 class Mailcode_Factory_Instantiator
 {
-    public function buildIf(string $ifType, string $params, string $type='') : Mailcode_Commands_IfBase
+    public function buildIf(string $ifType, string $params, string $type = '') : Mailcode_Commands_IfBase
     {
         $stringType = $type;
         
-        if(!empty($type))
+        if (!empty($type))
         {
             $stringType = ' '.$type;
         }
@@ -45,7 +45,7 @@ 
         
         $this->checkCommand($command);
         
-        if($command instanceof Mailcode_Commands_IfBase)
+        if ($command instanceof Mailcode_Commands_IfBase)
         {
             return $command;
         }
@@ -53,16 +53,16 @@ 
         throw $this->exceptionUnexpectedType('IfBase', $command);
     }
     
-    public function buildIfVar(string $ifType, string $variable, string $operand, string $value, bool $quoteValue=false, bool $insensitive=false) : Mailcode_Commands_IfBase
+    public function buildIfVar(string $ifType, string $variable, string $operand, string $value, bool $quoteValue = false, bool $insensitive = false) : Mailcode_Commands_IfBase
     {
         $variable = $this->filterVariableName($variable);
 
-        if($insensitive)
+        if ($insensitive)
         {
             $value = mb_strtolower($value);
         }
 
-        if($quoteValue)
+        if ($quoteValue)
         {
             $value = $this->quoteString($value);
         }
@@ -74,7 +74,7 @@ 
             $value
         );
 
-        if($insensitive)
+        if ($insensitive)
         {
             $condition .= ' '.Mailcode_Commands_Keywords::TYPE_INSENSITIVE;
         }
@@ -102,7 +102,7 @@ 
      * @return Mailcode_Commands_IfBase
      * @throws Mailcode_Factory_Exception
      */
-    public function buildIfContains(string $ifType, string $variable, array $searchTerms, bool $caseInsensitive=false, bool $regexEnabled=false, string $containsType='contains') : Mailcode_Commands_IfBase
+    public function buildIfContains(string $ifType, string $variable, array $searchTerms, bool $caseInsensitive = false, bool $regexEnabled = false, string $containsType = 'contains') : Mailcode_Commands_IfBase
     {
         $condition = sprintf(
             '%s%s"%s"',
@@ -114,23 +114,23 @@ 
         return $this->buildIf($ifType, $condition, $containsType);
     }
 
-    private function renderListKeywords(bool $caseInsensitive=false, bool $regexEnabled=false) : string
+    private function renderListKeywords(bool $caseInsensitive = false, bool $regexEnabled = false) : string
     {
         $keywords = array();
 
-        if($caseInsensitive)
+        if ($caseInsensitive)
         {
             $keywords[] = Mailcode_Commands_Keywords::TYPE_INSENSITIVE;
         }
 
-        if($regexEnabled)
+        if ($regexEnabled)
         {
             $keywords[] = Mailcode_Commands_Keywords::TYPE_REGEX;
         }
 
         $keywordsString = '';
 
-        if(!empty($keywords))
+        if (!empty($keywords))
         {
             $keywordsString = ' '.implode(' ', $keywords);
         }
@@ -147,7 +147,7 @@ 
      * @return Mailcode_Commands_IfBase
      * @throws Mailcode_Factory_Exception
      */
-    public function buildIfNotContains(string $ifType, string $variable, array $searchTerms, bool $caseInsensitive=false, bool $regexEnabled=false) : Mailcode_Commands_IfBase
+    public function buildIfNotContains(string $ifType, string $variable, array $searchTerms, bool $caseInsensitive = false, bool $regexEnabled = false) : Mailcode_Commands_IfBase
     {
         return $this->buildIfContains($ifType, $variable, $searchTerms, $caseInsensitive, $regexEnabled, 'not-contains');
     }
@@ -162,7 +162,7 @@ 
      * @return Mailcode_Commands_IfBase
      * @throws Mailcode_Factory_Exception
      */
-    public function buildIfListContains(string $ifType, string $variable, array $searchTerms, bool $caseInsensitive=false, bool $regexEnabled=false, string $containsType='list-contains') : Mailcode_Commands_IfBase
+    public function buildIfListContains(string $ifType, string $variable, array $searchTerms, bool $caseInsensitive = false, bool $regexEnabled = false, string $containsType = 'list-contains') : Mailcode_Commands_IfBase
     {
         return $this->buildIfContains($ifType, $variable, $searchTerms, $caseInsensitive, $regexEnabled, $containsType);
     }
@@ -176,17 +176,17 @@ 
      * @return Mailcode_Commands_IfBase
      * @throws Mailcode_Factory_Exception
      */
-    public function buildIfListNotContains(string $ifType, string $variable, array $searchTerms, bool $caseInsensitive=false, bool $regexEnabled=false) : Mailcode_Commands_IfBase
+    public function buildIfListNotContains(string $ifType, string $variable, array $searchTerms, bool $caseInsensitive = false, bool $regexEnabled = false) : Mailcode_Commands_IfBase
     {
         return $this->buildIfContains($ifType, $variable, $searchTerms, $caseInsensitive, $regexEnabled, 'list-not-contains');
     }
 
-    public function buildIfBeginsWith(string $ifType, string $variable, string $search, bool $caseInsensitive=false) : Mailcode_Commands_IfBase
+    public function buildIfBeginsWith(string $ifType, string $variable, string $search, bool $caseInsensitive = false) : Mailcode_Commands_IfBase
     {
         return $this->buildIfSearch($ifType, 'begins-with', $variable, $search, $caseInsensitive);
     }
     
-    public function buildIfEndsWith(string $ifType, string $variable, string $search, bool $caseInsensitive=false) : Mailcode_Commands_IfBase
+    public function buildIfEndsWith(string $ifType, string $variable, string $search, bool $caseInsensitive = false) : Mailcode_Commands_IfBase
     {
         return $this->buildIfSearch($ifType, 'ends-with', $variable, $search, $caseInsensitive);
     }
@@ -236,11 +236,11 @@ 
         );
     }
 
-    private function buildIfSearch(string $ifType, string $subType, string $variable, string $search, bool $caseInsensitive=false) : Mailcode_Commands_IfBase
+    private function buildIfSearch(string $ifType, string $subType, string $variable, string $search, bool $caseInsensitive = false) : Mailcode_Commands_IfBase
     {
         $keyword = ' ';
         
-        if($caseInsensitive)
+        if ($caseInsensitive)
         {
             $keyword = ' '.Mailcode_Commands_Keywords::TYPE_INSENSITIVE;
         }
@@ -274,7 +274,7 @@ 
     
     public function checkCommand(Mailcode_Commands_Command $command) : void
     {
-        if($command->isValid())
+        if ($command->isValid())
         {
             return;
         }
@@ -303,7 +303,7 @@ 
      */
     public function setEncoding(Mailcode_Commands_Command $cmd, string $urlEncoding) : void
     {
-        if($cmd instanceof Mailcode_Interfaces_Commands_Validation_URLEncode)
+        if ($cmd instanceof Mailcode_Interfaces_Commands_Validation_URLEncode)
         {
             $cmd->setURLEncoding(false);
 
@@ -313,7 +313,7 @@ 
             }
         }
 
-        if($cmd instanceof Mailcode_Interfaces_Commands_Validation_URLDecode)
+        if ($cmd instanceof Mailcode_Interfaces_Commands_Validation_URLDecode)
         {
             $cmd->setURLDecoding(false);
 
@@ -332,7 +332,7 @@ 
      */
     public function quoteString(string $string) : string
     {
-        if(substr($string, 0, 1) === '"' && substr($string, -1, 1) === '"') {
+        if (substr($string, 0, 1) === '"' && substr($string, -1, 1) === '"') {
             return $string;
         }