Mistralys / mailcode

src/Mailcode/Traits/Commands/Validation/ParseParams.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
             $this
         );
 
-        if(!$this->params->isValid() && $this->requiresParameters())
+        if (!$this->params->isValid() && $this->requiresParameters())
         {
             $error = $this->params->getValidationResult();
 

src/Mailcode/Traits/Commands/Type/Sibling.php

Spacing +3 added lines, -3 removed lines

@@ -40,15 +40,15 @@
     {
         $opening = $this->getOpeningCommand();
 
-        if($opening === null) {
+        if ($opening === null) {
             return array();
         }
 
         $siblings = $opening->getSiblingCommands();
         $result = array();
-        foreach($siblings as $sibling)
+        foreach ($siblings as $sibling)
         {
-            if($sibling !== $this) {
+            if ($sibling !== $this) {
                 $result[] = $sibling;
             }
         }

src/Mailcode/Collection/NestingValidator.php

Spacing +11 added lines, -11 removed lines

@@ -60,11 +60,11 @@ 
         
         $commands = $this->collection->getCommands();
         
-        foreach($commands as $command)
+        foreach ($commands as $command)
         {
             $method = 'validate_'.$command->getCommandType();
             
-            if(!method_exists($this, $method))
+            if (!method_exists($this, $method))
             {
                 throw new Mailcode_Exception(
                     'Unknown command type validation method.',
@@ -79,13 +79,13 @@ 
             
             $this->$method($command);
             
-            if(!$this->validationResult->isValid())
+            if (!$this->validationResult->isValid())
             {
                 break;
             }
         }
         
-        if($this->validationResult->isValid())
+        if ($this->validationResult->isValid())
         {
             $this->validate_Unclosed();
         }
@@ -97,7 +97,7 @@ 
     {
         $leftover = $this->getOpenCommand();
         
-        if($leftover === null)
+        if ($leftover === null)
         {
             return;
         }
@@ -113,7 +113,7 @@ 
     
     protected function getOpenCommand() : ?Mailcode_Commands_Command_Type_Opening
     {
-        if(empty($this->stack))
+        if (empty($this->stack))
         {
             return null;
         }
@@ -124,7 +124,7 @@ 
         
         $cmd = $this->stack[$idx];
         
-        if($cmd instanceof Mailcode_Commands_Command_Type_Opening)
+        if ($cmd instanceof Mailcode_Commands_Command_Type_Opening)
         {
             return $cmd;
         }
@@ -148,7 +148,7 @@ 
     {
         $parent = $this->getOpenCommand();
         
-        if($parent === null)
+        if ($parent === null)
         {
             $this->validationResult->makeError(
                 t(
@@ -162,7 +162,7 @@ 
             return;
         }
         
-        if($parent->getName() !== $command->getParentName())
+        if ($parent->getName() !== $command->getParentName())
         {
             $this->validationResult->makeError(
                 t(
@@ -184,7 +184,7 @@ 
     
     protected function validate_Closing(Mailcode_Commands_Command_Type_Closing $command) : void
     {
-        if(empty($this->stack))
+        if (empty($this->stack))
         {
             $this->validationResult->makeError(
                 t('All open commands have already been ended.'),
@@ -196,7 +196,7 @@ 
         
         $openingCommand = array_pop($this->stack);
 
-        if($openingCommand instanceof Mailcode_Commands_Command_Type_Opening)
+        if ($openingCommand instanceof Mailcode_Commands_Command_Type_Opening)
         {
             $this->log(sprintf('Closing command %s', $openingCommand->getName()));
 

Indentation +9 added lines, -9 removed lines

@@ -30,19 +30,19 @@
     public const VALIDATION_UNCLOSED_COMMAND = 49104;
     public const VALIDATION_CLOSING_NON_OPENING_COMMAND = 49105;
     
-   /**
-    * @var Mailcode_Collection
-    */
+    /**
+     * @var Mailcode_Collection
+     */
     protected $collection;
     
-   /**
-    * @var OperationResult
-    */
+    /**
+     * @var OperationResult
+     */
     protected $validationResult;
     
-   /**
-    * @var Mailcode_Commands_Command_Type[]
-    */
+    /**
+     * @var Mailcode_Commands_Command_Type[]
+     */
     protected $stack = array();
     
     public function __construct(Mailcode_Collection $collection)

src/Mailcode/Parser/Safeguard/Formatter/Type/PreProcessing.php

Spacing +3 added lines, -3 removed lines

@@ -25,16 +25,16 @@
     {
         $command = $location->getPlaceholder()->getCommand();
 
-        if($command instanceof Mailcode_Interfaces_Commands_PreProcessing)
+        if ($command instanceof Mailcode_Interfaces_Commands_PreProcessing)
         {
             return $command->preProcessOpening();
         }
 
-        if($command instanceof Mailcode_Commands_Command_Type_Closing)
+        if ($command instanceof Mailcode_Commands_Command_Type_Closing)
         {
             $opening = $command->getOpeningCommand();
 
-            if($opening instanceof Mailcode_Interfaces_Commands_PreProcessing) {
+            if ($opening instanceof Mailcode_Interfaces_Commands_PreProcessing) {
                 return $opening->preProcessClosing();
             }
         }

tools/prepend.php

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-if(!file_exists('../vendor/autoload.php')) {
+if (!file_exists('../vendor/autoload.php')) {
     die('Autoloader not present.');
 }
 

tools/translator.php

Braces +3 added lines, -6 removed lines

@@ -40,8 +40,7 @@ 
     try
     {
         $translated = translateMailcode($commandsText, $activeSyntax);
-    }
-    catch (Mailcode_Exception $e)
+    } catch (Mailcode_Exception $e)
     {
         $translated = '';
         $error = $e->getMessage();
@@ -146,8 +145,7 @@ 
                         <?php pt('No commands specified.') ?>
                     </div>
                 <?php
-            }
-            else if($error)
+            } else if($error)
             {
                 ?>
                     <div class="alert alert-danger">
@@ -155,8 +153,7 @@ 
                         <?php echo $error ?>
                     </div>
                 <?php
-            }
-            else
+            } else
             {
                 ?>
                     <textarea class="form-control" rows="10"><?php echo htmlspecialchars($translated) ?></textarea>

Spacing +6 added lines, -6 removed lines

@@ -31,7 +31,7 @@ 
 $error = null;
 $activeSyntax = $syntaxes[0]->getTypeID();
 
-if($request->getBool('translate'))
+if ($request->getBool('translate'))
 {
     $commandsText = $request->registerParam('mailcode')->getString();
     $activeSyntax = $request->registerParam('syntax')
@@ -48,12 +48,12 @@ 
         $error = $e->getMessage();
 
         $collection = $e->getCollection();
-        if($collection)
+        if ($collection)
         {
             $first = $collection->getFirstError();
             $error = $first->getMessage();
             $matched = $first->getMatchedText();
-            if(!empty($matched)) {
+            if (!empty($matched)) {
                 $error .= '<br>'.t('In command:').' <code>'.$matched.'</code>';
             }
         }
@@ -117,7 +117,7 @@ 
                             $selected = '';
                             $typeID = $syntax->getTypeID();
 
-                            if($typeID === $activeSyntax) {
+                            if ($typeID === $activeSyntax) {
                                 $selected = ' selected';
                             }
 
@@ -137,7 +137,7 @@ 
         <p></p><br>
         <h2><?php pt('Translated commands') ?></h2>
         <?php
-            if(empty($commandsText))
+            if (empty($commandsText))
             {
                 ?>
                     <div class="alert alert-info">
@@ -145,7 +145,7 @@ 
                     </div>
                 <?php
             }
-            else if($error)
+            else if ($error)
             {
                 ?>
                     <div class="alert alert-danger">

src/Mailcode/Factory/CommandSets/Set/ElseIf.php

Spacing +37 added lines, -37 removed lines

@@ -20,11 +20,11 @@ 
  */
 class Mailcode_Factory_CommandSets_Set_ElseIf extends Mailcode_Factory_CommandSets_IfBase
 {
-    public function elseIf(string $condition, string $type='') : Mailcode_Commands_Command_ElseIf
+    public function elseIf (string $condition, string $type = '') : Mailcode_Commands_Command_ElseIf
     {
         $command = $this->instantiator->buildIf('ElseIf', $condition, $type);
         
-        if($command instanceof Mailcode_Commands_Command_ElseIf)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf)
         {
             return $command;
         }
@@ -32,11 +32,11 @@ 
         throw $this->instantiator->exceptionUnexpectedType('ElseIf', $command);
     }
     
-    public function var(string $variable, string $operand, string $value, bool $quoteValue=false, bool $insensitive=false) : Mailcode_Commands_Command_ElseIf_Variable
+    public function var(string $variable, string $operand, string $value, bool $quoteValue = false, bool $insensitive = false) : Mailcode_Commands_Command_ElseIf_Variable
     {
         $command = $this->instantiator->buildIfVar('ElseIf', $variable, $operand, $value, $quoteValue, $insensitive);
         
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
         {
             return $command;
         }
@@ -44,11 +44,11 @@ 
         throw $this->instantiator->exceptionUnexpectedType('ElseIfVariable', $command);
     }
     
-    public function varString(string $variable, string $operand, string $value, bool $insensitive=false) : Mailcode_Commands_Command_ElseIf_Variable
+    public function varString(string $variable, string $operand, string $value, bool $insensitive = false) : Mailcode_Commands_Command_ElseIf_Variable
     {
         $command = $this->instantiator->buildIfVar('ElseIf', $variable, $operand, $value, true, $insensitive);
         
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
         {
             return $command;
         }
@@ -56,11 +56,11 @@ 
         throw $this->instantiator->exceptionUnexpectedType('ElseIfVarString', $command);
     }
     
-    public function varEquals(string $variable, string $value, bool $quoteValue=false, bool $insensitive=false) : Mailcode_Commands_Command_ElseIf_Variable
+    public function varEquals(string $variable, string $value, bool $quoteValue = false, bool $insensitive = false) : Mailcode_Commands_Command_ElseIf_Variable
     {
         $command = $this->instantiator->buildIfVar('ElseIf', $variable, '==', $value, $quoteValue, $insensitive);
         
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
         {
             return $command;
         }
@@ -68,11 +68,11 @@ 
         throw $this->instantiator->exceptionUnexpectedType('ElseIfVarEquals', $command);
     }
     
-    public function varEqualsString(string $variable, string $value, bool $insensitive=false) : Mailcode_Commands_Command_ElseIf_Variable
+    public function varEqualsString(string $variable, string $value, bool $insensitive = false) : Mailcode_Commands_Command_ElseIf_Variable
     {
         $command = $this->instantiator->buildIfVar('ElseIf', $variable, '==', $value, true, $insensitive);
         
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
         {
             return $command;
         }
@@ -80,11 +80,11 @@ 
         throw $this->instantiator->exceptionUnexpectedType('ElseIfVarEqualsString', $command);
     }
     
-    public function varNotEquals(string $variable, string $value, bool $quoteValue=false, bool $insensitive=false) : Mailcode_Commands_Command_ElseIf_Variable
+    public function varNotEquals(string $variable, string $value, bool $quoteValue = false, bool $insensitive = false) : Mailcode_Commands_Command_ElseIf_Variable
     {
         $command = $this->instantiator->buildIfVar('ElseIf', $variable, '!=', $value, $quoteValue, $insensitive);
         
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
         {
             return $command;
         }
@@ -92,11 +92,11 @@ 
         throw $this->instantiator->exceptionUnexpectedType('ElseIfVarNotEquals', $command);
     }
     
-    public function varNotEqualsString(string $variable, string $value, bool $insensitive=false) : Mailcode_Commands_Command_ElseIf_Variable
+    public function varNotEqualsString(string $variable, string $value, bool $insensitive = false) : Mailcode_Commands_Command_ElseIf_Variable
     {
         $command = $this->instantiator->buildIfVar('ElseIf', $variable, '!=', $value, true, $insensitive);
         
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
         {
             return $command;
         }
@@ -111,11 +111,11 @@ 
      * @return Mailcode_Commands_Command_ElseIf_Contains
      * @throws Mailcode_Factory_Exception
      */
-    public function contains(string $variable, array $searchTerms, bool $caseInsensitive=false) : Mailcode_Commands_Command_ElseIf_Contains
+    public function contains(string $variable, array $searchTerms, bool $caseInsensitive = false) : Mailcode_Commands_Command_ElseIf_Contains
     {
         $command = $this->instantiator->buildIfContains('ElseIf', $variable, $searchTerms, $caseInsensitive);
         
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Contains)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Contains)
         {
             return $command;
         }
@@ -130,11 +130,11 @@ 
      * @return Mailcode_Commands_Command_ElseIf_NotContains
      * @throws Mailcode_Factory_Exception
      */
-    public function notContains(string $variable, array $searchTerms, bool $caseInsensitive=false) : Mailcode_Commands_Command_ElseIf_NotContains
+    public function notContains(string $variable, array $searchTerms, bool $caseInsensitive = false) : Mailcode_Commands_Command_ElseIf_NotContains
     {
         $command = $this->instantiator->buildIfNotContains('ElseIf', $variable, $searchTerms, $caseInsensitive);
 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_NotContains)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_NotContains)
         {
             return $command;
         }
@@ -150,11 +150,11 @@ 
      * @return Mailcode_Commands_Command_ElseIf_ListContains
      * @throws Mailcode_Factory_Exception
      */
-    public function listContains(string $variable, array $searchTerms, bool $caseInsensitive=false, bool $regexEnabled=false) : Mailcode_Commands_Command_ElseIf_ListContains
+    public function listContains(string $variable, array $searchTerms, bool $caseInsensitive = false, bool $regexEnabled = false) : Mailcode_Commands_Command_ElseIf_ListContains
     {
         $command = $this->instantiator->buildIfListContains('ElseIf', $variable, $searchTerms, $caseInsensitive, $regexEnabled);
 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_ListContains)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_ListContains)
         {
             return $command;
         }
@@ -169,11 +169,11 @@ 
      * @return Mailcode_Commands_Command_ElseIf_ListEquals
      * @throws Mailcode_Factory_Exception
      */
-    public function listEquals(string $variable, array $searchTerms, bool $caseInsensitive=false) : Mailcode_Commands_Command_ElseIf_ListEquals
+    public function listEquals(string $variable, array $searchTerms, bool $caseInsensitive = false) : Mailcode_Commands_Command_ElseIf_ListEquals
     {
         $command = $this->instantiator->buildIfListContains('ElseIf', $variable, $searchTerms, $caseInsensitive, false, 'list-equals');
 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_ListEquals)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_ListEquals)
         {
             return $command;
         }
@@ -189,11 +189,11 @@ 
      * @return Mailcode_Commands_Command_ElseIf_ListNotContains
      * @throws Mailcode_Factory_Exception
      */
-    public function listNotContains(string $variable, array $searchTerms, bool $caseInsensitive=false, bool $regexEnabled=false) : Mailcode_Commands_Command_ElseIf_ListNotContains
+    public function listNotContains(string $variable, array $searchTerms, bool $caseInsensitive = false, bool $regexEnabled = false) : Mailcode_Commands_Command_ElseIf_ListNotContains
     {
         $command = $this->instantiator->buildIfListNotContains('ElseIf', $variable, $searchTerms, $caseInsensitive, $regexEnabled);
 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_ListNotContains)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_ListNotContains)
         {
             return $command;
         }
@@ -209,11 +209,11 @@ 
      * @return Mailcode_Commands_Command_ElseIf_ListBeginsWith
      * @throws Mailcode_Factory_Exception
      */
-    public function listBeginsWith(string $variable, array $searchTerms, bool $caseInsensitive=false, bool $regexEnabled=false) : Mailcode_Commands_Command_ElseIf_ListBeginsWith
+    public function listBeginsWith(string $variable, array $searchTerms, bool $caseInsensitive = false, bool $regexEnabled = false) : Mailcode_Commands_Command_ElseIf_ListBeginsWith
     {
         $command = $this->instantiator->buildIfListContains('ElseIf', $variable, $searchTerms, $caseInsensitive, $regexEnabled, 'list-begins-with');
 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_ListBeginsWith)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_ListBeginsWith)
         {
             return $command;
         }
@@ -229,11 +229,11 @@ 
      * @return Mailcode_Commands_Command_ElseIf_ListEndsWith
      * @throws Mailcode_Factory_Exception
      */
-    public function listEndsWith(string $variable, array $searchTerms, bool $caseInsensitive=false, bool $regexEnabled=false) : Mailcode_Commands_Command_ElseIf_ListEndsWith
+    public function listEndsWith(string $variable, array $searchTerms, bool $caseInsensitive = false, bool $regexEnabled = false) : Mailcode_Commands_Command_ElseIf_ListEndsWith
     {
         $command = $this->instantiator->buildIfListContains('ElseIf', $variable, $searchTerms, $caseInsensitive, $regexEnabled, 'list-ends-with');
 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_ListEndsWith)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_ListEndsWith)
         {
             return $command;
         }
@@ -241,11 +241,11 @@ 
         throw $this->instantiator->exceptionUnexpectedType('ElseIfListEndsWith', $command);
     }
 
-    public function beginsWith(string $variable, string $search, bool $caseInsensitive=false) : Mailcode_Commands_Command_ElseIf_BeginsWith
+    public function beginsWith(string $variable, string $search, bool $caseInsensitive = false) : Mailcode_Commands_Command_ElseIf_BeginsWith
     {
         $command = $this->instantiator->buildIfBeginsWith('ElseIf', $variable, $search, $caseInsensitive);
         
-        if($command instanceof Mailcode_Commands_Command_ElseIf_BeginsWith)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_BeginsWith)
         {
             return $command;
         }
@@ -253,11 +253,11 @@ 
         throw $this->instantiator->exceptionUnexpectedType('ElseIfBeginsWith', $command);
     }
     
-    public function endsWith(string $variable, string $search, bool $caseInsensitive=false) : Mailcode_Commands_Command_ElseIf_EndsWith
+    public function endsWith(string $variable, string $search, bool $caseInsensitive = false) : Mailcode_Commands_Command_ElseIf_EndsWith
     {
         $command = $this->instantiator->buildIfEndsWith('ElseIf', $variable, $search, $caseInsensitive);
         
-        if($command instanceof Mailcode_Commands_Command_ElseIf_EndsWith)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_EndsWith)
         {
             return $command;
         }
@@ -269,7 +269,7 @@ 
     {
         $command = $this->instantiator->buildIfEmpty('ElseIf', $variable);
         
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Empty)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Empty)
         {
             return $command;
         }
@@ -281,7 +281,7 @@ 
     {
         $command = $this->instantiator->buildIfNotEmpty('ElseIf', $variable);
         
-        if($command instanceof Mailcode_Commands_Command_ElseIf_NotEmpty)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_NotEmpty)
         {
             return $command;
         }
@@ -293,7 +293,7 @@ 
     {
         $command = $this->instantiator->buildIfBiggerThan('ElseIf', $variable, $value);
 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_BiggerThan)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_BiggerThan)
         {
             return $command;
         }
@@ -305,7 +305,7 @@ 
     {
         $command = $this->instantiator->buildIfSmallerThan('ElseIf', $variable, $value);
 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_SmallerThan)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_SmallerThan)
         {
             return $command;
         }
@@ -317,7 +317,7 @@ 
     {
         $command = $this->instantiator->buildIfEquals('ElseIf', $variable, $value);
 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_EqualsNumber)
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_EqualsNumber)
         {
             return $command;
         }

src/Mailcode/Commands/LogicKeywords.php

Indentation +45 added lines, -45 removed lines

@@ -31,27 +31,27 @@ 
     public const VALIDATION_CANNOT_MIX_LOGIC_KEYWORDS = 60701;
     public const VALIDATION_INVALID_SUB_COMMAND = 60702;
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     private $paramsString;
     
-   /**
-    * @var string[]
-    */
+    /**
+     * @var string[]
+     */
     private $names = array(
         'and', 
         'or'
     );
     
-   /**
-    * @var Mailcode_Commands_LogicKeywords_Keyword[]
-    */
+    /**
+     * @var Mailcode_Commands_LogicKeywords_Keyword[]
+     */
     private $keywords = array();
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     private $mainParams = '';
     
     public function __construct(Mailcode_Commands_Command $command, string $paramsString)
@@ -137,9 +137,9 @@ 
         }
     }
     
-   /**
-    * @return string[]
-    */
+    /**
+     * @return string[]
+     */
     private function detectParameters() : array
     {
         $params = $this->paramsString;
@@ -157,13 +157,13 @@ 
         return $stack;
     }
 
-   /**
-    * @param string $params
-    * @param Mailcode_Commands_LogicKeywords_Keyword $keyword
-    * @param string[] $stack
-    * @throws Mailcode_Exception
-    * @return string
-    */
+    /**
+     * @param string $params
+     * @param Mailcode_Commands_LogicKeywords_Keyword $keyword
+     * @param string[] $stack
+     * @throws Mailcode_Exception
+     * @return string
+     */
     private function detectParamsKeyword(string $params, Mailcode_Commands_LogicKeywords_Keyword $keyword, array &$stack) : string
     {
         $search = $keyword->getMatchedString();
@@ -188,22 +188,22 @@ 
         return $params;
     }
     
-   /**
-    * Extracts the parameters string to use for the 
-    * original command itself, omitting all the logic
-    * keywords for the sub-commands.
-    * 
-    * @return string
-    */
+    /**
+     * Extracts the parameters string to use for the 
+     * original command itself, omitting all the logic
+     * keywords for the sub-commands.
+     * 
+     * @return string
+     */
     public function getMainParamsString() : string
     {
         return $this->mainParams;
     }
     
-   /**
-    * Retrieves the detected keyword names.
-    * @return string[]
-    */
+    /**
+     * Retrieves the detected keyword names.
+     * @return string[]
+     */
     public function getDetectedNames() : array
     {
         $names = array();
@@ -221,23 +221,23 @@ 
         return $names;
     }
     
-   /**
-    * Retrieves all keywords that were detected in the
-    * command's parameters string, if any.
-    * 
-    * @return Mailcode_Commands_LogicKeywords_Keyword[]
-    */
+    /**
+     * Retrieves all keywords that were detected in the
+     * command's parameters string, if any.
+     * 
+     * @return Mailcode_Commands_LogicKeywords_Keyword[]
+     */
     public function getKeywords() : array
     {
         return $this->keywords;
     }
     
-   /**
-    * Detects any keyword statements in the parameters by keyword name.
-    * 
-    * @param string $name
-    * @return Mailcode_Commands_LogicKeywords_Keyword[]
-    */
+    /**
+     * Detects any keyword statements in the parameters by keyword name.
+     * 
+     * @param string $name
+     * @return Mailcode_Commands_LogicKeywords_Keyword[]
+     */
     private function detectKeywords(string $name) : array
     {
         $regex = sprintf('/%1$s\s+([a-z\-0-9]+):|%1$s:/x', $name);

Spacing +20 added lines, -20 removed lines

@@ -71,9 +71,9 @@ 
     
     private function parse() : void
     {
-        foreach($this->names as $name)
+        foreach ($this->names as $name)
         {
-            if(!stristr($this->paramsString, $name))
+            if (!stristr($this->paramsString, $name))
             {
                 continue;
             }
@@ -90,7 +90,7 @@ 
         $names = $this->getDetectedNames();
         $amount = count($names);
         
-        if($amount > 1)
+        if ($amount > 1)
         {
             $this->makeError(
                 t(
@@ -110,7 +110,7 @@ 
     
     private function splitParams() : void
     {
-        if(empty($this->keywords))
+        if (empty($this->keywords))
         {
             $this->mainParams = $this->paramsString;
             
@@ -119,13 +119,13 @@ 
         
         $params = $this->detectParameters();
         
-        foreach($this->keywords as $keyword)
+        foreach ($this->keywords as $keyword)
         {
             $kParams = (string)array_shift($params);
             
             $keyword->setParamsString($kParams);
             
-            if(!$keyword->isValid())
+            if (!$keyword->isValid())
             {
                 $this->makeError(
                     t('Error #%1$s:', $keyword->getCode()).' '.$keyword->getErrorMessage(),
@@ -145,7 +145,7 @@ 
         $params = $this->paramsString;
         $stack = array();
         
-        foreach($this->keywords as $keyword)
+        foreach ($this->keywords as $keyword)
         {
             $params = $this->detectParamsKeyword($params, $keyword, $stack);
         }
@@ -169,7 +169,7 @@ 
         $search = $keyword->getMatchedString();
         $pos = strpos($params, $search, 0);
         
-        if($pos === false)
+        if ($pos === false)
         {
             throw new Mailcode_Exception(
                 'Keyword matched string not found',
@@ -181,7 +181,7 @@ 
         $length = strlen($search);
         
         $store = substr($params, 0, $pos);
-        $params = trim(substr($params, $pos+$length));
+        $params = trim(substr($params, $pos + $length));
         
         $stack[] = $store;
         
@@ -208,11 +208,11 @@ 
     {
         $names = array();
         
-        foreach($this->keywords as $keyword)
+        foreach ($this->keywords as $keyword)
         {
             $name = $keyword->getName();
             
-            if(!in_array($name, $names))
+            if (!in_array($name, $names))
             {
                 $names[] = $name;
             }
@@ -245,7 +245,7 @@ 
         $matches = array();
         preg_match_all($regex, $this->paramsString, $matches, PREG_PATTERN_ORDER);
         
-        if(!isset($matches[0][0]) || empty($matches[0][0]))
+        if (!isset($matches[0][0]) || empty($matches[0][0]))
         {
             return array();
         }
@@ -253,7 +253,7 @@ 
         $amount = count($matches[0]);
         $result = array();
         
-        for($i=0; $i < $amount; $i++)
+        for ($i = 0; $i < $amount; $i++)
         {
             $result[] = $this->createKeyword(
                 $name, 
@@ -270,22 +270,22 @@ 
         return !empty($this->keywords);
     }
     
-    public function appendAND(string $paramsString, string $type='') : Mailcode_Commands_LogicKeywords_Keyword
+    public function appendAND(string $paramsString, string $type = '') : Mailcode_Commands_LogicKeywords_Keyword
     {
         return $this->appendKeyword('and', $paramsString, $type);
     }
     
-    public function appendOR(string $paramsString, string $type='') : Mailcode_Commands_LogicKeywords_Keyword
+    public function appendOR(string $paramsString, string $type = '') : Mailcode_Commands_LogicKeywords_Keyword
     {
         return $this->appendKeyword('or', $paramsString, $type);
     }
     
-    public function appendKeyword(string $name, string $paramsString, string $type='') : Mailcode_Commands_LogicKeywords_Keyword
+    public function appendKeyword(string $name, string $paramsString, string $type = '') : Mailcode_Commands_LogicKeywords_Keyword
     {
         $keyword = $this->createKeyword($name, $type);
         $keyword->setParamsString($paramsString);
         
-        if(!$keyword->isValid())
+        if (!$keyword->isValid())
         {
             throw new Mailcode_Exception(
                 'Cannot append invalid logic keyword',
@@ -306,13 +306,13 @@ 
         return $keyword;
     }
     
-    private function createKeyword(string $name, string $type='', string $matchedString='') : Mailcode_Commands_LogicKeywords_Keyword
+    private function createKeyword(string $name, string $type = '', string $matchedString = '') : Mailcode_Commands_LogicKeywords_Keyword
     {
-        if(empty($matchedString))
+        if (empty($matchedString))
         {
             $matchedString = $name;
             
-            if(!empty($type))
+            if (!empty($type))
             {
                 $matchedString .= ' '.$type;
             }

src/Mailcode/Variables/Collection/Invalid.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
 
     public function add(Mailcode_Variables_Variable $variable) : Mailcode_Variables_Collection
     {
-        if($variable->isValid())
+        if ($variable->isValid())
         {
             return $this;
         }
@@ -38,7 +38,7 @@ 
     {
         $first = $this->getFirst();
 
-        if($first)
+        if ($first)
         {
             return $first->getValidationResult();
         }