Mistralys / mailcode

src/Mailcode/Translator.php

Spacing +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
      */
     public function createSyntax(string $name) : Mailcode_Translator_Syntax
     {
-        if($this->syntaxExists($name))
+        if ($this->syntaxExists($name))
         {
             return new Mailcode_Translator_Syntax($name);
         }
@@ -59,7 +59,7 @@ 
         $names = $this->getSyntaxNames();
         $result = array();
 
-        foreach($names as $name)
+        foreach ($names as $name)
         {
             $result[] = $this->createSyntax($name);
         }

src/Mailcode/Parser/Statement.php

Indentation +13 added lines, -13 removed lines

@@ -26,24 +26,24 @@ 
     const VALIDATION_EMPTY = 48801;
     const VALIDATION_UNQUOTED_STRING_LITERALS = 48802;
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $statement;
     
-   /**
-    * @var OperationResult
-    */
+    /**
+     * @var OperationResult
+     */
     protected $result;
     
-   /**
-    * @var Mailcode_Parser_Statement_Tokenizer
-    */
+    /**
+     * @var Mailcode_Parser_Statement_Tokenizer
+     */
     protected $tokenizer;
     
-   /**
-    * @var Mailcode_Parser_Statement_Info|NULL
-    */
+    /**
+     * @var Mailcode_Parser_Statement_Info|NULL
+     */
     protected $info;
 
     /**
@@ -112,7 +112,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 +5 added lines, -5 removed lines

@@ -51,7 +51,7 @@ 
      */
     protected $freeform = false;
 
-    public function __construct(string $statement, bool $freeform=false)
+    public function __construct(string $statement, bool $freeform = false)
     {
         $this->statement = $statement;
         $this->result = new OperationResult($this);
@@ -80,7 +80,7 @@ 
     
     public function getInfo() : Mailcode_Parser_Statement_Info
     {
-        if($this->info instanceof Mailcode_Parser_Statement_Info)
+        if ($this->info instanceof Mailcode_Parser_Statement_Info)
         {
             return $this->info; 
         }
@@ -92,12 +92,12 @@ 
     
     protected function validate() : void
     {
-        if($this->freeform)
+        if ($this->freeform)
         {
             return;
         }
 
-        if(!$this->tokenizer->hasTokens())
+        if (!$this->tokenizer->hasTokens())
         {
             $this->result->makeError(
                 t('Empty statement'),
@@ -109,7 +109,7 @@ 
         
         $unknown = $this->tokenizer->getFirstUnknown();
         
-        if($unknown)
+        if ($unknown)
         {
             $this->result->makeError(
                t('Unquoted string literal found:').' ('.htmlspecialchars($unknown->getMatchedText()).')',

src/Mailcode/Commands/Command.php

Indentation +106 added lines, -106 removed lines

@@ -37,44 +37,44 @@ 
 
     const META_URL_ENCODING = 'url_encoding';
 
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $type = '';
 
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $paramsString = '';
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $matchedText = '';
 
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $hash = '';
     
-   /**
-    * @var OperationResult
-    */
+    /**
+     * @var OperationResult
+     */
     protected $validationResult = null;
     
-   /**
-    * @var \Mailcode\Mailcode
-    */
+    /**
+     * @var \Mailcode\Mailcode
+     */
     protected $mailcode;
     
-   /**
-    * @var \Mailcode\Mailcode_Parser_Statement
-    */
+    /**
+     * @var \Mailcode\Mailcode_Parser_Statement
+     */
     protected $params;
 
-   /**
-    * @var string[] 
-    */
+    /**
+     * @var string[] 
+     */
     protected $validations = array(
         'params_empty',
         'params_keywords',
@@ -83,24 +83,24 @@ 
         'type_unsupported'
     );
     
-   /**
-    * @var string
-    */
+    /**
+     * @var string
+     */
     protected $comment = '';
     
-   /**
-    * @var Mailcode_Commands_LogicKeywords|NULL
-    */
+    /**
+     * @var Mailcode_Commands_LogicKeywords|NULL
+     */
     protected $logicKeywords;
     
-   /**
-    * @var Mailcode_Parser_Statement_Validator
-    */
+    /**
+     * @var Mailcode_Parser_Statement_Validator
+     */
     protected $validator;
     
-   /**
-    * @var boolean
-    */
+    /**
+     * @var boolean
+     */
     private $validated = false;
 
     /**
@@ -130,13 +130,13 @@ 
         
     }
 
-   /**
-    * Sets the command's parent opening command, if any.
-    * NOTE: This is set automatically by the parser, and
-    * should not be called manually.
-    *
-    * @param Mailcode_Commands_Command $command
-    */
+    /**
+     * Sets the command's parent opening command, if any.
+     * NOTE: This is set automatically by the parser, and
+     * should not be called manually.
+     *
+     * @param Mailcode_Commands_Command $command
+     */
     public function setParent(Mailcode_Commands_Command $command) : void
     {
         $this->parent = $command;
@@ -152,9 +152,9 @@ 
         return $this->parent;
     }
     
-   /**
-    * @return string The ID of the command = the name of the command class file.
-    */
+    /**
+     * @return string The ID of the command = the name of the command class file.
+     */
     public function getID() : string
     {
         // account for commands with types: If_Variable should still return If.
@@ -163,14 +163,14 @@ 
         return array_shift($tokens);
     }
     
-   /**
-    * Sets an optional comment that is not used anywhere, but
-    * can be used by the application to track why a command is
-    * used somewhere. 
-    * 
-    * @param string $comment
-    * @return Mailcode_Commands_Command
-    */
+    /**
+     * Sets an optional comment that is not used anywhere, but
+     * can be used by the application to track why a command is
+     * used somewhere. 
+     * 
+     * @param string $comment
+     * @return Mailcode_Commands_Command
+     */
     public function setComment(string $comment) : Mailcode_Commands_Command
     {
         $this->comment = $comment;
@@ -178,34 +178,34 @@ 
         return $this;
     }
     
-   /**
-    * Retrieves the previously set comment, if any.
-    * 
-    * @return string
-    */
+    /**
+     * Retrieves the previously set comment, if any.
+     * 
+     * @return string
+     */
     public function getComment() : string
     {
         return $this->comment;
     }
     
-   /**
-    * Checks whether this is a dummy command, which is only
-    * used to access information on the command type. It cannot
-    * be used as an actual live command.
-    * 
-    * @return bool
-    */
+    /**
+     * Checks whether this is a dummy command, which is only
+     * used to access information on the command type. It cannot
+     * be used as an actual live command.
+     * 
+     * @return bool
+     */
     public function isDummy() : bool
     {
         return $this->type === '__dummy';
     }
     
-   /**
-    * Retrieves a hash of the actual matched command string,
-    * which is used in collections to detect duplicate commands.
-    * 
-    * @return string
-    */
+    /**
+     * Retrieves a hash of the actual matched command string,
+     * which is used in collections to detect duplicate commands.
+     * 
+     * @return string
+     */
     public function getHash() : string
     {
         $this->requireNonDummy();
@@ -307,9 +307,9 @@ 
         return $this->validationResult->isValid();
     }
     
-   /**
-    * @return string[]
-    */
+    /**
+     * @return string[]
+     */
     abstract protected function getValidations() : array;
     
     protected function validateSyntax_params_empty() : void
@@ -478,12 +478,12 @@ 
 
     abstract public function supportsURLEncoding() : bool;
     
-   /**
-    * Whether the command allows using logic keywords like "and:" or "or:"
-    * in the command parameters.
-    * 
-    * @return bool
-    */
+    /**
+     * Whether the command allows using logic keywords like "and:" or "or:"
+     * in the command parameters.
+     * 
+     * @return bool
+     */
     abstract public function supportsLogicKeywords() : bool;
     
     abstract public function generatesContent() : bool;
@@ -529,22 +529,22 @@ 
         return $normalizer->normalize();
     }
     
-   /**
-    * Retrieves the names of all the command's supported types: the part
-    * between the command name and the colon. Example: {command type: params}.
-    * 
-    * @return string[]
-    */
+    /**
+     * Retrieves the names of all the command's supported types: the part
+     * between the command name and the colon. Example: {command type: params}.
+     * 
+     * @return string[]
+     */
     public function getSupportedTypes() : array
     {
         return array();
     }
     
-   /**
-    * Retrieves all variable names used in the command.
-    * 
-    * @return Mailcode_Variables_Collection_Regular
-    */
+    /**
+     * Retrieves all variable names used in the command.
+     * 
+     * @return Mailcode_Variables_Collection_Regular
+     */
     public function getVariables() : Mailcode_Variables_Collection_Regular
     {
         return Mailcode::create()->findVariables($this->paramsString);
@@ -569,26 +569,26 @@ 
         );
     }
 
-   /**
-    * Sets a parameter for the translation backend. The backend can use
-    * these to allow command-specific configurations.
-    *
-    * @param string $name
-    * @param mixed $value
-    * @return $this
-    */
+    /**
+     * Sets a parameter for the translation backend. The backend can use
+     * these to allow command-specific configurations.
+     *
+     * @param string $name
+     * @param mixed $value
+     * @return $this
+     */
     public function setTranslationParam(string $name, $value)
     {
         $this->translationParams[$name] = $value;
         return $this;
     }
 
-   /**
-    * Retrieves a previously set translation parameter.
-    *
-    * @param string $name
-    * @return mixed
-    */
+    /**
+     * Retrieves a previously set translation parameter.
+     *
+     * @param string $name
+     * @return mixed
+     */
     public function getTranslationParam(string $name)
     {
         if(isset($this->translationParams[$name]))

Spacing +29 added lines, -29 removed lines

@@ -114,7 +114,7 @@ 
      */
     protected $parent = null;
 
-    public function __construct(string $type='', string $paramsString='', string $matchedText='')
+    public function __construct(string $type = '', string $paramsString = '', string $matchedText = '')
     {
         $this->type = $type;
         $this->paramsString = html_entity_decode($paramsString);
@@ -210,7 +210,7 @@ 
     {
         $this->requireNonDummy();
         
-        if($this->hash === '') {
+        if ($this->hash === '') {
             $this->hash = md5($this->matchedText);
         }
         
@@ -219,7 +219,7 @@ 
     
     protected function requireNonDummy() : void
     {
-        if(!$this->isDummy())
+        if (!$this->isDummy())
         {
             return;
         }
@@ -238,7 +238,7 @@ 
     
     protected function validate() : OperationResult
     {
-        if(!$this->validated)
+        if (!$this->validated)
         {
             $this->requireNonDummy();
             $this->validateSyntax();
@@ -251,7 +251,7 @@ 
     
     public function getValidationResult() :  OperationResult
     {
-        if(isset($this->validationResult)) 
+        if (isset($this->validationResult)) 
         {
             return $this->validationResult;
         }
@@ -267,10 +267,10 @@ 
     {
         $validations = $this->resolveValidations();
 
-        foreach($validations as $validation)
+        foreach ($validations as $validation)
         {
             // break off at the first validation issue
-            if(!$this->validateSyntaxMethod($validation))
+            if (!$this->validateSyntaxMethod($validation))
             {
                 return;
             }
@@ -289,7 +289,7 @@ 
     {
         $method = 'validateSyntax_'.$validation;
         
-        if(!method_exists($this, $method))
+        if (!method_exists($this, $method))
         {
             throw new Mailcode_Exception(
                 'Missing validation method ['.$validation.']',
@@ -314,12 +314,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.'),
@@ -331,14 +331,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(),
@@ -353,7 +353,7 @@ 
     
     protected function validateSyntax_params_parse() : void
     {
-        if(!$this->requiresParameters())
+        if (!$this->requiresParameters())
         {
             return;
         }
@@ -363,7 +363,7 @@ 
             $this->hasFreeformParameters()
         );
 
-        if(!$this->params->isValid())
+        if (!$this->params->isValid())
         {
             $error = $this->params->getValidationResult();
             
@@ -385,14 +385,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).' '.
@@ -406,7 +406,7 @@ 
     
     protected function validateSyntax_type_unsupported() : void
     {
-        if($this->supportsType() || empty($this->type))
+        if ($this->supportsType() || empty($this->type))
         {
             return;
         }
@@ -424,7 +424,7 @@ 
     
     public function getType() : string
     {
-        if($this->supportsType())
+        if ($this->supportsType())
         {
             return $this->type;
         }
@@ -444,7 +444,7 @@ 
     
     public function getHighlighted() : string
     {
-        if(!$this->isValid())
+        if (!$this->isValid())
         {
             return '';
         }
@@ -455,7 +455,7 @@ 
     
     public function getParamsString() : string
     {
-        if($this->requiresParameters())
+        if ($this->requiresParameters())
         {
             return $this->paramsString;
         }
@@ -492,22 +492,22 @@ 
     
     public final function getCommandType() : string
     {
-        if($this instanceof Mailcode_Commands_Command_Type_Closing)
+        if ($this instanceof Mailcode_Commands_Command_Type_Closing)
         {
             return 'Closing';
         }
         
-        if($this instanceof Mailcode_Commands_Command_Type_Opening)
+        if ($this instanceof Mailcode_Commands_Command_Type_Opening)
         {
             return 'Opening';
         }
         
-        if($this instanceof Mailcode_Commands_Command_Type_Sibling)
+        if ($this instanceof Mailcode_Commands_Command_Type_Sibling)
         {
             return 'Sibling';
         }
         
-        if($this instanceof Mailcode_Commands_Command_Type_Standalone)
+        if ($this instanceof Mailcode_Commands_Command_Type_Standalone)
         {
             return 'Standalone';
         }
@@ -557,7 +557,7 @@ 
     
     public function getLogicKeywords() : Mailcode_Commands_LogicKeywords
     {
-        if($this->supportsLogicKeywords() && isset($this->logicKeywords))
+        if ($this->supportsLogicKeywords() && isset($this->logicKeywords))
         {
             return $this->logicKeywords;
         }
@@ -591,7 +591,7 @@ 
     */
     public function getTranslationParam(string $name)
     {
-        if(isset($this->translationParams[$name]))
+        if (isset($this->translationParams[$name]))
         {
             return $this->translationParams[$name];
         }
@@ -603,7 +603,7 @@ 
      * @param bool $encoding
      * @return $this
      */
-    public function setURLEncoding(bool $encoding=true)
+    public function setURLEncoding(bool $encoding = true)
     {
         $this->requireURLEncoding();
 
@@ -619,7 +619,7 @@ 
      * @return $this
      * @throws Mailcode_Exception
      */
-    public function setURLDecoding(bool $decode=true)
+    public function setURLDecoding(bool $decode = true)
     {
         $this->requireURLEncoding();
 
@@ -630,7 +630,7 @@ 
 
     protected function requireURLEncoding() : void
     {
-        if($this->supportsURLEncoding()) {
+        if ($this->supportsURLEncoding()) {
             return;
         }
 

src/Mailcode/Commands/ShowBase.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
 
     protected function validateSyntax_urldeencode() : void
     {
-        if($this->isURLEncoded() && $this->getURLDecodeToken() !== null)
+        if ($this->isURLEncoded() && $this->getURLDecodeToken() !== null)
         {
             $this->validationResult->makeError(
                 t('Cannot enable URL decoding and encoding at the same time.'),

src/Mailcode/Commands/Command/ShowVariable.php

Spacing +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
         $tokens = $this->params->getInfo()->getTokens();
         $allowed = $this->resolveActiveTokens();
 
-        if(count($tokens) > count($allowed))
+        if (count($tokens) > count($allowed))
         {
             $this->validationResult->makeError(
                 t('Unknown parameters found:').' '.
@@ -69,13 +69,13 @@ 
         $allowed = array($this->getVariableToken());
 
         $token = $this->getURLEncodeToken();
-        if($token)
+        if ($token)
         {
             $allowed[] = $token;
         }
 
         $token = $this->getURLDecodeToken();
-        if($token)
+        if ($token)
         {
             $allowed[] = $token;
         }

src/Mailcode/Commands/Command/Code.php

Spacing +4 added lines, -4 removed lines

@@ -91,7 +91,7 @@ 
     {
         $lang = $this->params->getInfo()->getStringLiteralByIndex(0);
 
-        if($lang)
+        if ($lang)
         {
             $this->langToken = $lang;
             return;
@@ -115,7 +115,7 @@ 
     {
         // To keep PHPStan happy. If no token has been found, this
         // method will not be called.
-        if(!isset($this->langToken))
+        if (!isset($this->langToken))
         {
             return;
         }
@@ -123,7 +123,7 @@ 
         $name = $this->langToken->getText();
         $translator = new Mailcode_Translator();
 
-        if($translator->syntaxExists($name))
+        if ($translator->syntaxExists($name))
         {
             return;
         }
@@ -146,7 +146,7 @@ 
      */
     public function getSyntaxName() : string
     {
-        if(isset($this->langToken))
+        if (isset($this->langToken))
         {
             return $this->langToken->getText();
         }

src/Mailcode/Commands/Normalizer.php

Spacing +7 added lines, -7 removed lines

@@ -37,7 +37,7 @@ 
     
     public function normalize() : string
     {
-        if(!$this->command->isValid())
+        if (!$this->command->isValid())
         {
             return '';
         }
@@ -57,7 +57,7 @@ 
     
     private function addType() : void
     {
-        if(!$this->command->supportsType() || !$this->command->hasType())
+        if (!$this->command->supportsType() || !$this->command->hasType())
         {
             return;
         }
@@ -67,12 +67,12 @@ 
     
     private function addParams(Mailcode_Commands_Command $command) : void
     {
-        if(!$command->requiresParameters())
+        if (!$command->requiresParameters())
         {
             return;
         }
 
-        if($command->hasFreeformParameters())
+        if ($command->hasFreeformParameters())
         {
             $params = $command->getParams()->getStatementString();
         }
@@ -81,7 +81,7 @@ 
             $params = $command->getParams()->getNormalized();
         }
         
-        if(empty($params))
+        if (empty($params))
         {
             return;
         }
@@ -92,14 +92,14 @@ 
     
     private function addLogicKeywords() : void
     {
-        if(!$this->command->supportsLogicKeywords())
+        if (!$this->command->supportsLogicKeywords())
         {
             return;
         }
         
         $keywords = $this->command->getLogicKeywords()->getKeywords();
         
-        foreach($keywords as $keyword)
+        foreach ($keywords as $keyword)
         {
             $this->parts[] = ' ';
             $this->parts[] = $keyword->getKeywordString(); // e.g. "if variable"

Braces +1 added lines, -2 removed lines

@@ -75,8 +75,7 @@
         if($command->hasFreeformParameters())
         {
             $params = $command->getParams()->getStatementString();
-        }
-        else
+        } else
         {
             $params = $command->getParams()->getNormalized();
         }

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

Spacing +4 added lines, -4 removed lines

@@ -34,7 +34,7 @@ 
     
         $this->instantiator->checkCommand($cmd);
         
-        if($cmd instanceof Mailcode_Commands_Command_Comment)
+        if ($cmd instanceof Mailcode_Commands_Command_Comment)
         {
             return $cmd;
         }
@@ -42,7 +42,7 @@ 
         throw $this->instantiator->exceptionUnexpectedType('Comment', $cmd);
     }
     
-    public function for(string $sourceVariable, string $loopVariable) : Mailcode_Commands_Command_For
+    public function for (string $sourceVariable, string $loopVariable) : Mailcode_Commands_Command_For
     {
         $sourceVariable = '$'.ltrim($sourceVariable, '$');
         $loopVariable = '$'.ltrim($loopVariable, '$');
@@ -64,7 +64,7 @@ 
         
         $this->instantiator->checkCommand($cmd);
         
-        if($cmd instanceof Mailcode_Commands_Command_For)
+        if ($cmd instanceof Mailcode_Commands_Command_For)
         {
             return $cmd;
         }
@@ -89,7 +89,7 @@ 
 
         $this->instantiator->checkCommand($cmd);
 
-        if($cmd instanceof Mailcode_Commands_Command_Code)
+        if ($cmd instanceof Mailcode_Commands_Command_Code)
         {
             return $cmd;
         }

src/Mailcode/Interfaces/Commands/URLDecode.php

Spacing +1 added lines, -1 removed lines

@@ -12,5 +12,5 @@
      * @param bool $decode
      * @return $this
      */
-    public function setURLDecoding(bool $decode=true);
+    public function setURLDecoding(bool $decode = true);
 }