Passed
Push — master ( e9ba68...238ca7 )
by Jean-Christophe
02:39
created
Ajax/common/html/HtmlCollection.php 2 patches
Spacing   +52 added lines, -52 removed lines patch added patch discarded remove patch
@@ -14,44 +14,44 @@  discard block
 block discarded – undo
14 14
  */
15 15
 abstract class HtmlCollection extends HtmlDoubleElement {
16 16
 
17
-	public function __construct($identifier,$tagName="div"){
18
-		parent::__construct($identifier,$tagName);
17
+	public function __construct($identifier, $tagName="div") {
18
+		parent::__construct($identifier, $tagName);
19 19
 		$this->content=array();
20 20
 	}
21 21
 
22
-	public function addItems($items){
23
-		if(JArray::isAssociative($items)){
24
-			foreach ($items as $k=>$v){
25
-				$this->addItem([$k,$v]);
22
+	public function addItems($items) {
23
+		if (JArray::isAssociative($items)) {
24
+			foreach ($items as $k=>$v) {
25
+				$this->addItem([$k, $v]);
26 26
 			}
27
-		}else{
28
-			foreach ($items as $item){
27
+		} else {
28
+			foreach ($items as $item) {
29 29
 				$this->addItem($item);
30 30
 			}
31 31
 		}
32 32
 		return $this;
33 33
 	}
34 34
 
35
-	public function setItems($items){
35
+	public function setItems($items) {
36 36
 		$this->content=$items;
37 37
 		return $this;
38 38
 	}
39 39
 
40
-	public function getItems(){
40
+	public function getItems() {
41 41
 		return $this->content;
42 42
 	}
43 43
 
44
-	protected function getItemToAdd($item){
44
+	protected function getItemToAdd($item) {
45 45
 		$itemO=$item;
46
-		if($this->createCondition($item)===true){
46
+		if ($this->createCondition($item)===true) {
47 47
 			$itemO=$this->createItem($item);
48 48
 		}
49 49
 		return $itemO;
50 50
 	}
51 51
 
52
-	protected function setItemIdentifier($item,$classname,$index){
53
-		if($item instanceof BaseWidget){
54
-			if(JString::isNull($item->getIdentifier())){
52
+	protected function setItemIdentifier($item, $classname, $index) {
53
+		if ($item instanceof BaseWidget) {
54
+			if (JString::isNull($item->getIdentifier())) {
55 55
 				$item->setIdentifier($classname."-".$this->identifier."-".$index);
56 56
 			}
57 57
 		}
@@ -62,15 +62,15 @@  discard block
 block discarded – undo
62 62
 	 * @param HtmlDoubleElement|string|array $item
63 63
 	 * @return \Ajax\common\html\HtmlDoubleElement
64 64
 	 */
65
-	public function addItem($item){
65
+	public function addItem($item) {
66 66
 		$itemO=$this->getItemToAdd($item);
67 67
 		$this->addContent($itemO);
68 68
 		return $itemO;
69 69
 	}
70 70
 
71
-	public function insertItem($item,$position=0){
71
+	public function insertItem($item, $position=0) {
72 72
 		$itemO=$this->getItemToAdd($item);
73
-		\array_splice( $this->content, $position, 0, array($itemO));
73
+		\array_splice($this->content, $position, 0, array($itemO));
74 74
 		return $itemO;
75 75
 	}
76 76
 
@@ -93,11 +93,11 @@  discard block
 block discarded – undo
93 93
 		return $this;
94 94
 	}
95 95
 
96
-	public function removeItem($index){
96
+	public function removeItem($index) {
97 97
 		return array_splice($this->content, $index, 1);
98 98
 	}
99 99
 
100
-	public function count(){
100
+	public function count() {
101 101
 		return \sizeof($this->content);
102 102
 	}
103 103
 
@@ -108,8 +108,8 @@  discard block
 block discarded – undo
108 108
 		return $this->addItem($function($object));
109 109
 	}
110 110
 
111
-	public function apply($callBack){
112
-		foreach ($this->content as $item){
111
+	public function apply($callBack) {
112
+		foreach ($this->content as $item) {
113 113
 			$callBack($item);
114 114
 		}
115 115
 		return $this;
@@ -129,23 +129,23 @@  discard block
 block discarded – undo
129 129
 	 */
130 130
 	abstract protected function createItem($value);
131 131
 
132
-	protected function createCondition($value){
132
+	protected function createCondition($value) {
133 133
 		return \is_object($value)===false;
134 134
 	}
135 135
 
136
-	protected function contentAs($tagName){
137
-		foreach ($this->content as $item){
136
+	protected function contentAs($tagName) {
137
+		foreach ($this->content as $item) {
138 138
 			$item->setTagName($tagName);
139 139
 		}
140 140
 		return $this;
141 141
 	}
142 142
 
143
-	public function setProperties($properties){
143
+	public function setProperties($properties) {
144 144
 		$i=0;
145
-		foreach ($properties as $k=>$v){
145
+		foreach ($properties as $k=>$v) {
146 146
 			$c=$this->content[$i++];
147
-			if(isset($c))
148
-				$c->setProperty($k,$v);
147
+			if (isset($c))
148
+				$c->setProperty($k, $v);
149 149
 			else
150 150
 				return $this;
151 151
 		}
@@ -158,17 +158,17 @@  discard block
 block discarded – undo
158 158
 	 * @param array $values
159 159
 	 * @return HtmlCollection
160 160
 	 */
161
-	public function setPropertyValues($property,$values){
161
+	public function setPropertyValues($property, $values) {
162 162
 		$i=0;
163
-		if(\is_array($values)===false){
164
-			$values=\array_fill(0, $this->count(),$values);
163
+		if (\is_array($values)===false) {
164
+			$values=\array_fill(0, $this->count(), $values);
165 165
 		}
166
-		foreach ($values as $value){
166
+		foreach ($values as $value) {
167 167
 			$c=$this->content[$i++];
168
-			if(isset($c)){
169
-				$c->setProperty($property,$value);
168
+			if (isset($c)) {
169
+				$c->setProperty($property, $value);
170 170
 			}
171
-			else{
171
+			else {
172 172
 				return $this;
173 173
 			}
174 174
 		}
@@ -181,17 +181,17 @@  discard block
 block discarded – undo
181 181
 	 * @param array $values
182 182
 	 * @return HtmlCollection
183 183
 	 */
184
-	public function addPropertyValues($property,$values){
184
+	public function addPropertyValues($property, $values) {
185 185
 		$i=0;
186
-		if(\is_array($values)===false){
187
-			$values=\array_fill(0, $this->count(),$values);
186
+		if (\is_array($values)===false) {
187
+			$values=\array_fill(0, $this->count(), $values);
188 188
 		}
189
-		foreach ($values as $value){
189
+		foreach ($values as $value) {
190 190
 			$c=$this->content[$i++];
191
-			if(isset($c)){
192
-				$c->addToProperty($property,$value);
191
+			if (isset($c)) {
192
+				$c->addToProperty($property, $value);
193 193
 			}
194
-			else{
194
+			else {
195 195
 				return $this;
196 196
 			}
197 197
 		}
@@ -201,13 +201,13 @@  discard block
 block discarded – undo
201 201
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
202 202
 		$index=0;
203 203
 		$classname=\strtolower(JReflection::shortClassName($this));
204
-		foreach ($this->content as $item){
205
-			$this->setItemIdentifier($item,$classname,$index++);
204
+		foreach ($this->content as $item) {
205
+			$this->setItemIdentifier($item, $classname, $index++);
206 206
 		}
207
-		return parent::compile($js,$view);
207
+		return parent::compile($js, $view);
208 208
 	}
209 209
 
210
-	public function getItemById($identifier){
210
+	public function getItemById($identifier) {
211 211
 		return $this->getElementById($identifier, $this->content);
212 212
 	}
213 213
 
@@ -216,13 +216,13 @@  discard block
 block discarded – undo
216 216
 	 * @param string $target
217 217
 	 * @return HtmlCollection
218 218
 	 */
219
-	public function asLinks($hrefs=[],$target=NUll) {
220
-		foreach ( $this->content as $index=>$item ) {
221
-			if($item instanceof HtmlDoubleElement){
219
+	public function asLinks($hrefs=[], $target=NUll) {
220
+		foreach ($this->content as $index=>$item) {
221
+			if ($item instanceof HtmlDoubleElement) {
222 222
 				$href="";
223
-				if(isset($hrefs[$index]))
223
+				if (isset($hrefs[$index]))
224 224
 					$href=$hrefs[$index];
225
-				$item->asLink($href,$target);
225
+				$item->asLink($href, $target);
226 226
 			}
227 227
 		}
228 228
 		return $this;
Please login to merge, or discard this patch.
Braces   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 			foreach ($items as $k=>$v){
25 25
 				$this->addItem([$k,$v]);
26 26
 			}
27
-		}else{
27
+		} else{
28 28
 			foreach ($items as $item){
29 29
 				$this->addItem($item);
30 30
 			}
@@ -80,9 +80,9 @@  discard block
 block discarded – undo
80 80
 	 * @return \Ajax\common\html\HtmlDoubleElement
81 81
 	 */
82 82
 	public function getItem($index) {
83
-		if (is_int($index))
84
-			return $this->content[$index];
85
-		else {
83
+		if (is_int($index)) {
84
+					return $this->content[$index];
85
+		} else {
86 86
 			$elm=$this->getElementById($index, $this->content);
87 87
 			return $elm;
88 88
 		}
@@ -144,10 +144,11 @@  discard block
 block discarded – undo
144 144
 		$i=0;
145 145
 		foreach ($properties as $k=>$v){
146 146
 			$c=$this->content[$i++];
147
-			if(isset($c))
148
-				$c->setProperty($k,$v);
149
-			else
150
-				return $this;
147
+			if(isset($c)) {
148
+							$c->setProperty($k,$v);
149
+			} else {
150
+							return $this;
151
+			}
151 152
 		}
152 153
 		return $this;
153 154
 	}
@@ -167,8 +168,7 @@  discard block
 block discarded – undo
167 168
 			$c=$this->content[$i++];
168 169
 			if(isset($c)){
169 170
 				$c->setProperty($property,$value);
170
-			}
171
-			else{
171
+			} else{
172 172
 				return $this;
173 173
 			}
174 174
 		}
@@ -190,8 +190,7 @@  discard block
 block discarded – undo
190 190
 			$c=$this->content[$i++];
191 191
 			if(isset($c)){
192 192
 				$c->addToProperty($property,$value);
193
-			}
194
-			else{
193
+			} else{
195 194
 				return $this;
196 195
 			}
197 196
 		}
@@ -220,8 +219,9 @@  discard block
 block discarded – undo
220 219
 		foreach ( $this->content as $index=>$item ) {
221 220
 			if($item instanceof HtmlDoubleElement){
222 221
 				$href="";
223
-				if(isset($hrefs[$index]))
224
-					$href=$hrefs[$index];
222
+				if(isset($hrefs[$index])) {
223
+									$href=$hrefs[$index];
224
+				}
225 225
 				$item->asLink($href,$target);
226 226
 			}
227 227
 		}
Please login to merge, or discard this patch.
Ajax/semantic/html/elements/HtmlList.php 1 patch
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -20,17 +20,17 @@  discard block
 block discarded – undo
20 20
 
21 21
 	protected function createItem($value) {
22 22
 		$count=$this->count();
23
-		$item=new HtmlListItem("item-" . $this->identifier . "-" . $count, $value);
23
+		$item=new HtmlListItem("item-".$this->identifier."-".$count, $value);
24 24
 		return $item;
25 25
 	}
26 26
 
27 27
 	public function addHeader($niveau, $content) {
28
-		$header=new HtmlHeader("header-" . $this->identifier, $niveau, $content, "page");
28
+		$header=new HtmlHeader("header-".$this->identifier, $niveau, $content, "page");
29 29
 		$this->wrap($header);
30 30
 		return $header;
31 31
 	}
32 32
 
33
-	public function getItemPart($index,$partName="header"){
33
+	public function getItemPart($index, $partName="header") {
34 34
 		return $this->getItem($index)->getPart($partName);
35 35
 	}
36 36
 
@@ -38,9 +38,9 @@  discard block
 block discarded – undo
38 38
 		return $this->contentAs($tagName);
39 39
 	}
40 40
 
41
-	public function asLinks($hrefs=[],$target=NUll) {
42
-		$this->addToPropertyCtrl("class", "link", array ("link" ));
43
-		return parent::asLinks($hrefs,$target);
41
+	public function asLinks($hrefs=[], $target=NUll) {
42
+		$this->addToPropertyCtrl("class", "link", array("link"));
43
+		return parent::asLinks($hrefs, $target);
44 44
 	}
45 45
 
46 46
 	public function addList($items=array()) {
@@ -49,9 +49,9 @@  discard block
 block discarded – undo
49 49
 		return $this->addItem($list);
50 50
 	}
51 51
 
52
-	protected function getItemToAdd($item){
52
+	protected function getItemToAdd($item) {
53 53
 		$itemO=parent::getItemToAdd($item);
54
-		if($itemO instanceof AbstractCheckbox)
54
+		if ($itemO instanceof AbstractCheckbox)
55 55
 			$itemO->addClass("item");
56 56
 		return $itemO;
57 57
 	}
@@ -69,9 +69,9 @@  discard block
 block discarded – undo
69 69
 	}
70 70
 
71 71
 	public function run(JsUtils $js) {
72
-		if ($this->_hasCheckedList === true) {
73
-			$jsCode=include dirname(__FILE__) . '/../../components/jsTemplates/tplCheckedList.php';
74
-			$jsCode=\str_replace("%identifier%", "#" . $this->identifier, $jsCode);
72
+		if ($this->_hasCheckedList===true) {
73
+			$jsCode=include dirname(__FILE__).'/../../components/jsTemplates/tplCheckedList.php';
74
+			$jsCode=\str_replace("%identifier%", "#".$this->identifier, $jsCode);
75 75
 			$jsCode=\str_replace("%fireOnInit%", $this->_fireOnInit, $jsCode);
76 76
 			$this->executeOnRun($jsCode);
77 77
 		}
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 	 * @param boolean $fireOnInit
83 83
 	 */
84 84
 	public function setFireOnInit($fireOnInit) {
85
-		$this->_fireOnInit = $fireOnInit;
85
+		$this->_fireOnInit=$fireOnInit;
86 86
 	}
87 87
 
88 88
 	public function setRelaxed() {
@@ -109,42 +109,42 @@  discard block
 block discarded – undo
109 109
 	 * @param string $notAllChecked
110 110
 	 * @return HtmlList
111 111
 	 */
112
-	public function addCheckedList($items=array(), $masterItem=NULL, $values=array(),$notAllChecked=false,$name=null) {
112
+	public function addCheckedList($items=array(), $masterItem=NULL, $values=array(), $notAllChecked=false, $name=null) {
113 113
 		$count=$this->count();
114
-		$identifier=$this->identifier . "-" . $count;
114
+		$identifier=$this->identifier."-".$count;
115 115
 		if (isset($masterItem)) {
116
-			if(\is_array($masterItem)){
117
-				$masterO=new HtmlFormCheckbox("master-" . $identifier, @$masterItem[0],@$masterItem[1]);
118
-				if(isset($name))
116
+			if (\is_array($masterItem)) {
117
+				$masterO=new HtmlFormCheckbox("master-".$identifier, @$masterItem[0], @$masterItem[1]);
118
+				if (isset($name))
119 119
 					$masterO->setName($name);
120
-				if(isset($masterItem[1])){
121
-					if(\array_search($masterItem[1], $values)!==false){
120
+				if (isset($masterItem[1])) {
121
+					if (\array_search($masterItem[1], $values)!==false) {
122 122
 						$masterO->getDataField()->setProperty("checked", "");
123 123
 					}
124 124
 				}
125
-			}else{
126
-				$masterO=new HtmlFormCheckbox("master-" . $identifier, $masterItem);
125
+			} else {
126
+				$masterO=new HtmlFormCheckbox("master-".$identifier, $masterItem);
127 127
 			}
128
-			if($notAllChecked){
128
+			if ($notAllChecked) {
129 129
 				$masterO->getDataField()->addClass("_notAllChecked");
130 130
 			}
131 131
 			$masterO->getHtmlCk()->addToProperty("class", "master");
132 132
 			$masterO->setClass("item");
133 133
 			$this->addItem($masterO);
134 134
 		}
135
-		$fields=array ();
135
+		$fields=array();
136 136
 		$i=0;
137
-		foreach ( $items as $val => $caption ) {
138
-			$itemO=new HtmlFormCheckbox($identifier . "-" . $i++, $caption, $val, "child");
139
-			if (\array_search($val, $values) !== false) {
137
+		foreach ($items as $val => $caption) {
138
+			$itemO=new HtmlFormCheckbox($identifier."-".$i++, $caption, $val, "child");
139
+			if (\array_search($val, $values)!==false) {
140 140
 				$itemO->getDataField()->setProperty("checked", "");
141 141
 			}
142
-			if(isset($name))
142
+			if (isset($name))
143 143
 				$itemO->setName($name);
144 144
 			$itemO->setClass("item");
145 145
 			$fields[]=$itemO;
146 146
 		}
147
-		if (isset($masterO) === true) {
147
+		if (isset($masterO)===true) {
148 148
 			$list=new HtmlList("", $fields);
149 149
 			$list->setClass("list");
150 150
 			$masterO->addContent($list);
@@ -155,12 +155,12 @@  discard block
 block discarded – undo
155 155
 		return $this;
156 156
 	}
157 157
 
158
-	public function setIcons($icons){
159
-		if(!\is_array($icons)){
158
+	public function setIcons($icons) {
159
+		if (!\is_array($icons)) {
160 160
 			$icons=\array_fill(0, \sizeof($this->content), $icons);
161 161
 		}
162
-		$max=\min(\sizeof($icons),\sizeof($this->content));
163
-		for($i=0;$i<$max;$i++){
162
+		$max=\min(\sizeof($icons), \sizeof($this->content));
163
+		for ($i=0; $i<$max; $i++) {
164 164
 			$this->content[$i]->addIcon($icons[$i]);
165 165
 		}
166 166
 		return $this;
Please login to merge, or discard this patch.