Mistralys / mailcode

src/Mailcode/Commands/Command/ShowPhone.php

Spacing +6 added lines, -6 removed lines

@@ -72,7 +72,7 @@ 
     {
         $val = $this->validator->createStringLiteral();
 
-        if($val->isValid())
+        if ($val->isValid())
         {
             $this->sourceFormat = strtoupper($val->getToken()->getText());
             return;
@@ -94,7 +94,7 @@ 
     {
         $countries = self::getSupportedCountries();
 
-        if(isset($countries[$this->sourceFormat])) {
+        if (isset($countries[$this->sourceFormat])) {
             return;
         }
 
@@ -113,7 +113,7 @@ 
      */
     public static function getSupportedCountries() : array
     {
-        if(self::$countriesLoaded) {
+        if (self::$countriesLoaded) {
             return self::$supportedCountries;
         }
 
@@ -121,7 +121,7 @@ 
 
         $data = self::loadData();
 
-        foreach($data as $code => $def)
+        foreach ($data as $code => $def)
         {
             $code = (string)$code;
 
@@ -145,9 +145,9 @@ 
         $data = FileHelper::parseJSONFile(__DIR__.'/ShowPhone/numbers.json');
         $result = array();
 
-        foreach($data as $idx => $def)
+        foreach ($data as $idx => $def)
         {
-            if(!is_array($def) || !is_string($idx))
+            if (!is_array($def) || !is_string($idx))
             {
                 continue;
             }

src/Mailcode/Collection.php

Indentation +28 added lines, -28 removed lines

@@ -26,9 +26,9 @@ 
     public const ERROR_CANNOT_MODIFY_FINALIZED = 52302;
     public const ERROR_NO_VALIDATION_RESULT_AVAILABLE = 52303;
     
-   /**
-    * @var Mailcode_Commands_Command[]
-    */
+    /**
+     * @var Mailcode_Commands_Command[]
+     */
     protected array $commands = array();
     
     /**
@@ -63,21 +63,21 @@ 
         return $this;
     }
     
-   /**
-    * Whether there are any commands in the collection.
-    * 
-    * @return bool
-    */
+    /**
+     * Whether there are any commands in the collection.
+     * 
+     * @return bool
+     */
     public function hasCommands() : bool
     {
         return !empty($this->commands);
     }
     
-   /**
-    * Counts the amount of commands in the collection.
-    * 
-    * @return int
-    */
+    /**
+     * Counts the amount of commands in the collection.
+     * 
+     * @return int
+     */
     public function countCommands() : int
     {
         return count($this->commands);
@@ -98,12 +98,12 @@ 
         $this->commands = $keep;
     }
     
-   /**
-    * Retrieves all commands that were detected, in the exact order
-    * they were found.
-    * 
-    * @return Mailcode_Commands_Command[]
-    */
+    /**
+     * Retrieves all commands that were detected, in the exact order
+     * they were found.
+     * 
+     * @return Mailcode_Commands_Command[]
+     */
     public function getCommands() : array
     {
         $this->validate();
@@ -357,10 +357,10 @@ 
     }
 
     /**
-    * Retrieves only show variable commands in the collection, if any.
-    * 
-    * @return Mailcode_Commands_Command_ShowVariable[]
-    */
+     * Retrieves only show variable commands in the collection, if any.
+     * 
+     * @return Mailcode_Commands_Command_ShowVariable[]
+     */
     public function getShowVariableCommands(): array
     {
         return Mailcode_Collection_TypeFilter::getShowVariableCommands($this->commands);
@@ -374,11 +374,11 @@ 
         return Mailcode_Collection_TypeFilter::getForCommands($this->commands);
     }
 
-   /**
-    * Retrieves only show date commands in the collection, if any.
-    *
-    * @return Mailcode_Commands_Command_ShowDate[]
-    */
+    /**
+     * Retrieves only show date commands in the collection, if any.
+     *
+     * @return Mailcode_Commands_Command_ShowDate[]
+     */
     public function getShowDateCommands() : array
     {
         return Mailcode_Collection_TypeFilter::getShowDateCommands($this->commands);

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/Interfaces/Commands/Validation/NoHTML.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 {
     public const VALIDATION_NAME_NOHTML = 'nohtml';
 
-    public function setHTMLEnabled(bool $enabled=true) : self;
+    public function setHTMLEnabled(bool $enabled = true) : self;
 
     public function isHTMLEnabled() : bool;
 

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

Indentation +43 added lines, -43 removed lines

@@ -31,29 +31,29 @@ 
     public const PACKAGE_NAME = 'Mailcode';
 
     /**
-    * @var Mailcode_Parser|NULL
-    */
+     * @var Mailcode_Parser|NULL
+     */
     protected $parser = null;
     
-   /**
-    * @var Mailcode_Commands|NULL
-    */
+    /**
+     * @var Mailcode_Commands|NULL
+     */
     protected $commands = null;
     
-   /**
-    * @var Mailcode_Variables|NULL
-    */
+    /**
+     * @var Mailcode_Variables|NULL
+     */
     protected $variables = null;
     
-   /**
-    * @var Mailcode_Translator|NULL
-    */
+    /**
+     * @var Mailcode_Translator|NULL
+     */
     protected $translator = null;
     
-   /**
-    * Creates a new mailcode instance.
-    * @return Mailcode
-    */
+    /**
+     * Creates a new mailcode instance.
+     * @return Mailcode
+     */
     public static function create() : Mailcode
     {
         return new Mailcode();
@@ -65,11 +65,11 @@ 
     }
 
     /**
-    * Parses the string to detect all commands contained within.
-    * 
-    * @param string $string
-    * @return Mailcode_Collection
-    */
+     * Parses the string to detect all commands contained within.
+     * 
+     * @param string $string
+     * @return Mailcode_Collection
+     */
     public function parseString(string $string) : Mailcode_Collection
     {
         return $this->getParser()
@@ -77,11 +77,11 @@ 
             ->getCollection();
     }
     
-   /**
-    * Retrieves the string parser instance used to detect commands.
-    * 
-    * @return Mailcode_Parser
-    */
+    /**
+     * Retrieves the string parser instance used to detect commands.
+     * 
+     * @return Mailcode_Parser
+     */
     public function getParser() : Mailcode_Parser
     {
         if(!isset($this->parser)) 
@@ -92,12 +92,12 @@ 
         return $this->parser;
     }
     
-   /**
-    * Retrieves the commands collection, which is used to
-    * access information on the available commands.
-    * 
-    * @return Mailcode_Commands
-    */
+    /**
+     * Retrieves the commands collection, which is used to
+     * access information on the available commands.
+     * 
+     * @return Mailcode_Commands
+     */
     public function getCommands() : Mailcode_Commands
     {
         if(!isset($this->commands)) 
@@ -140,12 +140,12 @@ 
         return $this->variables;
     }
     
-   /**
-    * Creates the translator, which can be used to convert commands
-    * to another supported syntax.
-    * 
-    * @return Mailcode_Translator
-    */
+    /**
+     * Creates the translator, which can be used to convert commands
+     * to another supported syntax.
+     * 
+     * @return Mailcode_Translator
+     */
     public function createTranslator() : Mailcode_Translator
     {
         if(!isset($this->translator))
@@ -156,12 +156,12 @@ 
         return $this->translator;
     }
     
-   /**
-    * Creates the styler, which can be used to retrieve the 
-    * CSS required to style the highlighted commands in HTML.
-    * 
-    * @return Mailcode_Styler
-    */
+    /**
+     * Creates the styler, which can be used to retrieve the 
+     * CSS required to style the highlighted commands in HTML.
+     * 
+     * @return Mailcode_Styler
+     */
     public function createStyler() : Mailcode_Styler
     {
         return new Mailcode_Styler();

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/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/Parser/Match.php

Indentation +12 added lines, -12 removed lines

@@ -23,24 +23,24 @@
  */
 class Mailcode_Parser_Match
 {
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected string $name;
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected string $type;
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected string $params;
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected string $matchedString;
     
     public function __construct(string $name, string $type, string $params, string $matchedString)

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