Mistralys / mailcode

src/Mailcode/Renderer.php

Indentation +15 added lines, -15 removed lines

@@ -21,17 +21,17 @@ 
  */
 class Mailcode_Renderer
 {
-   /**
-    * @var boolean
-    */
+    /**
+     * @var boolean
+     */
     protected $highlighted = false;
     
-   /**
-    * Sets whether to output highlighted commands instead of the default plain text.
-    * 
-    * @param bool $highlighted
-    * @return Mailcode_Renderer
-    */
+    /**
+     * Sets whether to output highlighted commands instead of the default plain text.
+     * 
+     * @param bool $highlighted
+     * @return Mailcode_Renderer
+     */
     public function setOutputHighlighted(bool $highlighted=true) : Mailcode_Renderer
     {
         $this->highlighted = $highlighted;
@@ -39,12 +39,12 @@ 
         return $this;
     }
     
-   /**
-    * Converts a show variable command to string.
-    * 
-    * @param string $variableName The variable name, with or without $ sign.
-    * @return string
-    */
+    /**
+     * Converts a show variable command to string.
+     * 
+     * @param string $variableName The variable name, with or without $ sign.
+     * @return string
+     */
     public function showVar(string $variableName) : string
     {
         return $this->command2string(Mailcode_Factory::showVar($variableName));

Braces +9 added lines, -3 removed lines

@@ -62,7 +62,9 @@ 
     
     public function if(string $condition, string $type='') : string
     {
-        return $this->command2string(Mailcode_Factory::if($condition, $type));
+        return $this->command2string(Mailcode_Factory::if($condition, $type) {
+            );
+        }
     }
     
     public function ifVar(string $variable, string $operand, string $value, bool $quoteValue=false) : string
@@ -97,7 +99,9 @@ 
     
     public function elseIf(string $condition, string $type='') : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf($condition, $type));
+        return $this->command2string(Mailcode_Factory::elseIf($condition, $type) {
+            );
+        }
     }
     
     public function elseIfVar(string $variable, string $operand, string $value, bool $quoteValue=false) : string
@@ -130,10 +134,12 @@ 
         return $this->command2string(Mailcode_Factory::elseIfVarNotEqualsString($variable, $value));
     }
     
-    public function else() : string
+    public function else {
+        () : string
     {
         return $this->command2string(Mailcode_Factory::else());
     }
+    }
     
     public function end() : string
     {

Spacing +13 added lines, -13 removed lines

@@ -32,7 +32,7 @@ 
     * @param bool $highlighted
     * @return Mailcode_Renderer
     */
-    public function setOutputHighlighted(bool $highlighted=true) : Mailcode_Renderer
+    public function setOutputHighlighted(bool $highlighted = true) : Mailcode_Renderer
     {
         $this->highlighted = $highlighted;
         
@@ -55,7 +55,7 @@ 
         return $this->command2string(Mailcode_Factory::showSnippet($snippetName));
     }
     
-    public function setVar(string $variableName, string $value, bool $quoteValue=false) : string
+    public function setVar(string $variableName, string $value, bool $quoteValue = false) : string
     {
         return $this->command2string(Mailcode_Factory::setVar($variableName, $value, $quoteValue));
     }
@@ -65,12 +65,12 @@ 
         return $this->command2string(Mailcode_Factory::setVarString($variableName, $value));
     }
     
-    public function if(string $condition, string $type='') : string
+    public function if (string $condition, string $type = '') : string
     {
-        return $this->command2string(Mailcode_Factory::if($condition, $type));
+        return $this->command2string(Mailcode_Factory::if ($condition, $type));
     }
     
-    public function ifVar(string $variable, string $operand, string $value, bool $quoteValue=false) : string
+    public function ifVar(string $variable, string $operand, string $value, bool $quoteValue = false) : string
     {
         return $this->command2string(Mailcode_Factory::ifVar($variable, $operand, $value, $quoteValue));
     }
@@ -80,7 +80,7 @@ 
         return $this->command2string(Mailcode_Factory::ifVarString($variable, $operand, $value));
     }
     
-    public function ifVarEquals(string $variable, string $value, bool $quoteValue=false) : string
+    public function ifVarEquals(string $variable, string $value, bool $quoteValue = false) : string
     {
         return $this->command2string(Mailcode_Factory::ifVarEquals($variable, $value, $quoteValue));
     }
@@ -90,7 +90,7 @@ 
         return $this->command2string(Mailcode_Factory::ifVarEqualsString($variable, $value));    
     }
     
-    public function ifVarNotEquals(string $variable, string $value, bool $quoteValue=false) : string
+    public function ifVarNotEquals(string $variable, string $value, bool $quoteValue = false) : string
     {
         return $this->command2string(Mailcode_Factory::ifVarNotEquals($variable, $value, $quoteValue));
     }
@@ -100,12 +100,12 @@ 
         return $this->command2string(Mailcode_Factory::ifVarNotEqualsString($variable, $value));
     }
     
-    public function elseIf(string $condition, string $type='') : string
+    public function elseIf (string $condition, string $type = '') : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf($condition, $type));
+        return $this->command2string(Mailcode_Factory::elseIf ($condition, $type));
     }
     
-    public function elseIfVar(string $variable, string $operand, string $value, bool $quoteValue=false) : string
+    public function elseIfVar(string $variable, string $operand, string $value, bool $quoteValue = false) : string
     {
         return $this->command2string(Mailcode_Factory::elseIfVar($variable, $operand, $value, $quoteValue));
     }
@@ -115,7 +115,7 @@ 
         return $this->command2string(Mailcode_Factory::elseIfVarString($variable, $operand, $value));
     }
     
-    public function elseIfVarEquals(string $variable, string $value, bool $quoteValue=false) : string
+    public function elseIfVarEquals(string $variable, string $value, bool $quoteValue = false) : string
     {
         return $this->command2string(Mailcode_Factory::elseIfVarEquals($variable, $value, $quoteValue));
     }
@@ -125,7 +125,7 @@ 
         return $this->command2string(Mailcode_Factory::elseIfVarEqualsString($variable, $value));
     }
     
-    public function elseIfVarNotEquals(string $variable, string $value, bool $quoteValue=false) : string
+    public function elseIfVarNotEquals(string $variable, string $value, bool $quoteValue = false) : string
     {
         return $this->command2string(Mailcode_Factory::elseIfVarNotEquals($variable, $value, $quoteValue));
     }
@@ -152,7 +152,7 @@ 
     
     protected function command2string(Mailcode_Commands_Command $command) : string
     {
-        if($this->highlighted)
+        if ($this->highlighted)
         {
             return $command->getHighlighted();
         }

src/Mailcode/Variables.php

Indentation +9 added lines, -9 removed lines

@@ -22,17 +22,17 @@
 {
     const REGEX_VARIABLE_NAME = '/\$\s*([A-Z0-9_]+)\s*\.\s*([A-Z0-9_]+)|\$\s*([A-Z0-9_]+)/six';
     
-   /**
-    * @var Mailcode_Variables_Collection_Regular
-    */
+    /**
+     * @var Mailcode_Variables_Collection_Regular
+     */
     protected $collection;
     
-   /**
-    * Parses the specified string to find all variable names contained within, if any.
-    * 
-    * @param string $subject
-    * @return Mailcode_Variables_Collection_Regular
-    */
+    /**
+     * Parses the specified string to find all variable names contained within, if any.
+     * 
+     * @param string $subject
+     * @return Mailcode_Variables_Collection_Regular
+     */
     public function parseString(string $subject) : Mailcode_Variables_Collection_Regular
     {
         $this->collection = new Mailcode_Variables_Collection_Regular();

Spacing +5 added lines, -5 removed lines

@@ -40,14 +40,14 @@ 
         $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 $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);
             }
@@ -63,7 +63,7 @@ 
     protected function addSingle(string $name, string $matchedText) : void
     {
         // ignore US style numbers like $451
-        if(is_numeric($name))
+        if (is_numeric($name))
         {
             return;
         }
@@ -74,7 +74,7 @@ 
     protected function addPathed(string $path, string $name, string $matchedText) : void
     {
         // ignore US style numbers like $45.12
-        if(is_numeric($path.'.'.$name))
+        if (is_numeric($path.'.'.$name))
         {
             return;
         }

Braces +1 added lines, -2 removed lines

@@ -50,8 +50,7 @@
             if(!empty($matches[3][$idx]))
             {
                 $this->addSingle($matches[3][$idx], $matchedText);
-            }
-            else 
+            } else 
             {
                 $this->addPathed($matches[1][$idx], $matches[2][$idx], $matchedText);
             }

src/Mailcode.php

Indentation +45 added lines, -45 removed lines

@@ -26,51 +26,51 @@ 
  */
 class Mailcode
 {
-   /**
-    * @var Mailcode_Parser|NULL
-    */
+    /**
+     * @var Mailcode_Parser|NULL
+     */
     protected $parser = null;
     
-   /**
-    * @var Mailcode_Commands|NULL
-    */
+    /**
+     * @var Mailcode_Commands|NULL
+     */
     protected $commands = null;
     
-   /**
-    * @var Mailcode_Variables|NULL
-    */
+    /**
+     * @var Mailcode_Variables|NULL
+     */
     protected $variables = null;
     
-   /**
-    * @var Mailcode_Translator|NULL
-    */
+    /**
+     * @var Mailcode_Translator|NULL
+     */
     protected $translator = null;
     
-   /**
-    * Creates a new mailcode instance.
-    * @return Mailcode
-    */
+    /**
+     * Creates a new mailcode instance.
+     * @return Mailcode
+     */
     public static function create() : Mailcode
     {
         return new Mailcode();
     }
     
-   /**
-    * Parses the string to detect all commands contained within.
-    * 
-    * @param string $string
-    * @return Mailcode_Collection
-    */
+    /**
+     * Parses the string to detect all commands contained within.
+     * 
+     * @param string $string
+     * @return Mailcode_Collection
+     */
     public function parseString(string $string) : Mailcode_Collection
     {
         return $this->getParser()->parseString($string);
     }
     
-   /**
-    * Retrieves the string parser instance used to detect commands.
-    * 
-    * @return Mailcode_Parser
-    */
+    /**
+     * Retrieves the string parser instance used to detect commands.
+     * 
+     * @return Mailcode_Parser
+     */
     public function getParser() : Mailcode_Parser
     {
         if(!isset($this->parser)) 
@@ -81,12 +81,12 @@ 
         return $this->parser;
     }
     
-   /**
-    * Retrieves the commands collection, which is used to
-    * access information on the available commands.
-    * 
-    * @return Mailcode_Commands
-    */
+    /**
+     * Retrieves the commands collection, which is used to
+     * access information on the available commands.
+     * 
+     * @return Mailcode_Commands
+     */
     public function getCommands() : Mailcode_Commands
     {
         if(!isset($this->commands)) 
@@ -102,12 +102,12 @@ 
         return $this->getParser()->createSafeguard($subject);
     }
     
-   /**
-    * Attempts to find all variables in the target string.
-    * 
-    * @param string $subject
-    * @return Mailcode_Variables_Collection_Regular
-    */
+    /**
+     * Attempts to find all variables in the target string.
+     * 
+     * @param string $subject
+     * @return Mailcode_Variables_Collection_Regular
+     */
     public function findVariables(string $subject) : Mailcode_Variables_Collection_Regular
     {
         return $this->createVariables()->parseString($subject);
@@ -123,12 +123,12 @@ 
         return $this->variables;
     }
     
-   /**
-    * Creates the translator, which can be used to convert commands
-    * to another supported syntax.
-    * 
-    * @return Mailcode_Translator
-    */
+    /**
+     * Creates the translator, which can be used to convert commands
+     * to another supported syntax.
+     * 
+     * @return Mailcode_Translator
+     */
     public function createTranslator() : Mailcode_Translator
     {
         if(!isset($this->translator))

Spacing +4 added lines, -4 removed lines

@@ -73,7 +73,7 @@ 
     */
     public function getParser() : Mailcode_Parser
     {
-        if(!isset($this->parser)) 
+        if (!isset($this->parser)) 
         {
             $this->parser = new Mailcode_Parser($this);
         }
@@ -89,7 +89,7 @@ 
     */
     public function getCommands() : Mailcode_Commands
     {
-        if(!isset($this->commands)) 
+        if (!isset($this->commands)) 
         {
             $this->commands = new Mailcode_Commands();
         }
@@ -115,7 +115,7 @@ 
     
     public function createVariables() : Mailcode_Variables
     {
-        if(!isset($this->variables))
+        if (!isset($this->variables))
         {
             $this->variables = new Mailcode_Variables();
         }
@@ -131,7 +131,7 @@ 
     */
     public function createTranslator() : Mailcode_Translator
     {
-        if(!isset($this->translator))
+        if (!isset($this->translator))
         {
             $this->translator = new Mailcode_Translator(); 
         }

src/Mailcode/Parser/Statement.php

Indentation +13 added lines, -13 removed lines

@@ -29,24 +29,24 @@ 
     
     const VALIDATION_UNQUOTED_STRING_LITERALS = 48802;
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $statement;
     
-   /**
-    * @var OperationResult
-    */
+    /**
+     * @var OperationResult
+     */
     protected $result;
     
-   /**
-    * @var Mailcode_Parser_Statement_Tokenizer|NULL
-    */
+    /**
+     * @var Mailcode_Parser_Statement_Tokenizer|NULL
+     */
     protected $tokenizer;
     
-   /**
-    * @var Mailcode_Parser_Statement_Info|NULL
-    */
+    /**
+     * @var Mailcode_Parser_Statement_Info|NULL
+     */
     protected $info;
     
     public function __construct(string $statement)
@@ -102,7 +102,7 @@ 
         if($unknown)
         {
             $this->result->makeError(
-               t('Unquoted string literal found:').' ('.htmlspecialchars($unknown->getMatchedText()).')',
+                t('Unquoted string literal found:').' ('.htmlspecialchars($unknown->getMatchedText()).')',
                 self::VALIDATION_UNQUOTED_STRING_LITERALS
             );
         }

Spacing +3 added lines, -3 removed lines

@@ -75,7 +75,7 @@ 
     
     public function getInfo() : Mailcode_Parser_Statement_Info
     {
-        if(isset($this->info))
+        if (isset($this->info))
         {
             return $this->info; 
         }
@@ -87,7 +87,7 @@ 
     
     protected function validate() : void
     {
-        if(!$this->tokenizer->hasTokens())
+        if (!$this->tokenizer->hasTokens())
         {
             $this->result->makeError(
                 t('Empty statement'),
@@ -99,7 +99,7 @@ 
         
         $unknown = $this->tokenizer->getFirstUnknown();
         
-        if($unknown)
+        if ($unknown)
         {
             $this->result->makeError(
                t('Unquoted string literal found:').' ('.htmlspecialchars($unknown->getMatchedText()).')',

src/Mailcode/Commands/Command/Comment.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
         $this->paramsString = trim($this->paramsString);
         
         // automatically quote the parameters, since comments don't require any.
-        if(substr($this->paramsString, 0, 1) != '"')
+        if (substr($this->paramsString, 0, 1) != '"')
         {
             $this->paramsString = '"'.$this->paramsString.'"';
         }

src/Mailcode/Collection.php

Indentation +51 added lines, -51 removed lines

@@ -24,9 +24,9 @@ 
 {
     const ERROR_CANNOT_RETRIEVE_FIRST_ERROR = 52301;
     
-   /**
-    * @var Mailcode_Commands_Command[]
-    */
+    /**
+     * @var Mailcode_Commands_Command[]
+     */
     protected $commands = array();
     
     /**
@@ -34,17 +34,17 @@ 
      */
     protected $errors = array();
     
-   /**
-    * @var OperationResult|NULL
-    */
+    /**
+     * @var OperationResult|NULL
+     */
     protected $validationResult;
     
-   /**
-    * Adds a command to the collection.
-    * 
-    * @param Mailcode_Commands_Command $command
-    * @return Mailcode_Collection
-    */
+    /**
+     * Adds a command to the collection.
+     * 
+     * @param Mailcode_Commands_Command $command
+     * @return Mailcode_Collection
+     */
     public function addCommand(Mailcode_Commands_Command $command) : Mailcode_Collection
     {
         $this->commands[] = $command;
@@ -56,21 +56,21 @@ 
         return $this;
     }
     
-   /**
-    * Whether there are any commands in the collection.
-    * 
-    * @return bool
-    */
+    /**
+     * Whether there are any commands in the collection.
+     * 
+     * @return bool
+     */
     public function hasCommands() : bool
     {
         return !empty($this->commands);
     }
     
-   /**
-    * Counts the amount of commands in the collection.
-    * 
-    * @return int
-    */
+    /**
+     * Counts the amount of commands in the collection.
+     * 
+     * @return int
+     */
     public function countCommands() : int
     {
         return count($this->commands);
@@ -90,9 +90,9 @@ 
         $this->errors[] = new Mailcode_Collection_Error_Command($command);
     }
     
-   /**
-    * @return Mailcode_Collection_Error[]
-    */
+    /**
+     * @return Mailcode_Collection_Error[]
+     */
     public function getErrors()
     {
         $result = $this->getValidationResult();
@@ -134,25 +134,25 @@ 
         return empty($errors);
     }
     
-   /**
-    * Retrieves all commands that were detected, in the exact order
-    * they were found.
-    * 
-    * @return \Mailcode\Mailcode_Commands_Command[]
-    */
+    /**
+     * Retrieves all commands that were detected, in the exact order
+     * they were found.
+     * 
+     * @return \Mailcode\Mailcode_Commands_Command[]
+     */
     public function getCommands()
     {
-       return $this->commands;
+        return $this->commands;
     }
     
-   /**
-    * Retrieves all unique commands by their matched
-    * string hash: this ensures only commands that were
-    * written the exact same way (including spacing)
-    * are returned.
-    * 
-    * @return \Mailcode\Mailcode_Commands_Command[]
-    */
+    /**
+     * Retrieves all unique commands by their matched
+     * string hash: this ensures only commands that were
+     * written the exact same way (including spacing)
+     * are returned.
+     * 
+     * @return \Mailcode\Mailcode_Commands_Command[]
+     */
     public function getGroupedByHash()
     {
         $hashes = array();
@@ -170,12 +170,12 @@ 
         return array_values($hashes);
     }
 
-   /**
-    * Adds several commands at once.
-    * 
-    * @param Mailcode_Commands_Command[] $commands
-    * @return Mailcode_Collection
-    */
+    /**
+     * Adds several commands at once.
+     * 
+     * @param Mailcode_Commands_Command[] $commands
+     * @return Mailcode_Collection
+     */
     public function addCommands(array $commands) : Mailcode_Collection
     {
         foreach($commands as $command)
@@ -234,11 +234,11 @@ 
         return false;
     }
     
-   /**
-    * Retrieves only show variable commands in the collection, if any.
-    * 
-    * @return \Mailcode\Mailcode_Commands_Command_ShowVariable[]
-    */
+    /**
+     * Retrieves only show variable commands in the collection, if any.
+     * 
+     * @return \Mailcode\Mailcode_Commands_Command_ShowVariable[]
+     */
     public function getShowVariableCommands()
     {
         $result = array();

Spacing +12 added lines, -12 removed lines

@@ -99,7 +99,7 @@ 
         
         $errors = $this->errors;
         
-        if(!$result->isValid())
+        if (!$result->isValid())
         {
             $errors[] = new Mailcode_Collection_Error_Message(
                 '',
@@ -115,7 +115,7 @@ 
     {
         $errors = $this->getErrors();
         
-        if(!empty($errors))
+        if (!empty($errors))
         {
             return array_shift($errors);
         }
@@ -157,11 +157,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;
             }
@@ -178,7 +178,7 @@ 
     */
     public function addCommands(array $commands) : Mailcode_Collection
     {
-        foreach($commands as $command)
+        foreach ($commands as $command)
         {
             $this->addCommand($command);
         }
@@ -197,7 +197,7 @@ 
     {
         $collection = new Mailcode_Variables_Collection_Regular();
         
-        foreach($this->commands as $command)
+        foreach ($this->commands as $command)
         {
             $collection->mergeWith($command->getVariables());
         }
@@ -207,7 +207,7 @@ 
     
     public function getValidationResult() : OperationResult
     {
-        if(isset($this->validationResult))
+        if (isset($this->validationResult))
         {
             return $this->validationResult;
         }
@@ -223,9 +223,9 @@ 
     {
         $errors = $this->getErrors();
         
-        foreach($errors as $error)
+        foreach ($errors as $error)
         {
-            if($error->getCode() === $code)
+            if ($error->getCode() === $code)
             {
                 return true;
             }
@@ -243,9 +243,9 @@ 
     {
         $result = array();
         
-        foreach($this->commands as $command)
+        foreach ($this->commands as $command)
         {
-            if($command instanceof Mailcode_Commands_Command_ShowVariable)
+            if ($command instanceof Mailcode_Commands_Command_ShowVariable)
             {
                 $result[] = $command;
             }
@@ -258,7 +258,7 @@ 
     {
         $commands = $this->getCommands();
         
-        if(!empty($commands))
+        if (!empty($commands))
         {
             return array_shift($commands);
         }

src/Mailcode/Parser/Statement/Info.php

Indentation +25 added lines, -25 removed lines

@@ -21,14 +21,14 @@ 
  */
 class Mailcode_Parser_Statement_Info
 {
-   /**
-    * @var Mailcode_Parser_Statement_Tokenizer
-    */
+    /**
+     * @var Mailcode_Parser_Statement_Tokenizer
+     */
     protected $tokenizer;
     
-   /**
-    * @var Mailcode_Parser_Statement_Tokenizer_Token[]
-    */
+    /**
+     * @var Mailcode_Parser_Statement_Tokenizer_Token[]
+     */
     protected $tokens = array();
     
     public function __construct(Mailcode_Parser_Statement_Tokenizer $tokenizer)
@@ -37,11 +37,11 @@ 
         $this->tokens = $this->tokenizer->getTokens(); 
     }
     
-   /**
-    * Whether the whole statement is a variable being assigned a value.
-    * 
-    * @return bool
-    */
+    /**
+     * Whether the whole statement is a variable being assigned a value.
+     * 
+     * @return bool
+     */
     public function isVariableAssignment() : bool
     {
         $variable = $this->getVariableByIndex(0);
@@ -56,11 +56,11 @@ 
         return false;
     }
     
-   /**
-    * Whether the whole statement is a variable being compared to something.
-    * 
-    * @return bool
-    */
+    /**
+     * Whether the whole statement is a variable being compared to something.
+     * 
+     * @return bool
+     */
     public function isVariableComparison() : bool
     {
         $variable = $this->getVariableByIndex(0);
@@ -75,11 +75,11 @@ 
         return false;
     }
     
-   /**
-    * Retrieves all variables used in the statement.
-    * 
-    * @return \Mailcode\Mailcode_Variables_Variable[]
-    */
+    /**
+     * Retrieves all variables used in the statement.
+     * 
+     * @return \Mailcode\Mailcode_Variables_Variable[]
+     */
     public function getVariables()
     {
         $result = array();
@@ -158,10 +158,10 @@ 
         return isset($this->tokens[$index]);
     }
     
-   /**
-    * Retrieves all string literals that were found in the command.
-    * @return \Mailcode\Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral[]
-    */
+    /**
+     * Retrieves all string literals that were found in the command.
+     * @return \Mailcode\Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral[]
+     */
     public function getStringLiterals()
     {
         $result = array();

Spacing +11 added lines, -11 removed lines

@@ -48,7 +48,7 @@ 
         $operand = $this->getOperandByIndex(1);
         $value = $this->getTokenByIndex(2);
         
-        if($variable && $operand && $value && $operand->isAssignment())
+        if ($variable && $operand && $value && $operand->isAssignment())
         {
             return true;
         }
@@ -67,7 +67,7 @@ 
         $operand = $this->getOperandByIndex(1);
         $value = $this->getTokenByIndex(2);
         
-        if($variable && $operand && $value && $operand->isComparator())
+        if ($variable && $operand && $value && $operand->isComparator())
         {
             return true;
         }
@@ -84,9 +84,9 @@ 
     {
         $result = array();
         
-        foreach($this->tokens as $token)
+        foreach ($this->tokens as $token)
         {
-            if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Variable)
+            if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Variable)
             {
                 $result[] = $token->getVariable();
             }
@@ -99,7 +99,7 @@ 
     {
         $token = $this->getTokenByIndex($index);
         
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Variable)
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Variable)
         {
             return $token;
         }
@@ -111,7 +111,7 @@ 
     {
         $token = $this->getTokenByIndex($index);
         
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral)
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral)
         {
             return $token;
         }
@@ -123,7 +123,7 @@ 
     {
         $token = $this->getTokenByIndex($index);
         
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Keyword)
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Keyword)
         {
             return $token;
         }
@@ -135,7 +135,7 @@ 
     {
         $token = $this->getTokenByIndex($index);
         
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Operand)
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Operand)
         {
             return $token;
         }
@@ -145,7 +145,7 @@ 
     
     public function getTokenByIndex(int $index) : ?Mailcode_Parser_Statement_Tokenizer_Token
     {
-        if(isset($this->tokens[$index]))
+        if (isset($this->tokens[$index]))
         {
             return $this->tokens[$index];
         }
@@ -166,9 +166,9 @@ 
     {
         $result = array();
         
-        foreach($this->tokens as $token)
+        foreach ($this->tokens as $token)
         {
-            if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral)
+            if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral)
             {
                 $result[] = $token;
             }

src/Mailcode/Traits/Commands/IfVariable.php

Indentation +18 added lines, -18 removed lines

@@ -23,19 +23,19 @@ 
  */
 trait Mailcode_Traits_Commands_IfVariable
 {
-   /**
-    * @var Mailcode_Parser_Statement_Tokenizer_Token_Variable|NULL
-    */
+    /**
+     * @var Mailcode_Parser_Statement_Tokenizer_Token_Variable|NULL
+     */
     protected $variable;
     
-   /**
-    * @var Mailcode_Parser_Statement_Tokenizer_Token_Operand|NULL
-    */
+    /**
+     * @var Mailcode_Parser_Statement_Tokenizer_Token_Operand|NULL
+     */
     protected $comparator;
     
-   /**
-    * @var Mailcode_Parser_Statement_Tokenizer_Type_Value|NULL
-    */
+    /**
+     * @var Mailcode_Parser_Statement_Tokenizer_Type_Value|NULL
+     */
     protected $value;
     
     protected function getValidations() : array
@@ -123,11 +123,11 @@ 
         );
     }
     
-   /**
-    * Retrieves the variable being compared.
-    *
-    * @return Mailcode_Variables_Variable
-    */
+    /**
+     * Retrieves the variable being compared.
+     *
+     * @return Mailcode_Variables_Variable
+     */
     public function getVariable() : Mailcode_Variables_Variable
     {
         if(isset($this->variable))
@@ -156,10 +156,10 @@ 
         return $this->comparator->getOperand();
     }
     
-   /**
-    * Retrieves the unquoted value 
-    * @return string
-    */
+    /**
+     * Retrieves the unquoted value 
+     * @return string
+     */
     public function getValue() : string
     {
         return $this->value->getValue();

Spacing +7 added lines, -7 removed lines

@@ -53,7 +53,7 @@ 
         
         $var = $info->getVariableByIndex(0);
         
-        if($var)
+        if ($var)
         {
             $this->variable = $var;
             return;
@@ -71,7 +71,7 @@ 
         
         $operand = $info->getOperandByIndex(1);
         
-        if(!$operand)
+        if (!$operand)
         {
             $this->validationResult->makeError(
                 t('No operand sign after the variable name.'),
@@ -81,7 +81,7 @@ 
             return;
         }
         
-        if(!$operand->isComparator())
+        if (!$operand->isComparator())
         {
             $this->validationResult->makeError(
                 t('The operand sign is not a comparison operand.'),
@@ -100,7 +100,7 @@ 
         
         $token = $info->getTokenByIndex(2);
         
-        if(!$token)
+        if (!$token)
         {
             $this->validationResult->makeError(
                 t('Nothing found after the comparison operand.'),
@@ -110,7 +110,7 @@ 
             return;
         }
         
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Type_Value)
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Type_Value)
         {
             $this->value = $token;
             
@@ -130,7 +130,7 @@ 
     */
     public function getVariable() : Mailcode_Variables_Variable
     {
-        if(isset($this->variable))
+        if (isset($this->variable))
         {
             return $this->variable->getVariable();
         }
@@ -144,7 +144,7 @@ 
     
     public function getComparator() : string
     {
-        if(!isset($this->comparator))
+        if (!isset($this->comparator))
         {
             throw new Mailcode_Exception(
                 'No comparator available',

src/Mailcode/Traits/Commands/IfContains.php

Indentation +11 added lines, -11 removed lines

@@ -28,14 +28,14 @@ 
      */
     protected $variable;
 
-   /**
-    * @var Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral|NULL
-    */
+    /**
+     * @var Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral|NULL
+     */
     protected $literal;
     
-   /**
-    * @var boolean
-    */
+    /**
+     * @var boolean
+     */
     protected $caseInsensitive = false;
     
     protected function getValidations() : array
@@ -131,11 +131,11 @@ 
         );
     }
 
-   /**
-    * Retrieves the variable being compared.
-    *
-    * @return Mailcode_Variables_Variable
-    */
+    /**
+     * Retrieves the variable being compared.
+     *
+     * @return Mailcode_Variables_Variable
+     */
     public function getVariable() : Mailcode_Variables_Variable
     {
         if(isset($this->variable))

Spacing +7 added lines, -7 removed lines

@@ -52,7 +52,7 @@ 
         
         $variable = $info->getVariableByIndex(0);
         
-        if($variable)
+        if ($variable)
         {
             $this->variable = $variable;
             return;
@@ -71,7 +71,7 @@ 
         // first variant: variable "Search term"
         $string = $info->getStringLiteralByIndex(1);
         
-        if($string)
+        if ($string)
         {
             $this->literal = $string;
             return;
@@ -87,7 +87,7 @@ 
         // second variant: variable insensitive: "Search term"
         $keyword = $info->getKeywordByIndex(1);
         
-        if(!$keyword)
+        if (!$keyword)
         {
             $this->validationResult->makeError(
                 t('Expected a search term or the %1$s keyword after the variable name.', 'insensitive:'),
@@ -97,7 +97,7 @@ 
             return;
         }
         
-        if($keyword->getKeyword() !== 'insensitive:')
+        if ($keyword->getKeyword() !== 'insensitive:')
         {
             $this->validationResult->makeError(
                 t('Invalid keyword %1$s.', $keyword->getKeyword()).' '.
@@ -119,7 +119,7 @@ 
         
         $string = $info->getStringLiteralByIndex(2);
         
-        if($string)
+        if ($string)
         {
             $this->literal = $string;
             return;
@@ -138,7 +138,7 @@ 
     */
     public function getVariable() : Mailcode_Variables_Variable
     {
-        if(isset($this->variable))
+        if (isset($this->variable))
         {
             return $this->variable->getVariable();
         }
@@ -157,7 +157,7 @@ 
     
     public function getSearchTerm() : string
     {
-        if(!isset($this->literal))
+        if (!isset($this->literal))
         {
             throw new Mailcode_Exception(
                 'No string literal available',