Passed
Push — master ( 2851e6...6e90f8 )
by Sebastian
05:27
created
src/Mailcode/Parser/Safeguard.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
         $placeholders = $this->getPlaceholders();
181 181
         $string = $this->originalString;
182 182
         
183
-        foreach($placeholders as $placeholder)
183
+        foreach ($placeholders as $placeholder)
184 184
         {
185 185
             $string = $this->makePlaceholderSafe($string, $placeholder);
186 186
         }
@@ -210,14 +210,14 @@  discard block
 block discarded – undo
210 210
         $placeholders = $this->getPlaceholders();
211 211
         $total = count($placeholders);
212 212
 
213
-        for($i=0; $i < $total; $i++)
213
+        for ($i = 0; $i < $total; $i++)
214 214
         {
215 215
             $placeholder = $placeholders[$i];
216 216
             $command = $placeholder->getCommand();
217 217
 
218
-            if($command instanceof Mailcode_Interfaces_Commands_ProtectedContent)
218
+            if ($command instanceof Mailcode_Interfaces_Commands_ProtectedContent)
219 219
             {
220
-                $string = $command->protectContent($string, $placeholder, $placeholders[$i+1]);
220
+                $string = $command->protectContent($string, $placeholder, $placeholders[$i + 1]);
221 221
             }
222 222
         }
223 223
 
@@ -228,7 +228,7 @@  discard block
 block discarded – undo
228 228
     {
229 229
         $pos = mb_strpos($string, $placeholder->getOriginalText());
230 230
 
231
-        if($pos === false)
231
+        if ($pos === false)
232 232
         {
233 233
             throw new Mailcode_Exception(
234 234
                 'Placeholder original text not found',
@@ -267,7 +267,7 @@  discard block
 block discarded – undo
267 267
     */
268 268
     public function createFormatting($subject) : Mailcode_Parser_Safeguard_Formatting
269 269
     {
270
-        if(is_string($subject))
270
+        if (is_string($subject))
271 271
         {
272 272
             $subject = Mailcode::create()->createString($subject);
273 273
         }
@@ -283,7 +283,7 @@  discard block
 block discarded – undo
283 283
     */
284 284
     public function getPlaceholders()
285 285
     {
286
-        if(isset($this->placeholders))
286
+        if (isset($this->placeholders))
287 287
         {
288 288
             return $this->placeholders;
289 289
         }
@@ -292,7 +292,7 @@  discard block
 block discarded – undo
292 292
         
293 293
         $cmds = $this->getCollection()->getCommands();
294 294
         
295
-        foreach($cmds as $command)
295
+        foreach ($cmds as $command)
296 296
         {
297 297
             self::$counter++;
298 298
             
@@ -313,21 +313,21 @@  discard block
 block discarded – undo
313 313
      * @return string
314 314
      * @throws Mailcode_Exception
315 315
      */
316
-    protected function restore(string $string, bool $partial=false, bool $highlighted=false) : string
316
+    protected function restore(string $string, bool $partial = false, bool $highlighted = false) : string
317 317
     {
318
-        if(!$partial)
318
+        if (!$partial)
319 319
         {
320 320
             $this->requireValidCollection();
321 321
         }
322 322
         
323 323
         $formatting = $this->createFormatting($string);
324 324
 
325
-        if($partial)
325
+        if ($partial)
326 326
         {
327 327
             $formatting->makePartial();
328 328
         }
329 329
         
330
-        if($highlighted)
330
+        if ($highlighted)
331 331
         {
332 332
             $formatting->replaceWithHTMLHighlighting();
333 333
         }
@@ -347,7 +347,7 @@  discard block
 block discarded – undo
347 347
         {
348 348
             $command = $placeholder->getCommand();
349 349
 
350
-            if($command instanceof Mailcode_Interfaces_Commands_ProtectedContent)
350
+            if ($command instanceof Mailcode_Interfaces_Commands_ProtectedContent)
351 351
             {
352 352
                 $string = $command->restoreContent($string);
353 353
             }
@@ -440,7 +440,7 @@  discard block
 block discarded – undo
440 440
     */
441 441
     public function getCollection() : Mailcode_Collection
442 442
     {
443
-        if(isset($this->collection))
443
+        if (isset($this->collection))
444 444
         {
445 445
             return $this->collection;
446 446
         }
@@ -462,7 +462,7 @@  discard block
 block discarded – undo
462 462
     */
463 463
     protected function requireValidCollection() : void
464 464
     {
465
-        if($this->getCollection()->isValid())
465
+        if ($this->getCollection()->isValid())
466 466
         {
467 467
             return;
468 468
         }
@@ -485,7 +485,7 @@  discard block
 block discarded – undo
485 485
     */
486 486
     public function getPlaceholderStrings() : array
487 487
     {
488
-        if(is_array($this->placeholderStrings))
488
+        if (is_array($this->placeholderStrings))
489 489
         {
490 490
             return $this->placeholderStrings;
491 491
         }
@@ -494,7 +494,7 @@  discard block
 block discarded – undo
494 494
         
495 495
         $this->placeholderStrings = array();
496 496
         
497
-        foreach($placeholders as $placeholder)
497
+        foreach ($placeholders as $placeholder)
498 498
         {
499 499
             $this->placeholderStrings[] = $placeholder->getReplacementText();
500 500
         }
@@ -520,9 +520,9 @@  discard block
 block discarded – undo
520 520
     {
521 521
         $placeholders = $this->getPlaceholders();
522 522
         
523
-        foreach($placeholders as $placeholder)
523
+        foreach ($placeholders as $placeholder)
524 524
         {
525
-            if($placeholder->getID() === $id)
525
+            if ($placeholder->getID() === $id)
526 526
             {
527 527
                 return $placeholder;
528 528
             }
@@ -549,9 +549,9 @@  discard block
 block discarded – undo
549 549
     {
550 550
         $placeholders = $this->getPlaceholders();
551 551
         
552
-        foreach($placeholders as $placeholder)
552
+        foreach ($placeholders as $placeholder)
553 553
         {
554
-            if($placeholder->getReplacementText() === $string)
554
+            if ($placeholder->getReplacementText() === $string)
555 555
             {
556 556
                 return $placeholder;
557 557
             }
Please login to merge, or discard this patch.
src/Mailcode/Parser/Safeguard/DelimiterValidator.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 
45 45
     private function validate() : OperationResult
46 46
     {
47
-        if(empty($this->delimiter))
47
+        if (empty($this->delimiter))
48 48
         {
49 49
             return $this->makeError(
50 50
                 'Delimiters may not be empty.',
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
             );
53 53
         }
54 54
 
55
-        if(strlen($this->delimiter) < 2)
55
+        if (strlen($this->delimiter) < 2)
56 56
         {
57 57
             return $this->makeError(
58 58
                 'The delimiter must have at least 2 characters in length.',
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 
63 63
         $encoded = urlencode($this->delimiter);
64 64
 
65
-        if($encoded !== $this->delimiter)
65
+        if ($encoded !== $this->delimiter)
66 66
         {
67 67
             return $this->makeError(
68 68
                 'The delimiter is not URL encoding neutral: it must not be modified by a urlencode() call.',
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
 
76 76
     public function throwExceptionIfInvalid() : void
77 77
     {
78
-        if($this->isValid()) {
78
+        if ($this->isValid()) {
79 79
             return;
80 80
         }
81 81
 
Please login to merge, or discard this patch.
src/Mailcode/Parser/Safeguard/Placeholder.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@
 block discarded – undo
69 69
     
70 70
     public function getReplacementText() : string
71 71
     {
72
-        if(!empty($this->replacement))
72
+        if (!empty($this->replacement))
73 73
         {
74 74
             return $this->replacement;
75 75
         }
Please login to merge, or discard this patch.
src/Mailcode/Commands/Command/Break.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
 
70 70
     protected function _validateNesting() : void
71 71
     {
72
-        if($this->findParentFor($this))
72
+        if ($this->findParentFor($this))
73 73
         {
74 74
             return;
75 75
         }
@@ -84,12 +84,12 @@  discard block
 block discarded – undo
84 84
     {
85 85
         $parent = $command->getParent();
86 86
 
87
-        if($parent === null)
87
+        if ($parent === null)
88 88
         {
89 89
             return false;
90 90
         }
91 91
 
92
-        if($parent instanceof Mailcode_Commands_Command_For)
92
+        if ($parent instanceof Mailcode_Commands_Command_For)
93 93
         {
94 94
             return true;
95 95
         }
Please login to merge, or discard this patch.
src/Mailcode/Traits/Commands/Validation/EmptyParams.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,12 +29,12 @@
 block discarded – undo
29 29
 
30 30
     protected function validateSyntax_params_empty() : void
31 31
     {
32
-        if(!$this->requiresParameters())
32
+        if (!$this->requiresParameters())
33 33
         {
34 34
             return;
35 35
         }
36 36
 
37
-        if(empty($this->paramsString))
37
+        if (empty($this->paramsString))
38 38
         {
39 39
             $this->validationResult->makeError(
40 40
                 t('Parameters have to be specified.'),
Please login to merge, or discard this patch.
src/Mailcode/Traits/Commands/Validation/TypeSupported.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -35,14 +35,14 @@
 block discarded – undo
35 35
 
36 36
     protected function validateSyntax_type_supported() : void
37 37
     {
38
-        if(!$this->supportsType() || empty($this->type))
38
+        if (!$this->supportsType() || empty($this->type))
39 39
         {
40 40
             return;
41 41
         }
42 42
 
43 43
         $types = $this->getSupportedTypes();
44 44
 
45
-        if(!in_array($this->type, $types))
45
+        if (!in_array($this->type, $types))
46 46
         {
47 47
             $this->validationResult->makeError(
48 48
                 t('The command addon %1$s is not supported.', $this->type).' '.
Please login to merge, or discard this patch.
src/Mailcode/Traits/Commands/Validation/ParamKeywords.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -31,14 +31,14 @@
 block discarded – undo
31 31
 
32 32
     protected function validateSyntax_params_keywords() : void
33 33
     {
34
-        if(!$this->supportsLogicKeywords())
34
+        if (!$this->supportsLogicKeywords())
35 35
         {
36 36
             return;
37 37
         }
38 38
 
39 39
         $this->logicKeywords = new Mailcode_Commands_LogicKeywords($this, $this->paramsString);
40 40
 
41
-        if(!$this->logicKeywords->isValid())
41
+        if (!$this->logicKeywords->isValid())
42 42
         {
43 43
             $this->validationResult->makeError(
44 44
                 t('Invalid parameters:').' '.$this->logicKeywords->getErrorMessage(),
Please login to merge, or discard this patch.
src/Mailcode/Traits/Commands/Validation/TypeUnsupported.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@
 block discarded – undo
30 30
 
31 31
     protected function validateSyntax_type_unsupported() : void
32 32
     {
33
-        if($this->supportsType() || empty($this->type))
33
+        if ($this->supportsType() || empty($this->type))
34 34
         {
35 35
             return;
36 36
         }
Please login to merge, or discard this patch.
src/Mailcode/Factory/CommandSets/Set/Set.php 1 patch
Spacing   +3 added lines, -3 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_Set extends Mailcode_Factory_CommandSets_Set
22 22
 {
23
-    public function var(string $variableName, string $value, bool $quoteValue=true) : Mailcode_Commands_Command_SetVariable
23
+    public function var(string $variableName, string $value, bool $quoteValue = true) : Mailcode_Commands_Command_SetVariable
24 24
     {
25 25
         $variableName = $this->instantiator->filterVariableName($variableName);
26 26
         
27
-        if($quoteValue)
27
+        if ($quoteValue)
28 28
         {
29 29
             $value = $this->instantiator->quoteString($value);
30 30
         }
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
         
41 41
         $this->instantiator->checkCommand($cmd);
42 42
         
43
-        if($cmd instanceof Mailcode_Commands_Command_SetVariable)
43
+        if ($cmd instanceof Mailcode_Commands_Command_SetVariable)
44 44
         {
45 45
             return $cmd;
46 46
         }
Please login to merge, or discard this patch.