Mistralys / mailcode

src/Mailcode/Commands/Command/For.php

Spacing +3 added lines, -3 removed lines

@@ -146,7 +146,7 @@ 
 
         if (!$this->loopVar || !$this->keyword || !$this->sourceVar) {
             $this->validationResult->makeError(
-                t('Not a valid for loop.') . ' ' . t('Is the %1$s keyword missing?', 'in:'),
+                t('Not a valid for loop.').' '.t('Is the %1$s keyword missing?', 'in:'),
                 self::VALIDATION_INVALID_FOR_STATEMENT
             );
 
@@ -191,8 +191,8 @@ 
         }
 
         $this->validationResult->makeError(
-            t('The source variable is not a list variable:') . ' ' .
-            t('Expected a variable without dot, like %1$s.', '<code>$' . t('LIST') . '</code>'),
+            t('The source variable is not a list variable:').' '.
+            t('Expected a variable without dot, like %1$s.', '<code>$'.t('LIST').'</code>'),
             self::VALIDATION_VARIABLE_NAME_WITH_DOT
         );
     }

src/Mailcode/Commands/Command/SetVariable.php

Spacing +2 added lines, -2 removed lines

@@ -124,7 +124,7 @@ 
         // allowed ones.
         if (!in_array($sign, $allowed)) {
             $this->validationResult->makeError(
-                t('The %1$s sign is not allowed in this command.', '<code>' . $sign . '</code>'),
+                t('The %1$s sign is not allowed in this command.', '<code>'.$sign.'</code>'),
                 Mailcode_Commands_CommonConstants::VALIDATION_INVALID_OPERAND
             );
         }
@@ -150,7 +150,7 @@ 
 
         throw new Mailcode_Exception(
             'No variable found.',
-            'Statement does not start with a variable: [' . $this->paramsString . ']',
+            'Statement does not start with a variable: ['.$this->paramsString.']',
             Mailcode_Commands_CommonConstants::ERROR_NO_VARIABLE_AVAILABLE
         );
     }

src/Mailcode/Exception.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 {
     public function __construct(string $message, ?string $details = null, $code = null, $previous = null)
     {
-        if(defined('TESTS_ROOT') && !empty($details)) {
+        if (defined('TESTS_ROOT') && !empty($details)) {
             $message .= PHP_EOL.
                 'Details:'.PHP_EOL.
                 $details;

src/Mailcode/Parser/Statement/Tokenizer/Process/Variables.php

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
         $vars = Mailcode::create()->findVariables($this->tokenized, $this->tokenizer->getSourceCommand())->getGroupedByHash();
 
         $names = array();
-        foreach($vars as $var)
+        foreach ($vars as $var)
         {
             $names[$var->getMatchedText()] = $var;
         }
@@ -23,7 +23,7 @@ 
             return strlen($b) - strlen($a);
         });
 
-        foreach($names as $name => $var)
+        foreach ($names as $name => $var)
         {
             $this->registerToken('Variable', $name, $var);
         }

src/Mailcode/Variables/Collection.php

Spacing +11 added lines, -11 removed lines

@@ -62,9 +62,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;
             }
@@ -87,9 +87,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);
             }
@@ -119,7 +119,7 @@ 
     {
         $entries = array();
         
-        foreach($this->variables as $variable)
+        foreach ($this->variables as $variable)
         {
             $entries[$variable->getHash()] = $variable;
         }
@@ -136,7 +136,7 @@ 
     {
         $entries = array();
         
-        foreach($this->variables as $variable)
+        foreach ($this->variables as $variable)
         {
             $entries[$variable->getFullName()] = $variable;
         }
@@ -155,7 +155,7 @@ 
     {
         $entries = array();
 
-        foreach($this->variables as $variable)
+        foreach ($this->variables as $variable)
         {
             $entries[$variable->getUniqueName()] = $variable;
         }
@@ -180,11 +180,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;
             }
@@ -223,7 +223,7 @@ 
     {
         $variables = $collection->getGroupedByHash();
         
-        foreach($variables as $variable)
+        foreach ($variables as $variable)
         {
             $this->add($variable);
         }
@@ -235,7 +235,7 @@ 
     {
         $variables = $this->getAll();
         
-        if(!empty($variables))
+        if (!empty($variables))
         {
             return array_shift($variables);
         }

src/Mailcode/Translator/Syntax/ApacheVelocity/ShowSnippet.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     {
         $varName = undollarize($command->getVariableName());
 
-        if($command->isHTMLEnabled())
+        if ($command->isHTMLEnabled())
         {
             $statement = sprintf(
                 '%s.replaceAll($esc.newline, "<br/>")',

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

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
         // Split the variable name by the dot
         $parts = explode('.', undollarize($var->getFullName()));
 
-        if(count($parts) !== 2)
+        if (count($parts) !== 2)
         {
             $this->validationResult->makeError(
                 t('The variable %1$s is not a list property:', '<code>'.$var->getFullName().'</code>').' '.
@@ -76,7 +76,7 @@ 
      */
     public function getListVariable() : Mailcode_Variables_Variable
     {
-        if(isset($this->listVariable))
+        if (isset($this->listVariable))
         {
             return $this->listVariable;
         }

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

Spacing +2 added lines, -2 removed lines

@@ -59,12 +59,12 @@
      * @throws Mailcode_Exception
      * @throws Mailcode_Factory_Exception
      */
-    public function for(string $sourceVariable, string $loopVariable, $breakAtValue = ''): Mailcode_Commands_Command_For
+    public function for (string $sourceVariable, string $loopVariable, $breakAtValue = ''): Mailcode_Commands_Command_For
     {
         $sourceVariable = dollarize($sourceVariable);
         $loopVariable = dollarize($loopVariable);
 
-        if($breakAtValue instanceof Mailcode_Variables_Variable) {
+        if ($breakAtValue instanceof Mailcode_Variables_Variable) {
             $breakAtValue = $breakAtValue->getFullName();
         }
 

src/Mailcode/Variables.php

Spacing +11 added lines, -11 removed lines

@@ -34,21 +34,21 @@ 
      * @param Mailcode_Commands_Command|null $sourceCommand
      * @return Mailcode_Variables_Collection_Regular
      */
-    public function parseString(string $subject, ?Mailcode_Commands_Command $sourceCommand=null) : Mailcode_Variables_Collection_Regular
+    public function parseString(string $subject, ?Mailcode_Commands_Command $sourceCommand = null) : Mailcode_Variables_Collection_Regular
     {
         $this->collection = new Mailcode_Variables_Collection_Regular();
         
         $matches = array();
         preg_match_all(self::REGEX_VARIABLE_NAME, $subject, $matches, PREG_PATTERN_ORDER);
         
-        if(empty($matches[0]))
+        if (empty($matches[0]))
         {
             return $this->collection;
         }
         
-        foreach($matches[0] as $idx => $matchedText)
+        foreach ($matches[0] as $idx => $matchedText)
         {
-            if(!empty($matches[3][$idx]))
+            if (!empty($matches[3][$idx]))
             {
                 $this->addSingle($matches[3][$idx], $matchedText, $sourceCommand);
             }
@@ -61,10 +61,10 @@ 
         return $this->collection;
     }
     
-    protected function addSingle(string $name, string $matchedText, ?Mailcode_Commands_Command $sourceCommand=null) : void
+    protected function addSingle(string $name, string $matchedText, ?Mailcode_Commands_Command $sourceCommand = null) : void
     {
         // ignore US style numbers like $451
-        if(is_numeric($name))
+        if (is_numeric($name))
         {
             return;
         }
@@ -72,10 +72,10 @@ 
         $this->collection->add(new Mailcode_Variables_Variable('', $name, $matchedText, $sourceCommand));
     }
     
-    protected function addPathed(string $path, string $name, string $matchedText, ?Mailcode_Commands_Command $sourceCommand=null) : void
+    protected function addPathed(string $path, string $name, string $matchedText, ?Mailcode_Commands_Command $sourceCommand = null) : void
     {
         // ignore US style numbers like $45.12
-        if(is_numeric($path.'.'.$name))
+        if (is_numeric($path.'.'.$name))
         {
             return;
         }
@@ -93,9 +93,9 @@ 
         return ltrim($name, '$');
     }
 
-    public function createVariable(string $path, ?string $name='') : Mailcode_Variables_Variable
+    public function createVariable(string $path, ?string $name = '') : Mailcode_Variables_Variable
     {
-        if(empty($name)) {
+        if (empty($name)) {
             $fullName = dollarize($path);
             $name = $path;
             $path = '';
@@ -110,7 +110,7 @@ 
     {
         $parts = explode('.', undollarize($name));
 
-        if(count($parts) === 1) {
+        if (count($parts) === 1) {
             return $this->createVariable($parts[0]);
         }