@@ -126,10 +126,13 @@ discard block |
||
| 126 | 126 | $args = join(',', array_map(function($v) { |
| 127 | 127 | if(is_array($v) && $v['type'] ?? false === 'expr') { |
| 128 | 128 | return self::renderExpr($v); |
| 129 | - } else |
|
| 130 | - return self::varToCode($v); |
|
| 129 | + } else { |
|
| 130 | + return self::varToCode($v); |
|
| 131 | + } |
|
| 131 | 132 | } , $filter['args'])); |
| 132 | - if(strlen($args)) $args = ','.$args; |
|
| 133 | + if(strlen($args)) { |
|
| 134 | + $args = ','.$args; |
|
| 135 | + } |
|
| 133 | 136 | $o = "$func($o" . $args . ")"; |
| 134 | 137 | if($filter['chain'] ?? false) { |
| 135 | 138 | $o = self::addFilter($o, $filter['chain']); |
@@ -202,8 +205,9 @@ discard block |
||
| 202 | 205 | self::varToCode($limit), |
| 203 | 206 | self::varToCode($offset) |
| 204 | 207 | ]; |
| 205 | - if ($addGroup) |
|
| 206 | - $opt[] = self::varToCode($group); |
|
| 208 | + if ($addGroup) { |
|
| 209 | + $opt[] = self::varToCode($group); |
|
| 210 | + } |
|
| 207 | 211 | return join(',', $opt); |
| 208 | 212 | } |
| 209 | 213 | |
@@ -239,10 +243,12 @@ discard block |
||
| 239 | 243 | $id = null; |
| 240 | 244 | $type = []; |
| 241 | 245 | foreach ($n['query'] ?? [] as $q) { |
| 242 | - if ($q['q'] === 'id') |
|
| 243 | - $id = $q['id'] ?? null; |
|
| 244 | - if ($q['q'] === 'type') |
|
| 245 | - $type = $q['id']; |
|
| 246 | + if ($q['q'] === 'id') { |
|
| 247 | + $id = $q['id'] ?? null; |
|
| 248 | + } |
|
| 249 | + if ($q['q'] === 'type') { |
|
| 250 | + $type = $q['id']; |
|
| 251 | + } |
|
| 246 | 252 | } |
| 247 | 253 | if ($n['query'] === null) { |
| 248 | 254 | $o = 'Hamle\Scope::get(0)'; |
@@ -291,8 +297,9 @@ discard block |
||
| 291 | 297 | foreach ($this->tree as $node) { |
| 292 | 298 | switch ($node['type']) { |
| 293 | 299 | case 'string': |
| 294 | - if ($node['body'] !== '') |
|
| 295 | - $out .= $node['body']; |
|
| 300 | + if ($node['body'] !== '') { |
|
| 301 | + $out .= $node['body']; |
|
| 302 | + } |
|
| 296 | 303 | break; |
| 297 | 304 | case 'scopeName': |
| 298 | 305 | $out .= '<?=' . self::renderScopeName($node) . '?>'; |
@@ -322,8 +329,9 @@ discard block |
||
| 322 | 329 | foreach ($this->tree as $node) { |
| 323 | 330 | switch ($node['type']) { |
| 324 | 331 | case 'string': |
| 325 | - if ($node['body'] !== '') |
|
| 326 | - $out[] = self::varToCode($node['body']); |
|
| 332 | + if ($node['body'] !== '') { |
|
| 333 | + $out[] = self::varToCode($node['body']); |
|
| 334 | + } |
|
| 327 | 335 | break; |
| 328 | 336 | case 'scopeThis': |
| 329 | 337 | $out[] = self::renderScopeThis($node); |
@@ -82,21 +82,23 @@ discard block |
||
| 82 | 82 | $scopeName = $m[1]; |
| 83 | 83 | $lookup = substr($this->var, 0, strlen($this->var) - strlen($m[0])); |
| 84 | 84 | $hsv = new H\Text(trim($lookup), H\Text::TOKEN_CONTROL); |
| 85 | - } else |
|
| 86 | - $hsv = new H\Text($this->var, H\Text::TOKEN_CONTROL); |
|
| 85 | + } else { |
|
| 86 | + $hsv = new H\Text($this->var, H\Text::TOKEN_CONTROL); |
|
| 87 | + } |
|
| 87 | 88 | } |
| 88 | 89 | switch ($this->type) { |
| 89 | 90 | case "each": |
| 90 | - if ($this->var) |
|
| 91 | - $out .= "foreach(" . $hsv->toPHP() . " as {$this->o}) { \n"; |
|
| 92 | - else |
|
| 93 | - $out .= "foreach(Hamle\\Scope::get() as {$this->o}) { \n"; |
|
| 91 | + if ($this->var) { |
|
| 92 | + $out .= "foreach(" . $hsv->toPHP() . " as {$this->o}) { \n"; |
|
| 93 | + } else { |
|
| 94 | + $out .= "foreach(Hamle\\Scope::get() as {$this->o}) { \n"; |
|
| 95 | + } |
|
| 94 | 96 | $out .= "Hamle\\Scope::add({$this->o}); "; |
| 95 | 97 | break; |
| 96 | 98 | case "with": |
| 97 | - if ($scopeName) |
|
| 98 | - $out .= "Hamle\\Scope::add(" . $hsv->toPHP() . ", \"$scopeName\");\n;"; |
|
| 99 | - else { |
|
| 99 | + if ($scopeName) { |
|
| 100 | + $out .= "Hamle\\Scope::add(" . $hsv->toPHP() . ", \"$scopeName\");\n;"; |
|
| 101 | + } else { |
|
| 100 | 102 | $out .= "if(({$this->o} = " . $hsv->toPHP() . ") && " . |
| 101 | 103 | "{$this->o}->valid()) {\n"; |
| 102 | 104 | $out .= "Hamle\\Scope::add({$this->o});\n;"; |
@@ -123,8 +125,9 @@ discard block |
||
| 123 | 125 | case "each"; |
| 124 | 126 | $out .= 'Hamle\\Scope::done(); '; |
| 125 | 127 | $out .= '}'; |
| 126 | - if (!$this->var) |
|
| 127 | - $out .= "Hamle\\Scope::get()->rewind();\n"; |
|
| 128 | + if (!$this->var) { |
|
| 129 | + $out .= "Hamle\\Scope::get()->rewind();\n"; |
|
| 130 | + } |
|
| 128 | 131 | break; |
| 129 | 132 | case "if": |
| 130 | 133 | case "else": |
@@ -140,7 +143,9 @@ discard block |
||
| 140 | 143 | return ""; |
| 141 | 144 | break; |
| 142 | 145 | } |
| 143 | - if ($this->else) $out .= "else{"; |
|
| 146 | + if ($this->else) { |
|
| 147 | + $out .= "else{"; |
|
| 148 | + } |
|
| 144 | 149 | return $out . "\n?>"; |
| 145 | 150 | } |
| 146 | 151 | |
@@ -148,9 +153,12 @@ discard block |
||
| 148 | 153 | $ind = $minify ? '' : str_pad('', $indent); |
| 149 | 154 | $oneliner = (!(count($this->content) > 1 || $this->tags)); |
| 150 | 155 | $out = $this->renderStTag(); |
| 151 | - if ($this->content) $out .= $this->renderContent($ind, $oneliner || $minify); |
|
| 152 | - foreach ($this->tags as $tag) |
|
| 153 | - $out .= $tag->render($indent, $minify); |
|
| 156 | + if ($this->content) { |
|
| 157 | + $out .= $this->renderContent($ind, $oneliner || $minify); |
|
| 158 | + } |
|
| 159 | + foreach ($this->tags as $tag) { |
|
| 160 | + $out .= $tag->render($indent, $minify); |
|
| 161 | + } |
|
| 154 | 162 | $out .= $this->renderEnTag(); |
| 155 | 163 | return $out; |
| 156 | 164 | } |
@@ -301,14 +301,14 @@ |
||
| 301 | 301 | private function peg_f1($i) { $return = array(); |
| 302 | 302 | array_walk_recursive($i, function($a) use (&$return) { $return[] = $a; }); |
| 303 | 303 | return $return; |
| 304 | - } |
|
| 304 | + } |
|
| 305 | 305 | private function peg_f2($text) { return new \Seufert\Hamle\TextNode\StringLit(join('',$text)); } |
| 306 | 306 | private function peg_f3($body) { return $body; } |
| 307 | 307 | private function peg_f4($name) { |
| 308 | 308 | return new \Seufert\Hamle\TextNode\ScopeId(null, null, new \Seufert\Hamle\TextNode\ModelParam($name)); } |
| 309 | 309 | private function peg_f5($expr, $chain) { if(!$chain) return $expr; |
| 310 | - $top = array_pop($chain); |
|
| 311 | - while($chain) { $top = array_pop($chain)->withChain($top); } return $expr->withChain($top); } |
|
| 310 | + $top = array_pop($chain); |
|
| 311 | + while($chain) { $top = array_pop($chain)->withChain($top); } return $expr->withChain($top); } |
|
| 312 | 312 | private function peg_f6($sub) { return $sub; } |
| 313 | 313 | private function peg_f7($filter) { return $filter; } |
| 314 | 314 | private function peg_f8($name) { return new \Seufert\Hamle\TextNode\ModelParam($name, null); } |
@@ -306,7 +306,9 @@ |
||
| 306 | 306 | private function peg_f3($body) { return $body; } |
| 307 | 307 | private function peg_f4($name) { |
| 308 | 308 | return new \Seufert\Hamle\TextNode\ScopeId(null, null, new \Seufert\Hamle\TextNode\ModelParam($name)); } |
| 309 | - private function peg_f5($expr, $chain) { if(!$chain) return $expr; |
|
| 309 | + private function peg_f5($expr, $chain) { if(!$chain) { |
|
| 310 | + return $expr; |
|
| 311 | + } |
|
| 310 | 312 | $top = array_pop($chain); |
| 311 | 313 | while($chain) { $top = array_pop($chain)->withChain($top); } return $expr->withChain($top); } |
| 312 | 314 | private function peg_f6($sub) { return $sub; } |