Mistralys / mailcode

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

Spacing +2 added lines, -2 removed lines

@@ -29,12 +29,12 @@
 
     protected function validateSyntax_params_empty() : void
     {
-        if(!$this->requiresParameters())
+        if (!$this->requiresParameters())
         {
             return;
         }
 
-        if(empty($this->paramsString))
+        if (empty($this->paramsString))
         {
             $this->validationResult->makeError(
                 t('Parameters have to be specified.'),

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

Spacing +2 added lines, -2 removed lines

@@ -35,14 +35,14 @@
 
     protected function validateSyntax_type_supported() : void
     {
-        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).' '.

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

Spacing +2 added lines, -2 removed lines

@@ -31,14 +31,14 @@
 
     protected function validateSyntax_params_keywords() : void
     {
-        if(!$this->supportsLogicKeywords())
+        if (!$this->supportsLogicKeywords())
         {
             return;
         }
 
         $this->logicKeywords = new Mailcode_Commands_LogicKeywords($this, $this->paramsString);
 
-        if(!$this->logicKeywords->isValid())
+        if (!$this->logicKeywords->isValid())
         {
             $this->validationResult->makeError(
                 t('Invalid parameters:').' '.$this->logicKeywords->getErrorMessage(),

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

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 
     protected function validateSyntax_type_unsupported() : void
     {
-        if($this->supportsType() || empty($this->type))
+        if ($this->supportsType() || empty($this->type))
         {
             return;
         }

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

Spacing +3 added lines, -3 removed lines

@@ -20,11 +20,11 @@ 
  */
 class Mailcode_Factory_CommandSets_Set_Set extends Mailcode_Factory_CommandSets_Set
 {
-    public function var(string $variableName, string $value, bool $quoteValue=true) : Mailcode_Commands_Command_SetVariable
+    public function var(string $variableName, string $value, bool $quoteValue = true) : Mailcode_Commands_Command_SetVariable
     {
         $variableName = $this->instantiator->filterVariableName($variableName);
         
-        if($quoteValue)
+        if ($quoteValue)
         {
             $value = $this->instantiator->quoteString($value);
         }
@@ -40,7 +40,7 @@ 
         
         $this->instantiator->checkCommand($cmd);
         
-        if($cmd instanceof Mailcode_Commands_Command_SetVariable)
+        if ($cmd instanceof Mailcode_Commands_Command_SetVariable)
         {
             return $cmd;
         }

src/Mailcode/Factory.php

Spacing +3 added lines, -3 removed lines

@@ -53,7 +53,7 @@ 
      *
      * @return Mailcode_Factory_CommandSets_Set_If
      */
-    public static function if() : Mailcode_Factory_CommandSets_Set_If
+    public static function if () : Mailcode_Factory_CommandSets_Set_If
     {
         return self::$commandSets->if();
     }
@@ -63,7 +63,7 @@ 
      *
      * @return Mailcode_Factory_CommandSets_Set_ElseIf
      */
-    public static function elseIf() : Mailcode_Factory_CommandSets_Set_ElseIf
+    public static function elseIf () : Mailcode_Factory_CommandSets_Set_ElseIf
     {
         return self::$commandSets->elseIf();
     }
@@ -113,7 +113,7 @@ 
 
     public static function init() : void
     {
-        if(!isset(self::$commandSets))
+        if (!isset(self::$commandSets))
         {
             self::$commandSets = new Mailcode_Factory_CommandSets();
         }

Indentation +19 added lines, -19 removed lines

@@ -28,8 +28,8 @@ 
     const URL_ENCODING_DECODE = 'decode';
 
     /**
-    * @var Mailcode_Factory_CommandSets
-    */
+     * @var Mailcode_Factory_CommandSets
+     */
     private static $commandSets;
 
     /**
@@ -83,33 +83,33 @@ 
     }
 
     /**
-    * Creates a renderer instance, which can be used to easily
-    * create and convert commands to strings.
-    * 
-    * @return Mailcode_Renderer
-    */
+     * Creates a renderer instance, which can be used to easily
+     * create and convert commands to strings.
+     * 
+     * @return Mailcode_Renderer
+     */
     public static function createRenderer() : Mailcode_Renderer
     {
         return new Mailcode_Renderer();
     }
     
-   /**
-    * Creates a printer instance, which works like the renderer,
-    * but outputs the generated strings to standard output.
-    * 
-    * @return Mailcode_Printer
-    */
+    /**
+     * Creates a printer instance, which works like the renderer,
+     * but outputs the generated strings to standard output.
+     * 
+     * @return Mailcode_Printer
+     */
     public static function createPrinter() : Mailcode_Printer
     {
         return new Mailcode_Printer();
     }
     
-   /**
-    * Gets/creates the global instance of the date format info
-    * class, used to handle date formatting aspects.
-    * 
-    * @return Mailcode_Date_FormatInfo
-    */
+    /**
+     * Gets/creates the global instance of the date format info
+     * class, used to handle date formatting aspects.
+     * 
+     * @return Mailcode_Date_FormatInfo
+     */
     public static function createDateInfo() : Mailcode_Date_FormatInfo
     {
         return Mailcode_Date_FormatInfo::getInstance();

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::show()->var($variableName));

Spacing +26 added lines, -26 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::show()->snippet($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::set()->var($variableName, $value, $quoteValue));
     }
@@ -65,79 +65,79 @@ 
         return $this->command2string(Mailcode_Factory::set()->varString($variableName, $value));
     }
     
-    public function if(string $condition, string $type='') : string
+    public function if (string $condition, string $type = '') : string
     {
-        return $this->command2string(Mailcode_Factory::if()->if($condition, $type));
+        return $this->command2string(Mailcode_Factory::if ()->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::if()->var($variable, $operand, $value, $quoteValue));
+        return $this->command2string(Mailcode_Factory::if ()->var($variable, $operand, $value, $quoteValue));
     }
 
     public function ifVarString(string $variable, string $operand, string $value) : string
     {
-        return $this->command2string(Mailcode_Factory::if()->varString($variable, $operand, $value));
+        return $this->command2string(Mailcode_Factory::if ()->varString($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::if()->varEquals($variable, $value, $quoteValue));
+        return $this->command2string(Mailcode_Factory::if ()->varEquals($variable, $value, $quoteValue));
     }
 
     public function ifVarEqualsString(string $variable, string $value) : string
     {
-        return $this->command2string(Mailcode_Factory::if()->varEqualsString($variable, $value));
+        return $this->command2string(Mailcode_Factory::if ()->varEqualsString($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::if()->varNotEquals($variable, $value, $quoteValue));
+        return $this->command2string(Mailcode_Factory::if ()->varNotEquals($variable, $value, $quoteValue));
     }
 
     public function ifVarNotEqualsString(string $variable, string $value) : string
     {
-        return $this->command2string(Mailcode_Factory::if()->varNotEqualsString($variable, $value));
+        return $this->command2string(Mailcode_Factory::if ()->varNotEqualsString($variable, $value));
     }
     
-    public function elseIf(string $condition, string $type='') : string
+    public function elseIf (string $condition, string $type = '') : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf()->elseIf($condition, $type));
+        return $this->command2string(Mailcode_Factory::elseIf ()->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::elseIf()->var($variable, $operand, $value, $quoteValue));
+        return $this->command2string(Mailcode_Factory::elseIf ()->var($variable, $operand, $value, $quoteValue));
     }
 
     public function elseIfVarString(string $variable, string $operand, string $value) : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf()->varString($variable, $operand, $value));
+        return $this->command2string(Mailcode_Factory::elseIf ()->varString($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::elseIf()->varEquals($variable, $value, $quoteValue));
+        return $this->command2string(Mailcode_Factory::elseIf ()->varEquals($variable, $value, $quoteValue));
     }
 
     public function elseIfVarEqualsString(string $variable, string $value) : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf()->varEqualsString($variable, $value));
+        return $this->command2string(Mailcode_Factory::elseIf ()->varEqualsString($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::elseIf()->varNotEquals($variable, $value, $quoteValue));
+        return $this->command2string(Mailcode_Factory::elseIf ()->varNotEquals($variable, $value, $quoteValue));
     }
 
     public function elseIfVarNotEqualsString(string $variable, string $value) : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf()->varNotEqualsString($variable, $value));
+        return $this->command2string(Mailcode_Factory::elseIf ()->varNotEqualsString($variable, $value));
     }
     
     public function else() : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf()->else());
+        return $this->command2string(Mailcode_Factory::elseIf ()->else());
     }
     
     public function end() : string
@@ -152,7 +152,7 @@ 
     
     protected function command2string(Mailcode_Commands_Command $command) : string
     {
-        if($this->highlighted)
+        if ($this->highlighted)
         {
             return $command->getHighlighted();
         }

Braces +45 added lines, -15 removed lines

@@ -67,78 +67,108 @@
     
     public function if(string $condition, string $type='') : string
     {
-        return $this->command2string(Mailcode_Factory::if()->if($condition, $type));
+        return $this->command2string(Mailcode_Factory::if() {
+            ->if($condition, $type));
+        }
     }
     
     public function ifVar(string $variable, string $operand, string $value, bool $quoteValue=false) : string
     {
-        return $this->command2string(Mailcode_Factory::if()->var($variable, $operand, $value, $quoteValue));
+        return $this->command2string(Mailcode_Factory::if() {
+            ->var($variable, $operand, $value, $quoteValue));
+        }
     }
 
     public function ifVarString(string $variable, string $operand, string $value) : string
     {
-        return $this->command2string(Mailcode_Factory::if()->varString($variable, $operand, $value));
+        return $this->command2string(Mailcode_Factory::if() {
+            ->varString($variable, $operand, $value));
+        }
     }
     
     public function ifVarEquals(string $variable, string $value, bool $quoteValue=false) : string
     {
-        return $this->command2string(Mailcode_Factory::if()->varEquals($variable, $value, $quoteValue));
+        return $this->command2string(Mailcode_Factory::if() {
+            ->varEquals($variable, $value, $quoteValue));
+        }
     }
 
     public function ifVarEqualsString(string $variable, string $value) : string
     {
-        return $this->command2string(Mailcode_Factory::if()->varEqualsString($variable, $value));
+        return $this->command2string(Mailcode_Factory::if() {
+            ->varEqualsString($variable, $value));
+        }
     }
     
     public function ifVarNotEquals(string $variable, string $value, bool $quoteValue=false) : string
     {
-        return $this->command2string(Mailcode_Factory::if()->varNotEquals($variable, $value, $quoteValue));
+        return $this->command2string(Mailcode_Factory::if() {
+            ->varNotEquals($variable, $value, $quoteValue));
+        }
     }
 
     public function ifVarNotEqualsString(string $variable, string $value) : string
     {
-        return $this->command2string(Mailcode_Factory::if()->varNotEqualsString($variable, $value));
+        return $this->command2string(Mailcode_Factory::if() {
+            ->varNotEqualsString($variable, $value));
+        }
     }
     
     public function elseIf(string $condition, string $type='') : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf()->elseIf($condition, $type));
+        return $this->command2string(Mailcode_Factory::elseIf() {
+            ->elseIf($condition, $type));
+        }
     }
     
     public function elseIfVar(string $variable, string $operand, string $value, bool $quoteValue=false) : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf()->var($variable, $operand, $value, $quoteValue));
+        return $this->command2string(Mailcode_Factory::elseIf() {
+            ->var($variable, $operand, $value, $quoteValue));
+        }
     }
 
     public function elseIfVarString(string $variable, string $operand, string $value) : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf()->varString($variable, $operand, $value));
+        return $this->command2string(Mailcode_Factory::elseIf() {
+            ->varString($variable, $operand, $value));
+        }
     }
     
     public function elseIfVarEquals(string $variable, string $value, bool $quoteValue=false) : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf()->varEquals($variable, $value, $quoteValue));
+        return $this->command2string(Mailcode_Factory::elseIf() {
+            ->varEquals($variable, $value, $quoteValue));
+        }
     }
 
     public function elseIfVarEqualsString(string $variable, string $value) : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf()->varEqualsString($variable, $value));
+        return $this->command2string(Mailcode_Factory::elseIf() {
+            ->varEqualsString($variable, $value));
+        }
     }
     
     public function elseIfVarNotEquals(string $variable, string $value, bool $quoteValue=false) : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf()->varNotEquals($variable, $value, $quoteValue));
+        return $this->command2string(Mailcode_Factory::elseIf() {
+            ->varNotEquals($variable, $value, $quoteValue));
+        }
     }
 
     public function elseIfVarNotEqualsString(string $variable, string $value) : string
     {
-        return $this->command2string(Mailcode_Factory::elseIf()->varNotEqualsString($variable, $value));
+        return $this->command2string(Mailcode_Factory::elseIf() {
+            ->varNotEqualsString($variable, $value));
+        }
     }
     
-    public function else() : string
+    public function else {
+        () : string
     {
         return $this->command2string(Mailcode_Factory::elseIf()->else());
     }
+    }
     
     public function end() : string
     {

src/Mailcode/Translator/Syntax/ApacheVelocity/Contains/StatementBuilder.php

Indentation +22 added lines, -22 removed lines

@@ -176,27 +176,27 @@
      * @return array<string,string>
      * @throws Mailcode_Exception
      */
-     private function parseVarName() : array
-     {
-         $tokens = explode('.', ltrim($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('.', ltrim($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
+            );
+        }
 }
 

Braces +1 added lines, -2 removed lines

@@ -120,8 +120,7 @@
         if($this->isList())
         {
             $command = $this->renderListCommand($searchTerm);
-        }
-        else
+        } else
         {
             $command = $this->renderRegularCommand($searchTerm);
         }

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('.', ltrim($this->variable->getFullName(), '$'));
 
-         if(count($tokens) === 2)
+         if (count($tokens) === 2)
          {
              return array(
                  'path' => $tokens[0],

src/Mailcode/Translator/Syntax/ApacheVelocity.php

Indentation +12 added lines, -12 removed lines

@@ -20,9 +20,9 @@ 
  */
 abstract class Mailcode_Translator_Syntax_ApacheVelocity extends Mailcode_Translator_Command
 {
-   /**
-    * @var string[]
-    */
+    /**
+     * @var string[]
+     */
     private $regexSpecialChars = array(
         '?',
         '.',
@@ -47,15 +47,15 @@ 
     }
 
     /**
-    * Filters the string for use in an Apache Velocity (Java)
-    * regex string: escapes all special characters.
-    *
-    * Velocity does its own escaping, so no need to escape special
-    * characters as if they were a javascript string.
-    * 
-    * @param string $string
-    * @return string
-    */
+     * Filters the string for use in an Apache Velocity (Java)
+     * regex string: escapes all special characters.
+     *
+     * Velocity does its own escaping, so no need to escape special
+     * characters as if they were a javascript string.
+     * 
+     * @param string $string
+     * @return string
+     */
     public function filterRegexString(string $string) : string
     {
         // Special case: previously escaped quotes. 

Spacing +3 added lines, -3 removed lines

@@ -69,7 +69,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);
         }
@@ -83,7 +83,7 @@ 
 
     protected function addURLEncoding(Mailcode_Commands_Command $command, string $statement) : string
     {
-        if($command->isURLEncoded())
+        if ($command->isURLEncoded())
         {
             return sprintf(
                 '${esc.url($%s)}',
@@ -91,7 +91,7 @@ 
             );
         }
 
-        if($command->isURLDecoded())
+        if ($command->isURLDecoded())
         {
             return sprintf(
                 '${esc.unurl($%s)}',