Mistralys / mailcode

src/Mailcode.php

Spacing +8 added lines, -8 removed lines

@@ -77,7 +77,7 @@ 
     */
     public function getParser() : Mailcode_Parser
     {
-        if(!isset($this->parser)) 
+        if (!isset($this->parser)) 
         {
             $this->parser = new Mailcode_Parser($this);
         }
@@ -93,7 +93,7 @@ 
     */
     public function getCommands() : Mailcode_Commands
     {
-        if(!isset($this->commands)) 
+        if (!isset($this->commands)) 
         {
             $this->commands = new Mailcode_Commands();
         }
@@ -118,14 +118,14 @@ 
      * @param Mailcode_Commands_Command|null $sourceCommand
      * @return Mailcode_Variables_Collection_Regular
      */
-    public function findVariables(string $subject, ?Mailcode_Commands_Command $sourceCommand=null) : Mailcode_Variables_Collection_Regular
+    public function findVariables(string $subject, ?Mailcode_Commands_Command $sourceCommand = null) : Mailcode_Variables_Collection_Regular
     {
         return $this->createVariables()->parseString($subject, $sourceCommand);
     }
     
     public function createVariables() : Mailcode_Variables
     {
-        if(!isset($this->variables))
+        if (!isset($this->variables))
         {
             $this->variables = new Mailcode_Variables();
         }
@@ -141,7 +141,7 @@ 
     */
     public function createTranslator() : Mailcode_Translator
     {
-        if(!isset($this->translator))
+        if (!isset($this->translator))
         {
             $this->translator = new Mailcode_Translator(); 
         }
@@ -189,7 +189,7 @@ 
         return self::$logger;
     }
 
-    public static function setDebugging(bool $enabled=true) : void
+    public static function setDebugging(bool $enabled = true) : void
     {
         self::$debug = $enabled;
     }
@@ -204,9 +204,9 @@ 
      * @param array<string|int,mixed> $context
      * @return void
      */
-    public static function debug(string $message, array $context=array()) : void
+    public static function debug(string $message, array $context = array()) : void
     {
-        if(self::$debug && isset(self::$logger))
+        if (self::$debug && isset(self::$logger))
         {
             self::$logger->debug($message, $context);
         }

src/Mailcode/Commands/Command/ShowURL/AutoTrackingID.php

Spacing +3 added lines, -3 removed lines

@@ -48,7 +48,7 @@ 
     {
         $instanceID = $command->getInstanceID();
 
-        if(isset(self::$generated[$instanceID]))
+        if (isset(self::$generated[$instanceID]))
         {
             return self::$generated[$instanceID];
         }
@@ -85,11 +85,11 @@ 
 
     private static function generateIDString(TrackingIDInterface $command) : string
     {
-        if(isset(self::$customGenerator))
+        if (isset(self::$customGenerator))
         {
             $result = call_user_func(self::$customGenerator, $command);
 
-            if(is_string($result))
+            if (is_string($result))
             {
                 return $result;
             }

src/Mailcode/Parser.php

Spacing +10 added lines, -10 removed lines

@@ -72,7 +72,7 @@ 
 
         $result = new ParseResult($collection, $preParser);
 
-        if(!$collection->isValid())
+        if (!$collection->isValid())
         {
             return $result;
         }
@@ -84,7 +84,7 @@ 
         
         $total = count($matches[0]);
 
-        for($i=0; $i < $total; $i++)
+        for ($i = 0; $i < $total; $i++)
         {
             $match = $this->parseMatch($matches, $i);
             
@@ -154,17 +154,17 @@ 
     private function handleNesting(Mailcode_Commands_Command $cmd) : void
     {
         // Set the command's parent from the stack, if any is present.
-        if(!empty($this->stack))
+        if (!empty($this->stack))
         {
             $cmd->setParent($this->getStackLast());
         }
 
         // Handle opening and closing commands, adding and removing from the stack.
-        if($cmd instanceof Mailcode_Commands_Command_Type_Opening)
+        if ($cmd instanceof Mailcode_Commands_Command_Type_Opening)
         {
             $this->stack[] = $cmd;
         }
-        else if($cmd instanceof Mailcode_Commands_Command_Type_Closing)
+        else if ($cmd instanceof Mailcode_Commands_Command_Type_Closing)
         {
             array_pop($this->stack);
         }
@@ -178,7 +178,7 @@ 
     {
         $cmd = $this->stack[array_key_last($this->stack)];
 
-        if($cmd instanceof Mailcode_Commands_Command)
+        if ($cmd instanceof Mailcode_Commands_Command)
         {
             return $cmd;
         }
@@ -207,16 +207,16 @@ 
         // 5 = parameter type command, type
         // 6 = parameter type command, params
         
-        if(!empty($matches[1][$index]))
+        if (!empty($matches[1][$index]))
         {
             $name = $matches[1][$index];
         }
-        else if(!empty($matches[2][$index]))
+        else if (!empty($matches[2][$index]))
         {
             $name = $matches[2][$index];
             $params = $matches[3][$index];
         }
-        else if(!empty($matches[4][$index]))
+        else if (!empty($matches[4][$index]))
         {
             $name = $matches[4][$index];
             $type = $matches[5][$index];
@@ -253,7 +253,7 @@ 
      * @param Mailcode_Commands_Command|null $sourceCommand
      * @return Mailcode_Parser_Statement
      */
-    public function createStatement(string $statement, bool $freeform=false, ?Mailcode_Commands_Command $sourceCommand=null) : Mailcode_Parser_Statement
+    public function createStatement(string $statement, bool $freeform = false, ?Mailcode_Commands_Command $sourceCommand = null) : Mailcode_Parser_Statement
     {
         return new Mailcode_Parser_Statement($statement, $freeform, $sourceCommand);
     }

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;
         }