View GitHub Repository phpMv-UI

Schedule Inspection
Subscribe
Passed
Push — master ( 525a36...ddab02 )
by Jean-Christophe
13:28
created
Status
Category
Toggle Patches
Ajax/semantic/html/collections/form/HtmlFormFields.php 2 patches
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch 
@@ -35,7 +35,7 @@  discard block
 
 block discarded – undo
35 35 
 		}
36 36 
 		if (isset($label))
37 37 
 			$this->setLabel($label);
38 
-		foreach ( $fields as $field ) {
38 
+		foreach ($fields as $field) {
39 39 
 			$this->addItem($field);
40 40 
 		}
41 41 
 		return $this;
 
@@ -57,7 +57,7 @@  discard block
 
 block discarded – undo
57 57
58 58 
 	public function addItem($item) {
59 59 
 		$item=parent::addItem($item);
60 
-		if($item instanceof HtmlFormField)
60 
+		if ($item instanceof HtmlFormField)
61 61 
 			$item->setContainer($this);
62 62 
 		return $item;
63 63 
 	}
 
@@ -65,7 +65,7 @@  discard block
 
 block discarded – undo
65 65 
 	/**
66 66 
 	 * @return HtmlFormField
67 67 
 	 */
68 
-	public function getItem($index){
68 
+	public function getItem($index) {
69 69 
 		return parent::getItem($index);
70 70 
 	}
71 71 
 
@@ -104,10 +104,10 @@  discard block
 
 block discarded – undo
104 104 
 	}
105 105
106 106
107 
-	public static function radios($identifier,$name, $items=array(), $label=NULL, $value=null, $type=NULL) {
108 
-		$fields=array ();
107 
+	public static function radios($identifier, $name, $items=array(), $label=NULL, $value=null, $type=NULL) {
108 
+		$fields=array();
109 109 
 		$i=0;
110 
-		foreach ( $items as $val => $caption ) {
110 
+		foreach ($items as $val => $caption) {
111 111 
 			$itemO=new HtmlFormRadio($name."-".$i++, $name, $caption, $val, $type);
112 112 
 			if ($val===$value) {
113 113 
 				$itemO->getDataField()->setProperty("checked", "");
 
@@ -115,19 +115,19 @@  discard block
 
 block discarded – undo
115 115 
 			$fields[]=$itemO;
116 116 
 		}
117 117 
 		$radios=new HtmlFormFields($identifier, $fields);
118 
-		if (isset($label)){
118 
+		if (isset($label)) {
119 119 
 			$lbl=$radios->setLabel($label);
120 
-			if($lbl instanceof HtmlSemDoubleElement){
120 
+			if ($lbl instanceof HtmlSemDoubleElement) {
121 121 
 				$lbl->setProperty("for", $name);
122 122 
 			}
123 123 
 		}
124 124 
 		return $radios;
125 125 
 	}
126 126
127 
-	public static function checkeds($identifier,$name, $items=array(), $label=NULL, $values=array(), $type=NULL) {
128 
-		$fields=array ();
127 
+	public static function checkeds($identifier, $name, $items=array(), $label=NULL, $values=array(), $type=NULL) {
128 
+		$fields=array();
129 129 
 		$i=0;
130 
-		foreach ( $items as $val => $caption ) {
130 
+		foreach ($items as $val => $caption) {
131 131 
 			$itemO=new HtmlFormCheckbox($name."-".$i++, $caption, $val, $type);
132 132 
 			$itemO->setName($name);
133 133 
 			if (\array_search($val, $values)!==false) {
 
@@ -146,7 +146,7 @@  discard block
 
 block discarded – undo
146 146 
 		return $this;
147 147 
 	}
148 148
149 
-	public function run(JsUtils $js){
149 
+	public function run(JsUtils $js) {
150 150 
 		return parent::run($js);
151 151 
 		//return $result->setItemSelector("[data-value]");
152 152 
 	}
Please login to merge, or discard this patch.
Braces   +15 added lines, -10 removed lines patch added patch discarded remove patch 
@@ -29,12 +29,14 @@  discard block
 
 block discarded – undo
29 29 
 				if (\is_string($end)) {
30 30 
 					$label=$end;
31 31 
 					\array_pop($fields);
32 
-				} else
33 
-					$label=NULL;
32 
+				} else {
33 
+									$label=NULL;
34 
+				}
34 35 
 			}
35 36 
 		}
36 
-		if (isset($label))
37 
-			$this->setLabel($label);
37 
+		if (isset($label)) {
38 
+					$this->setLabel($label);
39 
+		}
38 40 
 		foreach ( $fields as $field ) {
39 41 
 			$this->addItem($field);
40 42 
 		}
 
@@ -57,8 +59,9 @@  discard block
 
 block discarded – undo
57 59
58 60 
 	public function addItem($item) {
59 61 
 		$item=parent::addItem($item);
60 
-		if($item instanceof HtmlFormField)
61 
-			$item->setContainer($this);
62 
+		if($item instanceof HtmlFormField) {
63 
+					$item->setContainer($this);
64 
+		}
62 65 
 		return $item;
63 66 
 	}
64 67 
 	
@@ -73,8 +76,9 @@  discard block
 
 block discarded – undo
73 76 
 		if ($this->_equalWidth) {
74 77 
 			$count=$this->count();
75 78 
 			$this->addToProperty("class", Wide::getConstants()["W".$count]." fields");
76 
-		} else
77 
-			$this->addToProperty("class", "fields");
79 
+		} else {
80 
+					$this->addToProperty("class", "fields");
81 
+		}
78 82 
 		return parent::compile($js, $view);
79 83 
 	}
80 84 
 
@@ -136,8 +140,9 @@  discard block
 
 block discarded – undo
136 140 
 			$fields[]=$itemO;
137 141 
 		}
138 142 
 		$radios=new HtmlFormFields($identifier, $fields);
139 
-		if (isset($label))
140 
-			$radios->setLabel($label)->setProperty("for", $name);
143 
+		if (isset($label)) {
144 
+					$radios->setLabel($label)->setProperty("for", $name);
145 
+		}
141 146 
 		return $radios;
142 147 
 	}
143 148
Please login to merge, or discard this patch.
Ajax/semantic/html/content/view/HtmlViewGroups.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -9,7 +9,7 @@  discard block
 
 block discarded – undo
9 9
10 10 
 abstract class HtmlViewGroups extends HtmlSemCollection {
11 11
12 
-	public function __construct($identifier, $uiClass,$items=array()) {
12 
+	public function __construct($identifier, $uiClass, $items=array()) {
13 13 
 		parent::__construct($identifier, "div", $uiClass);
14 14 
 		$this->addItems($items);
15 15 
 	}
 
@@ -41,7 +41,7 @@  discard block
 
 block discarded – undo
41 41 
 	 * @param int $wide
42 42 
 	 */
43 43 
 	public function setWide($wide) {
44 
-		$wide=Wide::getConstants()["W" . $wide];
44 
+		$wide=Wide::getConstants()["W".$wide];
45 45 
 		return $this->addToPropertyCtrl("class", $wide, Wide::getConstants());
46 46 
 	}
47 47 
 
@@ -50,7 +50,7 @@  discard block
 
 block discarded – undo
50 50 
 	/**
51 51 
 	 * @return HtmlViewGroups
52 52 
 	 */
53 
-	public function getItem($index){
53 
+	public function getItem($index) {
54 54 
 		return parent::getItem($index);
55 55 
 	}
56 56 
 
@@ -58,7 +58,7 @@  discard block
 
 block discarded – undo
58 58 
 		$item=$this->getItem($itemIndex);
59 59 
 		if (isset($item)) {
60 60 
 			$content=$item->getContent();
61 
-			if(isset($content[$contentIndex]))
61 
+			if (isset($content[$contentIndex]))
62 62 
 				return $content[$contentIndex];
63 63 
 			return $content;
64 64 
 		}
 
@@ -68,7 +68,7 @@  discard block
 
 block discarded – undo
68 68 
 		return $this->addItem($function($object));
69 69 
 	}
70 70
71 
-	public function run(JsUtils $js){
71 
+	public function run(JsUtils $js) {
72 72 
 		$result=parent::run($js);
73 73 
 		return $result->setItemSelector(".item");
74 74 
 	}
Please login to merge, or discard this patch.
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -31,8 +31,9 @@  discard block
 
 block discarded – undo
31 31 
 			if (isset($extra)) {
32 32 
 				$result->addExtraContent($extra);
33 33 
 			}
34 
-		} else
35 
-			$result->addItemContent($value);
34 
+		} else {
35 
+					$result->addItemContent($value);
36 
+		}
36 37 
 		return $result;
37 38 
 	}
38 39 
 
@@ -58,8 +59,9 @@  discard block
 
 block discarded – undo
58 59 
 		$item=$this->getItem($itemIndex);
59 60 
 		if (isset($item)) {
60 61 
 			$content=$item->getContent();
61 
-			if(isset($content[$contentIndex]))
62 
-				return $content[$contentIndex];
62 
+			if(isset($content[$contentIndex])) {
63 
+							return $content[$contentIndex];
64 
+			}
63 65 
 			return $content;
64 66 
 		}
65 67 
 	}
Please login to merge, or discard this patch.