@@ -50,10 +50,12 @@ discard block |
||
50 | 50 | */ |
51 | 51 | public function __construct($s) { |
52 | 52 | $m = array(); |
53 | - if (!preg_match('/^\$\((' . self::REGEX_FUNCSEL . '*)(.*)\)$/', $s, $m)) |
|
54 | - throw new ParseError("Unable to read \$ func in '$s'"); |
|
55 | - if (trim($m[2])) |
|
56 | - $this->sub = new FuncSub($m[2]); |
|
53 | + if (!preg_match('/^\$\((' . self::REGEX_FUNCSEL . '*)(.*)\)$/', $s, $m)) { |
|
54 | + throw new ParseError("Unable to read \$ func in '$s'"); |
|
55 | + } |
|
56 | + if (trim($m[2])) { |
|
57 | + $this->sub = new FuncSub($m[2]); |
|
58 | + } |
|
57 | 59 | if (!trim($m[1])) { |
58 | 60 | $this->scope = true; |
59 | 61 | return; |
@@ -70,16 +72,23 @@ discard block |
||
70 | 72 | $m = array(); |
71 | 73 | $att = array('id' => array(), 'tag' => array()); |
72 | 74 | foreach (explode(",", $s) as $str) { |
73 | - if (preg_match('/^[a-zA-Z0-9_]+/', $str, $m)) $type = $m[0]; |
|
74 | - else $type = "*"; |
|
75 | - if (preg_match('/#([a-zA-Z0-9_${}]+)/', $str, $m)) $att['id'][$type][] = $m[1]; |
|
76 | - elseif (preg_match_all('/\\.([a-zA-Z0-9_\-${}]+)/', $str, $m)) |
|
77 | - foreach ($m[1] as $tag) |
|
75 | + if (preg_match('/^[a-zA-Z0-9_]+/', $str, $m)) { |
|
76 | + $type = $m[0]; |
|
77 | + } else { |
|
78 | + $type = "*"; |
|
79 | + } |
|
80 | + if (preg_match('/#([a-zA-Z0-9_${}]+)/', $str, $m)) { |
|
81 | + $att['id'][$type][] = $m[1]; |
|
82 | + } elseif (preg_match_all('/\\.([a-zA-Z0-9_\-${}]+)/', $str, $m)) { |
|
83 | + foreach ($m[1] as $tag) |
|
78 | 84 | $att['tag'][$type][] = new Text($tag, Text::TOKEN_CODE); |
79 | - else $att['tag'][$type] = array(); |
|
85 | + } else { |
|
86 | + $att['tag'][$type] = array(); |
|
87 | + } |
|
88 | + } |
|
89 | + if (!(count($att['id']) xor count($att['tag']))) { |
|
90 | + throw new ParseError("Only tag, type or id can be combined"); |
|
80 | 91 | } |
81 | - if (!(count($att['id']) xor count($att['tag']))) |
|
82 | - throw new ParseError("Only tag, type or id can be combined"); |
|
83 | 92 | return $att; |
84 | 93 | } |
85 | 94 | |
@@ -92,14 +101,18 @@ discard block |
||
92 | 101 | } |
93 | 102 | $rand = false; |
94 | 103 | if (preg_match_all('/\\^(-?)([a-zA-Z0-9_]*)/', $s, $m)) { |
95 | - foreach($m[0] as $k=>$mv) |
|
96 | - if ($m[2][$k]) { |
|
104 | + foreach($m[0] as $k=>$mv) { |
|
105 | + if ($m[2][$k]) { |
|
97 | 106 | $dir = $m[1][$k] == "-"?Hamle\Hamle::SORT_DESCENDING:Hamle\Hamle::SORT_ASCENDING; |
107 | + } |
|
98 | 108 | $att['sort'][$m[2][$k]] = $dir; |
99 | - } else $rand = true; |
|
109 | + } else { |
|
110 | + $rand = true; |
|
111 | + } |
|
112 | + } |
|
113 | + if($rand) { |
|
114 | + $att['sort'] = [""=>$att['dir'] = Hamle\Hamle::SORT_RANDOM]; |
|
100 | 115 | } |
101 | - if($rand) |
|
102 | - $att['sort'] = [""=>$att['dir'] = Hamle\Hamle::SORT_RANDOM]; |
|
103 | 116 | return $att; |
104 | 117 | } |
105 | 118 | |
@@ -124,17 +137,19 @@ discard block |
||
124 | 137 | } |
125 | 138 | $limit = Text::varToCode($this->sortlimit['sort']) . "," . |
126 | 139 | $this->sortlimit['limit'] . "," . $this->sortlimit['offset']; |
127 | - if (count($this->filt['tag'])) |
|
128 | - return "Hamle\\Run::modelTypeTags(" . |
|
140 | + if (count($this->filt['tag'])) { |
|
141 | + return "Hamle\\Run::modelTypeTags(" . |
|
129 | 142 | Text::varToCode($this->filt['tag']) . ",$limit)$sub"; |
130 | - if (count($this->filt['id'])) |
|
131 | - if (isset($this->filt['id']['*']) && count($this->filt['id']['*']) == 1) |
|
143 | + } |
|
144 | + if (count($this->filt['id'])) { |
|
145 | + if (isset($this->filt['id']['*']) && count($this->filt['id']['*']) == 1) |
|
132 | 146 | return "Hamle\\Run::modelId(" . |
133 | 147 | Text::varToCode(current($this->filt['id']['*'])) . |
134 | 148 | ",$limit)$sub"; |
135 | - else |
|
136 | - return "Hamle\\Run::modelTypeId(" . |
|
149 | + } else { |
|
150 | + return "Hamle\\Run::modelTypeId(" . |
|
137 | 151 | Text::varToCode($this->filt['id']) . ",$limit)$sub"; |
152 | + } |
|
138 | 153 | return ""; |
139 | 154 | } |
140 | 155 | |
@@ -145,34 +160,39 @@ discard block |
||
145 | 160 | public function getOrCreateModel(Model $parent = null) { |
146 | 161 | if($this->scope instanceof Scope) { |
147 | 162 | $parent = $this->scope->getOrCreateModel(); |
148 | - } elseif ($this->scope === true) |
|
149 | - $parent = \Seufert\Hamle\Scope::get(0); |
|
150 | - if ($this->filt && count($this->filt['tag'])) |
|
151 | - $parent = \Seufert\Hamle\Run::modelTypeTags( |
|
163 | + } elseif ($this->scope === true) { |
|
164 | + $parent = \Seufert\Hamle\Scope::get(0); |
|
165 | + } |
|
166 | + if ($this->filt && count($this->filt['tag'])) { |
|
167 | + $parent = \Seufert\Hamle\Run::modelTypeTags( |
|
152 | 168 | $this->filt['tag'], |
153 | 169 | $this->sortlimit['sort'], |
154 | 170 | $this->sortlimit['limit'], |
155 | 171 | $this->sortlimit['offset'] |
156 | 172 | ); |
157 | - if ($this->filt && count($this->filt['id'])) |
|
158 | - if (isset($this->filt['id']['*']) && count($this->filt['id']['*']) === 1) |
|
173 | + } |
|
174 | + if ($this->filt && count($this->filt['id'])) { |
|
175 | + if (isset($this->filt['id']['*']) && count($this->filt['id']['*']) === 1) |
|
159 | 176 | $parent = \Seufert\Hamle\Run::modelId( |
160 | 177 | current($this->filt['id']['*']), |
161 | 178 | $this->sortlimit['sort'], |
162 | 179 | $this->sortlimit['limit'], |
163 | 180 | $this->sortlimit['offset'] |
164 | 181 | ); |
165 | - else |
|
166 | - $parent = \Seufert\Hamle\Run::modelTypeId( |
|
182 | + } else { |
|
183 | + $parent = \Seufert\Hamle\Run::modelTypeId( |
|
167 | 184 | $this->filt['id'], |
168 | 185 | $this->sortlimit['sort'], |
169 | 186 | $this->sortlimit['limit'], |
170 | 187 | $this->sortlimit['offset'] |
171 | 188 | ); |
172 | - if($this->sub) |
|
173 | - return $this->sub->getOrCreateModel($parent)->current(); |
|
174 | - if(!$parent) |
|
175 | - throw new \RuntimeException('Unable to create model with no relation'); |
|
189 | + } |
|
190 | + if($this->sub) { |
|
191 | + return $this->sub->getOrCreateModel($parent)->current(); |
|
192 | + } |
|
193 | + if(!$parent) { |
|
194 | + throw new \RuntimeException('Unable to create model with no relation'); |
|
195 | + } |
|
176 | 196 | return $parent->current(); |
177 | 197 | } |
178 | 198 |
@@ -43,20 +43,24 @@ discard block |
||
43 | 43 | } |
44 | 44 | $s = preg_split("/-[>!]/", $s); |
45 | 45 | // if(count($s) == 1) $s = explode("-!",$s[0]); |
46 | - if (!$s[0]) throw new ParseError("Unable to parse Complex Expression"); |
|
47 | - if ($s[0][1] === '(') |
|
48 | - $this->func = new Text\Func($s[0]); |
|
49 | - elseif ($s[0][1] === '[') |
|
50 | - $this->func = new Text\Scope($s[0]); |
|
51 | - else |
|
52 | - $this->func = new SimpleVar($s[0]); |
|
46 | + if (!$s[0]) { |
|
47 | + throw new ParseError("Unable to parse Complex Expression"); |
|
48 | + } |
|
49 | + if ($s[0][1] === '(') { |
|
50 | + $this->func = new Text\Func($s[0]); |
|
51 | + } elseif ($s[0][1] === '[') { |
|
52 | + $this->func = new Text\Scope($s[0]); |
|
53 | + } else { |
|
54 | + $this->func = new SimpleVar($s[0]); |
|
55 | + } |
|
53 | 56 | array_shift($s); |
54 | 57 | $this->sel = $s; |
55 | 58 | } |
56 | 59 | |
57 | 60 | function toHTML($escape = false) { |
58 | - if($escape) |
|
59 | - return "<?=htmlspecialchars(" .$this->toPHP() . ")?>"; |
|
61 | + if($escape) { |
|
62 | + return "<?=htmlspecialchars(" .$this->toPHP() . ")?>"; |
|
63 | + } |
|
60 | 64 | return "<?=" . $this->toPHP() . "?>"; |
61 | 65 | } |
62 | 66 | function toPHP() { |
@@ -65,18 +69,22 @@ discard block |
||
65 | 69 | function toPHPVar() { |
66 | 70 | if ($this->sel) { |
67 | 71 | $sel = array(); |
68 | - foreach ($this->sel as $s) |
|
69 | - $sel[] = "hamleGet('$s')"; |
|
72 | + foreach ($this->sel as $s) { |
|
73 | + $sel[] = "hamleGet('$s')"; |
|
74 | + } |
|
70 | 75 | return $this->func->toPHP() . "->" . implode('->', $sel); |
71 | - } else |
|
72 | - return $this->func->toPHP(); |
|
76 | + } else { |
|
77 | + return $this->func->toPHP(); |
|
78 | + } |
|
73 | 79 | } |
74 | 80 | |
75 | 81 | function getOrCreateModel(Model $parent = null) { |
76 | - if($this->func instanceof Text\Scope) |
|
77 | - return $this->func->getOrCreateModel($parent); |
|
78 | - if($this->func instanceof Text\Func) |
|
79 | - return $this->func->getOrCreateModel($parent); |
|
82 | + if($this->func instanceof Text\Scope) { |
|
83 | + return $this->func->getOrCreateModel($parent); |
|
84 | + } |
|
85 | + if($this->func instanceof Text\Func) { |
|
86 | + return $this->func->getOrCreateModel($parent); |
|
87 | + } |
|
80 | 88 | throw new RuntimeException('Unsupported func type encountered:'.get_class($this->func)); |
81 | 89 | } |
82 | 90 | |
@@ -85,11 +93,13 @@ discard block |
||
85 | 93 | * @return WriteModel |
86 | 94 | */ |
87 | 95 | function setValue($value) { |
88 | - if(!$this->sel || count($this->sel) != 1) |
|
89 | - throw new RuntimeException('Can only set values, when one var name is present'); |
|
96 | + if(!$this->sel || count($this->sel) != 1) { |
|
97 | + throw new RuntimeException('Can only set values, when one var name is present'); |
|
98 | + } |
|
90 | 99 | $model = $this->getOrCreateModel(); |
91 | - if(!$model instanceof WriteModel) |
|
92 | - throw new RuntimeException('Can only set values on WriteModel, got '.get_class($model)); |
|
100 | + if(!$model instanceof WriteModel) { |
|
101 | + throw new RuntimeException('Can only set values on WriteModel, got '.get_class($model)); |
|
102 | + } |
|
93 | 103 | $model->hamleSet($this->sel[0], $value); |
94 | 104 | return $model; |
95 | 105 | } |
@@ -45,8 +45,9 @@ |
||
45 | 45 | if ($this->escape) { |
46 | 46 | $parse = new H\Text($s, $strtype); |
47 | 47 | $this->content[] = $parse->toHTML($this->escapeVars); |
48 | - } else |
|
49 | - $this->content[] = $s; |
|
48 | + } else { |
|
49 | + $this->content[] = $s; |
|
50 | + } |
|
50 | 51 | } |
51 | 52 | } |
52 | 53 |
@@ -21,12 +21,14 @@ discard block |
||
21 | 21 | static $scopeHook; |
22 | 22 | |
23 | 23 | static function add($model, $name = null) { |
24 | - if (!$model instanceOf Model) |
|
25 | - throw new Unsupported("Unsupported Model (".get_class($model)."), Needs to implement hamleModel Interface"); |
|
26 | - if ($name) |
|
27 | - self::$namedScopes[$name] = $model; |
|
28 | - else |
|
29 | - self::$scopes[] = $model; |
|
24 | + if (!$model instanceOf Model) { |
|
25 | + throw new Unsupported("Unsupported Model (".get_class($model)."), Needs to implement hamleModel Interface"); |
|
26 | + } |
|
27 | + if ($name) { |
|
28 | + self::$namedScopes[$name] = $model; |
|
29 | + } else { |
|
30 | + self::$scopes[] = $model; |
|
31 | + } |
|
30 | 32 | if(self::$scopeHook) { |
31 | 33 | (self::$scopeHook)($model); |
32 | 34 | } |
@@ -47,15 +49,21 @@ discard block |
||
47 | 49 | */ |
48 | 50 | static function get($id = 0) { |
49 | 51 | if (0 == $id) { |
50 | - if ($scope = end(self::$scopes)) |
|
51 | - return $scope; |
|
52 | + if ($scope = end(self::$scopes)) { |
|
53 | + return $scope; |
|
54 | + } |
|
52 | 55 | throw new OutOfScope("Unable to find Scope ($id)"); |
53 | 56 | } |
54 | 57 | $key = $id - 1; |
55 | - if ($id < 0) $key = count(self::$scopes) + $id - 1; |
|
56 | - if ($id == 0) $key = count(self::$scopes) - 1; |
|
57 | - if (!isset(self::$scopes[$key])) |
|
58 | - throw new OutOfScope("Unable to find Scope ($id) or $key"); |
|
58 | + if ($id < 0) { |
|
59 | + $key = count(self::$scopes) + $id - 1; |
|
60 | + } |
|
61 | + if ($id == 0) { |
|
62 | + $key = count(self::$scopes) - 1; |
|
63 | + } |
|
64 | + if (!isset(self::$scopes[$key])) { |
|
65 | + throw new OutOfScope("Unable to find Scope ($id) or $key"); |
|
66 | + } |
|
59 | 67 | return self::$scopes[$key]; |
60 | 68 | } |
61 | 69 | |
@@ -73,8 +81,9 @@ discard block |
||
73 | 81 | self::$namedScopes[$name]->rewind(); |
74 | 82 | return self::$namedScopes[$name]; |
75 | 83 | } else |
76 | - if(self::$returnZeroOnNoScope) |
|
77 | - return new Model\Zero(); |
|
84 | + if(self::$returnZeroOnNoScope) { |
|
85 | + return new Model\Zero(); |
|
86 | + } |
|
78 | 87 | throw new RunTime("Unable to find scope ($name)"); |
79 | 88 | } |
80 | 89 |
@@ -76,9 +76,10 @@ discard block |
||
76 | 76 | } |
77 | 77 | array_shift($path); |
78 | 78 | } |
79 | - foreach ($this->tags as $tag) |
|
80 | - if ($found = $tag->find($path)) |
|
81 | - $list = array_merge($list, $found); |
|
79 | + foreach ($this->tags as $tag) { |
|
80 | + if ($found = $tag->find($path)) |
|
81 | + $list = array_merge($list, $found); |
|
82 | + } |
|
82 | 83 | return $list; |
83 | 84 | } |
84 | 85 | |
@@ -90,7 +91,9 @@ discard block |
||
90 | 91 | */ |
91 | 92 | function replace($path, Tag $newTag) { |
92 | 93 | if ($this->compare($path[0])) { |
93 | - if (count($path) == 1) return $newTag; |
|
94 | + if (count($path) == 1) { |
|
95 | + return $newTag; |
|
96 | + } |
|
94 | 97 | array_shift($path); |
95 | 98 | } |
96 | 99 | foreach ($this->tags as $k => $tag) { |
@@ -103,19 +106,23 @@ discard block |
||
103 | 106 | } |
104 | 107 | |
105 | 108 | function addSnipContent($contentTag, &$tagArray = array(), $key = 0) { |
106 | - foreach ($this->tags as $k => $tag) |
|
107 | - $tag->addSnipContent($contentTag, $this->tags, $k); |
|
109 | + foreach ($this->tags as $k => $tag) { |
|
110 | + $tag->addSnipContent($contentTag, $this->tags, $k); |
|
111 | + } |
|
108 | 112 | } |
109 | 113 | |
110 | 114 | function compare($tic) { |
111 | - if (isset($tic['type']) && $this->type != $tic['type']) |
|
112 | - return false; |
|
115 | + if (isset($tic['type']) && $this->type != $tic['type']) { |
|
116 | + return false; |
|
117 | + } |
|
113 | 118 | if (isset($tic['id']) && |
114 | 119 | !(isset($this->opt['id']) && $tic['id'] == $this->opt['id']) |
115 | - ) |
|
116 | - return false; |
|
117 | - if (array_diff($tic['class'] ?? [], $this->opt['class'] ?? [])) |
|
118 | - return false; |
|
120 | + ) { |
|
121 | + return false; |
|
122 | + } |
|
123 | + if (array_diff($tic['class'] ?? [], $this->opt['class'] ?? [])) { |
|
124 | + return false; |
|
125 | + } |
|
119 | 126 | return true; |
120 | 127 | } |
121 | 128 | |
@@ -125,10 +132,11 @@ discard block |
||
125 | 132 | * @param string $mode Mode to add child [append|prepend] |
126 | 133 | */ |
127 | 134 | function addChild(Tag $tag, $mode = "append") { |
128 | - if ($mode == "prepend") |
|
129 | - array_unshift($this->tags, $tag); |
|
130 | - else |
|
131 | - $this->tags[] = $tag; |
|
135 | + if ($mode == "prepend") { |
|
136 | + array_unshift($this->tags, $tag); |
|
137 | + } else { |
|
138 | + $this->tags[] = $tag; |
|
139 | + } |
|
132 | 140 | } |
133 | 141 | |
134 | 142 | /** |
@@ -142,9 +150,12 @@ discard block |
||
142 | 150 | $ind = $minify ? '' : str_pad('', $indent); |
143 | 151 | $oneliner = (!(count($this->content) > 1 || $this->tags)); |
144 | 152 | $out = $ind . $this->renderStTag() . ($oneliner || $minify ? '' : "\n"); |
145 | - if ($this->content) $out .= $this->renderContent($ind, $oneliner || $minify); |
|
146 | - foreach ($this->tags as $tag) |
|
147 | - $out .= $tag->render($indent + self::INDENT_SIZE, $minify); |
|
153 | + if ($this->content) { |
|
154 | + $out .= $this->renderContent($ind, $oneliner || $minify); |
|
155 | + } |
|
156 | + foreach ($this->tags as $tag) { |
|
157 | + $out .= $tag->render($indent + self::INDENT_SIZE, $minify); |
|
158 | + } |
|
148 | 159 | $out .= ($minify || $oneliner ? '' : $ind) . $this->renderEnTag() . ($minify ? '' : "\n"); |
149 | 160 | return $out; |
150 | 161 | } |
@@ -158,8 +169,9 @@ discard block |
||
158 | 169 | */ |
159 | 170 | function renderContent($pad = "", $oneliner = false) { |
160 | 171 | $out = ""; |
161 | - foreach ($this->content as $c) |
|
162 | - $out .= ($oneliner ? '' : $pad) . $c . ($oneliner ? '' : "\n"); |
|
172 | + foreach ($this->content as $c) { |
|
173 | + $out .= ($oneliner ? '' : $pad) . $c . ($oneliner ? '' : "\n"); |
|
174 | + } |
|
163 | 175 | return $out; |
164 | 176 | } |
165 | 177 |
@@ -32,25 +32,33 @@ discard block |
||
32 | 32 | |
33 | 33 | function __construct($params) { |
34 | 34 | parent::__construct(); |
35 | - if (!preg_match('/^(append|content|prepend|replace)(?: (.*))?$/', $params, $m)) |
|
36 | - throw new Hamle\Exception\ParseError("Unable to parse Snippet($params)"); |
|
35 | + if (!preg_match('/^(append|content|prepend|replace)(?: (.*))?$/', $params, $m)) { |
|
36 | + throw new Hamle\Exception\ParseError("Unable to parse Snippet($params)"); |
|
37 | + } |
|
37 | 38 | $this->type = $m[1]; |
38 | - if (isset($m[2])) |
|
39 | - $this->path = explode(" ", $m[2]); |
|
40 | - else |
|
41 | - $this->path = array(); |
|
42 | - foreach ($this->path as $k => $v) |
|
43 | - $this->path[$k] = self::decodeClassId($v); |
|
39 | + if (isset($m[2])) { |
|
40 | + $this->path = explode(" ", $m[2]); |
|
41 | + } else { |
|
42 | + $this->path = array(); |
|
43 | + } |
|
44 | + foreach ($this->path as $k => $v) { |
|
45 | + $this->path[$k] = self::decodeClassId($v); |
|
46 | + } |
|
44 | 47 | } |
45 | 48 | |
46 | 49 | static function decodeClassId($s) { |
47 | 50 | $out = $m = array(); |
48 | - if(preg_match('/^[a-zA-Z0-9\_]+/', $s, $m)) |
|
49 | - $out['type'] = $m[0]; |
|
51 | + if(preg_match('/^[a-zA-Z0-9\_]+/', $s, $m)) { |
|
52 | + $out['type'] = $m[0]; |
|
53 | + } |
|
50 | 54 | preg_match_all('/[#\.][a-zA-Z0-9\-\_]+/m', $s, $m); |
51 | - if (isset($m[0])) foreach ($m[0] as $ss) { |
|
55 | + if (isset($m[0])) { |
|
56 | + foreach ($m[0] as $ss) { |
|
52 | 57 | if ($ss[0] === "#") $out['id'] = substr($ss, 1); |
53 | - if ($ss[0] === ".") $out['class'][] = substr($ss, 1); |
|
58 | + } |
|
59 | + if ($ss[0] === ".") { |
|
60 | + $out['class'][] = substr($ss, 1); |
|
61 | + } |
|
54 | 62 | } |
55 | 63 | return $out; |
56 | 64 | } |
@@ -62,21 +70,24 @@ discard block |
||
62 | 70 | function addSnipContent($contentTag, &$tagArray = array(), $key = 0) { |
63 | 71 | if ($this->type == "content") { |
64 | 72 | $tagArray[$key] = $contentTag; |
65 | - } else |
|
66 | - parent::addSnipContent($contentTag, $tagArray, $key); |
|
73 | + } else { |
|
74 | + parent::addSnipContent($contentTag, $tagArray, $key); |
|
75 | + } |
|
67 | 76 | } |
68 | 77 | |
69 | 78 | function apply(Hamle\Tag $rootTag) { |
70 | 79 | if ($this->type == "append" or $this->type == "prepend") { |
71 | 80 | $matchTags = $rootTag->find($this->path); |
72 | - foreach ($matchTags as $tag) |
|
73 | - foreach ($this->tags as $t) { |
|
81 | + foreach ($matchTags as $tag) { |
|
82 | + foreach ($this->tags as $t) { |
|
74 | 83 | $tag->addChild($t, $this->type); |
84 | + } |
|
75 | 85 | } |
76 | 86 | } elseif ($this->type == "replace") { |
77 | 87 | $rootTag->replace($this->path, $this); |
78 | - } else |
|
79 | - throw new Hamle\Exception\ParseError("Cant Apply snippet to document '{$this->type}'"); |
|
88 | + } else { |
|
89 | + throw new Hamle\Exception\ParseError("Cant Apply snippet to document '{$this->type}'"); |
|
90 | + } |
|
80 | 91 | } |
81 | 92 | |
82 | 93 | } |
@@ -109,22 +109,25 @@ discard block |
||
109 | 109 | $revSnip = array(); |
110 | 110 | /** @var Tag[] $roots */ |
111 | 111 | $roots = array(); |
112 | - foreach ($this->root as $snip) |
|
113 | - if ($snip instanceOf Tag\Snippet) { |
|
112 | + foreach ($this->root as $snip) { |
|
113 | + if ($snip instanceOf Tag\Snippet) { |
|
114 | 114 | if ($snip->getType() == "append") { |
115 | 115 | array_unshift($revSnip, $snip); |
116 | + } |
|
116 | 117 | } else { |
117 | 118 | $fwdSnip[] = $snip; |
118 | 119 | } |
119 | 120 | } else { |
120 | 121 | $roots[] = $snip; |
121 | 122 | } |
122 | - foreach ($fwdSnip as $snip) |
|
123 | - foreach ($roots as $root) |
|
123 | + foreach ($fwdSnip as $snip) { |
|
124 | + foreach ($roots as $root) |
|
124 | 125 | $snip->apply($root); |
125 | - foreach ($revSnip as $snip) |
|
126 | - foreach ($roots as $root) |
|
126 | + } |
|
127 | + foreach ($revSnip as $snip) { |
|
128 | + foreach ($roots as $root) |
|
127 | 129 | $snip->apply($root); |
130 | + } |
|
128 | 131 | $this->root = $roots; |
129 | 132 | } |
130 | 133 | |
@@ -143,9 +146,11 @@ discard block |
||
143 | 146 | $heir = array(); |
144 | 147 | while ($this->lineNo < $this->lineCount) { |
145 | 148 | $line = $this->lines[$this->lineNo]; |
146 | - if (trim($line)) if (preg_match(self::REGEX_PARSE_LINE, $line, $m)) { |
|
149 | + if (trim($line)) { |
|
150 | + if (preg_match(self::REGEX_PARSE_LINE, $line, $m)) { |
|
147 | 151 | if (FALSE !== strpos($m[1], "\t")) |
148 | 152 | throw new ParseError("Tabs are not supported in templates at this time"); |
153 | + } |
|
149 | 154 | $indent = strlen($m[1]); |
150 | 155 | $tag = isset($m[2]) ? $tag = $m[2] : ""; |
151 | 156 | $classid = isset($m[3]) ? $m[3] : ""; |
@@ -157,13 +162,13 @@ discard block |
||
157 | 162 | unset($m[0]); |
158 | 163 | switch (strlen($code) ? $code[0] : ($textcode ? $textcode : "")) { |
159 | 164 | case "|": //Control Tag |
160 | - if ($code == "|snippet") |
|
161 | - $hTag = new Tag\Snippet($text); |
|
162 | - elseif ($code == "|form") |
|
163 | - $hTag = new Tag\Form($text); |
|
164 | - elseif ($code == "|formhint") |
|
165 | - $hTag = new Tag\FormHint(); |
|
166 | - elseif ($code == "|else") { |
|
165 | + if ($code == "|snippet") { |
|
166 | + $hTag = new Tag\Snippet($text); |
|
167 | + } elseif ($code == "|form") { |
|
168 | + $hTag = new Tag\Form($text); |
|
169 | + } elseif ($code == "|formhint") { |
|
170 | + $hTag = new Tag\FormHint(); |
|
171 | + } elseif ($code == "|else") { |
|
167 | 172 | $hTag = new Tag\Control(substr($code, 1), $heir[$i - 1]); |
168 | 173 | $hTag->setVar($text); |
169 | 174 | } else { |
@@ -174,8 +179,9 @@ discard block |
||
174 | 179 | case ":": //Filter Tag |
175 | 180 | $hTag = new Tag\Filter(substr($code, 1)); |
176 | 181 | $hTag->addContent($text, Text::TOKEN_CODE); |
177 | - foreach ($this->consumeBlock($indent) as $l) |
|
178 | - $hTag->addContent($l, Text::TOKEN_CODE); |
|
182 | + foreach ($this->consumeBlock($indent) as $l) { |
|
183 | + $hTag->addContent($l, Text::TOKEN_CODE); |
|
184 | + } |
|
179 | 185 | break; |
180 | 186 | case "_": //String Tag |
181 | 187 | case "__": //Unescape String Tag |
@@ -191,8 +197,9 @@ discard block |
||
191 | 197 | case "//": // Non Printed Comment |
192 | 198 | $hTag = new Tag\Comment($textcode); |
193 | 199 | $hTag->addContent($text); |
194 | - foreach ($this->consumeBlock($indent) as $l) |
|
195 | - $hTag->addContent($l, Text::TOKEN_CODE); |
|
200 | + foreach ($this->consumeBlock($indent) as $l) { |
|
201 | + $hTag->addContent($l, Text::TOKEN_CODE); |
|
202 | + } |
|
196 | 203 | break; |
197 | 204 | default: |
198 | 205 | $attr = array(); |
@@ -204,25 +211,34 @@ discard block |
||
204 | 211 | } |
205 | 212 | $class = array(); $id = ""; $ref = ""; |
206 | 213 | preg_match_all('/[#\.!][a-zA-Z0-9\-\_]+/m', $classid, $cid); |
207 | - if (isset($cid[0])) foreach ($cid[0] as $s) { |
|
214 | + if (isset($cid[0])) { |
|
215 | + foreach ($cid[0] as $s) { |
|
208 | 216 | if ($s[0] == "#") $id = substr($s, 1); |
209 | - if ($s[0] == ".") $class[] = substr($s, 1); |
|
210 | - if ($s[0] == "!") $ref = substr($s, 1); |
|
211 | 217 | } |
212 | - if($ref) |
|
213 | - $hTag = new Tag\DynHtml($tag, $class, $attr, $id, $ref); |
|
214 | - else |
|
215 | - $hTag = new Tag\Html($tag, $class, $attr, $id); |
|
218 | + if ($s[0] == ".") { |
|
219 | + $class[] = substr($s, 1); |
|
220 | + } |
|
221 | + if ($s[0] == "!") { |
|
222 | + $ref = substr($s, 1); |
|
223 | + } |
|
224 | + } |
|
225 | + if($ref) { |
|
226 | + $hTag = new Tag\DynHtml($tag, $class, $attr, $id, $ref); |
|
227 | + } else { |
|
228 | + $hTag = new Tag\Html($tag, $class, $attr, $id); |
|
229 | + } |
|
216 | 230 | $hTag->addContent($text); |
217 | 231 | break; |
218 | 232 | } |
219 | 233 | $heir[$i] = $hTag; |
220 | - if ($i > 0) |
|
221 | - $heir[$i - 1]->addChild($hTag); |
|
222 | - else |
|
223 | - $this->root[] = $hTag; |
|
224 | - } else |
|
225 | - throw new ParseError("Unable to parse line {$this->lineNo}\n\"$line\"/" . preg_last_error()); |
|
234 | + if ($i > 0) { |
|
235 | + $heir[$i - 1]->addChild($hTag); |
|
236 | + } else { |
|
237 | + $this->root[] = $hTag; |
|
238 | + } |
|
239 | + } else { |
|
240 | + throw new ParseError("Unable to parse line {$this->lineNo}\n\"$line\"/" . preg_last_error()); |
|
241 | + } |
|
226 | 242 | $this->lineNo++; |
227 | 243 | } |
228 | 244 | } |
@@ -238,8 +254,9 @@ discard block |
||
238 | 254 | |
239 | 255 | function output($minify = false) { |
240 | 256 | $out = "<?php\nuse Seufert\\Hamle;\n?>"; |
241 | - foreach ($this->root as $tag) |
|
242 | - $out .= $tag->render(0, $minify); |
|
257 | + foreach ($this->root as $tag) { |
|
258 | + $out .= $tag->render(0, $minify); |
|
259 | + } |
|
243 | 260 | return $out; |
244 | 261 | |
245 | 262 | } |
@@ -251,15 +268,18 @@ discard block |
||
251 | 268 | (!trim($this->lines[$this->lineNo + 1]) || |
252 | 269 | preg_match('/^(\s){' . $indent . '}((\s)+[^\s].*)$/', |
253 | 270 | $this->lines[$this->lineNo + 1], $m))) { |
254 | - if (trim($this->lines[$this->lineNo + 1])) |
|
255 | - $out[] = $m[2]; |
|
271 | + if (trim($this->lines[$this->lineNo + 1])) { |
|
272 | + $out[] = $m[2]; |
|
273 | + } |
|
256 | 274 | $this->lineNo++; |
257 | 275 | } |
258 | 276 | return $out; |
259 | 277 | } |
260 | 278 | |
261 | 279 | function indentLevel($indent) { |
262 | - if (!isset($this->indents)) $this->indents = array(); |
|
280 | + if (!isset($this->indents)) { |
|
281 | + $this->indents = array(); |
|
282 | + } |
|
263 | 283 | if (!count($this->indents)) { |
264 | 284 | $this->indents = array(0 => $indent); |
265 | 285 | // Key = indent level, Value = Depth in spaces |
@@ -49,8 +49,9 @@ discard block |
||
49 | 49 | if(preg_match("/^([a-z_]+)(?:\\((?P<vars>.*)\\))?(?:\\|(?P<chained>.+?))?$/", $s, $m)) { |
50 | 50 | $this->filter = $m[1]; |
51 | 51 | $this->vars = isset($m['vars']) && strlen($m['vars']) ? explode(',', $m['vars']) : []; |
52 | - foreach($this->vars as $k=>$v) |
|
53 | - $this->vars[$k] = str_replace(",",',',$v); |
|
52 | + foreach($this->vars as $k=>$v) { |
|
53 | + $this->vars[$k] = str_replace(",",',',$v); |
|
54 | + } |
|
54 | 55 | if(isset($m['chained']) && strlen($m['chained'])) { |
55 | 56 | $this->chained = new Filter($m['chained'],$what); |
56 | 57 | } |
@@ -71,25 +72,29 @@ discard block |
||
71 | 72 | } |
72 | 73 | |
73 | 74 | function toHTML($escape = false) { |
74 | - if($escape) |
|
75 | - return "<?=htmlspecialchars(" .$this->toPHP() . ")?>"; |
|
75 | + if($escape) { |
|
76 | + return "<?=htmlspecialchars(" .$this->toPHP() . ")?>"; |
|
77 | + } |
|
76 | 78 | return "<?=" . $this->toPHP() . "?>"; |
77 | 79 | } |
78 | 80 | |
79 | 81 | function toPHPpre() { |
80 | 82 | $pre = ''; |
81 | - if($this->chained) |
|
82 | - $pre = $this->chained->toPHPpre(); |
|
83 | + if($this->chained) { |
|
84 | + $pre = $this->chained->toPHPpre(); |
|
85 | + } |
|
83 | 86 | return "$pre{$this->filter}("; |
84 | 87 | } |
85 | 88 | |
86 | 89 | function toPHPpost() { |
87 | 90 | $post = ''; |
88 | - if($this->chained) |
|
89 | - $post = $this->chained->toPHPpost(); |
|
91 | + if($this->chained) { |
|
92 | + $post = $this->chained->toPHPpost(); |
|
93 | + } |
|
90 | 94 | $o = ''; |
91 | - foreach($this->vars as $v) |
|
92 | - $o .= ','.$this->varToCode($v); |
|
95 | + foreach($this->vars as $v) { |
|
96 | + $o .= ','.$this->varToCode($v); |
|
97 | + } |
|
93 | 98 | return "$o)$post"; |
94 | 99 | } |
95 | 100 | |
@@ -104,8 +109,9 @@ discard block |
||
104 | 109 | static function itersplit($v, $sep = ",") { |
105 | 110 | $o = []; |
106 | 111 | foreach(explode($sep, $v) as $k=>$i) { |
107 | - if($i) |
|
108 | - $o[] = ['v'=>trim($i), 'value'=>trim($i), 'k'=>$k,'key'=>$k]; |
|
112 | + if($i) { |
|
113 | + $o[] = ['v'=>trim($i), 'value'=>trim($i), 'k'=>$k,'key'=>$k]; |
|
114 | + } |
|
109 | 115 | } |
110 | 116 | return new WrapArray($o); |
111 | 117 | } |
@@ -66,7 +66,9 @@ discard block |
||
66 | 66 | } |
67 | 67 | |
68 | 68 | function __call($name, $valarray) { |
69 | - if (count($valarray) < 1) return $this->__get($name); |
|
69 | + if (count($valarray) < 1) { |
|
70 | + return $this->__get($name); |
|
71 | + } |
|
70 | 72 | $val = count($valarray) == 1 ? current($valarray) : $valarray; |
71 | 73 | switch ($name) { |
72 | 74 | case "name": |
@@ -111,7 +113,9 @@ discard block |
||
111 | 113 | } |
112 | 114 | |
113 | 115 | function getValue() { |
114 | - if (!is_null($this->setValue)) return $this->setValue; |
|
116 | + if (!is_null($this->setValue)) { |
|
117 | + return $this->setValue; |
|
118 | + } |
|
115 | 119 | if (isset($_REQUEST[$this->form . "_" . $this->name])) { |
116 | 120 | return $_REQUEST[$this->form . "_" . $this->name]; |
117 | 121 | } |
@@ -130,12 +134,15 @@ discard block |
||
130 | 134 | if (!$this->valid) { |
131 | 135 | $atts['class'][] = "hamleFormError"; |
132 | 136 | } |
133 | - if ($this->opt["disabled"]) |
|
134 | - $atts['disabled'] = "disabled"; |
|
135 | - if ($this->opt['required']) |
|
136 | - $atts['required'] = "required"; |
|
137 | - if ($this->opt['help']) |
|
138 | - $atts['title'] = $this->opt['help']; |
|
137 | + if ($this->opt["disabled"]) { |
|
138 | + $atts['disabled'] = "disabled"; |
|
139 | + } |
|
140 | + if ($this->opt['required']) { |
|
141 | + $atts['required'] = "required"; |
|
142 | + } |
|
143 | + if ($this->opt['help']) { |
|
144 | + $atts['title'] = $this->opt['help']; |
|
145 | + } |
|
139 | 146 | } |
140 | 147 | |
141 | 148 | function getLabelAttStatic(&$atts, &$type, &$content) { |
@@ -173,10 +180,12 @@ discard block |
||
173 | 180 | function doProcess($submit) { |
174 | 181 | if ($submit) { |
175 | 182 | $value = $this->getValue(); |
176 | - if ($this->opt['required']) |
|
177 | - $this->valid = $this->valid && strlen($value); |
|
178 | - if ($this->opt['regex']) |
|
179 | - $this->valid = $this->valid && preg_match($this->opt['regex'], $value); |
|
183 | + if ($this->opt['required']) { |
|
184 | + $this->valid = $this->valid && strlen($value); |
|
185 | + } |
|
186 | + if ($this->opt['regex']) { |
|
187 | + $this->valid = $this->valid && preg_match($this->opt['regex'], $value); |
|
188 | + } |
|
180 | 189 | } |
181 | 190 | } |
182 | 191 |