Mistralys / mailcode

src/Mailcode/Variables/Collection.php

Indentation +56 added lines, -56 removed lines

@@ -20,9 +20,9 @@ 
  */
 abstract class Mailcode_Variables_Collection
 {
-   /**
-    * @var Mailcode_Variables_Variable[]
-    */
+    /**
+     * @var Mailcode_Variables_Variable[]
+     */
     protected array $variables = array();
     
     public function __construct()
@@ -52,12 +52,12 @@ 
         return count($this->variables);
     }
     
-   /**
-    * Checks whether the collection contains a variable with the specified name.
-    * 
-    * @param string $fullName The variable name, with or without $ sign.
-    * @return bool
-    */
+    /**
+     * Checks whether the collection contains a variable with the specified name.
+     * 
+     * @param string $fullName The variable name, with or without $ sign.
+     * @return bool
+     */
     public function hasVariableName(string $fullName) : bool
     {
         $fullName = $this->fixName($fullName);
@@ -73,14 +73,14 @@ 
         return false;
     }
     
-   /**
-    * Retrieves a collection of all variable instances for
-    * the specified name (there can be several with differing
-    * matched texts because of spacing).
-    * 
-    * @param string $fullName
-    * @return Mailcode_Variables_Collection
-    */
+    /**
+     * Retrieves a collection of all variable instances for
+     * the specified name (there can be several with differing
+     * matched texts because of spacing).
+     * 
+     * @param string $fullName
+     * @return Mailcode_Variables_Collection
+     */
     public function getByFullName(string $fullName) : Mailcode_Variables_Collection
     {
         $fullName = $this->fixName($fullName);
@@ -98,23 +98,23 @@ 
         return $collection;
     }
     
-   /**
-    * Prepends the $ sign to a variable name if it does not have it.
-    * 
-    * @param string $fullName
-    * @return string
-    */
+    /**
+     * Prepends the $ sign to a variable name if it does not have it.
+     * 
+     * @param string $fullName
+     * @return string
+     */
     protected function fixName(string $fullName) : string
     {
         return dollarize($fullName);
     }
     
-   /**
-    * Retrieves all variables, grouped by their hash - meaning
-    * unique matched strings.
-    * 
-    * @return Mailcode_Variables_Variable[]
-    */
+    /**
+     * Retrieves all variables, grouped by their hash - meaning
+     * unique matched strings.
+     * 
+     * @return Mailcode_Variables_Variable[]
+     */
     public function getGroupedByHash() : array
     {
         $entries = array();
@@ -127,11 +127,11 @@ 
         return $this->sortVariables($entries);
     }
     
-   /**
-    * Retrieves all variables, grouped by their name. 
-    * 
-    * @return Mailcode_Variables_Variable[]
-    */
+    /**
+     * Retrieves all variables, grouped by their name. 
+     * 
+     * @return Mailcode_Variables_Variable[]
+     */
     public function getGroupedByName() : array
     {
         $entries = array();
@@ -163,19 +163,19 @@ 
         return $this->sortVariables($entries);
     }
     
-   /**
-    * Retrieves all variables, in the order they were addded.
-    * @return Mailcode_Variables_Variable[]
-    */
+    /**
+     * Retrieves all variables, in the order they were addded.
+     * @return Mailcode_Variables_Variable[]
+     */
     public function getAll()
     {
         return $this->variables;
     }
     
-   /**
-    * Retrieves the full names of the variables that are present in the collection.
-    * @return string[]
-    */
+    /**
+     * Retrieves the full names of the variables that are present in the collection.
+     * @return string[]
+     */
     public function getNames() : array
     {
         $result = array();
@@ -193,13 +193,13 @@ 
         return $result;
     }
     
-   /**
-    * Takes a list of variables and sorts them, throwing away
-    * the source array's keys.
-    * 
-    * @param Mailcode_Variables_Variable[] $entries
-    * @return Mailcode_Variables_Variable[]
-    */
+    /**
+     * Takes a list of variables and sorts them, throwing away
+     * the source array's keys.
+     * 
+     * @param Mailcode_Variables_Variable[] $entries
+     * @return Mailcode_Variables_Variable[]
+     */
     protected function sortVariables(array $entries)
     {
         $result = array_values($entries);
@@ -212,13 +212,13 @@ 
         return $result;
     }
 
-   /**
-    *  Merges the variables collection with the target collection
-    *  by inheriting all that collection's variables.
-    *  
-    * @param Mailcode_Variables_Collection $collection
-    * @return Mailcode_Variables_Collection
-    */
+    /**
+     *  Merges the variables collection with the target collection
+     *  by inheriting all that collection's variables.
+     *  
+     * @param Mailcode_Variables_Collection $collection
+     * @return Mailcode_Variables_Collection
+     */
     public function mergeWith(Mailcode_Variables_Collection $collection) : Mailcode_Variables_Collection
     {
         $variables = $collection->getGroupedByHash();

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/Variables/Variable.php

Spacing +10 added lines, -10 removed lines

@@ -72,7 +72,7 @@ 
      */
     private ?Mailcode_Commands_Command $command;
 
-    public function __construct(string $path, string $name, string $matchedText, ?Mailcode_Commands_Command $sourceCommand=null)
+    public function __construct(string $path, string $name, string $matchedText, ?Mailcode_Commands_Command $sourceCommand = null)
     {
         $this->path = $path;
         $this->name = $name;
@@ -95,7 +95,7 @@ 
     
     public function getFullName() : string
     {
-        if(empty($this->path))
+        if (empty($this->path))
         {
             return dollarize($this->name);
         }
@@ -113,7 +113,7 @@ 
      */
     public function getUniqueName() : string
     {
-        if(isset($this->command))
+        if (isset($this->command))
         {
             return $this->getFullName().'/'.$this->command->getHash();
         }
@@ -138,7 +138,7 @@ 
     
     public function getHash() : string
     {
-        if(empty($this->hash))
+        if (empty($this->hash))
         {
             $this->hash = md5($this->matchedText);
         }
@@ -157,7 +157,7 @@ 
      */
     public function getValidationResult() : OperationResult
     {
-        if(isset($this->validationResult))
+        if (isset($this->validationResult))
         {
             return $this->validationResult;
         }
@@ -171,11 +171,11 @@ 
 
     protected function validate(OperationResult $result) : 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',
@@ -220,7 +220,7 @@ 
     
     protected function validateNumber(string $value, int $errorCode) : void
     {
-        if(!is_numeric(substr($value, 0, 1)))
+        if (!is_numeric(substr($value, 0, 1)))
         {
             return;
         }
@@ -238,7 +238,7 @@ 
     protected function validateUnderscore(string $value, int $errorCode) : void
     {
         // allow empty paths
-        if(empty($value))
+        if (empty($value))
         {
             return;
         }
@@ -246,7 +246,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/Translator/Syntax/ApacheVelocity/Contains/StatementBuilder.php

Indentation +22 added lines, -22 removed lines

@@ -225,27 +225,27 @@
      * @return array<string,string>
      * @throws Mailcode_Exception
      */
-     private function parseVarName() : array
-     {
-         $tokens = explode('.', undollarize($this->variable->getFullName()));
-
-         if(count($tokens) === 2)
-         {
-             return array(
-                 'path' => $tokens[0],
-                 'name' => $tokens[1]
-             );
-         }
-
-         throw new Mailcode_Exception(
-             'Invalid variable name for a list property.',
-             sprintf(
-                 'Exactly 2 parts are required, variable [%s] has [%s].',
-                 $this->variable->getFullName(),
-                 count($tokens)
-             ),
-             self::ERROR_INVALID_LIST_VARIABLE_NAME
-         );
-     }
+        private function parseVarName() : array
+        {
+            $tokens = explode('.', undollarize($this->variable->getFullName()));
+
+            if(count($tokens) === 2)
+            {
+                return array(
+                    'path' => $tokens[0],
+                    'name' => $tokens[1]
+                );
+            }
+
+            throw new Mailcode_Exception(
+                'Invalid variable name for a list property.',
+                sprintf(
+                    'Exactly 2 parts are required, variable [%s] has [%s].',
+                    $this->variable->getFullName(),
+                    count($tokens)
+                ),
+                self::ERROR_INVALID_LIST_VARIABLE_NAME
+            );
+        }
 }
 

Spacing +10 added lines, -10 removed lines

@@ -84,7 +84,7 @@ 
      */
     public function getSign() : string
     {
-        if($this->isNotContains())
+        if ($this->isNotContains())
         {
             return '!';
         }
@@ -101,7 +101,7 @@ 
      */
     public function getConnector()
     {
-        if($this->isNotContains())
+        if ($this->isNotContains())
         {
             return '&&';
         }
@@ -117,7 +117,7 @@ 
     {
         $parts = array();
 
-        foreach($this->searchTerms as $token)
+        foreach ($this->searchTerms as $token)
         {
             $parts[] = $this->renderCommand($token);
         }
@@ -132,7 +132,7 @@ 
      */
     private function renderCommand(Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral $searchTerm) : string
     {
-        if($this->isList())
+        if ($this->isList())
         {
             $command = $this->renderListCommand($searchTerm);
         }
@@ -173,14 +173,14 @@ 
     private function renderRegex(Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral $searchTerm) : string
     {
         $opts = 's';
-        if($this->caseSensitive)
+        if ($this->caseSensitive)
         {
             $opts = 'is';
         }
 
         $filtered = trim($searchTerm->getNormalized(), '"');
 
-        if(!$this->regexEnabled)
+        if (!$this->regexEnabled)
         {
             $filtered = $this->translator->filterRegexString($filtered);
             $filtered = $this->addWildcards($filtered);
@@ -203,17 +203,17 @@ 
      */
     private function addWildcards(string $searchTerm) : string
     {
-        if($this->containsType === 'list-begins-with')
+        if ($this->containsType === 'list-begins-with')
         {
             return $searchTerm.'.*';
         }
 
-        if($this->containsType === 'list-ends-with')
+        if ($this->containsType === 'list-ends-with')
         {
             return '.*'.$searchTerm;
         }
 
-        if($this->containsType === 'list-equals')
+        if ($this->containsType === 'list-equals')
         {
             return '\\\A'.$searchTerm.'\\\Z';
         }
@@ -229,7 +229,7 @@ 
      {
          $tokens = explode('.', undollarize($this->variable->getFullName()));
 
-         if(count($tokens) === 2)
+         if (count($tokens) === 2)
          {
              return array(
                  'path' => $tokens[0],

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/Translator/Syntax/ApacheVelocity.php

Spacing +2 added lines, -2 removed lines

@@ -71,7 +71,7 @@ 
 
         // All other special characters have to be escaped
         // with two backslashes.
-        foreach($this->regexSpecialChars as $char)
+        foreach ($this->regexSpecialChars as $char)
         {
             $string = str_replace($char, '\\'.$char, $string);
         }
@@ -99,7 +99,7 @@ 
     {
         $varName = dollarize($varName);
 
-        if($absolute)
+        if ($absolute)
         {
             $varName = sprintf('${numeric.abs(%s)}', $varName);
         }

src/Mailcode/Variables.php

Spacing +8 added lines, -8 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;
         }

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/Traits/Commands/Validation/BreakAtTrait.php

Spacing +6 added lines, -6 removed lines

@@ -43,13 +43,13 @@ 
 
         $val = $this->validator->createKeyword(Mailcode_Commands_Keywords::TYPE_BREAK_AT);
 
-        $this->breakAtEnabled = $val->isValid() && $this->breakAtToken != null;;
+        $this->breakAtEnabled = $val->isValid() && $this->breakAtToken != null; ;
 
         if ($this->breakAtEnabled) {
             if (!$this->breakAtToken instanceof Mailcode_Parser_Statement_Tokenizer_Token_Number &&
                 !$this->breakAtToken instanceof Mailcode_Parser_Statement_Tokenizer_Token_Variable) {
                 $this->validationResult->makeError(
-                    t('Invalid break-at type.' . ' ' . 'Expected Number or Variable.'),
+                    t('Invalid break-at type.'.' '.'Expected Number or Variable.'),
                     BreakAtInterface::VALIDATION_BREAK_AT_CODE_WRONG_TYPE
                 );
                 return;
@@ -73,19 +73,19 @@ 
     public function getBreakAt()
     {
         $token = $this->getBreakAtToken();
-        if($token === null) {
+        if ($token === null) {
             return null;
         }
 
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Variable) {
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Variable) {
             return $token->getVariable();
         }
 
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Number) {
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Number) {
             return (int)$token->getValue();
         }
 
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral) {
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral) {
             return (int)$token->getText();
         }
 

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