Test Failed
Push — master ( 81711e...bf8774 )
by Sebastian
03:41
created
src/Mailcode/Traits/Formatting/HTMLHighlighting.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
     {
43 43
         $tagName = strtolower($tagName);
44 44
         
45
-        if(!in_array($tagName, $this->excludeTags))
45
+        if (!in_array($tagName, $this->excludeTags))
46 46
         {
47 47
             $this->excludeTags[] = $tagName;
48 48
         }
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
     */
59 59
     public function excludeTags(array $tagNames) : Mailcode_Parser_Safeguard_Formatter_Type_HTMLHighlighting
60 60
     {
61
-        foreach($tagNames as $tagName)
61
+        foreach ($tagNames as $tagName)
62 62
         {
63 63
             $this->excludeTag((string)$tagName);
64 64
         }
Please login to merge, or discard this patch.
src/Mailcode/Parser/Safeguard/Formatter/ReplacerType.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
     {
30 30
         $locations = $this->resolveLocations();
31 31
         
32
-        foreach($locations as $location)
32
+        foreach ($locations as $location)
33 33
         {
34 34
             $location->replaceWith($this->resolveReplacement($location));
35 35
         }
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
     */
45 45
     private function resolveReplacement(Mailcode_Parser_Safeguard_Formatter_Location $location) : string
46 46
     {
47
-        if($location->requiresAdjustment())
47
+        if ($location->requiresAdjustment())
48 48
         {
49 49
             return $this->getReplaceString($location);
50 50
         }
Please login to merge, or discard this patch.
src/Mailcode/Parser/Safeguard/Formatter/Type/SingleLines/Location.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -54,26 +54,26 @@  discard block
 block discarded – undo
54 54
         $position = $this->getStartPosition();
55 55
         
56 56
         // we're at the beginning of the string
57
-        if($position === false || $position === 0)
57
+        if ($position === false || $position === 0)
58 58
         {
59 59
             return;
60 60
         }
61 61
         
62 62
         $prevPos = $position - $this->eolLength;
63 63
         
64
-        if($prevPos < 0)
64
+        if ($prevPos < 0)
65 65
         {
66 66
             $prevPos = 0;
67 67
         }
68 68
         
69
-        if($this->isWithinCommand($prevPos))
69
+        if ($this->isWithinCommand($prevPos))
70 70
         {
71 71
             return;
72 72
         }
73 73
         
74 74
         $prev = $this->subject->getSubstr($prevPos, $this->eolLength);
75 75
         
76
-        if($prev !== $this->formatter->getEOLChar())
76
+        if ($prev !== $this->formatter->getEOLChar())
77 77
         {
78 78
             $this->prepend = $this->eol;
79 79
         }
@@ -86,21 +86,21 @@  discard block
 block discarded – undo
86 86
         $position = $this->getEndPosition();
87 87
         
88 88
         // we're at the end of the string
89
-        if($position === false || $position >= $subjectLength)
89
+        if ($position === false || $position >= $subjectLength)
90 90
         {
91 91
             return;
92 92
         }
93 93
         
94 94
         $nextPos = $position + $this->eolLength;
95 95
         
96
-        if($nextPos > $subjectLength)
96
+        if ($nextPos > $subjectLength)
97 97
         {
98 98
             $nextPos = $subjectLength - $this->eolLength;
99 99
         }
100 100
         
101 101
         $next = $this->subject->getSubstr($nextPos, $this->eolLength);
102 102
         
103
-        if($next !== $this->formatter->getEOLChar())
103
+        if ($next !== $this->formatter->getEOLChar())
104 104
         {
105 105
             $this->append = $this->eol;
106 106
         }
Please login to merge, or discard this patch.
src/Mailcode/Parser/Safeguard/Formatter/Type/MarkVariables/Location.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@
 block discarded – undo
32 32
     
33 33
     public function requiresAdjustment(): bool
34 34
     {
35
-        if(!parent::requiresAdjustment())
35
+        if (!parent::requiresAdjustment())
36 36
         {
37 37
             return false;
38 38
         }
Please login to merge, or discard this patch.
src/Mailcode/Parser/Safeguard/Formatter/Type/HTMLHighlighting/Location.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
     {
54 54
         $pos = $this->getStartPosition();
55 55
         
56
-        if(is_bool($pos))
56
+        if (is_bool($pos))
57 57
         {
58 58
             return '';
59 59
         }
@@ -75,9 +75,9 @@  discard block
 block discarded – undo
75 75
     {
76 76
         $tagNames = $this->getParentTags();
77 77
         
78
-        foreach($tagNames as $tagName)
78
+        foreach ($tagNames as $tagName)
79 79
         {
80
-            if($this->formatter->isTagExcluded($tagName))
80
+            if ($this->formatter->isTagExcluded($tagName))
81 81
             {
82 82
                 return true;
83 83
             }
@@ -99,11 +99,11 @@  discard block
 block discarded – undo
99 99
         // Create a stack of all direct parent tags of the command.
100 100
         // Handles closing tags as well.
101 101
         $stack = array();
102
-        foreach($tags[2] as $idx => $tagName)
102
+        foreach ($tags[2] as $idx => $tagName)
103 103
         {
104 104
             $closing = $tags[1][$idx] === '/';
105 105
             
106
-            if($closing)
106
+            if ($closing)
107 107
             {
108 108
                 array_pop($stack);
109 109
             }
@@ -128,9 +128,9 @@  discard block
 block discarded – undo
128 128
         
129 129
         // This check is easy: if the command is in the attribute
130 130
         // of any of the tags, we will find the command marker in there.
131
-        foreach($tags[3] as $attributes)
131
+        foreach ($tags[3] as $attributes)
132 132
         {
133
-            if(strstr($attributes, self::COMMAND_MARKER))
133
+            if (strstr($attributes, self::COMMAND_MARKER))
134 134
             {
135 135
                 return true;
136 136
             }
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
     
142 142
     private function getTagAncestry() : array
143 143
     {
144
-        if($this->ancestryChecked)
144
+        if ($this->ancestryChecked)
145 145
         {
146 146
             return $this->tagAncestry;
147 147
         }
Please login to merge, or discard this patch.
src/Mailcode/Parser/Safeguard/Formatter/Type/MarkVariables.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@
 block discarded – undo
40 40
     
41 41
     public function setTemplate(string $template) : void
42 42
     {
43
-        if(substr_count($template, '%s') !== 1)
43
+        if (substr_count($template, '%s') !== 1)
44 44
         {
45 45
             throw new Mailcode_Exception(
46 46
                 'Invalid tag template',
Please login to merge, or discard this patch.
src/Mailcode/Parser/Safeguard/Formatter/FormatType.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@
 block discarded – undo
34 34
     {
35 35
         $locations = $this->resolveLocations();
36 36
         
37
-        foreach($locations as $location)
37
+        foreach ($locations as $location)
38 38
         {
39 39
             $location->format();
40 40
         }
Please login to merge, or discard this patch.
src/Mailcode/Parser/Safeguard/Formatter/Location.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -84,23 +84,23 @@  discard block
 block discarded – undo
84 84
         
85 85
         $placeholderID = $this->placeholder->getID();
86 86
         
87
-        foreach($placeholders as $placeholder)
87
+        foreach ($placeholders as $placeholder)
88 88
         {
89
-            if($placeholder->getID() === $placeholderID)
89
+            if ($placeholder->getID() === $placeholderID)
90 90
             {
91 91
                 continue;
92 92
             }
93 93
             
94 94
             $start = $this->subject->getSubstrPosition($placeholder->getReplacementText());
95 95
             
96
-            if($start === false)
96
+            if ($start === false)
97 97
             {
98 98
                 continue;
99 99
             }
100 100
             
101 101
             $end = $start + $placeholder->getLength();
102 102
             
103
-            if($position >= $start && $position <= $end)
103
+            if ($position >= $start && $position <= $end)
104 104
             {
105 105
                 return true;
106 106
             }
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
     {
117 117
         $start = $this->getStartPosition();
118 118
         
119
-        if($start !== false)
119
+        if ($start !== false)
120 120
         {
121 121
             return $start + $this->placeholder->getLength();
122 122
         }
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
     {
147 147
         $needle = $this->placeholder->getReplacementText();
148 148
         
149
-        if($this->subject->replaceSubstrings($needle, $replacementText))
149
+        if ($this->subject->replaceSubstrings($needle, $replacementText))
150 150
         {
151 151
             return;
152 152
         }
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
     
164 164
     public function format() : void
165 165
     {
166
-        if($this->requiresAdjustment() && (!empty($this->prepend) || !empty($this->append)))
166
+        if ($this->requiresAdjustment() && (!empty($this->prepend) || !empty($this->append)))
167 167
         {
168 168
             $this->replaceWith(sprintf(
169 169
                 '%s%s%s',
Please login to merge, or discard this patch.
src/Mailcode/Parser/Safeguard/Formatter.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
         
73 73
         $instance = new $class($this, $placeholder);
74 74
         
75
-        if($instance instanceof Mailcode_Parser_Safeguard_Formatter_Location)
75
+        if ($instance instanceof Mailcode_Parser_Safeguard_Formatter_Location)
76 76
         {
77 77
             return $instance;
78 78
         }
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
         
101 101
         $result = array();
102 102
         
103
-        foreach($placeholders as $placeholder)
103
+        foreach ($placeholders as $placeholder)
104 104
         {
105 105
             $result[] = $this->createLocation($placeholder);
106 106
         }
@@ -118,7 +118,7 @@  discard block
 block discarded – undo
118 118
     {
119 119
         $eol = ConvertHelper::detectEOLCharacter($subject);
120 120
         
121
-        if($eol)
121
+        if ($eol)
122 122
         {
123 123
             return $eol->getCharacter();
124 124
         }
Please login to merge, or discard this patch.