Mistralys / mailcode

src/Mailcode/Collection/NestingValidator.php

Spacing +10 added lines, -10 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(
@@ -181,7 +181,7 @@ 
     
     protected function validate_Closing(Mailcode_Commands_Command_Type_Closing $command)
     {
-        if(empty($this->stack))
+        if (empty($this->stack))
         {
             $this->validationResult->makeError(
                 t('All open commands have already been ended.'),

src/Mailcode/Commands/Command.php

Spacing +15 added lines, -15 removed lines

@@ -59,7 +59,7 @@ 
     */
     protected $mailcode;
     
-    public function __construct(string $type='', string $paramsString='', string $matchedText='')
+    public function __construct(string $type = '', string $paramsString = '', string $matchedText = '')
     {
         $this->type = $type;
         $this->paramsString = $paramsString;
@@ -98,7 +98,7 @@ 
     {
         $this->requireNonDummy();
         
-        if($this->hash === '') {
+        if ($this->hash === '') {
             $this->hash = md5($this->matchedText);
         }
         
@@ -107,7 +107,7 @@ 
     
     protected function requireNonDummy() : void
     {
-        if(!$this->isDummy())
+        if (!$this->isDummy())
         {
             return;
         }
@@ -128,7 +128,7 @@ 
     {
         $this->requireNonDummy();
         
-        if(isset($this->validationResult)) 
+        if (isset($this->validationResult)) 
         {
             return $this->validationResult;
         }
@@ -142,7 +142,7 @@ 
     
     public function getValidationResult() :  \AppUtils\OperationResult
     {
-        if(isset($this->validationResult)) 
+        if (isset($this->validationResult)) 
         {
             return $this->validationResult;
         }
@@ -165,11 +165,11 @@ 
     {
         $validations = array_merge($this->validations, $this->getValidations());
         
-        foreach($validations as $validation)
+        foreach ($validations as $validation)
         {
             $method = 'validateSyntax_'.$validation;
             
-            if(!method_exists($this, $method))
+            if (!method_exists($this, $method))
             {
                 throw new Mailcode_Exception(
                     'Missing validation method',
@@ -185,7 +185,7 @@ 
             $this->$method();
             
             // break off at the first validation issue
-            if(!$this->validationResult->isValid())
+            if (!$this->validationResult->isValid())
             {
                 return;
             }
@@ -196,7 +196,7 @@ 
     
     protected function validateSyntax_params()
     {
-        if($this->requiresParameters() && empty($this->paramsString))
+        if ($this->requiresParameters() && empty($this->paramsString))
         {
             $this->validationResult->makeError(
                 t('Parameters have to be specified.'),
@@ -208,14 +208,14 @@ 
     
     protected function validateSyntax_type_supported()
     {
-        if(!$this->supportsType() || empty($this->type))
+        if (!$this->supportsType() || empty($this->type))
         {
             return;
         }
         
         $types = $this->getSupportedTypes();
 
-        if(!in_array($this->type, $types))
+        if (!in_array($this->type, $types))
         {
             $this->validationResult->makeError(
                 t('The command addon %1$s is not supported.', $this->type).' '.
@@ -229,7 +229,7 @@ 
     
     protected function validateSyntax_type_unsupported()
     {
-        if($this->supportsType() || empty($this->type))
+        if ($this->supportsType() || empty($this->type))
         {
             return;
         }
@@ -244,7 +244,7 @@ 
     {
         $vars = $this->getVariables();
         
-        if(!$vars->hasInvalid()) 
+        if (!$vars->hasInvalid()) 
         {
             return;
         }
@@ -264,7 +264,7 @@ 
     
     public function getType() : string
     {
-        if($this->supportsType())
+        if ($this->supportsType())
         {
             return $this->type;
         }
@@ -290,7 +290,7 @@ 
     
     public function getParamsString() : string
     {
-        if($this->requiresParameters())
+        if ($this->requiresParameters())
         {
             return $this->paramsString;
         }

src/Mailcode/Commands/Command/SetVariable.php

Spacing +5 added lines, -5 removed lines

@@ -65,12 +65,12 @@ 
     {
         $amount = substr_count($this->paramsString, '=');
         
-        if($amount === 1)
+        if ($amount === 1)
         {
             return;
         }
         
-        if($amount < 1)
+        if ($amount < 1)
         {
             $this->validationResult->makeError(
                 t('The quality operator (=) is missing.'),
@@ -90,7 +90,7 @@ 
     {
         $this->parts = \AppUtils\ConvertHelper::explodeTrim('=', $this->paramsString);
         
-        if(count($this->parts) === 2) 
+        if (count($this->parts) === 2) 
         {
             return;
         }
@@ -106,7 +106,7 @@ 
         // any variables we may find have already been validated.
         $vars = $this->mailcode->findVariables($this->parts[0])->getGroupedByName();
         
-        if(count($vars) === 1)
+        if (count($vars) === 1)
         {
             $this->leftVar = array_shift($vars);
             return;
@@ -122,7 +122,7 @@ 
     {
         $this->statement = $this->mailcode->getParser()->createStatement($this->parts[1]);
         
-        if($this->statement->isValid())
+        if ($this->statement->isValid())
         {
             return;
         }

src/Mailcode/Commands/Command/If.php

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
     {
         $amount = $this->getVariables()->countVariables();
         
-        if($amount >= 1)
+        if ($amount >= 1)
         {
             return;
         }
@@ -61,7 +61,7 @@ 
     {
         $validations = array();
         
-        if($this->getType() === 'variable')
+        if ($this->getType() === 'variable')
         {
             $validations[] = 'require_variable';
         }

src/Mailcode/Variables/Variable.php

Spacing +7 added lines, -7 removed lines

@@ -86,7 +86,7 @@ 
     
     public function getHash() : string
     {
-        if(empty($this->hash))
+        if (empty($this->hash))
         {
             $this->hash = md5($this->matchedText);
         }
@@ -101,7 +101,7 @@ 
     
     public function getValidationResult() : OperationResult
     {
-        if(isset($this->validationResult))
+        if (isset($this->validationResult))
         {
             return $this->validationResult;
         }
@@ -122,11 +122,11 @@ 
     
     protected function validate() : void
     {
-        foreach($this->validations as $validation)
+        foreach ($this->validations as $validation)
         {
             $method = 'validate_'.$validation;
             
-            if(!method_exists($this, $method))
+            if (!method_exists($this, $method))
             {
                 throw new Mailcode_Exception(
                     'Unknown validation method',
@@ -141,7 +141,7 @@ 
             
             $this->$method();
             
-            if(!$this->validationResult->isValid())
+            if (!$this->validationResult->isValid())
             {
                 return;
             }
@@ -170,7 +170,7 @@ 
     
     protected function validateNumber(string $value, int $errorCode)
     {
-        if(!is_numeric(substr($value, 0, 1)))
+        if (!is_numeric(substr($value, 0, 1)))
         {
             return;
         }
@@ -190,7 +190,7 @@ 
         $length = strlen($value);
         
         // trimming underscores does not change the length: no underscores at start or end of string.
-        if(strlen(trim($value, '_')) == $length)
+        if (strlen(trim($value, '_')) == $length)
         {
             return;
         }

src/Mailcode/Parser/Statement.php

Spacing +10 added lines, -10 removed lines

@@ -63,7 +63,7 @@ 
     
     public function getValidationResult() : OperationResult
     {
-        if(isset($this->result))
+        if (isset($this->result))
         {
             return $this->result;
         }
@@ -79,7 +79,7 @@ 
     {
         $statement = trim($this->statement);
         
-        if(empty($statement))
+        if (empty($statement))
         {
             $this->result->makeError(
                 t('Empty statement'),
@@ -94,7 +94,7 @@ 
         $leftover = $this->removeTokens($tokenized);
         $leftover = str_replace(' ', '', $leftover);
          
-        if(!empty($leftover))
+        if (!empty($leftover))
         {
             $this->result->makeError(
                t('Unquoted string literals found:').' "'.$leftover.'"',
@@ -118,7 +118,7 @@ 
         $matches = array();
         preg_match_all('/'.sprintf($this->token, '[A-Z0-9_]+').'/sx', $statement, $matches, PREG_PATTERN_ORDER);
         
-        foreach($matches[0] as $match)
+        foreach ($matches[0] as $match)
         {
             $statement = str_replace($match, '', $statement);
         }
@@ -140,11 +140,11 @@ 
     {
         $tokenized = trim($statement);
         
-        foreach($this->tokens as $token)
+        foreach ($this->tokens as $token)
         {
             $method = 'tokenize_'.$token;
             
-            if(!method_exists($this, $method))
+            if (!method_exists($this, $method))
             {
                 throw new Mailcode_Exception(
                     'Unknown statement token.',    
@@ -172,7 +172,7 @@ 
     {
         $vars = Mailcode::create()->findVariables($tokenized)->getGroupedByHash();
         
-        foreach($vars as $var)
+        foreach ($vars as $var)
         {
             $tokenized = str_replace($var->getMatchedText(), $this->getToken('VARIABLE'), $tokenized);
         }
@@ -182,7 +182,7 @@ 
     
     protected function tokenize_operands(string $tokenized) : string
     {
-        foreach($this->operands as $operand)
+        foreach ($this->operands as $operand)
         {
             $tokenized = str_replace($operand, $this->getToken('OPERAND'), $tokenized);
         }
@@ -195,7 +195,7 @@ 
         $matches = array();
         preg_match_all('/"(.*)"/sx', $tokenized, $matches, PREG_PATTERN_ORDER);
         
-        foreach($matches[0] as $match)
+        foreach ($matches[0] as $match)
         {
             $tokenized = str_replace($match, $this->getToken('STRING_LITERAL'), $tokenized);
         }
@@ -208,7 +208,7 @@ 
         $matches = array();
         preg_match_all('/[0-9]+\s*[.,]\s*[0-9]+|[0-9]+/sx', $tokenized, $matches, PREG_PATTERN_ORDER);
         
-        foreach($matches[0] as $match)
+        foreach ($matches[0] as $match)
         {
             $tokenized = str_replace($match, $this->getToken('NUMBER'), $tokenized);
         }

src/Mailcode/Collection.php

Spacing +8 added lines, -8 removed lines

@@ -97,7 +97,7 @@ 
         
         $errors = $this->errors;
         
-        if(!$result->isValid())
+        if (!$result->isValid())
         {
             $errors[] = new Mailcode_Collection_Error_Message(
                 '',
@@ -139,11 +139,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;
             }
@@ -154,7 +154,7 @@ 
     
     public function addCommands(array $commands) : Mailcode_Collection
     {
-        foreach($commands as $command)
+        foreach ($commands as $command)
         {
             $this->addCommand($command);
         }
@@ -173,7 +173,7 @@ 
     {
         $collection = new Mailcode_Variables_Collection_Regular();
         
-        foreach($this->commands as $command)
+        foreach ($this->commands as $command)
         {
             $collection->mergeWith($command->getVariables());
         }
@@ -183,7 +183,7 @@ 
     
     public function getValidationResult() : OperationResult
     {
-        if(isset($this->validationResult))
+        if (isset($this->validationResult))
         {
             return $this->validationResult;
         }
@@ -199,9 +199,9 @@ 
     {
         $errors = $this->getErrors();
         
-        foreach($errors as $error)
+        foreach ($errors as $error)
         {
-            if($error->getCode() === $code)
+            if ($error->getCode() === $code)
             {
                 return true;
             }