Passed
Pull Request — master (#18)
by
unknown
03:40
created
tools/translator.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 $error = null;
32 32
 $activeSyntax = $syntaxes[0]->getTypeID();
33 33
 
34
-if($request->getBool('translate'))
34
+if ($request->getBool('translate'))
35 35
 {
36 36
     $commandsText = $request->registerParam('mailcode')->getString();
37 37
     $activeSyntax = $request->registerParam('syntax')
@@ -48,12 +48,12 @@  discard block
 block discarded – undo
48 48
         $error = $e->getMessage();
49 49
 
50 50
         $collection = $e->getCollection();
51
-        if($collection)
51
+        if ($collection)
52 52
         {
53 53
             $first = $collection->getFirstError();
54 54
             $error = $first->getMessage();
55 55
             $matched = $first->getMatchedText();
56
-            if(!empty($matched)) {
56
+            if (!empty($matched)) {
57 57
                 $error .= '<br>'.t('In command:').' <code>'.$matched.'</code>';
58 58
             }
59 59
         }
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
                             $selected = '';
118 118
                             $typeID = $syntax->getTypeID();
119 119
 
120
-                            if($typeID === $activeSyntax) {
120
+                            if ($typeID === $activeSyntax) {
121 121
                                 $selected = ' selected';
122 122
                             }
123 123
 
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
         <p></p><br>
138 138
         <h2><?php pt('Translated commands') ?></h2>
139 139
         <?php
140
-            if(empty($commandsText))
140
+            if (empty($commandsText))
141 141
             {
142 142
                 ?>
143 143
                     <div class="alert alert-info">
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
                     </div>
146 146
                 <?php
147 147
             }
148
-            else if($error)
148
+            else if ($error)
149 149
             {
150 150
                 ?>
151 151
                     <div class="alert alert-danger">
Please login to merge, or discard this patch.
src/Mailcode/Translator/Syntax/HubL/Base/AbstractIfBase.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
         // IF type.
45 45
         $method = 'translate'.$this->getIfType($command);
46 46
 
47
-        if(method_exists($this, $method))
47
+        if (method_exists($this, $method))
48 48
         {
49 49
             return (string)$this->$method($command);
50 50
         }
@@ -64,11 +64,11 @@  discard block
 block discarded – undo
64 64
 
65 65
         $keywords = $command->getLogicKeywords()->getKeywords();
66 66
         
67
-        foreach($keywords as $keyword)
67
+        foreach ($keywords as $keyword)
68 68
         {
69 69
             $keyCommand = $keyword->getCommand();
70 70
             
71
-            if($keyCommand instanceof Mailcode_Commands_IfBase)
71
+            if ($keyCommand instanceof Mailcode_Commands_IfBase)
72 72
             {
73 73
                 $body .= ' '.$this->getSign($keyword).' '.$this->translateBody($keyCommand);
74 74
             }
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
      */
97 97
     protected function getSign(Mailcode_Commands_LogicKeywords_Keyword $keyword) : string
98 98
     {
99
-        switch($keyword->getName())
99
+        switch ($keyword->getName())
100 100
         {
101 101
             case 'and':
102 102
                 return 'and';
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
     {
130 130
         $sign = '!';
131 131
         
132
-        if($notEmpty)
132
+        if ($notEmpty)
133 133
         {
134 134
             $sign = '';
135 135
         }
@@ -145,12 +145,12 @@  discard block
 block discarded – undo
145 145
     {
146 146
         $params = $command->getParams();
147 147
 
148
-        if(!$params)
148
+        if (!$params)
149 149
         {
150 150
             return '';
151 151
         }
152 152
 
153
-        if($command->hasFreeformParameters())
153
+        if ($command->hasFreeformParameters())
154 154
         {
155 155
             return $params->getStatementString();
156 156
         }
@@ -158,11 +158,11 @@  discard block
 block discarded – undo
158 158
         return $params->getNormalized();
159 159
     }
160 160
     
161
-    protected function _translateVariable(Mailcode_Variables_Variable $variable, string $comparator, string $value, bool $insensitive=false) : string
161
+    protected function _translateVariable(Mailcode_Variables_Variable $variable, string $comparator, string $value, bool $insensitive = false) : string
162 162
     {
163
-        $fullName =  $this->formatVariableName($variable->getFullName());
163
+        $fullName = $this->formatVariableName($variable->getFullName());
164 164
 
165
-        if($insensitive)
165
+        if ($insensitive)
166 166
         {
167 167
             $fullName .= '|lower';
168 168
             $value = mb_strtolower($value);
Please login to merge, or discard this patch.
src/Mailcode/Factory/CommandSets/Set/If.php 1 patch
Spacing   +39 added lines, -39 removed lines patch added patch discarded remove patch
@@ -20,11 +20,11 @@  discard block
 block discarded – undo
20 20
  */
21 21
 class Mailcode_Factory_CommandSets_Set_If extends Mailcode_Factory_CommandSets_IfBase
22 22
 {
23
-    public function if(string $condition, string $type='') : Mailcode_Commands_Command_If
23
+    public function if (string $condition, string $type = '') : Mailcode_Commands_Command_If
24 24
     {
25 25
         $command = $this->instantiator->buildIf('If', $condition, $type);
26 26
         
27
-        if($command instanceof Mailcode_Commands_Command_If)
27
+        if ($command instanceof Mailcode_Commands_Command_If)
28 28
         {
29 29
             return $command;
30 30
         }
@@ -32,11 +32,11 @@  discard block
 block discarded – undo
32 32
         throw $this->instantiator->exceptionUnexpectedType('If', $command);
33 33
     }
34 34
     
35
-    public function var(string $variable, string $operand, string $value, bool $quoteValue=false, bool $insensitive=false) : Mailcode_Commands_Command_If_Variable
35
+    public function var(string $variable, string $operand, string $value, bool $quoteValue = false, bool $insensitive = false) : Mailcode_Commands_Command_If_Variable
36 36
     {
37 37
         $command = $this->instantiator->buildIfVar('If', $variable, $operand, $value, $quoteValue, $insensitive);
38 38
         
39
-        if($command instanceof Mailcode_Commands_Command_If_Variable)
39
+        if ($command instanceof Mailcode_Commands_Command_If_Variable)
40 40
         {
41 41
             return $command;
42 42
         }
@@ -44,11 +44,11 @@  discard block
 block discarded – undo
44 44
         throw $this->instantiator->exceptionUnexpectedType('IfVar', $command);
45 45
     }
46 46
     
47
-    public function varString(string $variable, string $operand, string $value, bool $insensitive=false) : Mailcode_Commands_Command_If_Variable
47
+    public function varString(string $variable, string $operand, string $value, bool $insensitive = false) : Mailcode_Commands_Command_If_Variable
48 48
     {
49 49
         $command = $this->instantiator->buildIfVar('If', $variable, $operand, $value, true, $insensitive);
50 50
         
51
-        if($command instanceof Mailcode_Commands_Command_If_Variable)
51
+        if ($command instanceof Mailcode_Commands_Command_If_Variable)
52 52
         {
53 53
             return $command;
54 54
         }
@@ -56,11 +56,11 @@  discard block
 block discarded – undo
56 56
         throw $this->instantiator->exceptionUnexpectedType('IfVarString', $command);
57 57
     }
58 58
     
59
-    public function varEquals(string $variable, string $value, bool $quoteValue=false, bool $insensitive=false) : Mailcode_Commands_Command_If_Variable
59
+    public function varEquals(string $variable, string $value, bool $quoteValue = false, bool $insensitive = false) : Mailcode_Commands_Command_If_Variable
60 60
     {
61 61
         $command = $this->instantiator->buildIfVar('If', $variable, '==', $value, $quoteValue, $insensitive);
62 62
         
63
-        if($command instanceof Mailcode_Commands_Command_If_Variable)
63
+        if ($command instanceof Mailcode_Commands_Command_If_Variable)
64 64
         {
65 65
             return $command;
66 66
         }
@@ -68,11 +68,11 @@  discard block
 block discarded – undo
68 68
         throw $this->instantiator->exceptionUnexpectedType('IfVarEquals', $command);
69 69
     }
70 70
     
71
-    public function varEqualsString(string $variable, string $value, bool $insensitive=false) : Mailcode_Commands_Command_If
71
+    public function varEqualsString(string $variable, string $value, bool $insensitive = false) : Mailcode_Commands_Command_If
72 72
     {
73 73
         $command = $this->instantiator->buildIfVar('If', $variable, '==', $value, true, $insensitive);
74 74
         
75
-        if($command instanceof Mailcode_Commands_Command_If_Variable)
75
+        if ($command instanceof Mailcode_Commands_Command_If_Variable)
76 76
         {
77 77
             return $command;
78 78
         }
@@ -80,11 +80,11 @@  discard block
 block discarded – undo
80 80
         throw $this->instantiator->exceptionUnexpectedType('IfVarEqualsString', $command);
81 81
     }
82 82
     
83
-    public function varNotEquals(string $variable, string $value, bool $quoteValue=false, bool $insensitive=false) : Mailcode_Commands_Command_If_Variable
83
+    public function varNotEquals(string $variable, string $value, bool $quoteValue = false, bool $insensitive = false) : Mailcode_Commands_Command_If_Variable
84 84
     {
85 85
         $command = $this->instantiator->buildIfVar('If', $variable, '!=', $value, $quoteValue, $insensitive);
86 86
         
87
-        if($command instanceof Mailcode_Commands_Command_If_Variable)
87
+        if ($command instanceof Mailcode_Commands_Command_If_Variable)
88 88
         {
89 89
             return $command;
90 90
         }
@@ -92,11 +92,11 @@  discard block
 block discarded – undo
92 92
         throw $this->instantiator->exceptionUnexpectedType('IfVarNotEquals', $command);
93 93
     }
94 94
     
95
-    public function varNotEqualsString(string $variable, string $value, bool $insensitive=false) : Mailcode_Commands_Command_If_Variable
95
+    public function varNotEqualsString(string $variable, string $value, bool $insensitive = false) : Mailcode_Commands_Command_If_Variable
96 96
     {
97 97
         $command = $this->instantiator->buildIfVar('If', $variable, '!=', $value, true, $insensitive);
98 98
         
99
-        if($command instanceof Mailcode_Commands_Command_If_Variable)
99
+        if ($command instanceof Mailcode_Commands_Command_If_Variable)
100 100
         {
101 101
             return $command;
102 102
         }
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
     {
109 109
         $command = $this->instantiator->buildIfEmpty('If', $variable);
110 110
         
111
-        if($command instanceof Mailcode_Commands_Command_If_Empty)
111
+        if ($command instanceof Mailcode_Commands_Command_If_Empty)
112 112
         {
113 113
             return $command;
114 114
         }
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
     {
121 121
         $command = $this->instantiator->buildIfNotEmpty('If', $variable);
122 122
         
123
-        if($command instanceof Mailcode_Commands_Command_If_NotEmpty)
123
+        if ($command instanceof Mailcode_Commands_Command_If_NotEmpty)
124 124
         {
125 125
             return $command;
126 126
         }
@@ -135,11 +135,11 @@  discard block
 block discarded – undo
135 135
      * @return Mailcode_Commands_Command_If_Contains
136 136
      * @throws Mailcode_Factory_Exception
137 137
      */
138
-    public function contains(string $variable, array $searchTerms, bool $caseInsensitive=false) : Mailcode_Commands_Command_If_Contains
138
+    public function contains(string $variable, array $searchTerms, bool $caseInsensitive = false) : Mailcode_Commands_Command_If_Contains
139 139
     {
140 140
         $command = $this->instantiator->buildIfContains('If', $variable, $searchTerms, $caseInsensitive);
141 141
         
142
-        if($command instanceof Mailcode_Commands_Command_If_Contains)
142
+        if ($command instanceof Mailcode_Commands_Command_If_Contains)
143 143
         {
144 144
             return $command;
145 145
         }
@@ -154,11 +154,11 @@  discard block
 block discarded – undo
154 154
      * @return Mailcode_Commands_Command_If_NotContains
155 155
      * @throws Mailcode_Factory_Exception
156 156
      */
157
-    public function notContains(string $variable, array $searchTerms, bool $caseInsensitive=false) : Mailcode_Commands_Command_If_NotContains
157
+    public function notContains(string $variable, array $searchTerms, bool $caseInsensitive = false) : Mailcode_Commands_Command_If_NotContains
158 158
     {
159 159
         $command = $this->instantiator->buildIfNotContains('If', $variable, $searchTerms, $caseInsensitive);
160 160
 
161
-        if($command instanceof Mailcode_Commands_Command_If_NotContains)
161
+        if ($command instanceof Mailcode_Commands_Command_If_NotContains)
162 162
         {
163 163
             return $command;
164 164
         }
@@ -174,11 +174,11 @@  discard block
 block discarded – undo
174 174
      * @return Mailcode_Commands_Command_If_ListContains
175 175
      * @throws Mailcode_Factory_Exception
176 176
      */
177
-    public function listContains(string $variable, array $searchTerms, bool $caseInsensitive=false, bool $regexEnabled=false) : Mailcode_Commands_Command_If_ListContains
177
+    public function listContains(string $variable, array $searchTerms, bool $caseInsensitive = false, bool $regexEnabled = false) : Mailcode_Commands_Command_If_ListContains
178 178
     {
179 179
         $command = $this->instantiator->buildIfListContains('If', $variable, $searchTerms, $caseInsensitive, $regexEnabled, 'list-contains');
180 180
 
181
-        if($command instanceof Mailcode_Commands_Command_If_ListContains)
181
+        if ($command instanceof Mailcode_Commands_Command_If_ListContains)
182 182
         {
183 183
             return $command;
184 184
         }
@@ -193,11 +193,11 @@  discard block
 block discarded – undo
193 193
      * @return Mailcode_Commands_Command_If_ListEquals
194 194
      * @throws Mailcode_Factory_Exception
195 195
      */
196
-    public function listEquals(string $variable, array $searchTerms, bool $caseInsensitive=false) : Mailcode_Commands_Command_If_ListEquals
196
+    public function listEquals(string $variable, array $searchTerms, bool $caseInsensitive = false) : Mailcode_Commands_Command_If_ListEquals
197 197
     {
198 198
         $command = $this->instantiator->buildIfListContains('If', $variable, $searchTerms, $caseInsensitive, false, 'list-equals');
199 199
 
200
-        if($command instanceof Mailcode_Commands_Command_If_ListEquals)
200
+        if ($command instanceof Mailcode_Commands_Command_If_ListEquals)
201 201
         {
202 202
             return $command;
203 203
         }
@@ -213,11 +213,11 @@  discard block
 block discarded – undo
213 213
      * @return Mailcode_Commands_Command_If_ListNotContains
214 214
      * @throws Mailcode_Factory_Exception
215 215
      */
216
-    public function listNotContains(string $variable, array $searchTerms, bool $caseInsensitive=false, bool $regexEnabled=false) : Mailcode_Commands_Command_If_ListNotContains
216
+    public function listNotContains(string $variable, array $searchTerms, bool $caseInsensitive = false, bool $regexEnabled = false) : Mailcode_Commands_Command_If_ListNotContains
217 217
     {
218 218
         $command = $this->instantiator->buildIfListNotContains('If', $variable, $searchTerms, $caseInsensitive, $regexEnabled);
219 219
 
220
-        if($command instanceof Mailcode_Commands_Command_If_ListNotContains)
220
+        if ($command instanceof Mailcode_Commands_Command_If_ListNotContains)
221 221
         {
222 222
             return $command;
223 223
         }
@@ -233,11 +233,11 @@  discard block
 block discarded – undo
233 233
      * @return Mailcode_Commands_Command_If_ListBeginsWith
234 234
      * @throws Mailcode_Factory_Exception
235 235
      */
236
-    public function listBeginsWith(string $variable, array $searchTerms, bool $caseInsensitive=false, bool $regexEnabled=false) : Mailcode_Commands_Command_If_ListBeginsWith
236
+    public function listBeginsWith(string $variable, array $searchTerms, bool $caseInsensitive = false, bool $regexEnabled = false) : Mailcode_Commands_Command_If_ListBeginsWith
237 237
     {
238
-        $command = $this->instantiator->buildIfListContains('If', $variable, $searchTerms, $caseInsensitive, $regexEnabled,'list-begins-with');
238
+        $command = $this->instantiator->buildIfListContains('If', $variable, $searchTerms, $caseInsensitive, $regexEnabled, 'list-begins-with');
239 239
 
240
-        if($command instanceof Mailcode_Commands_Command_If_ListBeginsWith)
240
+        if ($command instanceof Mailcode_Commands_Command_If_ListBeginsWith)
241 241
         {
242 242
             return $command;
243 243
         }
@@ -253,11 +253,11 @@  discard block
 block discarded – undo
253 253
      * @return Mailcode_Commands_Command_If_ListEndsWith
254 254
      * @throws Mailcode_Factory_Exception
255 255
      */
256
-    public function listEndsWith(string $variable, array $searchTerms, bool $caseInsensitive=false, bool $regexEnabled=false) : Mailcode_Commands_Command_If_ListEndsWith
256
+    public function listEndsWith(string $variable, array $searchTerms, bool $caseInsensitive = false, bool $regexEnabled = false) : Mailcode_Commands_Command_If_ListEndsWith
257 257
     {
258
-        $command = $this->instantiator->buildIfListContains('If', $variable, $searchTerms, $caseInsensitive, $regexEnabled,'list-ends-with');
258
+        $command = $this->instantiator->buildIfListContains('If', $variable, $searchTerms, $caseInsensitive, $regexEnabled, 'list-ends-with');
259 259
 
260
-        if($command instanceof Mailcode_Commands_Command_If_ListEndsWith)
260
+        if ($command instanceof Mailcode_Commands_Command_If_ListEndsWith)
261 261
         {
262 262
             return $command;
263 263
         }
@@ -265,11 +265,11 @@  discard block
 block discarded – undo
265 265
         throw $this->instantiator->exceptionUnexpectedType('IfListEndsWith', $command);
266 266
     }
267 267
 
268
-    public function beginsWith(string $variable, string $search, bool $caseInsensitive=false) : Mailcode_Commands_Command_If_BeginsWith
268
+    public function beginsWith(string $variable, string $search, bool $caseInsensitive = false) : Mailcode_Commands_Command_If_BeginsWith
269 269
     {
270 270
         $command = $this->instantiator->buildIfBeginsWith('If', $variable, $search, $caseInsensitive);
271 271
         
272
-        if($command instanceof Mailcode_Commands_Command_If_BeginsWith)
272
+        if ($command instanceof Mailcode_Commands_Command_If_BeginsWith)
273 273
         {
274 274
             return $command;
275 275
         }
@@ -277,11 +277,11 @@  discard block
 block discarded – undo
277 277
         throw $this->instantiator->exceptionUnexpectedType('IfBeginsWith', $command);
278 278
     }
279 279
     
280
-    public function endsWith(string $variable, string $search, bool $caseInsensitive=false) : Mailcode_Commands_Command_If_EndsWith
280
+    public function endsWith(string $variable, string $search, bool $caseInsensitive = false) : Mailcode_Commands_Command_If_EndsWith
281 281
     {
282 282
         $command = $this->instantiator->buildIfEndsWith('If', $variable, $search, $caseInsensitive);
283 283
         
284
-        if($command instanceof Mailcode_Commands_Command_If_EndsWith)
284
+        if ($command instanceof Mailcode_Commands_Command_If_EndsWith)
285 285
         {
286 286
             return $command;
287 287
         }
@@ -293,7 +293,7 @@  discard block
 block discarded – undo
293 293
     {
294 294
         $command = $this->instantiator->buildIfBiggerThan('If', $variable, $value);
295 295
 
296
-        if($command instanceof Mailcode_Commands_Command_If_BiggerThan)
296
+        if ($command instanceof Mailcode_Commands_Command_If_BiggerThan)
297 297
         {
298 298
             return $command;
299 299
         }
@@ -305,7 +305,7 @@  discard block
 block discarded – undo
305 305
     {
306 306
         $command = $this->instantiator->buildIfSmallerThan('If', $variable, $value);
307 307
 
308
-        if($command instanceof Mailcode_Commands_Command_If_SmallerThan)
308
+        if ($command instanceof Mailcode_Commands_Command_If_SmallerThan)
309 309
         {
310 310
             return $command;
311 311
         }
@@ -317,7 +317,7 @@  discard block
 block discarded – undo
317 317
     {
318 318
         $command = $this->instantiator->buildIfEquals('If', $variable, $value);
319 319
 
320
-        if($command instanceof Mailcode_Commands_Command_If_EqualsNumber)
320
+        if ($command instanceof Mailcode_Commands_Command_If_EqualsNumber)
321 321
         {
322 322
             return $command;
323 323
         }
Please login to merge, or discard this patch.
src/Mailcode/Factory/CommandSets/Set/Show/Price.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
         $cmd = $this->commands->createCommand(
32 32
             'ShowPrice',
33 33
             '',
34
-            $variableName . $paramsString,
34
+            $variableName.$paramsString,
35 35
             sprintf(
36 36
                 '{showprice: %s%s}',
37 37
                 $variableName,
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
         }
62 62
 
63 63
         if (!empty($params)) {
64
-            return ' ' . implode(' ', $params);
64
+            return ' '.implode(' ', $params);
65 65
         }
66 66
 
67 67
         return '';
Please login to merge, or discard this patch.