View GitHub Repository mailcode

Schedule Inspection
Subscribe
Passed
Push — master ( 1b5df6...213867 )
by Sebastian
13:22
created
Status
Category
Toggle Patches
src/Mailcode/Parser/Statement/Info.php 2 patches
Indentation   +25 added lines, -25 removed lines patch added patch discarded remove patch 
@@ -21,14 +21,14 @@  discard block
 
 block discarded – undo
21 21 
  */
22 22 
 class Mailcode_Parser_Statement_Info
23 23 
 {
24 
-   /**
25 
-    * @var Mailcode_Parser_Statement_Tokenizer
26 
-    */
24 
+    /**
25 
+     * @var Mailcode_Parser_Statement_Tokenizer
26 
+     */
27 27 
     protected $tokenizer;
28 28
29 
-   /**
30 
-    * @var Mailcode_Parser_Statement_Tokenizer_Token[]
31 
-    */
29 
+    /**
30 
+     * @var Mailcode_Parser_Statement_Tokenizer_Token[]
31 
+     */
32 32 
     protected $tokens = array();
33 33
34 34 
     public function __construct(Mailcode_Parser_Statement_Tokenizer $tokenizer)
 
@@ -37,11 +37,11 @@  discard block
 
 block discarded – undo
37 37 
         $this->tokens = $this->tokenizer->getTokens();
38 38 
     }
39 39
40 
-   /**
41 
-    * Whether the whole statement is a variable being assigned a value.
42 
-    *
43 
-    * @return bool
44 
-    */
40 
+    /**
41 
+     * Whether the whole statement is a variable being assigned a value.
42 
+     *
43 
+     * @return bool
44 
+     */
45 45 
     public function isVariableAssignment() : bool
46 46 
     {
47 47 
         $variable = $this->getVariableByIndex(0);
 
@@ -56,11 +56,11 @@  discard block
 
 block discarded – undo
56 56 
         return false;
57 57 
     }
58 58
59 
-   /**
60 
-    * Whether the whole statement is a variable being compared to something.
61 
-    *
62 
-    * @return bool
63 
-    */
59 
+    /**
60 
+     * Whether the whole statement is a variable being compared to something.
61 
+     *
62 
+     * @return bool
63 
+     */
64 64 
     public function isVariableComparison() : bool
65 65 
     {
66 66 
         $variable = $this->getVariableByIndex(0);
 
@@ -75,11 +75,11 @@  discard block
 
 block discarded – undo
75 75 
         return false;
76 76 
     }
77 77
78 
-   /**
79 
-    * Retrieves all variables used in the statement.
80 
-    *
81 
-    * @return \Mailcode\Mailcode_Variables_Variable[]
82 
-    */
78 
+    /**
79 
+     * Retrieves all variables used in the statement.
80 
+     *
81 
+     * @return \Mailcode\Mailcode_Variables_Variable[]
82 
+     */
83 83 
     public function getVariables()
84 84 
     {
85 85 
         $result = array();
 
@@ -158,10 +158,10 @@  discard block
 
 block discarded – undo
158 158 
         return isset($this->tokens[$index]);
159 159 
     }
160 160
161 
-   /**
162 
-    * Retrieves all string literals that were found in the command.
163 
-    * @return \Mailcode\Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral[]
164 
-    */
161 
+    /**
162 
+     * Retrieves all string literals that were found in the command.
163 
+     * @return \Mailcode\Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral[]
164 
+     */
165 165 
     public function getStringLiterals()
166 166 
     {
167 167 
         $result = array();
Please login to merge, or discard this patch.
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch 
@@ -48,7 +48,7 @@  discard block
 
 block discarded – undo
48 48 
         $operand = $this->getOperandByIndex(1);
49 49 
         $value = $this->getTokenByIndex(2);
50 50
51 
-        if($variable && $operand && $value && $operand->isAssignment())
51 
+        if ($variable && $operand && $value && $operand->isAssignment())
52 52 
         {
53 53 
             return true;
54 54 
         }
 
@@ -67,7 +67,7 @@  discard block
 
 block discarded – undo
67 67 
         $operand = $this->getOperandByIndex(1);
68 68 
         $value = $this->getTokenByIndex(2);
69 69
70 
-        if($variable && $operand && $value && $operand->isComparator())
70 
+        if ($variable && $operand && $value && $operand->isComparator())
71 71 
         {
72 72 
             return true;
73 73 
         }
 
@@ -84,9 +84,9 @@  discard block
 
 block discarded – undo
84 84 
     {
85 85 
         $result = array();
86 86
87 
-        foreach($this->tokens as $token)
87 
+        foreach ($this->tokens as $token)
88 88 
         {
89 
-            if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Variable)
89 
+            if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Variable)
90 90 
             {
91 91 
                 $result[] = $token->getVariable();
92 92 
             }
 
@@ -99,7 +99,7 @@  discard block
 
 block discarded – undo
99 99 
     {
100 100 
         $token = $this->getTokenByIndex($index);
101 101
102 
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Variable)
102 
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Variable)
103 103 
         {
104 104 
             return $token;
105 105 
         }
 
@@ -111,7 +111,7 @@  discard block
 
 block discarded – undo
111 111 
     {
112 112 
         $token = $this->getTokenByIndex($index);
113 113
114 
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral)
114 
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral)
115 115 
         {
116 116 
             return $token;
117 117 
         }
 
@@ -123,7 +123,7 @@  discard block
 
 block discarded – undo
123 123 
     {
124 124 
         $token = $this->getTokenByIndex($index);
125 125
126 
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Keyword)
126 
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Keyword)
127 127 
         {
128 128 
             return $token;
129 129 
         }
 
@@ -135,7 +135,7 @@  discard block
 
 block discarded – undo
135 135 
     {
136 136 
         $token = $this->getTokenByIndex($index);
137 137
138 
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Operand)
138 
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_Operand)
139 139 
         {
140 140 
             return $token;
141 141 
         }
 
@@ -145,7 +145,7 @@  discard block
 
 block discarded – undo
145 145
146 146 
     public function getTokenByIndex(int $index) : ?Mailcode_Parser_Statement_Tokenizer_Token
147 147 
     {
148 
-        if(isset($this->tokens[$index]))
148 
+        if (isset($this->tokens[$index]))
149 149 
         {
150 150 
             return $this->tokens[$index];
151 151 
         }
 
@@ -166,9 +166,9 @@  discard block
 
 block discarded – undo
166 166 
     {
167 167 
         $result = array();
168 168
169 
-        foreach($this->tokens as $token)
169 
+        foreach ($this->tokens as $token)
170 170 
         {
171 
-            if($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral)
171 
+            if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral)
172 172 
             {
173 173 
                 $result[] = $token;
174 174 
             }
Please login to merge, or discard this patch.
src/Mailcode/Parser/Statement/Tokenizer.php 1 patch
Indentation   +51 added lines, -51 removed lines patch added patch discarded remove patch 
@@ -23,9 +23,9 @@  discard block
 
 block discarded – undo
23 23 
 {
24 24 
     const ERROR_TOKENIZE_METHOD_MISSING = 49801;
25 25
26 
-   /**
27 
-    * @var string[]
28 
-    */
26 
+    /**
27 
+     * @var string[]
28 
+     */
29 29 
     protected $operands = array(
30 30 
         '==',
31 31 
         '<=',
 
@@ -40,17 +40,17 @@  discard block
 
 block discarded – undo
40 40 
         '<'
41 41 
     );
42 42
43 
-   /**
44 
-    * @var string[]
45 
-    */
43 
+    /**
44 
+     * @var string[]
45 
+     */
46 46 
     protected $keywords = array(
47 47 
         'in:',
48 48 
         'insensitive:'
49 49 
     );
50 50
51 
-   /**
52 
-    * @var string
53 
-    */
51 
+    /**
52 
+     * @var string
53 
+     */
54 54 
     protected $delimiter = '§§';
55 55
56 56 
     /**
 
@@ -66,19 +66,19 @@  discard block
 
 block discarded – undo
66 66 
         'extract_tokens'
67 67 
     );
68 68
69 
-   /**
70 
-    * @var Mailcode_Parser_Statement
71 
-    */
69 
+    /**
70 
+     * @var Mailcode_Parser_Statement
71 
+     */
72 72 
     protected $statement;
73 73
74 
-   /**
75 
-    * @var string
76 
-    */
74 
+    /**
75 
+     * @var string
76 
+     */
77 77 
     protected $tokenized;
78 78
79 
-   /**
80 
-    * @var Mailcode_Parser_Statement_Tokenizer_Token[]
81 
-    */
79 
+    /**
80 
+     * @var Mailcode_Parser_Statement_Tokenizer_Token[]
81 
+     */
82 82 
     protected $tokensTemporary = array();
83 83
84 84 
     /**
 
@@ -86,9 +86,9 @@  discard block
 
 block discarded – undo
86 86 
      */
87 87 
     protected $tokensOrdered = array();
88 88
89 
-   /**
90 
-    * @var string[]
91 
-    */
89 
+    /**
90 
+     * @var string[]
91 
+     */
92 92 
     protected static $ids = array();
93 93
94 94 
     public function __construct(Mailcode_Parser_Statement $statement)
 
@@ -98,12 +98,12 @@  discard block
 
 block discarded – undo
98 98 
         $this->tokenize($statement->getStatementString());
99 99 
     }
100 100
101 
-   /**
102 
-    * Retrieves all tokens detected in the statement string, in
103 
-    * the order they were found.
104 
-    *
105 
-    * @return Mailcode_Parser_Statement_Tokenizer_Token[]
106 
-    */
101 
+    /**
102 
+     * Retrieves all tokens detected in the statement string, in
103 
+     * the order they were found.
104 
+     *
105 
+     * @return Mailcode_Parser_Statement_Tokenizer_Token[]
106 
+     */
107 107 
     public function getTokens()
108 108 
     {
109 109 
         return $this->tokensOrdered;
 
@@ -114,11 +114,11 @@  discard block
 
 block discarded – undo
114 114 
         return !empty($this->tokensOrdered);
115 115 
     }
116 116
117 
-   /**
118 
-    * Whether there were any unknown tokens in the statement.
119 
-    *
120 
-    * @return bool
121 
-    */
117 
+    /**
118 
+     * Whether there were any unknown tokens in the statement.
119 
+     *
120 
+     * @return bool
121 
+     */
122 122 
     public function hasUnknown() : bool
123 123 
     {
124 124 
         $unknown = $this->getUnknown();
 
@@ -126,11 +126,11 @@  discard block
 
 block discarded – undo
126 126 
         return !empty($unknown);
127 127 
     }
128 128
129 
-   /**
130 
-    * Retrieves all unknown content tokens, if any.
131 
-    *
132 
-    * @return \Mailcode\Mailcode_Parser_Statement_Tokenizer_Token_Unknown[]
133 
-    */
129 
+    /**
130 
+     * Retrieves all unknown content tokens, if any.
131 
+     *
132 
+     * @return \Mailcode\Mailcode_Parser_Statement_Tokenizer_Token_Unknown[]
133 
+     */
134 134 
     public function getUnknown()
135 135 
     {
136 136 
         $result = array();
 
@@ -200,13 +200,13 @@  discard block
 
 block discarded – undo
200 200 
         }
201 201 
     }
202 202
203 
-   /**
204 
-    * Registers a token to add in the statement string.
205 
-    *
206 
-    * @param string $type
207 
-    * @param string $matchedText
208 
-    * @param mixed $subject
209 
-    */
203 
+    /**
204 
+     * Registers a token to add in the statement string.
205 
+     *
206 
+     * @param string $type
207 
+     * @param string $matchedText
208 
+     * @param mixed $subject
209 
+     */
210 210 
     protected function registerToken(string $type, string $matchedText, $subject=null) : void
211 211 
     {
212 212 
         $tokenID = $this->generateID();
 
@@ -318,13 +318,13 @@  discard block
 
 block discarded – undo
318 318 
         }
319 319 
     }
320 320
321 
-   /**
322 
-    * Generates a unique alphabet-based ID without numbers
323 
-    * to use as token name, to avoid conflicts with the
324 
-    * numbers detection.
325 
-    *
326 
-    * @return string
327 
-    */
321 
+    /**
322 
+     * Generates a unique alphabet-based ID without numbers
323 
+     * to use as token name, to avoid conflicts with the
324 
+     * numbers detection.
325 
+     *
326 
+     * @return string
327 
+     */
328 328 
     protected function generateID() : string
329 329 
     {
330 330 
         static $alphas;
Please login to merge, or discard this patch.
src/Mailcode/Traits/Commands/IfVariable.php 2 patches
Indentation   +18 added lines, -18 removed lines patch added patch discarded remove patch 
@@ -23,19 +23,19 @@  discard block
 
 block discarded – undo
23 23 
  */
24 24 
 trait Mailcode_Traits_Commands_IfVariable
25 25 
 {
26 
-   /**
27 
-    * @var Mailcode_Parser_Statement_Tokenizer_Token_Variable|NULL
28 
-    */
26 
+    /**
27 
+     * @var Mailcode_Parser_Statement_Tokenizer_Token_Variable|NULL
28 
+     */
29 29 
     protected $variable;
30 30
31 
-   /**
32 
-    * @var Mailcode_Parser_Statement_Tokenizer_Token_Operand|NULL
33 
-    */
31 
+    /**
32 
+     * @var Mailcode_Parser_Statement_Tokenizer_Token_Operand|NULL
33 
+     */
34 34 
     protected $comparator;
35 35
36 
-   /**
37 
-    * @var Mailcode_Parser_Statement_Tokenizer_Type_Value|NULL
38 
-    */
36 
+    /**
37 
+     * @var Mailcode_Parser_Statement_Tokenizer_Type_Value|NULL
38 
+     */
39 39 
     protected $value;
40 40
41 41 
     protected function getValidations() : array
 
@@ -123,11 +123,11 @@  discard block
 
 block discarded – undo
123 123 
         );
124 124 
     }
125 125
126 
-   /**
127 
-    * Retrieves the variable being compared.
128 
-    *
129 
-    * @return Mailcode_Variables_Variable
130 
-    */
126 
+    /**
127 
+     * Retrieves the variable being compared.
128 
+     *
129 
+     * @return Mailcode_Variables_Variable
130 
+     */
131 131 
     public function getVariable() : Mailcode_Variables_Variable
132 132 
     {
133 133 
         if(isset($this->variable))
 
@@ -156,10 +156,10 @@  discard block
 
 block discarded – undo
156 156 
         return $this->comparator->getOperand();
157 157 
     }
158 158
159 
-   /**
160 
-    * Retrieves the unquoted value
161 
-    * @return string
162 
-    */
159 
+    /**
160 
+     * Retrieves the unquoted value
161 
+     * @return string
162 
+     */
163 163 
     public function getValue() : string
164 164 
     {
165 165 
         return $this->value->getValue();
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch 
@@ -53,7 +53,7 @@  discard block
 
 block discarded – undo
53 53
54 54 
         $var = $info->getVariableByIndex(0);
55 55
56 
-        if($var)
56 
+        if ($var)
57 57 
         {
58 58 
             $this->variable = $var;
59 59 
             return;
 
@@ -71,7 +71,7 @@  discard block
 
 block discarded – undo
71 71
72 72 
         $operand = $info->getOperandByIndex(1);
73 73
74 
-        if(!$operand)
74 
+        if (!$operand)
75 75 
         {
76 76 
             $this->validationResult->makeError(
77 77 
                 t('No operand sign after the variable name.'),
 
@@ -81,7 +81,7 @@  discard block
 
 block discarded – undo
81 81 
             return;
82 82 
         }
83 83
84 
-        if(!$operand->isComparator())
84 
+        if (!$operand->isComparator())
85 85 
         {
86 86 
             $this->validationResult->makeError(
87 87 
                 t('The operand sign is not a comparison operand.'),
 
@@ -100,7 +100,7 @@  discard block
 
 block discarded – undo
100 100
101 101 
         $token = $info->getTokenByIndex(2);
102 102
103 
-        if(!$token)
103 
+        if (!$token)
104 104 
         {
105 105 
             $this->validationResult->makeError(
106 106 
                 t('Nothing found after the comparison operand.'),
 
@@ -110,7 +110,7 @@  discard block
 
 block discarded – undo
110 110 
             return;
111 111 
         }
112 112
113 
-        if($token instanceof Mailcode_Parser_Statement_Tokenizer_Type_Value)
113 
+        if ($token instanceof Mailcode_Parser_Statement_Tokenizer_Type_Value)
114 114 
         {
115 115 
             $this->value = $token;
116 116 
             
@@ -130,7 +130,7 @@  discard block
 
 block discarded – undo
130 130 
     */
131 131 
     public function getVariable() : Mailcode_Variables_Variable
132 132 
     {
133 
-        if(isset($this->variable))
133 
+        if (isset($this->variable))
134 134 
         {
135 135 
             return $this->variable->getVariable();
136 136 
         }
 
@@ -144,7 +144,7 @@  discard block
 
 block discarded – undo
144 144
145 145 
     public function getComparator() : string
146 146 
     {
147 
-        if(!isset($this->comparator))
147 
+        if (!isset($this->comparator))
148 148 
         {
149 149 
             throw new Mailcode_Exception(
150 150 
                 'No comparator available',
Please login to merge, or discard this patch.
src/Mailcode/Traits/Commands/IfContains.php 2 patches
Indentation   +11 added lines, -11 removed lines patch added patch discarded remove patch 
@@ -28,14 +28,14 @@  discard block
 
 block discarded – undo
28 28 
      */
29 29 
     protected $variable;
30 30
31 
-   /**
32 
-    * @var Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral|NULL
33 
-    */
31 
+    /**
32 
+     * @var Mailcode_Parser_Statement_Tokenizer_Token_StringLiteral|NULL
33 
+     */
34 34 
     protected $literal;
35 35
36 
-   /**
37 
-    * @var boolean
38 
-    */
36 
+    /**
37 
+     * @var boolean
38 
+     */
39 39 
     protected $caseInsensitive = false;
40 40
41 41 
     protected function getValidations() : array
 
@@ -131,11 +131,11 @@  discard block
 
 block discarded – undo
131 131 
         );
132 132 
     }
133 133
134 
-   /**
135 
-    * Retrieves the variable being compared.
136 
-    *
137 
-    * @return Mailcode_Variables_Variable
138 
-    */
134 
+    /**
135 
+     * Retrieves the variable being compared.
136 
+     *
137 
+     * @return Mailcode_Variables_Variable
138 
+     */
139 139 
     public function getVariable() : Mailcode_Variables_Variable
140 140 
     {
141 141 
         if(isset($this->variable))
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch 
@@ -52,7 +52,7 @@  discard block
 
 block discarded – undo
52 52
53 53 
         $variable = $info->getVariableByIndex(0);
54 54
55 
-        if($variable)
55 
+        if ($variable)
56 56 
         {
57 57 
             $this->variable = $variable;
58 58 
             return;
 
@@ -71,7 +71,7 @@  discard block
 
 block discarded – undo
71 71 
         // first variant: variable "Search term"
72 72 
         $string = $info->getStringLiteralByIndex(1);
73 73
74 
-        if($string)
74 
+        if ($string)
75 75 
         {
76 76 
             $this->literal = $string;
77 77 
             return;
 
@@ -87,7 +87,7 @@  discard block
 
 block discarded – undo
87 87 
         // second variant: variable insensitive: "Search term"
88 88 
         $keyword = $info->getKeywordByIndex(1);
89 89
90 
-        if(!$keyword)
90 
+        if (!$keyword)
91 91 
         {
92 92 
             $this->validationResult->makeError(
93 93 
                 t('Expected a search term or the %1$s keyword after the variable name.', 'insensitive:'),
 
@@ -97,7 +97,7 @@  discard block
 
 block discarded – undo
97 97 
             return;
98 98 
         }
99 99
100 
-        if($keyword->getKeyword() !== 'insensitive:')
100 
+        if ($keyword->getKeyword() !== 'insensitive:')
101 101 
         {
102 102 
             $this->validationResult->makeError(
103 103 
                 t('Invalid keyword %1$s.', $keyword->getKeyword()).' '.
 
@@ -119,7 +119,7 @@  discard block
 
 block discarded – undo
119 119
120 120 
         $string = $info->getStringLiteralByIndex(2);
121 121
122 
-        if($string)
122 
+        if ($string)
123 123 
         {
124 124 
             $this->literal = $string;
125 125 
             return;
 
@@ -138,7 +138,7 @@  discard block
 
 block discarded – undo
138 138 
     */
139 139 
     public function getVariable() : Mailcode_Variables_Variable
140 140 
     {
141 
-        if(isset($this->variable))
141 
+        if (isset($this->variable))
142 142 
         {
143 143 
             return $this->variable->getVariable();
144 144 
         }
 
@@ -157,7 +157,7 @@  discard block
 
 block discarded – undo
157 157
158 158 
     public function getSearchTerm() : string
159 159 
     {
160 
-        if(!isset($this->literal))
160 
+        if (!isset($this->literal))
161 161 
         {
162 162 
             throw new Mailcode_Exception(
163 163 
                 'No string literal available',
Please login to merge, or discard this patch.
src/Mailcode/Commands.php 2 patches
Indentation   +46 added lines, -46 removed lines patch added patch discarded remove patch 
@@ -25,21 +25,21 @@  discard block
 
 block discarded – undo
25 25
26 26 
     const ERROR_COMMAND_DOES_NOT_EXIST = 45901;
27 27
28 
-   /**
29 
-    * @var Mailcode_Commands_Command[]
30 
-    */
28 
+    /**
29 
+     * @var Mailcode_Commands_Command[]
30 
+     */
31 31 
     private $commands = array();
32 32
33 
-   /**
34 
-    * @var string[]Mailcode_Commands_Command
35 
-    */
33 
+    /**
34 
+     * @var string[]Mailcode_Commands_Command
35 
+     */
36 36 
     private static $dummyCommands = array();
37 37
38 
-   /**
39 
-    * Retrieves a list of all available command IDs.
40 
-    *
41 
-    * @return string[]
42 
-    */
38 
+    /**
39 
+     * Retrieves a list of all available command IDs.
40 
+     *
41 
+     * @return string[]
42 
+     */
43 43 
     public function getIDs() : array
44 44 
     {
45 45 
         static $ids = array();
 
@@ -52,13 +52,13 @@  discard block
 
 block discarded – undo
52 52 
         return $ids;
53 53 
     }
54 54
55 
-   /**
56 
-    * Retrieves a list of all available commands, sorted by label.
57 
-    *
58 
-    * NOTE: These instances are only used for information purposes.
59 
-    *
60 
-    * @return Mailcode_Commands_Command[]
61 
-    */
55 
+    /**
56 
+     * Retrieves a list of all available commands, sorted by label.
57 
+     *
58 
+     * NOTE: These instances are only used for information purposes.
59 
+     *
60 
+     * @return Mailcode_Commands_Command[]
61 
+     */
62 62 
     public function getAll()
63 63 
     {
64 64 
         if(!empty($this->commands)) {
 
@@ -84,12 +84,12 @@  discard block
 
 block discarded – undo
84 84 
         return $result;
85 85 
     }
86 86
87 
-   /**
88 
-    * Gets an available command by its ID.
89 
-    *
90 
-    * @param string $id
91 
-    * @return Mailcode_Commands_Command
92 
-    */
87 
+    /**
88 
+     * Gets an available command by its ID.
89 
+     *
90 
+     * @param string $id
91 
+     * @return Mailcode_Commands_Command
92 
+     */
93 93 
     public function getByID(string $id) : Mailcode_Commands_Command
94 94 
     {
95 95 
         static $instances = array();
 
@@ -102,15 +102,15 @@  discard block
 
 block discarded – undo
102 102 
         return $instances[$id];
103 103 
     }
104 104
105 
-   /**
106 
-    * Retrieves the ID of a command by its name.
107 
-    *
108 
-    * @param string $name
109 
-    * @throws Mailcode_Exception
110 
-    * @return string
111 
-    *
112 
-    * @see Mailcode_Commands::ERROR_COMMAND_NAME_DOES_NOT_EXIST
113 
-    */
105 
+    /**
106 
+     * Retrieves the ID of a command by its name.
107 
+     *
108 
+     * @param string $name
109 
+     * @throws Mailcode_Exception
110 
+     * @return string
111 
+     *
112 
+     * @see Mailcode_Commands::ERROR_COMMAND_NAME_DOES_NOT_EXIST
113 
+     */
114 114 
     public function getIDByName(string $name) : string
115 115 
     {
116 116 
         $items = $this->getAll();
 
@@ -139,12 +139,12 @@  discard block
 
 block discarded – undo
139 139 
         return in_array($id, $ids);
140 140 
     }
141 141
142 
-   /**
143 
-    * Checks wether the specified name exists.
144 
-    *
145 
-    * @param string $name For example: "showvar".
146 
-    * @return bool
147 
-    */
142 
+    /**
143 
+     * Checks wether the specified name exists.
144 
+     *
145 
+     * @param string $name For example: "showvar".
146 
+     * @return bool
147 
+     */
148 148 
     public function nameExists(string $name) : bool
149 149 
     {
150 150 
         $items = $this->getAll();
 
@@ -198,13 +198,13 @@  discard block
 
 block discarded – undo
198 198 
         return $class;
199 199 
     }
200 200
201 
-   /**
202 
-    * Retrieves the dummy command of the specified type, which
203 
-    * is used to retrieve information on the command's capabilities.
204 
-    *
205 
-    * @param string $id
206 
-    * @return Mailcode_Commands_Command
207 
-    */
201 
+    /**
202 
+     * Retrieves the dummy command of the specified type, which
203 
+     * is used to retrieve information on the command's capabilities.
204 
+     *
205 
+     * @param string $id
206 
+     * @return Mailcode_Commands_Command
207 
+     */
208 208 
     private function getDummyCommand(string $id) : Mailcode_Commands_Command
209 209 
     {
210 210 
         if(!isset(self::$dummyCommands[$id]))
Please login to merge, or discard this patch.
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch 
@@ -44,7 +44,7 @@  discard block
 
 block discarded – undo
44 44 
     {
45 45 
         static $ids = array();
46 46
47 
-        if(empty($ids)) {
47 
+        if (empty($ids)) {
48 48 
             $ids = \AppUtils\FileHelper::createFileFinder(__DIR__.'/Commands/Command')
49 49 
             ->getPHPClassNames();
50 50 
         }
 
@@ -61,7 +61,7 @@  discard block
 
 block discarded – undo
61 61 
     */
62 62 
     public function getAll()
63 63 
     {
64 
-        if(!empty($this->commands)) {
64 
+        if (!empty($this->commands)) {
65 65 
             return $this->commands;
66 66 
         }
67 67 
         
@@ -69,7 +69,7 @@  discard block
 
 block discarded – undo
69 69
70 70 
         $result = array();
71 71
72 
-        foreach($ids as $id)
72 
+        foreach ($ids as $id)
73 73 
         {
74 74 
             $result[] = $this->getDummyCommand($id);
75 75 
         }
 
@@ -94,7 +94,7 @@  discard block
 
 block discarded – undo
94 94 
     {
95 95 
         static $instances = array();
96 96
97 
-        if(!isset($instances[$id]))
97 
+        if (!isset($instances[$id]))
98 98 
         {
99 99 
             $instances[$id] = $this->createCommand($id, '__dummy', '', '');
100 100 
         }
 
@@ -115,9 +115,9 @@  discard block
 
 block discarded – undo
115 115 
     {
116 116 
         $items = $this->getAll();
117 117
118 
-        foreach($items as $item)
118 
+        foreach ($items as $item)
119 119 
         {
120 
-            if($item->getName() === $name) {
120 
+            if ($item->getName() === $name) {
121 121 
                 return $item->getID();
122 122 
             }
123 123 
         }
 
@@ -149,9 +149,9 @@  discard block
 
 block discarded – undo
149 149 
     {
150 150 
         $items = $this->getAll();
151 151
152 
-        foreach($items as $item)
152 
+        foreach ($items as $item)
153 153 
         {
154 
-            if($item->getName() === $name) {
154 
+            if ($item->getName() === $name) {
155 155 
                 return true;
156 156 
             }
157 157 
         }
 
@@ -163,7 +163,7 @@  discard block
 
 block discarded – undo
163 163 
     {
164 164 
         $class = $this->resolveClassName($id, $type);
165 165
166 
-        if(!class_exists($class))
166 
+        if (!class_exists($class))
167 167 
         {
168 168 
             throw new Mailcode_Exception(
169 169 
                 'No such command',
 
@@ -185,9 +185,9 @@  discard block
 
 block discarded – undo
185 185
186 186 
         $dummy = $this->getDummyCommand($id);
187 187
188 
-        if($dummy->supportsType())
188 
+        if ($dummy->supportsType())
189 189 
         {
190 
-            if(empty($type))
190 
+            if (empty($type))
191 191 
             {
192 192 
                 $type = $dummy->getDefaultType();
193 193 
             }
 
@@ -207,7 +207,7 @@  discard block
 
 block discarded – undo
207 207 
     */
208 208 
     private function getDummyCommand(string $id) : Mailcode_Commands_Command
209 209 
     {
210 
-        if(!isset(self::$dummyCommands[$id]))
210 
+        if (!isset(self::$dummyCommands[$id]))
211 211 
         {
212 212 
             $class = 'Mailcode\Mailcode_Commands_Command_'.$id;
213 213 
             self::$dummyCommands[$id] = new $class('__dummy');
Please login to merge, or discard this patch.
src/Mailcode/Factory.php 3 patches
Indentation   +52 added lines, -52 removed lines patch added patch discarded remove patch 
@@ -24,17 +24,17 @@  discard block
 
 block discarded – undo
24 24
25 25 
     const ERROR_UNEXPECTED_COMMAND_TYPE = 50002;
26 26
27 
-   /**
28 
-    * @var Mailcode_Renderer
29 
-    */
27 
+    /**
28 
+     * @var Mailcode_Renderer
29 
+     */
30 30 
     protected static $renderer;
31 31
32 
-   /**
33 
-    * Creates a ShowVariable command.
34 
-    *
35 
-    * @param string $variableName A variable name, with or without the $ sign prepended.
36 
-    * @return Mailcode_Commands_Command_ShowVariable
37 
-    */
32 
+    /**
33 
+     * Creates a ShowVariable command.
34 
+     *
35 
+     * @param string $variableName A variable name, with or without the $ sign prepended.
36 
+     * @return Mailcode_Commands_Command_ShowVariable
37 
+     */
38 38 
     public static function showVar(string $variableName) : Mailcode_Commands_Command_ShowVariable
39 39 
     {
40 40 
         $variableName = self::_filterVariableName($variableName);
 
@@ -56,12 +56,12 @@  discard block
 
 block discarded – undo
56 56 
         throw self::_exceptionUnexpectedType('ShowVariable', $cmd);
57 57 
     }
58 58
59 
-   /**
60 
-    * Creates a ShowSnippet command.
61 
-    *
62 
-    * @param string $snippetName A snippet name, with or without the $ sign prepended.
63 
-    * @return Mailcode_Commands_Command_ShowSnippet
64 
-    */
59 
+    /**
60 
+     * Creates a ShowSnippet command.
61 
+     *
62 
+     * @param string $snippetName A snippet name, with or without the $ sign prepended.
63 
+     * @return Mailcode_Commands_Command_ShowSnippet
64 
+     */
65 65 
     public static function showSnippet(string $snippetName) : Mailcode_Commands_Command_ShowSnippet
66 66 
     {
67 67 
         $snippetName = self::_filterVariableName($snippetName);
 
@@ -83,17 +83,17 @@  discard block
 
 block discarded – undo
83 83 
         throw self::_exceptionUnexpectedType('ShowSnippet', $cmd);
84 84 
     }
85 85
86 
-   /**
87 
-    * Creates a SetVariable command.
88 
-    *
89 
-    * @param string $variableName A variable name, with or without the $ sign prepended.
90 
-    * @param string $value
91 
-    * @param bool $quoteValue Whether to treat the value as a string literal, and add quotes to it.
92 
-    * @return Mailcode_Commands_Command_SetVariable
93 
-    * @throws Mailcode_Factory_Exception
94 
-    *
95 
-    * @see Mailcode_Factory::ERROR_INVALID_COMMAND_CREATED
96 
-    */
86 
+    /**
87 
+     * Creates a SetVariable command.
88 
+     *
89 
+     * @param string $variableName A variable name, with or without the $ sign prepended.
90 
+     * @param string $value
91 
+     * @param bool $quoteValue Whether to treat the value as a string literal, and add quotes to it.
92 
+     * @return Mailcode_Commands_Command_SetVariable
93 
+     * @throws Mailcode_Factory_Exception
94 
+     *
95 
+     * @see Mailcode_Factory::ERROR_INVALID_COMMAND_CREATED
96 
+     */
97 97 
     public static function setVar(string $variableName, string $value, bool $quoteValue=true) : Mailcode_Commands_Command_SetVariable
98 98 
     {
99 99 
         $variableName = self::_filterVariableName($variableName);
 
@@ -122,14 +122,14 @@  discard block
 
 block discarded – undo
122 122 
         throw self::_exceptionUnexpectedType('SetVariable', $cmd);
123 123 
     }
124 124
125 
-   /**
126 
-    * Like setVar(), but treats the value as a string literal
127 
-    * and automatically adds quotes to it.
128 
-    *
129 
-    * @param string $variableName
130 
-    * @param string $value
131 
-    * @return Mailcode_Commands_Command_SetVariable
132 
-    */
125 
+    /**
126 
+     * Like setVar(), but treats the value as a string literal
127 
+     * and automatically adds quotes to it.
128 
+     *
129 
+     * @param string $variableName
130 
+     * @param string $value
131 
+     * @return Mailcode_Commands_Command_SetVariable
132 
+     */
133 133 
     public static function setVarString(string $variableName, string $value) : Mailcode_Commands_Command_SetVariable
134 134 
     {
135 135 
         return self::setVar($variableName, $value, true);
 
@@ -461,12 +461,12 @@  discard block
 
 block discarded – undo
461 461 
         return '$'.ltrim($name, '$');
462 462 
     }
463 463
464 
-   /**
465 
-    * Quotes a string literal: adds the quotes, and escapes any quotes already present in it.
466 
-    *
467 
-    * @param string $string
468 
-    * @return string
469 
-    */
464 
+    /**
465 
+     * Quotes a string literal: adds the quotes, and escapes any quotes already present in it.
466 
+     *
467 
+     * @param string $string
468 
+     * @return string
469 
+     */
470 470 
     protected static function _quoteString(string $string) : string
471 471 
     {
472 472 
         return '"'.str_replace('"', '\"', $string).'"';
 
@@ -499,23 +499,23 @@  discard block
 
 block discarded – undo
499 499 
         );
500 500 
     }
501 501
502 
-   /**
503 
-    * Creates a renderer instance, which can be used to easily
504 
-    * create and convert commands to strings.
505 
-    *
506 
-    * @return Mailcode_Renderer
507 
-    */
502 
+    /**
503 
+     * Creates a renderer instance, which can be used to easily
504 
+     * create and convert commands to strings.
505 
+     *
506 
+     * @return Mailcode_Renderer
507 
+     */
508 508 
     public static function createRenderer() : Mailcode_Renderer
509 509 
     {
510 510 
         return new Mailcode_Renderer();
511 511 
     }
512 512
513 
-   /**
514 
-    * Creates a printer instance, which works like the renderer,
515 
-    * but outputs the generated strings to standard output.
516 
-    *
517 
-    * @return Mailcode_Printer
518 
-    */
513 
+    /**
514 
+     * Creates a printer instance, which works like the renderer,
515 
+     * but outputs the generated strings to standard output.
516 
+     *
517 
+     * @return Mailcode_Printer
518 
+     */
519 519 
     public static function createPrinter() : Mailcode_Printer
520 520 
     {
521 521 
         return new Mailcode_Printer();
Please login to merge, or discard this patch.
Spacing   +42 added lines, -42 removed lines patch added patch discarded remove patch 
@@ -48,7 +48,7 @@  discard block
 
 block discarded – undo
48 48
49 49 
         self::_checkCommand($cmd);
50 50
51 
-        if($cmd instanceof Mailcode_Commands_Command_ShowVariable)
51 
+        if ($cmd instanceof Mailcode_Commands_Command_ShowVariable)
52 52 
         {
53 53 
             return $cmd;
54 54 
         }
 
@@ -75,7 +75,7 @@  discard block
 
 block discarded – undo
75 75
76 76 
         self::_checkCommand($cmd);
77 77
78 
-        if($cmd instanceof Mailcode_Commands_Command_ShowSnippet)
78 
+        if ($cmd instanceof Mailcode_Commands_Command_ShowSnippet)
79 79 
         {
80 80 
             return $cmd;
81 81 
         }
 
@@ -94,11 +94,11 @@  discard block
 
 block discarded – undo
94 94 
     *
95 95 
     * @see Mailcode_Factory::ERROR_INVALID_COMMAND_CREATED
96 96 
     */
97 
-    public static function setVar(string $variableName, string $value, bool $quoteValue=true) : Mailcode_Commands_Command_SetVariable
97 
+    public static function setVar(string $variableName, string $value, bool $quoteValue = true) : Mailcode_Commands_Command_SetVariable
98 98 
     {
99 99 
         $variableName = self::_filterVariableName($variableName);
100 100
101 
-        if($quoteValue)
101 
+        if ($quoteValue)
102 102 
         {
103 103 
             $value = self::_quoteString($value);
104 104 
         }
 
@@ -114,7 +114,7 @@  discard block
 
 block discarded – undo
114 114
115 115 
         self::_checkCommand($cmd);
116 116
117 
-        if($cmd instanceof Mailcode_Commands_Command_SetVariable)
117 
+        if ($cmd instanceof Mailcode_Commands_Command_SetVariable)
118 118 
         {
119 119 
             return $cmd;
120 120 
         }
 
@@ -149,7 +149,7 @@  discard block
 
 block discarded – undo
149 149
150 150 
         self::_checkCommand($cmd);
151 151
152 
-        if($cmd instanceof Mailcode_Commands_Command_Comment)
152 
+        if ($cmd instanceof Mailcode_Commands_Command_Comment)
153 153 
         {
154 154 
             return $cmd;
155 155 
         }
 
@@ -168,7 +168,7 @@  discard block
 
 block discarded – undo
168 168
169 169 
         self::_checkCommand($cmd);
170 170
171 
-        if($cmd instanceof Mailcode_Commands_Command_Else)
171 
+        if ($cmd instanceof Mailcode_Commands_Command_Else)
172 172 
         {
173 173 
             return $cmd;
174 174 
         }
 
@@ -187,7 +187,7 @@  discard block
 
 block discarded – undo
187 187
188 188 
         self::_checkCommand($cmd);
189 189
190 
-        if($cmd instanceof Mailcode_Commands_Command_End)
190 
+        if ($cmd instanceof Mailcode_Commands_Command_End)
191 191 
         {
192 192 
             return $cmd;
193 193 
         }
 
@@ -195,11 +195,11 @@  discard block
 
 block discarded – undo
195 195 
         throw self::_exceptionUnexpectedType('End', $cmd);
196 196 
     }
197 197
198 
-    protected static function _buildIf(string $ifType, string $params, string $type='') : Mailcode_Commands_IfBase
198 
+    protected static function _buildIf(string $ifType, string $params, string $type = '') : Mailcode_Commands_IfBase
199 199 
     {
200 200 
         $stringType = $type;
201 201
202 
-        if(!empty($type))
202 
+        if (!empty($type))
203 203 
         {
204 204 
             $stringType = ' '.$type;
205 205 
         }
 
@@ -218,7 +218,7 @@  discard block
 
 block discarded – undo
218 218
219 219 
         self::_checkCommand($command);
220 220
221 
-        if($command instanceof Mailcode_Commands_IfBase)
221 
+        if ($command instanceof Mailcode_Commands_IfBase)
222 222 
         {
223 223 
             return $command;
224 224 
         }
 
@@ -226,9 +226,9 @@  discard block
 
 block discarded – undo
226 226 
         throw self::_exceptionUnexpectedType('IfBase', $command);
227 227 
     }
228 228
229 
-    protected static function _buildIfVar(string $ifType, string $variable, string $operand, string $value, bool $quoteValue=false) : Mailcode_Commands_IfBase
229 
+    protected static function _buildIfVar(string $ifType, string $variable, string $operand, string $value, bool $quoteValue = false) : Mailcode_Commands_IfBase
230 230 
     {
231 
-        if($quoteValue)
231 
+        if ($quoteValue)
232 232 
         {
233 233 
             $value = self::_quoteString($value);
234 234 
         }
 
@@ -243,11 +243,11 @@  discard block
 
 block discarded – undo
243 243 
         return self::_buildIf($ifType, $condition, 'variable');
244 244 
     }
245 245
246 
-    public static function if(string $condition, string $type='') : Mailcode_Commands_Command_If
246 
+    public static function if (string $condition, string $type = '') : Mailcode_Commands_Command_If
247 247 
     {
248 248 
         $command = self::_buildIf('If', $condition, $type);
249 249
250 
-        if($command instanceof Mailcode_Commands_Command_If)
250 
+        if ($command instanceof Mailcode_Commands_Command_If)
251 251 
         {
252 252 
             return $command;
253 253 
         }
 
@@ -255,11 +255,11 @@  discard block
 
 block discarded – undo
255 255 
         throw self::_exceptionUnexpectedType('If', $command);
256 256 
     }
257 257
258 
-    public static function ifVar(string $variable, string $operand, string $value, bool $quoteValue=false) : Mailcode_Commands_Command_If_Variable
258 
+    public static function ifVar(string $variable, string $operand, string $value, bool $quoteValue = false) : Mailcode_Commands_Command_If_Variable
259 259 
     {
260 260 
         $command = self::_buildIfVar('If', $variable, $operand, $value, $quoteValue);
261 261
262 
-        if($command instanceof Mailcode_Commands_Command_If_Variable)
262 
+        if ($command instanceof Mailcode_Commands_Command_If_Variable)
263 263 
         {
264 264 
             return $command;
265 265 
         }
 
@@ -271,7 +271,7 @@  discard block
 
 block discarded – undo
271 271 
     {
272 272 
         $command = self::_buildIfVar('If', $variable, $operand, $value, true);
273 273
274 
-        if($command instanceof Mailcode_Commands_Command_If_Variable)
274 
+        if ($command instanceof Mailcode_Commands_Command_If_Variable)
275 275 
         {
276 276 
             return $command;
277 277 
         }
 
@@ -279,11 +279,11 @@  discard block
 
 block discarded – undo
279 279 
         throw self::_exceptionUnexpectedType('IfVarString', $command);
280 280 
     }
281 281
282 
-    public static function ifVarEquals(string $variable, string $value, bool $quoteValue=false) : Mailcode_Commands_Command_If_Variable
282 
+    public static function ifVarEquals(string $variable, string $value, bool $quoteValue = false) : Mailcode_Commands_Command_If_Variable
283 283 
     {
284 284 
         $command = self::_buildIfVar('If', $variable, '==', $value, $quoteValue);
285 285
286 
-        if($command instanceof Mailcode_Commands_Command_If_Variable)
286 
+        if ($command instanceof Mailcode_Commands_Command_If_Variable)
287 287 
         {
288 288 
             return $command;
289 289 
         }
 
@@ -295,7 +295,7 @@  discard block
 
 block discarded – undo
295 295 
     {
296 296 
         $command = self::_buildIfVar('If', $variable, '==', $value, true);
297 297
298 
-        if($command instanceof Mailcode_Commands_Command_If_Variable)
298 
+        if ($command instanceof Mailcode_Commands_Command_If_Variable)
299 299 
         {
300 300 
             return $command;
301 301 
         }
 
@@ -303,11 +303,11 @@  discard block
 
 block discarded – undo
303 303 
         throw self::_exceptionUnexpectedType('IfarEqualsString', $command);
304 304 
     }
305 305
306 
-    public static function ifVarNotEquals(string $variable, string $value, bool $quoteValue=false) : Mailcode_Commands_Command_If_Variable
306 
+    public static function ifVarNotEquals(string $variable, string $value, bool $quoteValue = false) : Mailcode_Commands_Command_If_Variable
307 307 
     {
308 308 
         $command = self::_buildIfVar('If', $variable, '!=', $value, $quoteValue);
309 309
310 
-        if($command instanceof Mailcode_Commands_Command_If_Variable)
310 
+        if ($command instanceof Mailcode_Commands_Command_If_Variable)
311 311 
         {
312 312 
             return $command;
313 313 
         }
 
@@ -319,7 +319,7 @@  discard block
 
 block discarded – undo
319 319 
     {
320 320 
         $command = self::_buildIfVar('If', $variable, '!=', $value, true);
321 321
322 
-        if($command instanceof Mailcode_Commands_Command_If_Variable)
322 
+        if ($command instanceof Mailcode_Commands_Command_If_Variable)
323 323 
         {
324 324 
             return $command;
325 325 
         }
 
@@ -327,11 +327,11 @@  discard block
 
 block discarded – undo
327 327 
         throw self::_exceptionUnexpectedType('IfVarNotEqualsString', $command);
328 328 
     }
329 329
330 
-    public static function elseIf(string $condition, string $type='') : Mailcode_Commands_Command_ElseIf
330 
+    public static function elseIf (string $condition, string $type = '') : Mailcode_Commands_Command_ElseIf
331 331 
     {
332 332 
         $command = self::_buildIf('ElseIf', $condition, $type);
333 333
334 
-        if($command instanceof Mailcode_Commands_Command_ElseIf)
334 
+        if ($command instanceof Mailcode_Commands_Command_ElseIf)
335 335 
         {
336 336 
             return $command;
337 337 
         }
 
@@ -339,11 +339,11 @@  discard block
 
 block discarded – undo
339 339 
         throw self::_exceptionUnexpectedType('ElseIf', $command);
340 340 
     }
341 341
342 
-    public static function elseIfVar(string $variable, string $operand, string $value, bool $quoteValue=false) : Mailcode_Commands_Command_ElseIf_Variable
342 
+    public static function elseIfVar(string $variable, string $operand, string $value, bool $quoteValue = false) : Mailcode_Commands_Command_ElseIf_Variable
343 343 
     {
344 344 
         $command = self::_buildIfVar('ElseIf', $variable, $operand, $value, $quoteValue);
345 345
346 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
346 
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
347 347 
         {
348 348 
             return $command;
349 349 
         }
 
@@ -355,7 +355,7 @@  discard block
 
 block discarded – undo
355 355 
     {
356 356 
         $command = self::_buildIfVar('ElseIf', $variable, $operand, $value, true);
357 357
358 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
358 
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
359 359 
         {
360 360 
             return $command;
361 361 
         }
 
@@ -363,11 +363,11 @@  discard block
 
 block discarded – undo
363 363 
         throw self::_exceptionUnexpectedType('ElseIfVarString', $command);
364 364 
     }
365 365
366 
-    public static function elseIfVarEquals(string $variable, string $value, bool $quoteValue=false) : Mailcode_Commands_Command_ElseIf_Variable
366 
+    public static function elseIfVarEquals(string $variable, string $value, bool $quoteValue = false) : Mailcode_Commands_Command_ElseIf_Variable
367 367 
     {
368 368 
         $command = self::_buildIfVar('ElseIf', $variable, '==', $value, $quoteValue);
369 369
370 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
370 
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
371 371 
         {
372 372 
             return $command;
373 373 
         }
 
@@ -379,7 +379,7 @@  discard block
 
 block discarded – undo
379 379 
     {
380 380 
         $command = self::_buildIfVar('ElseIf', $variable, '==', $value, true);
381 381
382 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
382 
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
383 383 
         {
384 384 
             return $command;
385 385 
         }
 
@@ -387,11 +387,11 @@  discard block
 
 block discarded – undo
387 387 
         throw self::_exceptionUnexpectedType('ElseIfVarEqualsString', $command);
388 388 
     }
389 389
390 
-    public static function elseIfVarNotEquals(string $variable, string $value, bool $quoteValue=false) : Mailcode_Commands_Command_ElseIf_Variable
390 
+    public static function elseIfVarNotEquals(string $variable, string $value, bool $quoteValue = false) : Mailcode_Commands_Command_ElseIf_Variable
391 391 
     {
392 392 
         $command = self::_buildIfVar('ElseIf', $variable, '!=', $value, $quoteValue);
393 393
394 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
394 
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
395 395 
         {
396 396 
             return $command;
397 397 
         }
 
@@ -403,7 +403,7 @@  discard block
 
 block discarded – undo
403 403 
     {
404 404 
         $command = self::_buildIfVar('ElseIf', $variable, '!=', $value, true);
405 405
406 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
406 
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
407 407 
         {
408 408 
             return $command;
409 409 
         }
 
@@ -411,11 +411,11 @@  discard block
 
 block discarded – undo
411 411 
         throw self::_exceptionUnexpectedType('ElseIfVarNotEqualsString', $command);
412 412 
     }
413 413
414 
-    public static function ifContains(string $variable, string $search, bool $caseInsensitive=false) : Mailcode_Commands_Command_If_Contains
414 
+    public static function ifContains(string $variable, string $search, bool $caseInsensitive = false) : Mailcode_Commands_Command_If_Contains
415 415 
     {
416 416 
         $command = self::_buildIfContains('If', $variable, $search, $caseInsensitive);
417 417
418 
-        if($command instanceof Mailcode_Commands_Command_If_Contains)
418 
+        if ($command instanceof Mailcode_Commands_Command_If_Contains)
419 419 
         {
420 420 
             return $command;
421 421 
         }
 
@@ -423,11 +423,11 @@  discard block
 
 block discarded – undo
423 423 
         throw self::_exceptionUnexpectedType('ElseIfContains', $command);
424 424 
     }
425 425
426 
-    public static function elseIfContains(string $variable, string $search, bool $caseInsensitive=false) : Mailcode_Commands_Command_ElseIf_Contains
426 
+    public static function elseIfContains(string $variable, string $search, bool $caseInsensitive = false) : Mailcode_Commands_Command_ElseIf_Contains
427 427 
     {
428 428 
         $command = self::_buildIfContains('ElseIf', $variable, $search, $caseInsensitive);
429 429
430 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Contains)
430 
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Contains)
431 431 
         {
432 432 
             return $command;
433 433 
         }
 
@@ -435,11 +435,11 @@  discard block
 
 block discarded – undo
435 435 
         throw self::_exceptionUnexpectedType('ElseIfContains', $command);
436 436 
     }
437 437
438 
-    protected static function _buildIfContains(string $ifType, string $variable, string $search, bool $caseInsensitive=false) : Mailcode_Commands_IfBase
438 
+    protected static function _buildIfContains(string $ifType, string $variable, string $search, bool $caseInsensitive = false) : Mailcode_Commands_IfBase
439 439 
     {
440 440 
         $keyword = ' ';
441 441
442 
-        if($caseInsensitive)
442 
+        if ($caseInsensitive)
443 443 
         {
444 444 
             $keyword = ' insensitive: ';
445 445 
         }
 
@@ -474,7 +474,7 @@  discard block
 
 block discarded – undo
474 474
475 475 
     protected static function _checkCommand(Mailcode_Commands_Command $command) : void
476 476 
     {
477 
-        if($command->isValid())
477 
+        if ($command->isValid())
478 478 
         {
479 479 
             return;
480 480 
         }
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -157,7 +157,8 @@  discard block
 
 block discarded – undo
157 157 
         throw self::_exceptionUnexpectedType('Comment', $cmd);
158 158 
     }
159 159
160 
-    public static function else() : Mailcode_Commands_Command_Else
160 
+    public static function else {
161 
+        () : Mailcode_Commands_Command_Else
161 162 
     {
162 163 
         $cmd = Mailcode::create()->getCommands()->createCommand(
163 164 
             'Else', 
@@ -165,6 +166,7 @@  discard block
 
 block discarded – undo
165 166 
             '',
166 167 
             '{else}'
167 168 
         );
169 
+    }
168 170
169 171 
         self::_checkCommand($cmd);
170 172
Please login to merge, or discard this patch.
src/Mailcode/Translator/Syntax/ApacheVelocity.php 2 patches
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch 
@@ -20,13 +20,13 @@
 
 block discarded – undo
20 20 
  */
21 21 
 abstract class Mailcode_Translator_Syntax_ApacheVelocity extends Mailcode_Translator_Command
22 22 
 {
23 
-   /**
24 
-    * Filters the string for use in an Apache Velocity (Java)
25 
-    * regex string: escapes all special characters.
26 
-    *
27 
-    * @param string $string
28 
-    * @return string
29 
-    */
23 
+    /**
24 
+     * Filters the string for use in an Apache Velocity (Java)
25 
+     * regex string: escapes all special characters.
26 
+     *
27 
+     * @param string $string
28 
+     * @return string
29 
+     */
30 30 
     protected function filterRegexString(string $string) : string
31 31 
     {
32 32 
         $escape = array(
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -48,7 +48,7 @@
 
 block discarded – undo
48 48 
             ')'
49 49 
         );
50 50
51 
-        foreach($escape as $char)
51 
+        foreach ($escape as $char)
52 52 
         {
53 53 
             $string = str_replace($char, '\\'.$char, $string);
54 54 
         }
Please login to merge, or discard this patch.
src/Mailcode/Translator/Syntax/ApacheVelocity/If.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -22,17 +22,17 @@  discard block
 
 block discarded – undo
22 22 
 {
23 23 
     public function translate(Mailcode_Commands_Command_If $command): string
24 24 
     {
25 
-        if($command instanceof Mailcode_Commands_Command_If_Command)
25 
+        if ($command instanceof Mailcode_Commands_Command_If_Command)
26 26 
         {
27 27 
             return $this->translateCommand($command);
28 28 
         }
29 29
30 
-        if($command instanceof Mailcode_Commands_Command_If_Variable)
30 
+        if ($command instanceof Mailcode_Commands_Command_If_Variable)
31 31 
         {
32 32 
             return $this->translateVariable($command);
33 33 
         }
34 34
35 
-        if($command instanceof Mailcode_Commands_Command_If_Contains)
35 
+        if ($command instanceof Mailcode_Commands_Command_If_Contains)
36 36 
         {
37 37 
             return $this->translateContains($command);
38 38 
         }
 
@@ -44,7 +44,7 @@  discard block
 
 block discarded – undo
44 44 
     {
45 45 
         $params = $command->getParams();
46 46
47 
-        if(!$params)
47 
+        if (!$params)
48 48 
         {
49 49 
             return '';
50 50 
         }
 
@@ -68,7 +68,7 @@  discard block
 
 block discarded – undo
68 68 
     protected function translateContains(Mailcode_Commands_Command_If_Contains $command) : string
69 69 
     {
70 70 
         $opts = 's';
71 
-        if($command->isCaseInsensitive())
71 
+        if ($command->isCaseInsensitive())
72 72 
         {
73 73 
             $opts = 'is';
74 74 
         }
Please login to merge, or discard this patch.
src/Mailcode/Translator/Syntax/ApacheVelocity/ElseIf.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -22,17 +22,17 @@  discard block
 
 block discarded – undo
22 22 
 {
23 23 
     public function translate(Mailcode_Commands_Command_ElseIf $command): string
24 24 
     {
25 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Command)
25 
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Command)
26 26 
         {
27 27 
             return $this->translateCommand($command);
28 28 
         }
29 29
30 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
30 
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Variable)
31 31 
         {
32 32 
             return $this->translateVariable($command);
33 33 
         }
34 34
35 
-        if($command instanceof Mailcode_Commands_Command_ElseIf_Contains)
35 
+        if ($command instanceof Mailcode_Commands_Command_ElseIf_Contains)
36 36 
         {
37 37 
             return $this->translateContains($command);
38 38 
         }
 
@@ -44,7 +44,7 @@  discard block
 
 block discarded – undo
44 44 
     {
45 45 
         $params = $command->getParams();
46 46
47 
-        if(!$params)
47 
+        if (!$params)
48 48 
         {
49 49 
             return '';
50 50 
         }
 
@@ -68,7 +68,7 @@  discard block
 
 block discarded – undo
68 68 
     protected function translateContains(Mailcode_Commands_Command_ElseIf_Contains $command) : string
69 69 
     {
70 70 
         $opts = 's';
71 
-        if($command->isCaseInsensitive())
71 
+        if ($command->isCaseInsensitive())
72 72 
         {
73 73 
             $opts = 'is';
74 74 
         }
Please login to merge, or discard this patch.