Passed
Push — master ( a80c4b...f51294 )
by Sebastian
02:39
created
localization/index.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@
 block discarded – undo
12 12
     $autoload = realpath($root.'/../vendor/autoload.php');
13 13
     
14 14
     // we need the autoloader to be present
15
-    if($autoload === false) 
15
+    if ($autoload === false) 
16 16
     {
17 17
         die('<b>ERROR:</b> Autoloader not present. Run composer update first.');
18 18
     }
Please login to merge, or discard this patch.
src/Mailcode/Parser/Safeguard/Placeholder.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
     
65 65
     public function getReplacementText() : string
66 66
     {
67
-        if(!empty($this->replacement))
67
+        if (!empty($this->replacement))
68 68
         {
69 69
             return $this->replacement;
70 70
         }
@@ -82,13 +82,13 @@  discard block
 block discarded – undo
82 82
         // to total amount of zeroes to pad with to obtain the total length
83 83
         $padLength = $this->getLength() - $length;
84 84
         
85
-        if($padLength < 0) 
85
+        if ($padLength < 0) 
86 86
         {
87 87
             $padLength = 0;
88 88
         }
89 89
         
90 90
         // create the zero-padded ID to fill the format string with 
91
-        $paddedID  = str_pad((string)$this->id, $padLength, '0');
91
+        $paddedID = str_pad((string)$this->id, $padLength, '0');
92 92
         
93 93
         $this->replacement = sprintf($format, $paddedID);
94 94
         
Please login to merge, or discard this patch.
src/Mailcode/Parser.php 2 patches
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -141,13 +141,11 @@
 block discarded – undo
141 141
         if(!empty($matches[1][$index]))
142 142
         {
143 143
             $name = $matches[1][$index];
144
-        }
145
-        else if(!empty($matches[2][$index]))
144
+        } else if(!empty($matches[2][$index]))
146 145
         {
147 146
             $name = $matches[2][$index];
148 147
             $params = $matches[3][$index];
149
-        }
150
-        else if(!empty($matches[4][$index]))
148
+        } else if(!empty($matches[4][$index]))
151 149
         {
152 150
             $name = $matches[4][$index];
153 151
             $type = $matches[5][$index];
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
         
74 74
         $total = count($matches[0]);
75 75
         
76
-        for($i=0; $i < $total; $i++)
76
+        for ($i = 0; $i < $total; $i++)
77 77
         {
78 78
             $match = $this->parseMatch($matches, $i);
79 79
             
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
     
86 86
     protected function prepareString(string $subject) : string
87 87
     {
88
-        if(!ConvertHelper::isStringHTML($subject))
88
+        if (!ConvertHelper::isStringHTML($subject))
89 89
         {
90 90
             return $subject;
91 91
         }
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
     {
107 107
         $name = $match->getName();
108 108
         
109
-        if(!$this->commands->nameExists($name))
109
+        if (!$this->commands->nameExists($name))
110 110
         {
111 111
             $collection->addErrorMessage(
112 112
                 $match->getMatchedString(),
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
             $match->getMatchedString()
124 124
         );
125 125
         
126
-        if($cmd->isValid())
126
+        if ($cmd->isValid())
127 127
         {
128 128
             $collection->addCommand($cmd);
129 129
             return;
@@ -153,16 +153,16 @@  discard block
 block discarded – undo
153 153
         // 5 = parameter type command, type
154 154
         // 6 = parameter type command, params
155 155
         
156
-        if(!empty($matches[1][$index]))
156
+        if (!empty($matches[1][$index]))
157 157
         {
158 158
             $name = $matches[1][$index];
159 159
         }
160
-        else if(!empty($matches[2][$index]))
160
+        else if (!empty($matches[2][$index]))
161 161
         {
162 162
             $name = $matches[2][$index];
163 163
             $params = $matches[3][$index];
164 164
         }
165
-        else if(!empty($matches[4][$index]))
165
+        else if (!empty($matches[4][$index]))
166 166
         {
167 167
             $name = $matches[4][$index];
168 168
             $type = $matches[5][$index];
Please login to merge, or discard this patch.
src/Mailcode/Parser/Safeguard.php 2 patches
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -158,8 +158,7 @@
 block discarded – undo
158 158
             if($highlighted)
159 159
             {
160 160
                 $replace = $placeholder->getHighlightedText();
161
-            }
162
-            else 
161
+            } else 
163 162
             {
164 163
                 $replace = $placeholder->getOriginalText();
165 164
             }
Please login to merge, or discard this patch.
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
     */
135 135
     public function setDelimiter(string $delimiter) : Mailcode_Parser_Safeguard
136 136
     {
137
-        if(empty($delimiter))
137
+        if (empty($delimiter))
138 138
         {
139 139
             throw new Mailcode_Exception(
140 140
                 'Empty delimiter',
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
         $safe = str_replace(array_values($replaces), array_keys($replaces), $this->originalString);
187 187
 
188 188
         // If a formatter has been selected, let it modify the string.
189
-        if(isset($this->formatter))
189
+        if (isset($this->formatter))
190 190
         {
191 191
             $safe = $this->formatter->format($safe);
192 192
         }
@@ -198,7 +198,7 @@  discard block
 block discarded – undo
198 198
     {
199 199
         $class = 'Mailcode\Mailcode_Parser_Safeguard_Formatter_'.$formatterID;
200 200
         
201
-        if(class_exists($class))
201
+        if (class_exists($class))
202 202
         {
203 203
             $this->formatter = new $class($this);
204 204
             
@@ -224,7 +224,7 @@  discard block
 block discarded – undo
224 224
     {
225 225
         $formatter = $this->selectFormatter('SingleLines');
226 226
         
227
-        if($formatter instanceof Mailcode_Parser_Safeguard_Formatter_SingleLines)
227
+        if ($formatter instanceof Mailcode_Parser_Safeguard_Formatter_SingleLines)
228 228
         {
229 229
             return $formatter;
230 230
         }
@@ -246,17 +246,17 @@  discard block
 block discarded – undo
246 246
     * @param bool $highlighted
247 247
     * @return string[]string
248 248
     */
249
-    protected function getReplaces(bool $highlighted=false) : array
249
+    protected function getReplaces(bool $highlighted = false) : array
250 250
     {
251 251
         $placeholders = $this->getPlaceholders();
252 252
         
253 253
         $replaces = array();
254 254
         
255
-        foreach($placeholders as $placeholder)
255
+        foreach ($placeholders as $placeholder)
256 256
         {
257 257
             $replace = '';
258 258
             
259
-            if($highlighted)
259
+            if ($highlighted)
260 260
             {
261 261
                 $replace = $placeholder->getHighlightedText();
262 262
             }
@@ -280,7 +280,7 @@  discard block
 block discarded – undo
280 280
     */
281 281
     public function getPlaceholders()
282 282
     {
283
-        if(isset($this->placeholders))
283
+        if (isset($this->placeholders))
284 284
         {
285 285
             return $this->placeholders;
286 286
         }
@@ -289,7 +289,7 @@  discard block
 block discarded – undo
289 289
         
290 290
         $cmds = $this->getCollection()->getGroupedByHash();
291 291
         
292
-        foreach($cmds as $command)
292
+        foreach ($cmds as $command)
293 293
         {
294 294
             self::$counter++;
295 295
             
@@ -303,9 +303,9 @@  discard block
 block discarded – undo
303 303
         return $this->placeholders;
304 304
     }
305 305
     
306
-    protected function restore(string $string, bool $partial=false, bool $highlighted=false) : string
306
+    protected function restore(string $string, bool $partial = false, bool $highlighted = false) : string
307 307
     {
308
-        if(!$partial)
308
+        if (!$partial)
309 309
         {
310 310
             $this->requireValidCollection();
311 311
         }
@@ -314,9 +314,9 @@  discard block
 block discarded – undo
314 314
         
315 315
         $placeholderStrings = array_keys($replaces);
316 316
         
317
-        foreach($placeholderStrings as $search)
317
+        foreach ($placeholderStrings as $search)
318 318
         {
319
-            if(!$partial && !strstr($string, $search))
319
+            if (!$partial && !strstr($string, $search))
320 320
             {
321 321
                 throw new Mailcode_Exception(
322 322
                     'Command placeholder not found',
@@ -418,7 +418,7 @@  discard block
 block discarded – undo
418 418
     */
419 419
     public function getCollection() : Mailcode_Collection
420 420
     {
421
-        if(isset($this->collection))
421
+        if (isset($this->collection))
422 422
         {
423 423
             return $this->collection;
424 424
         }
@@ -440,7 +440,7 @@  discard block
 block discarded – undo
440 440
     */
441 441
     protected function requireValidCollection() : void
442 442
     {
443
-        if($this->getCollection()->isValid())
443
+        if ($this->getCollection()->isValid())
444 444
         {
445 445
             return;
446 446
         }
@@ -463,7 +463,7 @@  discard block
 block discarded – undo
463 463
     */
464 464
     public function getPlaceholderStrings() : array
465 465
     {
466
-        if(is_array($this->placeholderStrings))
466
+        if (is_array($this->placeholderStrings))
467 467
         {
468 468
             return $this->placeholderStrings;
469 469
         }
@@ -472,7 +472,7 @@  discard block
 block discarded – undo
472 472
         
473 473
         $this->placeholderStrings = array();
474 474
         
475
-        foreach($placeholders as $placeholder)
475
+        foreach ($placeholders as $placeholder)
476 476
         {
477 477
             $this->placeholderStrings[] = $placeholder->getReplacementText();
478 478
         }
@@ -498,9 +498,9 @@  discard block
 block discarded – undo
498 498
     {
499 499
         $placeholders = $this->getPlaceholders();
500 500
         
501
-        foreach($placeholders as $placeholder)
501
+        foreach ($placeholders as $placeholder)
502 502
         {
503
-            if($placeholder->getID() === $id)
503
+            if ($placeholder->getID() === $id)
504 504
             {
505 505
                 return $placeholder;
506 506
             }
@@ -527,9 +527,9 @@  discard block
 block discarded – undo
527 527
     {
528 528
         $placeholders = $this->getPlaceholders();
529 529
         
530
-        foreach($placeholders as $placeholder)
530
+        foreach ($placeholders as $placeholder)
531 531
         {
532
-            if($placeholder->getReplacementText() === $string)
532
+            if ($placeholder->getReplacementText() === $string)
533 533
             {
534 534
                 return $placeholder;
535 535
             }
Please login to merge, or discard this patch.
src/Mailcode/Variables/Collection/Regular.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
     
33 33
     public function add(Mailcode_Variables_Variable $variable) : Mailcode_Variables_Collection
34 34
     {
35
-        if(!$variable->isValid())
35
+        if (!$variable->isValid())
36 36
         {
37 37
             return $this->addInvalid($variable);
38 38
         }
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
         parent::mergeWith($collection);
77 77
         
78 78
         // also inherit any invalid variables
79
-        if($collection instanceof Mailcode_Variables_Collection_Regular)
79
+        if ($collection instanceof Mailcode_Variables_Collection_Regular)
80 80
         {
81 81
             $invalid = $collection->getInvalid();
82 82
             
Please login to merge, or discard this patch.
src/Mailcode/Variables/Collection/Invalid.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
 {
25 25
     public function add(Mailcode_Variables_Variable $variable) : Mailcode_Variables_Collection
26 26
     {
27
-        if($variable->isValid())
27
+        if ($variable->isValid())
28 28
         {
29 29
             return $this;
30 30
         }
Please login to merge, or discard this patch.
src/functions.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
  */
24 24
 function init() : void
25 25
 {
26
-    if(!class_exists('\AppLocalize\Localization')) {
26
+    if (!class_exists('\AppLocalize\Localization')) {
27 27
         return;
28 28
     }
29 29
     
Please login to merge, or discard this patch.
src/Mailcode/Collection/NestingValidator.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -60,11 +60,11 @@  discard block
 block discarded – undo
60 60
         
61 61
         $commands = $this->collection->getCommands();
62 62
         
63
-        foreach($commands as $command)
63
+        foreach ($commands as $command)
64 64
         {
65 65
             $method = 'validate_'.$command->getCommandType();
66 66
             
67
-            if(!method_exists($this, $method))
67
+            if (!method_exists($this, $method))
68 68
             {
69 69
                 throw new Mailcode_Exception(
70 70
                     'Unknown command type validation method.',
@@ -79,13 +79,13 @@  discard block
 block discarded – undo
79 79
             
80 80
             $this->$method($command);
81 81
             
82
-            if(!$this->validationResult->isValid())
82
+            if (!$this->validationResult->isValid())
83 83
             {
84 84
                 break;
85 85
             }
86 86
         }
87 87
         
88
-        if($this->validationResult->isValid())
88
+        if ($this->validationResult->isValid())
89 89
         {
90 90
             $this->validate_Unclosed();
91 91
         }
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
     {
98 98
         $leftover = $this->getOpenCommand();
99 99
         
100
-        if($leftover === null)
100
+        if ($leftover === null)
101 101
         {
102 102
             return;
103 103
         }
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
     
114 114
     protected function getOpenCommand() : ?Mailcode_Commands_Command_Type_Opening
115 115
     {
116
-        if(empty($this->stack))
116
+        if (empty($this->stack))
117 117
         {
118 118
             return null;
119 119
         }
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
         
125 125
         $cmd = $this->stack[$idx];
126 126
         
127
-        if($cmd instanceof Mailcode_Commands_Command_Type_Opening)
127
+        if ($cmd instanceof Mailcode_Commands_Command_Type_Opening)
128 128
         {
129 129
             return $cmd;
130 130
         }
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
     {
149 149
         $parent = $this->getOpenCommand();
150 150
         
151
-        if($parent === null)
151
+        if ($parent === null)
152 152
         {
153 153
             $this->validationResult->makeError(
154 154
                 t(
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
             return;
163 163
         }
164 164
         
165
-        if($parent->getName() !== $command->getParentName())
165
+        if ($parent->getName() !== $command->getParentName())
166 166
         {
167 167
             $this->validationResult->makeError(
168 168
                 t(
@@ -181,7 +181,7 @@  discard block
 block discarded – undo
181 181
     
182 182
     protected function validate_Closing(Mailcode_Commands_Command_Type_Closing $command) : void
183 183
     {
184
-        if(empty($this->stack))
184
+        if (empty($this->stack))
185 185
         {
186 186
             $this->validationResult->makeError(
187 187
                 t('All open commands have already been ended.'),
Please login to merge, or discard this patch.
src/Mailcode/Commands/Command/For.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@
 block discarded – undo
62 62
         $keyword = $info->getKeywordByIndex(1);
63 63
         $container = $info->getVariableByIndex(2);
64 64
         
65
-        if($variable && $keyword && $container && $keyword->isForIn())
65
+        if ($variable && $keyword && $container && $keyword->isForIn())
66 66
         {
67 67
             return;
68 68
         }
Please login to merge, or discard this patch.