@@ -52,8 +52,8 @@ |
||
52 | 52 | |
53 | 53 | if ($commonPath) { |
54 | 54 | return [ |
55 | - 'commonPath' => $this->getCommonPath($files), |
|
56 | - 'files' => $files |
|
55 | + 'commonPath' => $this->getCommonPath($files), |
|
56 | + 'files' => $files |
|
57 | 57 | ]; |
58 | 58 | } |
59 | 59 |
@@ -100,7 +100,7 @@ |
||
100 | 100 | foreach ($subStrings as $string) { |
101 | 101 | if (($type === self::PREFIX && \strpos($filename, $string) === 0) || |
102 | 102 | ($type === self::SUFFIX && |
103 | - \substr($filename, -1 * \strlen($string)) === $string)) { |
|
103 | + \substr($filename, -1*\strlen($string)) === $string)) { |
|
104 | 104 | $matched = true; |
105 | 105 | |
106 | 106 | break; |
@@ -224,7 +224,7 @@ discard block |
||
224 | 224 | } |
225 | 225 | |
226 | 226 | $html = HTMLPurifier_Encoder::convertFromUTF8($html, $config, $context); |
227 | - $this->context =& $context; |
|
227 | + $this->context = & $context; |
|
228 | 228 | return $html; |
229 | 229 | } |
230 | 230 | |
@@ -241,7 +241,7 @@ discard block |
||
241 | 241 | { |
242 | 242 | $context_array = array(); |
243 | 243 | $array = array(); |
244 | - foreach($array_of_html as $key=>$value){ |
|
244 | + foreach ($array_of_html as $key=>$value) { |
|
245 | 245 | if (is_array($value)) { |
246 | 246 | $array[$key] = $this->purifyArray($value, $config); |
247 | 247 | } else { |
@@ -40,7 +40,9 @@ discard block |
||
40 | 40 | $c = strcspn($css, ";'\"", $i); |
41 | 41 | $accum .= substr($css, $i, $c); |
42 | 42 | $i += $c; |
43 | - if ($i == $len) break; |
|
43 | + if ($i == $len) { |
|
44 | + break; |
|
45 | + } |
|
44 | 46 | $d = $css[$i]; |
45 | 47 | if ($quoted) { |
46 | 48 | $accum .= $d; |
@@ -57,7 +59,9 @@ discard block |
||
57 | 59 | } |
58 | 60 | } |
59 | 61 | } |
60 | - if ($accum != "") $declarations[] = $accum; |
|
62 | + if ($accum != "") { |
|
63 | + $declarations[] = $accum; |
|
64 | + } |
|
61 | 65 | |
62 | 66 | $propvalues = array(); |
63 | 67 | $new_declarations = ''; |
@@ -79,8 +79,8 @@ |
||
79 | 79 | // Based off of RFC 1738, but amended so that |
80 | 80 | // as per RFC 3696, the top label need only not be all numeric. |
81 | 81 | // The productions describing this are: |
82 | - $a = '[a-z]'; // alpha |
|
83 | - $an = '[a-z0-9]'; // alphanum |
|
82 | + $a = '[a-z]'; // alpha |
|
83 | + $an = '[a-z0-9]'; // alphanum |
|
84 | 84 | $and = "[a-z0-9-$underscore]"; // alphanum | "-" |
85 | 85 | // domainlabel = alphanum | alphanum *( alphanum | "-" ) alphanum |
86 | 86 | $domainlabel = "$an(?:$and*$an)?"; |
@@ -64,7 +64,7 @@ |
||
64 | 64 | } |
65 | 65 | |
66 | 66 | if (!$this->selector) { |
67 | - $id_accumulator =& $context->get('IDAccumulator'); |
|
67 | + $id_accumulator = & $context->get('IDAccumulator'); |
|
68 | 68 | if (isset($id_accumulator->ids[$id])) { |
69 | 69 | return false; |
70 | 70 | } |
@@ -232,7 +232,7 @@ discard block |
||
232 | 232 | if ($this->bcmath) { |
233 | 233 | return bcmul($s1, $s2, $scale); |
234 | 234 | } else { |
235 | - return $this->scale((float)$s1 * (float)$s2, $scale); |
|
235 | + return $this->scale((float)$s1*(float)$s2, $scale); |
|
236 | 236 | } |
237 | 237 | } |
238 | 238 | |
@@ -248,7 +248,7 @@ discard block |
||
248 | 248 | if ($this->bcmath) { |
249 | 249 | return bcdiv($s1, $s2, $scale); |
250 | 250 | } else { |
251 | - return $this->scale((float)$s1 / (float)$s2, $scale); |
|
251 | + return $this->scale((float)$s1/(float)$s2, $scale); |
|
252 | 252 | } |
253 | 253 | } |
254 | 254 |
@@ -103,9 +103,9 @@ |
||
103 | 103 | $parent_def = $definition->info_parent_def; |
104 | 104 | $stack = array( |
105 | 105 | array($top_node, |
106 | - $parent_def->descendants_are_inline, |
|
107 | - $parent_def->excludes, // exclusions |
|
108 | - 0) |
|
106 | + $parent_def->descendants_are_inline, |
|
107 | + $parent_def->excludes, // exclusions |
|
108 | + 0) |
|
109 | 109 | ); |
110 | 110 | |
111 | 111 | while (!empty($stack)) { |
@@ -61,7 +61,7 @@ |
||
61 | 61 | $context->register('IsInline', $is_inline); |
62 | 62 | |
63 | 63 | // setup error collector |
64 | - $e =& $context->get('ErrorCollector', true); |
|
64 | + $e = & $context->get('ErrorCollector', true); |
|
65 | 65 | |
66 | 66 | //####################################################################// |
67 | 67 | // Loop initialization |
@@ -128,19 +128,25 @@ discard block |
||
128 | 128 | break; |
129 | 129 | } |
130 | 130 | }; |
131 | - if ($go) continue; |
|
131 | + if ($go) { |
|
132 | + continue; |
|
133 | + } |
|
132 | 134 | list($token, $d) = $node->toTokenPair(); |
133 | 135 | // base case |
134 | 136 | if ($excludes_enabled && isset($excludes[$node->name])) { |
135 | 137 | $node->dead = true; |
136 | - if ($e) $e->send(E_ERROR, 'Strategy_FixNesting: Node excluded'); |
|
138 | + if ($e) { |
|
139 | + $e->send(E_ERROR, 'Strategy_FixNesting: Node excluded'); |
|
140 | + } |
|
137 | 141 | } else { |
138 | 142 | // XXX I suppose it would be slightly more efficient to |
139 | 143 | // avoid the allocation here and have children |
140 | 144 | // strategies handle it |
141 | 145 | $children = array(); |
142 | 146 | foreach ($node->children as $child) { |
143 | - if (!$child->dead) $children[] = $child; |
|
147 | + if (!$child->dead) { |
|
148 | + $children[] = $child; |
|
149 | + } |
|
144 | 150 | } |
145 | 151 | $result = $def->child->validateChildren($children, $config, $context); |
146 | 152 | if ($result === true) { |
@@ -148,7 +154,9 @@ discard block |
||
148 | 154 | $node->children = $children; |
149 | 155 | } elseif ($result === false) { |
150 | 156 | $node->dead = true; |
151 | - if ($e) $e->send(E_ERROR, 'Strategy_FixNesting: Node removed'); |
|
157 | + if ($e) { |
|
158 | + $e->send(E_ERROR, 'Strategy_FixNesting: Node removed'); |
|
159 | + } |
|
152 | 160 | } else { |
153 | 161 | $node->children = $result; |
154 | 162 | if ($e) { |
@@ -151,8 +151,8 @@ |
||
151 | 151 | |
152 | 152 | // isset is in loop because $tokens size changes during loop exec |
153 | 153 | for (;; |
154 | - // only increment if we don't need to reprocess |
|
155 | - $reprocess ? $reprocess = false : $token = $zipper->next($token)) { |
|
154 | + // only increment if we don't need to reprocess |
|
155 | + $reprocess ? $reprocess = false : $token = $zipper->next($token)) { |
|
156 | 156 | |
157 | 157 | // check for a rewind |
158 | 158 | if (is_int($i)) { |
@@ -82,10 +82,10 @@ |
||
82 | 82 | $stack = array(); |
83 | 83 | |
84 | 84 | // member variables |
85 | - $this->stack =& $stack; |
|
86 | - $this->tokens =& $tokens; |
|
87 | - $this->token =& $token; |
|
88 | - $this->zipper =& $zipper; |
|
85 | + $this->stack = & $stack; |
|
86 | + $this->tokens = & $tokens; |
|
87 | + $this->token = & $token; |
|
88 | + $this->zipper = & $zipper; |
|
89 | 89 | $this->config = $config; |
90 | 90 | $this->context = $context; |
91 | 91 |
@@ -162,7 +162,9 @@ |
||
162 | 162 | $rewind_offset = $this->injectors[$i]->getRewindOffset(); |
163 | 163 | if (is_int($rewind_offset)) { |
164 | 164 | for ($j = 0; $j < $rewind_offset; $j++) { |
165 | - if (empty($zipper->front)) break; |
|
165 | + if (empty($zipper->front)) { |
|
166 | + break; |
|
167 | + } |
|
166 | 168 | $token = $zipper->prev($token); |
167 | 169 | // indicate that other injectors should not process this token, |
168 | 170 | // but we need to reprocess it. See Note [Injector skips] |