Mistralys / mailcode

src/Mailcode.php

Spacing +3 added lines, -3 removed lines

@@ -68,7 +68,7 @@ 
     */
     public function getParser() : Mailcode_Parser
     {
-        if(!isset($this->parser)) 
+        if (!isset($this->parser)) 
         {
             $this->parser = new Mailcode_Parser($this);
         }
@@ -84,7 +84,7 @@ 
     */
     public function getCommands() : Mailcode_Commands
     {
-        if(!isset($this->commands)) 
+        if (!isset($this->commands)) 
         {
             $this->commands = new Mailcode_Commands();
         }
@@ -110,7 +110,7 @@ 
     
     public function createVariables() : Mailcode_Variables
     {
-        if(!isset($this->variables))
+        if (!isset($this->variables))
         {
             $this->variables = new Mailcode_Variables();
         }

src/Mailcode/Commands/Command.php

Spacing +15 added lines, -15 removed lines

@@ -59,7 +59,7 @@ 
     */
     protected $validationResult = null;
     
-    public function __construct(string $type='', string $paramsString='', string $matchedText='')
+    public function __construct(string $type = '', string $paramsString = '', string $matchedText = '')
     {
         $this->type = $type;
         $this->paramsString = $paramsString;
@@ -97,7 +97,7 @@ 
     {
         $this->requireNonDummy();
         
-        if($this->hash === '') {
+        if ($this->hash === '') {
             $this->hash = md5($this->matchedText);
         }
         
@@ -106,7 +106,7 @@ 
     
     protected function requireNonDummy() : void
     {
-        if(!$this->isDummy())
+        if (!$this->isDummy())
         {
             return;
         }
@@ -127,7 +127,7 @@ 
     {
         $this->requireNonDummy();
         
-        if(isset($this->validationResult)) 
+        if (isset($this->validationResult)) 
         {
             return $this->validationResult;
         }
@@ -141,7 +141,7 @@ 
     
     public function getValidationResult() :  \AppUtils\OperationResult
     {
-        if(isset($this->validationResult)) 
+        if (isset($this->validationResult)) 
         {
             return $this->validationResult;
         }
@@ -164,11 +164,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',
@@ -184,7 +184,7 @@ 
             $this->$method();
             
             // break off at the first validation issue
-            if(!$this->validationResult->isValid())
+            if (!$this->validationResult->isValid())
             {
                 return;
             }
@@ -195,7 +195,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.'),
@@ -207,14 +207,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).' '.
@@ -228,7 +228,7 @@ 
     
     protected function validateSyntax_type_unsupported()
     {
-        if($this->supportsType() || empty($this->type))
+        if ($this->supportsType() || empty($this->type))
         {
             return;
         }
@@ -243,7 +243,7 @@ 
     {
         $vars = $this->getVariables();
         
-        if(!$vars->hasInvalid()) 
+        if (!$vars->hasInvalid()) 
         {
             return;
         }
@@ -263,7 +263,7 @@ 
     
     public function getType() : string
     {
-        if($this->supportsType())
+        if ($this->supportsType())
         {
             return $this->type;
         }
@@ -289,7 +289,7 @@ 
     
     public function getParamsString() : string
     {
-        if($this->requiresParameters())
+        if ($this->requiresParameters())
         {
             return $this->paramsString;
         }

src/Mailcode/Commands/Command/ShowVariable.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     {
          $amount = $this->getVariables()->countVariables();
          
-         if($amount === 1)
+         if ($amount === 1)
          {
              return;
          }

src/Mailcode/Variables/Variable.php

Spacing +4 added lines, -4 removed lines

@@ -83,7 +83,7 @@ 
     
     public function getHash() : string
     {
-        if(empty($this->hash))
+        if (empty($this->hash))
         {
             $this->hash = md5($this->matchedText);
         }
@@ -107,14 +107,14 @@ 
     
     protected function validate() : void
     {
-        if(isset($this->validationResult))
+        if (isset($this->validationResult))
         {
             return;
         }
         
         $this->validationResult = new OperationResult($this);
         
-        if(is_numeric(substr($this->path, 0, 1))) 
+        if (is_numeric(substr($this->path, 0, 1))) 
         {
             $this->validationResult->makeError(
                 t(
@@ -126,7 +126,7 @@ 
             );
         }
         
-        if(is_numeric(substr($this->name, 0, 1)))
+        if (is_numeric(substr($this->name, 0, 1)))
         {
             $this->validationResult->makeError(
                 t(

src/Mailcode/Variables/Collection/Regular.php

Spacing +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
     
     public function add(Mailcode_Variables_Variable $variable) : Mailcode_Variables_Collection
     {
-        if(!$variable->isValid())
+        if (!$variable->isValid())
         {
             return $this->addInvalid($variable);
         }
@@ -76,7 +76,7 @@ 
         parent::mergeWith($collection);
         
         // also inherit any invalid variables
-        if($collection instanceof Mailcode_Variables_Collection_Regular)
+        if ($collection instanceof Mailcode_Variables_Collection_Regular)
         {
             $invalid = $collection->getInvalid();
             

src/Mailcode/Variables/Collection/Invalid.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 {
     public function add(Mailcode_Variables_Variable $variable) : Mailcode_Variables_Collection
     {
-        if($variable->isValid())
+        if ($variable->isValid())
         {
             return $this;
         }

src/Mailcode/Variables/Collection.php

Spacing +9 added lines, -9 removed lines

@@ -66,9 +66,9 @@ 
     {
         $fullName = $this->fixName($fullName);
         
-        foreach($this->variables as $variable)
+        foreach ($this->variables as $variable)
         {
-            if($variable->getFullName() === $fullName)
+            if ($variable->getFullName() === $fullName)
             {
                 return true;
             }
@@ -91,9 +91,9 @@ 
         
         $collection = new Mailcode_Variables_Collection_Regular();
         
-        foreach($this->variables as $variable)
+        foreach ($this->variables as $variable)
         {
-            if($variable->getFullName() === $fullName)
+            if ($variable->getFullName() === $fullName)
             {
                 $collection->add($variable);
             }
@@ -110,7 +110,7 @@ 
     */
     protected function fixName(string $fullName) : string
     {
-        if(substr($fullName, 0, 1) === '$')
+        if (substr($fullName, 0, 1) === '$')
         {
             return $fullName;
         }
@@ -138,7 +138,7 @@ 
     {
         $entries = array();
         
-        foreach($this->variables as $variable)
+        foreach ($this->variables as $variable)
         {
             $entries[$variable->getFullName()] = $variable;
         }
@@ -154,11 +154,11 @@ 
     {
         $result = array();
         
-        foreach($this->variables as $variable)
+        foreach ($this->variables as $variable)
         {
             $name = $variable->getFullName();
             
-            if(!in_array($name, $result))
+            if (!in_array($name, $result))
             {
                 $result[] = $name;
             }
@@ -197,7 +197,7 @@ 
     {
         $variables = $collection->getGroupedByHash();
         
-        foreach($variables as $variable)
+        foreach ($variables as $variable)
         {
             $this->add($variable);
         }

src/Mailcode/Collection.php

Spacing +3 added lines, -3 removed lines

@@ -40,7 +40,7 @@ 
     {
         $hash = $command->getHash();
         
-        if(!isset($this->commands[$hash])) {
+        if (!isset($this->commands[$hash])) {
             $this->commands[$hash] = $command;
         }
         
@@ -106,7 +106,7 @@ 
     
     public function addCommands(array $commands) : Mailcode_Collection
     {
-        foreach($commands as $command)
+        foreach ($commands as $command)
         {
             $this->addCommand($command);
         }
@@ -125,7 +125,7 @@ 
     {
         $collection = new Mailcode_Variables_Collection_Regular();
         
-        foreach($this->commands as $command)
+        foreach ($this->commands as $command)
         {
             $collection->mergeWith($command->getVariables());
         }

src/Mailcode/Variables.php

Spacing +2 added lines, -2 removed lines

@@ -35,12 +35,12 @@
         $matches = array();
         preg_match_all(self::REGEX_VARIABLE_NAME, $subject, $matches, PREG_PATTERN_ORDER);
         
-        if(!isset($matches[0]) || empty($matches[0]))
+        if (!isset($matches[0]) || empty($matches[0]))
         {
             return $collection;
         }
         
-        foreach($matches[0] as $idx => $matchedText)
+        foreach ($matches[0] as $idx => $matchedText)
         {
             $path = $matches[1][$idx];
             $name = $matches[2][$idx];