Passed
Push — master ( 21fb75...28f925 )
by Jean-Christophe
02:07
created
Ajax/bootstrap/html/HtmlPagination.php 2 patches
Spacing   +34 added lines, -34 removed lines patch added patch discarded remove patch
@@ -45,70 +45,70 @@  discard block
 block discarded – undo
45 45
 	/**
46 46
 	 * @param string $identifier
47 47
 	 */
48
-	public function __construct($identifier,$from=1,$to=1,$active=NULL,$countVisible=NULL){
49
-		parent::__construct($identifier,"ul");
48
+	public function __construct($identifier, $from=1, $to=1, $active=NULL, $countVisible=NULL) {
49
+		parent::__construct($identifier, "ul");
50 50
 		$this->setProperty("class", "pagination");
51 51
 		$this->active=$active;
52 52
 		$this->from=$from;
53 53
 		$this->to=$to;
54 54
 		$this->urlMask="%page%";
55
-		if(!isset($countVisible))
55
+		if (!isset($countVisible))
56 56
 			$this->countVisible=$to-$from+1;
57 57
 		else
58 58
 			$this->countVisible=$countVisible;
59 59
 		$this->createContent();
60 60
 	}
61 61
 
62
-	private function createElement($num,$content,$disabled=false,$current=false){
62
+	private function createElement($num, $content, $disabled=false, $current=false) {
63 63
 		$count=sizeof($this->content)+1;
64
-		$elem=new HtmlBsDoubleElement("li-".$this->identifier."-".$count,"li");
65
-		if($disabled){
64
+		$elem=new HtmlBsDoubleElement("li-".$this->identifier."-".$count, "li");
65
+		if ($disabled) {
66 66
 			$elem->setProperty("class", "disabled");
67 67
 		}
68
-		if($current){
68
+		if ($current) {
69 69
 			$content.="<span class='sr-only'>(current)</span>";
70 70
 			$elem->setProperty("class", "active");
71 71
 		}
72
-		if(!$disabled){
72
+		if (!$disabled) {
73 73
 			$url=$this->getUrl($num);
74
-			$href=new HtmlLink("a-".$this->identifier."-".$count,$url,$content);
74
+			$href=new HtmlLink("a-".$this->identifier."-".$count, $url, $content);
75 75
 			$href->setProperty($this->attr, $url);
76 76
 			$elem->setContent($href);
77
-		}else{
77
+		} else {
78 78
 			$elem->setContent($content);
79 79
 		}
80 80
 		$this->content[]=$elem;
81 81
 		return $this;
82 82
 	}
83 83
 
84
-	protected function createContent(){
84
+	protected function createContent() {
85 85
 		$this->content=array();
86
-		$this->createElement($this->active-1,"<span aria-hidden='true'>&laquo;</span>",$this->active===1);
86
+		$this->createElement($this->active-1, "<span aria-hidden='true'>&laquo;</span>", $this->active===1);
87 87
 		$start=$this->getStart();
88
-		$end=min($start+$this->countVisible-1,$this->to);
89
-		for($index=$start;$index<=$end;$index++){
90
-			$this->createElement($index,$index,false,$index===$this->active);
88
+		$end=min($start+$this->countVisible-1, $this->to);
89
+		for ($index=$start; $index<=$end; $index++) {
90
+			$this->createElement($index, $index, false, $index===$this->active);
91 91
 		}
92
-		$this->createElement($this->active+1,"<span aria-hidden='true'>&raquo;</span>",$this->active===$this->to);
92
+		$this->createElement($this->active+1, "<span aria-hidden='true'>&raquo;</span>", $this->active===$this->to);
93 93
 	}
94 94
 
95
-	protected function half(){
95
+	protected function half() {
96 96
 		return (int)($this->countVisible/2);
97 97
 	}
98 98
 
99
-	protected function getStart(){
99
+	protected function getStart() {
100 100
 		$result=1;
101
-		if($this->countVisible!==$this->to-$this->from+1){
102
-			$result=max($this->active-$this->half(),$result);
101
+		if ($this->countVisible!==$this->to-$this->from+1) {
102
+			$result=max($this->active-$this->half(), $result);
103 103
 		}
104 104
 		return $result;
105 105
 	}
106 106
 
107 107
 	public function _addEvent($event, $jsCode) {
108
-		foreach ($this->content as $li){
108
+		foreach ($this->content as $li) {
109 109
 			$content=$li->getContent();
110
-			if($content instanceof BaseHtml)
111
-				$content->_addEvent($event,$jsCode);
110
+			if ($content instanceof BaseHtml)
111
+				$content->_addEvent($event, $jsCode);
112 112
 		}
113 113
 	}
114 114
 	/**
@@ -117,14 +117,14 @@  discard block
 block discarded – undo
117 117
 	 * @param object $dispatcher the request dispatcher
118 118
 	 * @return \Ajax\bootstrap\html\HtmlPagination
119 119
 	 */
120
-	public function fromDispatcher(JsUtils $js,$dispatcher,$startIndex=0){
120
+	public function fromDispatcher(JsUtils $js, $dispatcher, $startIndex=0) {
121 121
 		$items=$js->fromDispatcher($dispatcher);
122 122
 		$url=implode("/", $items);
123
-		if($this->urlMask==="%page%"){
123
+		if ($this->urlMask==="%page%") {
124 124
 			$this->urlMask=preg_replace("/[0-9]/", "%page%", $url);
125 125
 		}
126
-		for($index=$this->from;$index<=$this->to;$index++){
127
-			if($this->getUrl($index)==$url){
126
+		for ($index=$this->from; $index<=$this->to; $index++) {
127
+			if ($this->getUrl($index)==$url) {
128 128
 				$this->setActive($index);
129 129
 				break;
130 130
 			}
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
 		return $this;
133 133
 	}
134 134
 
135
-	public function getUrl($index){
135
+	public function getUrl($index) {
136 136
 		return str_ireplace("%page%", $index, $this->urlMask);
137 137
 	}
138 138
 
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
 		if (is_int($size)) {
147 147
 			return $this->addToPropertyUnique("class", CssRef::sizes("pagination")[$size], CssRef::sizes("pagination"));
148 148
 		}
149
-		if(!JString::startsWith($size, "pagination-") && $size!=="")
149
+		if (!JString::startsWith($size, "pagination-") && $size!=="")
150 150
 			$size="pagination-".$size;
151 151
 		return $this->addToPropertyCtrl("class", $size, CssRef::sizes("pagination"));
152 152
 	}
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
 		return $this->from;
156 156
 	}
157 157
 	public function setFrom($from) {
158
-		$this->from = $from;
158
+		$this->from=$from;
159 159
 		$this->createContent();
160 160
 		return $this;
161 161
 	}
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
 		return $this->to;
164 164
 	}
165 165
 	public function setTo($to) {
166
-		$this->to = $to;
166
+		$this->to=$to;
167 167
 		$this->createContent();
168 168
 		return $this;
169 169
 	}
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
 		return $this->active;
172 172
 	}
173 173
 	public function setActive($active) {
174
-		$this->active = $active;
174
+		$this->active=$active;
175 175
 		$this->createContent();
176 176
 		return $this;
177 177
 	}
@@ -179,7 +179,7 @@  discard block
 block discarded – undo
179 179
 		return $this->urlMask;
180 180
 	}
181 181
 	public function setUrlMask($urlMask) {
182
-		$this->urlMask = $urlMask;
182
+		$this->urlMask=$urlMask;
183 183
 		$this->createContent();
184 184
 		return $this;
185 185
 	}
@@ -187,7 +187,7 @@  discard block
 block discarded – undo
187 187
 		return $this->countVisible;
188 188
 	}
189 189
 	public function setCountVisible($countVisible) {
190
-		$this->countVisible = $countVisible;
190
+		$this->countVisible=$countVisible;
191 191
 		$this->createContent();
192 192
 		return $this;
193 193
 	}
Please login to merge, or discard this patch.
Braces   +12 added lines, -9 removed lines patch added patch discarded remove patch
@@ -52,10 +52,11 @@  discard block
 block discarded – undo
52 52
 		$this->from=$from;
53 53
 		$this->to=$to;
54 54
 		$this->urlMask="%page%";
55
-		if(!isset($countVisible))
56
-			$this->countVisible=$to-$from+1;
57
-		else
58
-			$this->countVisible=$countVisible;
55
+		if(!isset($countVisible)) {
56
+					$this->countVisible=$to-$from+1;
57
+		} else {
58
+					$this->countVisible=$countVisible;
59
+		}
59 60
 		$this->createContent();
60 61
 	}
61 62
 
@@ -74,7 +75,7 @@  discard block
 block discarded – undo
74 75
 			$href=new HtmlLink("a-".$this->identifier."-".$count,$url,$content);
75 76
 			$href->setProperty($this->attr, $url);
76 77
 			$elem->setContent($href);
77
-		}else{
78
+		} else{
78 79
 			$elem->setContent($content);
79 80
 		}
80 81
 		$this->content[]=$elem;
@@ -107,8 +108,9 @@  discard block
 block discarded – undo
107 108
 	public function _addEvent($event, $jsCode) {
108 109
 		foreach ($this->content as $li){
109 110
 			$content=$li->getContent();
110
-			if($content instanceof BaseHtml)
111
-				$content->_addEvent($event,$jsCode);
111
+			if($content instanceof BaseHtml) {
112
+							$content->_addEvent($event,$jsCode);
113
+			}
112 114
 		}
113 115
 	}
114 116
 	/**
@@ -146,8 +148,9 @@  discard block
 block discarded – undo
146 148
 		if (is_int($size)) {
147 149
 			return $this->addToPropertyUnique("class", CssRef::sizes("pagination")[$size], CssRef::sizes("pagination"));
148 150
 		}
149
-		if(!JString::startsWith($size, "pagination-") && $size!=="")
150
-			$size="pagination-".$size;
151
+		if(!JString::startsWith($size, "pagination-") && $size!=="") {
152
+					$size="pagination-".$size;
153
+		}
151 154
 		return $this->addToPropertyCtrl("class", $size, CssRef::sizes("pagination"));
152 155
 	}
153 156
 
Please login to merge, or discard this patch.
Ajax/common/html/html5/HtmlUtils.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -3,11 +3,11 @@
 block discarded – undo
3 3
 namespace Ajax\common\html\html5;
4 4
 
5 5
 class HtmlUtils {
6
-	public static function javascriptInclude($url){
6
+	public static function javascriptInclude($url) {
7 7
 		return '<script src="'.$url.'"></script>';
8 8
 	}
9 9
 	
10
-	public static function stylesheetInclude($url){
10
+	public static function stylesheetInclude($url) {
11 11
 		return '<link rel="stylesheet" type="text/css" href="'.$url.'">';
12 12
 	}
13 13
 }
Please login to merge, or discard this patch.
Ajax/common/html/traits/BaseHtmlPropertiesTrait.php 2 patches
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -8,12 +8,12 @@  discard block
 block discarded – undo
8 8
  * @author jc
9 9
  * @property BaseWidget $_self
10 10
  */
11
-trait BaseHtmlPropertiesTrait{
11
+trait BaseHtmlPropertiesTrait {
12 12
 
13
-	protected $properties=array ();
13
+	protected $properties=array();
14 14
 	abstract protected function ctrl($name, $value, $typeCtrl);
15 15
 	abstract protected function removeOldValues(&$oldValue, $allValues);
16
-	abstract protected function _getElementBy($callback,$elements);
16
+	abstract protected function _getElementBy($callback, $elements);
17 17
 	public function getProperties() {
18 18
 		return $this->_self->properties;
19 19
 	}
@@ -45,14 +45,14 @@  discard block
 block discarded – undo
45 45
 	 */
46 46
 	public function addToProperty($name, $value, $separator=" ") {
47 47
 		if (\is_array($value)) {
48
-			foreach ( $value as $v ) {
48
+			foreach ($value as $v) {
49 49
 				$this->_self->addToProperty($name, $v, $separator);
50 50
 			}
51
-		} else if ($value !== "" && $this->_self->propertyContains($name, $value) === false) {
52
-			if(isset($this->_self->properties[$name])){
51
+		} else if ($value!=="" && $this->_self->propertyContains($name, $value)===false) {
52
+			if (isset($this->_self->properties[$name])) {
53 53
 				$v=$this->_self->properties[$name];
54
-				if (isset($v) && $v !== "")
55
-					$v=$v . $separator . $value;
54
+				if (isset($v) && $v!=="")
55
+					$v=$v.$separator.$value;
56 56
 					else
57 57
 						$v=$value;
58 58
 	
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
 	}
92 92
 
93 93
 	public function addToPropertyCtrlCheck($name, $value, $typeCtrl) {
94
-		if ($this->_self->ctrl($name, $value, $typeCtrl) === true) {
94
+		if ($this->_self->ctrl($name, $value, $typeCtrl)===true) {
95 95
 			return $this->_self->addToProperty($name, $value);
96 96
 		}
97 97
 		return $this;
@@ -112,12 +112,12 @@  discard block
 block discarded – undo
112 112
 	}
113 113
 
114 114
 	protected function setPropertyCtrl($name, $value, $typeCtrl) {
115
-		if ($this->_self->ctrl($name, $value, $typeCtrl) === true)
115
+		if ($this->_self->ctrl($name, $value, $typeCtrl)===true)
116 116
 			return $this->_self->setProperty($name, $value);
117 117
 			return $this;
118 118
 	}
119 119
 
120
-	protected function getElementByPropertyValue($propertyName,$value, $elements) {
121
-		return $this->_self->_getElementBy(function(BaseHtml $element) use ($propertyName,$value){return $element->propertyContains($propertyName, $value) === true;}, $elements);
120
+	protected function getElementByPropertyValue($propertyName, $value, $elements) {
121
+		return $this->_self->_getElementBy(function(BaseHtml $element) use ($propertyName, $value){return $element->propertyContains($propertyName, $value)===true; }, $elements);
122 122
 	}
123 123
 }
Please login to merge, or discard this patch.
Braces   +17 added lines, -12 removed lines patch added patch discarded remove patch
@@ -33,8 +33,9 @@  discard block
 block discarded – undo
33 33
 	}
34 34
 
35 35
 	public function getProperty($name) {
36
-		if (array_key_exists($name, $this->_self->properties))
37
-			return $this->_self->properties[$name];
36
+		if (array_key_exists($name, $this->_self->properties)) {
37
+					return $this->_self->properties[$name];
38
+		}
38 39
 	}
39 40
 
40 41
 	/**
@@ -51,10 +52,11 @@  discard block
 block discarded – undo
51 52
 		} else if ($value !== "" && $this->_self->propertyContains($name, $value) === false) {
52 53
 			if(isset($this->_self->properties[$name])){
53 54
 				$v=$this->_self->properties[$name];
54
-				if (isset($v) && $v !== "")
55
-					$v=$v . $separator . $value;
56
-					else
57
-						$v=$value;
55
+				if (isset($v) && $v !== "") {
56
+									$v=$v . $separator . $value;
57
+				} else {
58
+											$v=$value;
59
+					}
58 60
 	
59 61
 				return $this->_self->setProperty($name, $v);
60 62
 			}
@@ -78,8 +80,9 @@  discard block
 block discarded – undo
78 80
 	}
79 81
 
80 82
 	protected function addToPropertyUnique($name, $value, $typeCtrl) {
81
-		if (is_string($typeCtrl) && @class_exists($typeCtrl, true))
82
-			$typeCtrl=$typeCtrl::getConstants();
83
+		if (is_string($typeCtrl) && @class_exists($typeCtrl, true)) {
84
+					$typeCtrl=$typeCtrl::getConstants();
85
+		}
83 86
 			if (\is_array($typeCtrl)) {
84 87
 				$this->_self->removeOldValues($this->_self->properties[$name], $typeCtrl);
85 88
 			}
@@ -98,8 +101,9 @@  discard block
 block discarded – undo
98 101
 	}
99 102
 
100 103
 	public function removeProperty($name) {
101
-		if (\array_key_exists($name, $this->_self->properties))
102
-			unset($this->_self->properties[$name]);
104
+		if (\array_key_exists($name, $this->_self->properties)) {
105
+					unset($this->_self->properties[$name]);
106
+		}
103 107
 			return $this;
104 108
 	}
105 109
 
@@ -112,8 +116,9 @@  discard block
 block discarded – undo
112 116
 	}
113 117
 
114 118
 	protected function setPropertyCtrl($name, $value, $typeCtrl) {
115
-		if ($this->_self->ctrl($name, $value, $typeCtrl) === true)
116
-			return $this->_self->setProperty($name, $value);
119
+		if ($this->_self->ctrl($name, $value, $typeCtrl) === true) {
120
+					return $this->_self->setProperty($name, $value);
121
+		}
117 122
 			return $this;
118 123
 	}
119 124
 
Please login to merge, or discard this patch.
Ajax/bootstrap/html/HtmlPanel.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -89,8 +89,9 @@  discard block
 block discarded – undo
89 89
 	 * @return \Ajax\bootstrap\html\HtmlPanel default : "panel-default"
90 90
 	 */
91 91
 	public function setStyle($cssStyle) {
92
-		if (!JString::startsWith($cssStyle, "panel"))
93
-			$cssStyle="panel".$cssStyle;
92
+		if (!JString::startsWith($cssStyle, "panel")) {
93
+					$cssStyle="panel".$cssStyle;
94
+		}
94 95
 		return $this->addToPropertyCtrl("class", $cssStyle, CssRef::Styles("panel"));
95 96
 	}
96 97
 
@@ -132,7 +133,8 @@  discard block
 block discarded – undo
132 133
 	 * @return $this default : false
133 134
 	 */
134 135
 	public function show($value) {
135
-		if ($this->_collapsable)
136
-			$this->_showOnStartup=$value;
136
+		if ($this->_collapsable) {
137
+					$this->_showOnStartup=$value;
138
+		}
137 139
 	}
138 140
 }
Please login to merge, or discard this patch.
Ajax/common/Widget.php 2 patches
Spacing   +81 added lines, -81 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
 use Ajax\semantic\html\modules\HtmlModal;
21 21
 
22 22
 abstract class Widget extends HtmlDoubleElement {
23
-	use FieldAsTrait,FormTrait;
23
+	use FieldAsTrait, FormTrait;
24 24
 
25 25
 	/**
26 26
 	 * @var string classname
@@ -53,17 +53,17 @@  discard block
 block discarded – undo
53 53
 	protected $_generated;
54 54
 
55 55
 
56
-	public function __construct($identifier,$model,$modelInstance=NULL) {
56
+	public function __construct($identifier, $model, $modelInstance=NULL) {
57 57
 		parent::__construct($identifier);
58 58
 		$this->_template="%wrapContentBefore%%content%%wrapContentAfter%";
59 59
 		$this->setModel($model);
60
-		if(isset($modelInstance)){
60
+		if (isset($modelInstance)) {
61 61
 			$this->show($modelInstance);
62 62
 		}
63 63
 		$this->_generated=false;
64 64
 	}
65 65
 
66
-	protected function _init($instanceViewer,$contentKey,$content,$edition){
66
+	protected function _init($instanceViewer, $contentKey, $content, $edition) {
67 67
 		$this->_instanceViewer=$instanceViewer;
68 68
 		$this->content=[$contentKey=>$content];
69 69
 		$this->_self=$content;
@@ -75,31 +75,31 @@  discard block
 block discarded – undo
75 75
 	 * @param int|string $fieldName
76 76
 	 * @return int|string|boolean
77 77
 	 */
78
-	protected function _getIndex($fieldName){
78
+	protected function _getIndex($fieldName) {
79 79
 		$index=$fieldName;
80
-		if(\is_string($fieldName)){
80
+		if (\is_string($fieldName)) {
81 81
 			$fields=$this->_instanceViewer->getVisibleProperties();
82 82
 			$index=\array_search($fieldName, $fields);
83 83
 		}
84 84
 		return $index;
85 85
 	}
86 86
 
87
-	protected function _getFieldIdentifier($prefix,$name=""){
87
+	protected function _getFieldIdentifier($prefix, $name="") {
88 88
 		return $this->identifier."-{$prefix}-".$this->_instanceViewer->getIdentifier();
89 89
 	}
90 90
 
91
-	protected function _getFieldName($index){
91
+	protected function _getFieldName($index) {
92 92
 		return $this->_instanceViewer->getFieldName($index);
93 93
 	}
94 94
 
95
-	protected function _getFieldCaption($index){
95
+	protected function _getFieldCaption($index) {
96 96
 		return $this->_instanceViewer->getCaption($index);
97 97
 	}
98 98
 
99
-	abstract protected  function _setToolbarPosition($table,$captions=NULL);
99
+	abstract protected  function _setToolbarPosition($table, $captions=NULL);
100 100
 
101
-	public function show($modelInstance){
102
-		if(\is_array($modelInstance)){
101
+	public function show($modelInstance) {
102
+		if (\is_array($modelInstance)) {
103 103
 			$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
104 104
 		}
105 105
 		$this->_modelInstance=$modelInstance;
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
 
126 126
 	abstract public function getHtmlComponent();
127 127
 
128
-	public function setAttached($value=true){
128
+	public function setAttached($value=true) {
129 129
 		return $this->getHtmlComponent()->setAttached($value);
130 130
 	}
131 131
 
@@ -136,59 +136,59 @@  discard block
 block discarded – undo
136 136
 	 * @param callable $callback function called after the field compilation
137 137
 	 * @return Widget
138 138
 	 */
139
-	public function afterCompile($index,$callback){
139
+	public function afterCompile($index, $callback) {
140 140
 		$index=$this->_getIndex($index);
141 141
 		$this->_instanceViewer->afterCompile($index, $callback);
142 142
 		return $this;
143 143
 	}
144 144
 
145
-	public function setColor($color){
145
+	public function setColor($color) {
146 146
 		return $this->getHtmlComponent()->setColor($color);
147 147
 	}
148 148
 
149 149
 
150
-	public function setCaptions($captions){
150
+	public function setCaptions($captions) {
151 151
 		$this->_instanceViewer->setCaptions($captions);
152 152
 		return $this;
153 153
 	}
154 154
 
155
-	public function setCaption($index,$caption){
155
+	public function setCaption($index, $caption) {
156 156
 		$this->_instanceViewer->setCaption($this->_getIndex($index), $caption);
157 157
 		return $this;
158 158
 	}
159 159
 
160
-	public function setFields($fields){
160
+	public function setFields($fields) {
161 161
 		$this->_instanceViewer->setVisibleProperties($fields);
162 162
 		return $this;
163 163
 	}
164 164
 
165
-	public function addField($field){
165
+	public function addField($field) {
166 166
 		$this->_instanceViewer->addField($field);
167 167
 		return $this;
168 168
 	}
169 169
 
170
-	public function addFields($fields){
170
+	public function addFields($fields) {
171 171
 		$this->_instanceViewer->addFields($fields);
172 172
 		return $this;
173 173
 	}
174 174
 
175
-	public function addMessage($attributes=NULL,$fieldName="message"){
175
+	public function addMessage($attributes=NULL, $fieldName="message") {
176 176
 		$this->_instanceViewer->addField($fieldName);
177 177
 		$count=$this->_instanceViewer->visiblePropertiesCount();
178
-		return $this->fieldAsMessage($count-1,$attributes);
178
+		return $this->fieldAsMessage($count-1, $attributes);
179 179
 	}
180 180
 
181
-	public function addErrorMessage(){
182
-		return $this->addMessage(["error"=>true],"message");
181
+	public function addErrorMessage() {
182
+		return $this->addMessage(["error"=>true], "message");
183 183
 	}
184 184
 
185
-	public function insertField($index,$field){
185
+	public function insertField($index, $field) {
186 186
 		$index=$this->_getIndex($index);
187 187
 		$this->_instanceViewer->insertField($index, $field);
188 188
 		return $this;
189 189
 	}
190 190
 
191
-	public function insertInField($index,$field){
191
+	public function insertInField($index, $field) {
192 192
 		$index=$this->_getIndex($index);
193 193
 		$this->_instanceViewer->insertInField($index, $field);
194 194
 		return $this;
@@ -200,13 +200,13 @@  discard block
 block discarded – undo
200 200
 	 * @param callable $callback function parameters are : $value : the field value, $instance : the active instance of model, $fieldIndex : the field index, $rowIndex : the row index
201 201
 	 * @return Widget
202 202
 	 */
203
-	public function setValueFunction($index,$callback){
203
+	public function setValueFunction($index, $callback) {
204 204
 		$index=$this->_getIndex($index);
205 205
 		$this->_instanceViewer->setValueFunction($index, $callback);
206 206
 		return $this;
207 207
 	}
208 208
 
209
-	public function setIdentifierFunction($callback){
209
+	public function setIdentifierFunction($callback) {
210 210
 		$this->_instanceViewer->setIdentifierFunction($callback);
211 211
 		return $this;
212 212
 	}
@@ -214,8 +214,8 @@  discard block
 block discarded – undo
214 214
 	/**
215 215
 	 * @return \Ajax\semantic\html\collections\menus\HtmlMenu
216 216
 	 */
217
-	public function getToolbar(){
218
-		if(isset($this->_toolbar)===false){
217
+	public function getToolbar() {
218
+		if (isset($this->_toolbar)===false) {
219 219
 			$this->_toolbar=new HtmlMenu("toolbar-".$this->identifier);
220 220
 		}
221 221
 		return $this->_toolbar;
@@ -227,15 +227,15 @@  discard block
 block discarded – undo
227 227
 	 * @param callable $callback function to call on $element
228 228
 	 * @return \Ajax\common\html\HtmlDoubleElement
229 229
 	 */
230
-	public function addInToolbar($element,$callback=NULL){
230
+	public function addInToolbar($element, $callback=NULL) {
231 231
 		$tb=$this->getToolbar();
232
-		if($element instanceof BaseWidget){
233
-			if($element->getIdentifier()===""){
232
+		if ($element instanceof BaseWidget) {
233
+			if ($element->getIdentifier()==="") {
234 234
 				$element->setIdentifier("tb-item-".$this->identifier."-".$tb->count());
235 235
 			}
236 236
 		}
237
-		if(isset($callback)){
238
-			if(\is_callable($callback)){
237
+		if (isset($callback)) {
238
+			if (\is_callable($callback)) {
239 239
 				$callback($element);
240 240
 			}
241 241
 		}
@@ -248,9 +248,9 @@  discard block
 block discarded – undo
248 248
 	 * @param callable $callback function($element)
249 249
 	 * @return \Ajax\common\html\HtmlDoubleElement
250 250
 	 */
251
-	public function addItemInToolbar($caption,$icon=NULL,$callback=NULL){
252
-		$result=$this->addInToolbar($caption,$callback);
253
-		if(isset($icon) && method_exists($result, "addIcon"))
251
+	public function addItemInToolbar($caption, $icon=NULL, $callback=NULL) {
252
+		$result=$this->addInToolbar($caption, $callback);
253
+		if (isset($icon) && method_exists($result, "addIcon"))
254 254
 			$result->addIcon($icon);
255 255
 		return $result;
256 256
 	}
@@ -260,14 +260,14 @@  discard block
 block discarded – undo
260 260
 	 * @param callable $callback function($element)
261 261
 	 * @return \Ajax\common\Widget
262 262
 	 */
263
-	public function addItemsInToolbar(array $items,$callback=NULL){
264
-		if(JArray::isAssociative($items)){
265
-			foreach ($items as $icon=>$item){
266
-				$this->addItemInToolbar($item,$icon,$callback);
263
+	public function addItemsInToolbar(array $items, $callback=NULL) {
264
+		if (JArray::isAssociative($items)) {
265
+			foreach ($items as $icon=>$item) {
266
+				$this->addItemInToolbar($item, $icon, $callback);
267 267
 			}
268
-		}else{
269
-			foreach ($items as $item){
270
-				$this->addItemInToolbar($item,null,$callback);
268
+		} else {
269
+			foreach ($items as $item) {
270
+				$this->addItemInToolbar($item, null, $callback);
271 271
 			}
272 272
 		}
273 273
 		return $this;
@@ -279,12 +279,12 @@  discard block
 block discarded – undo
279 279
 	 * @param callable $callback function($element)
280 280
 	 * @return \Ajax\common\html\HtmlDoubleElement
281 281
 	 */
282
-	public function addDropdownInToolbar($value,$items,$callback=NULL){
282
+	public function addDropdownInToolbar($value, $items, $callback=NULL) {
283 283
 		$dd=$value;
284 284
 		if (\is_string($value)) {
285
-			$dd=new HtmlDropdown("dropdown-". $this->identifier."-".$value, $value, $items);
285
+			$dd=new HtmlDropdown("dropdown-".$this->identifier."-".$value, $value, $items);
286 286
 		}
287
-		return $this->addInToolbar($dd,$callback);
287
+		return $this->addInToolbar($dd, $callback);
288 288
 	}
289 289
 
290 290
 	/**
@@ -293,9 +293,9 @@  discard block
 block discarded – undo
293 293
 	 * @param callable $callback function($element)
294 294
 	 * @return \Ajax\common\html\HtmlDoubleElement
295 295
 	 */
296
-	public function addButtonInToolbar($caption,$cssStyle=null,$callback=NULL){
297
-		$bt=new HtmlButton("bt-".$caption,$caption,$cssStyle);
298
-		return $this->addInToolbar($bt,$callback);
296
+	public function addButtonInToolbar($caption, $cssStyle=null, $callback=NULL) {
297
+		$bt=new HtmlButton("bt-".$caption, $caption, $cssStyle);
298
+		return $this->addInToolbar($bt, $callback);
299 299
 	}
300 300
 
301 301
 	/**
@@ -304,9 +304,9 @@  discard block
 block discarded – undo
304 304
 	 * @param callable $callback function($element)
305 305
 	 * @return \Ajax\common\html\HtmlDoubleElement
306 306
 	 */
307
-	public function addButtonsInToolbar(array $captions,$asIcon=false,$callback=NULL){
308
-		$bts=new HtmlButtonGroups("",$captions,$asIcon);
309
-		return $this->addInToolbar($bts,$callback);
307
+	public function addButtonsInToolbar(array $captions, $asIcon=false, $callback=NULL) {
308
+		$bts=new HtmlButtonGroups("", $captions, $asIcon);
309
+		return $this->addInToolbar($bts, $callback);
310 310
 	}
311 311
 
312 312
 	/**
@@ -316,15 +316,15 @@  discard block
 block discarded – undo
316 316
 	 * @param boolean $labeled
317 317
 	 * @return \Ajax\common\html\HtmlDoubleElement
318 318
 	 */
319
-	public function addLabelledIconButtonInToolbar($caption,$icon,$before=true,$labeled=false){
320
-		$bt=new HtmlButton("",$caption);
321
-		$bt->addIcon($icon,$before,$labeled);
319
+	public function addLabelledIconButtonInToolbar($caption, $icon, $before=true, $labeled=false) {
320
+		$bt=new HtmlButton("", $caption);
321
+		$bt->addIcon($icon, $before, $labeled);
322 322
 		return $this->addInToolbar($bt);
323 323
 	}
324 324
 
325
-	public function addSubmitInToolbar($identifier,$value,$cssStyle=NULL,$url=NULL,$responseElement=NULL,$parameters=NULL){
326
-		$button=new HtmlButton($identifier,$value,$cssStyle);
327
-		$this->_buttonAsSubmit($button,"click",$url,$responseElement,$parameters);
325
+	public function addSubmitInToolbar($identifier, $value, $cssStyle=NULL, $url=NULL, $responseElement=NULL, $parameters=NULL) {
326
+		$button=new HtmlButton($identifier, $value, $cssStyle);
327
+		$this->_buttonAsSubmit($button, "click", $url, $responseElement, $parameters);
328 328
 		return $this->addInToolbar($button);
329 329
 	}
330 330
 
@@ -354,7 +354,7 @@  discard block
 block discarded – undo
354 354
 	 * @param callable $defaultValueFunction function parameters are : $name : the field name, $value : the field value ,$index : the field index, $instance : the active instance of model
355 355
 	 * @return \Ajax\common\Widget
356 356
 	 */
357
-	public function setDefaultValueFunction($defaultValueFunction){
357
+	public function setDefaultValueFunction($defaultValueFunction) {
358 358
 		$this->_instanceViewer->setDefaultValueFunction($defaultValueFunction);
359 359
 		return $this;
360 360
 	}
@@ -363,7 +363,7 @@  discard block
 block discarded – undo
363 363
 	 * @param string|boolean $disable
364 364
 	 * @return string
365 365
 	 */
366
-	public function jsDisabled($disable=true){
366
+	public function jsDisabled($disable=true) {
367 367
 		return "$('#".$this->identifier." .ui.input,#".$this->identifier." .ui.dropdown,#".$this->identifier." .ui.checkbox').toggleClass('disabled',".$disable.");";
368 368
 	}
369 369
 
@@ -372,12 +372,12 @@  discard block
 block discarded – undo
372 372
 	 * @param callable $callback function($element)
373 373
 	 * @return \Ajax\common\html\HtmlDoubleElement
374 374
 	 */
375
-	public function addEditButtonInToolbar($caption,$callback=NULL){
376
-		$bt=new HtmlButton($this->identifier."-editBtn",$caption);
375
+	public function addEditButtonInToolbar($caption, $callback=NULL) {
376
+		$bt=new HtmlButton($this->identifier."-editBtn", $caption);
377 377
 		$bt->setToggle();
378 378
 		$bt->setActive($this->_edition);
379 379
 		$bt->onClick($this->jsDisabled(Javascript::prep_value("!$(event.target).hasClass('active')")));
380
-		return $this->addInToolbar($bt,$callback);
380
+		return $this->addInToolbar($bt, $callback);
381 381
 	}
382 382
 
383 383
 	public function setToolbar(HtmlMenu $_toolbar) {
@@ -391,34 +391,34 @@  discard block
 block discarded – undo
391 391
 	}
392 392
 
393 393
 	public function getForm() {
394
-		if(!isset($this->_form)){
394
+		if (!isset($this->_form)) {
395 395
 			$this->_form=new HtmlForm("frm-".$this->identifier);
396 396
 			$this->setEdition(true);
397 397
 		}
398 398
 		return $this->_form;
399 399
 	}
400 400
 
401
-	public function run(JsUtils $js){
401
+	public function run(JsUtils $js) {
402 402
 		parent::run($js);
403
-		if(isset($this->_form)){
403
+		if (isset($this->_form)) {
404 404
 			$this->runForm($js);
405 405
 		}
406 406
 	}
407 407
 
408
-	protected function runForm(JsUtils $js){
408
+	protected function runForm(JsUtils $js) {
409 409
 		$fields=$this->getContentInstances(HtmlFormField::class);
410
-		foreach ($fields as $field){
410
+		foreach ($fields as $field) {
411 411
 			$this->_form->addField($field);
412 412
 		}
413 413
 		return $this->_form->run($js);
414 414
 	}
415 415
 
416
-	protected function _compileForm(){
417
-		if(isset($this->_form)){
416
+	protected function _compileForm() {
417
+		if (isset($this->_form)) {
418 418
 			$noValidate="";
419
-			if(\sizeof($this->_form->getValidationParams())>0)
419
+			if (\sizeof($this->_form->getValidationParams())>0)
420 420
 				$noValidate="novalidate";
421
-			$this->wrapContent("<form class='ui form' id='frm-".$this->identifier."' name='frm-".$this->identifier."' ".$noValidate.">","</form>");
421
+			$this->wrapContent("<form class='ui form' id='frm-".$this->identifier."' name='frm-".$this->identifier."' ".$noValidate.">", "</form>");
422 422
 		}
423 423
 	}
424 424
 
@@ -433,32 +433,32 @@  discard block
 block discarded – undo
433 433
 		return $this;
434 434
 	}
435 435
 
436
-	public function moveFieldTo($from,$to){
436
+	public function moveFieldTo($from, $to) {
437 437
 		return $this->_instanceViewer->moveFieldTo($from, $to);
438 438
 	}
439 439
 
440
-	public function swapFields($index1,$index2){
440
+	public function swapFields($index1, $index2) {
441 441
 		$index1=$this->_getIndex($index1);
442 442
 		$index2=$this->_getIndex($index2);
443 443
 		return $this->_instanceViewer->swapFields($index1, $index2);
444 444
 	}
445 445
 
446
-	public function removeField($index){
446
+	public function removeField($index) {
447 447
 		$index=$this->_getIndex($index);
448 448
 		$this->_instanceViewer->removeField($index);
449 449
 		return $this;
450 450
 	}
451 451
 
452
-	public function asModal($header=null){
453
-		$modal=new HtmlModal("modal-".$this->identifier,$header);
452
+	public function asModal($header=null) {
453
+		$modal=new HtmlModal("modal-".$this->identifier, $header);
454 454
 		$modal->setContent($this);
455
-		if(isset($this->_form)){
455
+		if (isset($this->_form)) {
456 456
 			$this->_form->onSuccess($modal->jsHide());
457 457
 		}
458 458
 		return $modal;
459 459
 	}
460 460
 
461 461
 	public function addToProperty($name, $value, $separator=" ") {
462
-		return $this->getHtmlComponent()->addToProperty($name,$value,$separator);
462
+		return $this->getHtmlComponent()->addToProperty($name, $value, $separator);
463 463
 	}
464 464
 }
Please login to merge, or discard this patch.
Braces   +7 added lines, -5 removed lines patch added patch discarded remove patch
@@ -250,8 +250,9 @@  discard block
 block discarded – undo
250 250
 	 */
251 251
 	public function addItemInToolbar($caption,$icon=NULL,$callback=NULL){
252 252
 		$result=$this->addInToolbar($caption,$callback);
253
-		if(isset($icon) && method_exists($result, "addIcon"))
254
-			$result->addIcon($icon);
253
+		if(isset($icon) && method_exists($result, "addIcon")) {
254
+					$result->addIcon($icon);
255
+		}
255 256
 		return $result;
256 257
 	}
257 258
 
@@ -265,7 +266,7 @@  discard block
 block discarded – undo
265 266
 			foreach ($items as $icon=>$item){
266 267
 				$this->addItemInToolbar($item,$icon,$callback);
267 268
 			}
268
-		}else{
269
+		} else{
269 270
 			foreach ($items as $item){
270 271
 				$this->addItemInToolbar($item,null,$callback);
271 272
 			}
@@ -416,8 +417,9 @@  discard block
 block discarded – undo
416 417
 	protected function _compileForm(){
417 418
 		if(isset($this->_form)){
418 419
 			$noValidate="";
419
-			if(\sizeof($this->_form->getValidationParams())>0)
420
-				$noValidate="novalidate";
420
+			if(\sizeof($this->_form->getValidationParams())>0) {
421
+							$noValidate="novalidate";
422
+			}
421 423
 			$this->wrapContent("<form class='ui form' id='frm-".$this->identifier."' name='frm-".$this->identifier."' ".$noValidate.">","</form>");
422 424
 		}
423 425
 	}
Please login to merge, or discard this patch.
Ajax/semantic/widgets/datatable/DataTableFieldAsTrait.php 2 patches
Spacing   +57 added lines, -57 removed lines patch added patch discarded remove patch
@@ -13,21 +13,21 @@  discard block
 block discarded – undo
13 13
  * @property boolean $_visibleHover
14 14
  * @property InstanceViewer $_instanceViewer
15 15
  */
16
-trait DataTableFieldAsTrait{
16
+trait DataTableFieldAsTrait {
17 17
 	abstract public function addField($field);
18
-	abstract public function insertField($index,$field);
19
-	abstract public function insertInField($index,$field);
20
-	abstract public function fieldAs($index,$type,$attributes=NULL);
18
+	abstract public function insertField($index, $field);
19
+	abstract public function insertInField($index, $field);
20
+	abstract public function fieldAs($index, $type, $attributes=NULL);
21 21
 	abstract protected function cleanIdentifier($id);
22
-	abstract protected function _fieldAs($elementCallback,&$index,$attributes=NULL,$prefix=null);
22
+	abstract protected function _fieldAs($elementCallback, &$index, $attributes=NULL, $prefix=null);
23 23
 	/**
24 24
 	 * @param string $caption
25 25
 	 * @param callable $callback
26 26
 	 * @param boolean $visibleHover
27 27
 	 * @return callable
28 28
 	 */
29
-	private function getFieldButtonCallable($caption,$visibleHover=true,$callback=null){
30
-		return $this->getCallable("getFieldButton",[$caption,$visibleHover],$callback);
29
+	private function getFieldButtonCallable($caption, $visibleHover=true, $callback=null) {
30
+		return $this->getCallable("getFieldButton", [$caption, $visibleHover], $callback);
31 31
 	}
32 32
 
33 33
 	/**
@@ -36,20 +36,20 @@  discard block
 block discarded – undo
36 36
 	 * @param callable $callback
37 37
 	 * @return callable
38 38
 	 */
39
-	private function getCallable($thisCallback,$parameters,$callback=null){
40
-		$result=function($instance) use($thisCallback,$parameters,$callback){
41
-			$object=call_user_func_array(array($this,$thisCallback), $parameters);
42
-			if(isset($callback)){
43
-				if(\is_callable($callback)){
44
-					$callback($object,$instance,$this->_instanceViewer->count()+1);
39
+	private function getCallable($thisCallback, $parameters, $callback=null) {
40
+		$result=function($instance) use($thisCallback, $parameters, $callback){
41
+			$object=call_user_func_array(array($this, $thisCallback), $parameters);
42
+			if (isset($callback)) {
43
+				if (\is_callable($callback)) {
44
+					$callback($object, $instance, $this->_instanceViewer->count()+1);
45 45
 				}
46 46
 			}
47
-			if($object instanceof HtmlSemDoubleElement){
47
+			if ($object instanceof HtmlSemDoubleElement) {
48 48
 				$id=$this->_instanceViewer->getIdentifier();
49
-				$object->setProperty("data-ajax",$id);
50
-				if($object->propertyContains("class","visibleover")){
49
+				$object->setProperty("data-ajax", $id);
50
+				if ($object->propertyContains("class", "visibleover")) {
51 51
 					$this->_visibleHover=true;
52
-					$object->setProperty("style","visibility:hidden;");
52
+					$object->setProperty("style", "visibility:hidden;");
53 53
 				}
54 54
 			}
55 55
 			return $object;
@@ -61,16 +61,16 @@  discard block
 block discarded – undo
61 61
 	 * @param string $caption
62 62
 	 * @return HtmlButton
63 63
 	 */
64
-	private function getFieldButton($caption,$visibleHover=true){
65
-		$bt= new HtmlButton($this->cleanIdentifier($caption),$caption);
66
-		if($visibleHover)
64
+	private function getFieldButton($caption, $visibleHover=true) {
65
+		$bt=new HtmlButton($this->cleanIdentifier($caption), $caption);
66
+		if ($visibleHover)
67 67
 			$this->_visibleOver($bt);
68 68
 		return $bt;
69 69
 	}
70 70
 
71
-	private function getFieldButtons($buttons,$visibleHover=true){
72
-		$bts=new HtmlButtonGroups("",$buttons);
73
-		if($visibleHover)
71
+	private function getFieldButtons($buttons, $visibleHover=true) {
72
+		$bts=new HtmlButtonGroups("", $buttons);
73
+		if ($visibleHover)
74 74
 			$this->_visibleOver($bts);
75 75
 		return $bts;
76 76
 	}
@@ -84,19 +84,19 @@  discard block
 block discarded – undo
84 84
 	 * @param array $attributes associative array (<b>ajax</b> key is for ajax post)
85 85
 	 * @return DataTable
86 86
 	 */
87
-	public function fieldAsSubmit($index,$cssStyle=NULL,$url=NULL,$responseElement=NULL,$attributes=NULL){
88
-		return $this->_fieldAs(function($id,$name,$value,$caption) use ($url,$responseElement,$cssStyle,$attributes){
89
-			$button=new HtmlButton($id,$value,$cssStyle);
90
-			$button->postOnClick($url,"$(event.target).closest('tr').find(':input').serialize()",$responseElement,$attributes["ajax"]);
91
-			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"])
87
+	public function fieldAsSubmit($index, $cssStyle=NULL, $url=NULL, $responseElement=NULL, $attributes=NULL) {
88
+		return $this->_fieldAs(function($id, $name, $value, $caption) use ($url, $responseElement, $cssStyle, $attributes){
89
+			$button=new HtmlButton($id, $value, $cssStyle);
90
+			$button->postOnClick($url, "$(event.target).closest('tr').find(':input').serialize()", $responseElement, $attributes["ajax"]);
91
+			if (!isset($attributes["visibleHover"]) || $attributes["visibleHover"])
92 92
 				$this->_visibleOver($button);
93 93
 				return $button;
94
-		}, $index,$attributes);
94
+		}, $index, $attributes);
95 95
 	}
96 96
 
97
-	protected function _visibleOver(BaseHtml $element){
97
+	protected function _visibleOver(BaseHtml $element) {
98 98
 		$this->_visibleHover=true;
99
-		return $element->addToProperty("class", "visibleover")->setProperty("style","visibility:hidden;");
99
+		return $element->addToProperty("class", "visibleover")->setProperty("style", "visibility:hidden;");
100 100
 	}
101 101
 
102 102
 	/**
@@ -106,8 +106,8 @@  discard block
 block discarded – undo
106 106
 	 * @param boolean $visibleHover
107 107
 	 * @return DataTable
108 108
 	 */
109
-	public function addFieldButton($caption,$visibleHover=true,$callback=null){
110
-		$this->addField($this->getCallable("getFieldButton",[$caption,$visibleHover],$callback));
109
+	public function addFieldButton($caption, $visibleHover=true, $callback=null) {
110
+		$this->addField($this->getCallable("getFieldButton", [$caption, $visibleHover], $callback));
111 111
 		return $this;
112 112
 	}
113 113
 
@@ -118,8 +118,8 @@  discard block
 block discarded – undo
118 118
 	 * @param boolean $visibleHover
119 119
 	 * @return DataTable
120 120
 	 */
121
-	public function addFieldButtons($buttons,$visibleHover=true,$callback=null){
122
-		$this->addField($this->getCallable("getFieldButtons",[$buttons,$visibleHover],$callback));
121
+	public function addFieldButtons($buttons, $visibleHover=true, $callback=null) {
122
+		$this->addField($this->getCallable("getFieldButtons", [$buttons, $visibleHover], $callback));
123 123
 		return $this;
124 124
 	}
125 125
 
@@ -130,8 +130,8 @@  discard block
 block discarded – undo
130 130
 	 * @param callable $callback
131 131
 	 * @return DataTable
132 132
 	 */
133
-	public function insertFieldButton($index,$caption,$visibleHover=true,$callback=null){
134
-		$this->insertField($index, $this->getFieldButtonCallable($caption,$visibleHover,$callback));
133
+	public function insertFieldButton($index, $caption, $visibleHover=true, $callback=null) {
134
+		$this->insertField($index, $this->getFieldButtonCallable($caption, $visibleHover, $callback));
135 135
 		return $this;
136 136
 	}
137 137
 
@@ -142,25 +142,25 @@  discard block
 block discarded – undo
142 142
 	 * @param callable $callback
143 143
 	 * @return DataTable
144 144
 	 */
145
-	public function insertInFieldButton($index,$caption,$visibleHover=true,$callback=null){
146
-		$this->insertInField($index, $this->getFieldButtonCallable($caption,$visibleHover,$callback));
145
+	public function insertInFieldButton($index, $caption, $visibleHover=true, $callback=null) {
146
+		$this->insertInField($index, $this->getFieldButtonCallable($caption, $visibleHover, $callback));
147 147
 		return $this;
148 148
 	}
149 149
 
150
-	private function addDefaultButton($icon,$class=null,$visibleHover=true,$callback=null){
151
-		$this->addField($this->getCallable("getDefaultButton",[$icon,$class,$visibleHover],$callback));
150
+	private function addDefaultButton($icon, $class=null, $visibleHover=true, $callback=null) {
151
+		$this->addField($this->getCallable("getDefaultButton", [$icon, $class, $visibleHover], $callback));
152 152
 		return $this;
153 153
 	}
154 154
 
155
-	private function insertDefaultButtonIn($index,$icon,$class=null,$visibleHover=true,$callback=null){
156
-		$this->insertInField($index,$this->getCallable("getDefaultButton",[$icon,$class,$visibleHover],$callback));
155
+	private function insertDefaultButtonIn($index, $icon, $class=null, $visibleHover=true, $callback=null) {
156
+		$this->insertInField($index, $this->getCallable("getDefaultButton", [$icon, $class, $visibleHover], $callback));
157 157
 		return $this;
158 158
 	}
159 159
 
160
-	private function getDefaultButton($icon,$class=null,$visibleHover=true){
161
-		$bt=$this->getFieldButton("",$visibleHover);
160
+	private function getDefaultButton($icon, $class=null, $visibleHover=true) {
161
+		$bt=$this->getFieldButton("", $visibleHover);
162 162
 		$bt->asIcon($icon);
163
-		if(isset($class))
163
+		if (isset($class))
164 164
 			$bt->addClass($class);
165 165
 		return $bt;
166 166
 	}
@@ -172,9 +172,9 @@  discard block
 block discarded – undo
172 172
 	 * @param callable $callback this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
173 173
 	 * @return DataTable
174 174
 	 */
175
-	public function addDeleteButton($visibleHover=true,$deleteBehavior=[],$callback=null){
175
+	public function addDeleteButton($visibleHover=true, $deleteBehavior=[], $callback=null) {
176 176
 		$this->_deleteBehavior=$deleteBehavior;
177
-		return $this->addDefaultButton("remove","_delete red basic",$visibleHover,$callback);
177
+		return $this->addDefaultButton("remove", "_delete red basic", $visibleHover, $callback);
178 178
 	}
179 179
 
180 180
 	/**
@@ -184,9 +184,9 @@  discard block
 block discarded – undo
184 184
 	 * @param callable $callback this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
185 185
 	 * @return DataTable
186 186
 	 */
187
-	public function addEditButton($visibleHover=true,$editBehavior=[],$callback=null){
187
+	public function addEditButton($visibleHover=true, $editBehavior=[], $callback=null) {
188 188
 		$this->_editBehavior=$editBehavior;
189
-		return $this->addDefaultButton("edit","_edit basic",$visibleHover,$callback);
189
+		return $this->addDefaultButton("edit", "_edit basic", $visibleHover, $callback);
190 190
 	}
191 191
 
192 192
 	/**
@@ -197,20 +197,20 @@  discard block
 block discarded – undo
197 197
 	 * @param callable $callbackDelete this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
198 198
 	 * @return DataTable
199 199
 	 */
200
-	public function addEditDeleteButtons($visibleHover=true,$behavior=[],$callbackEdit=null,$callbackDelete=null){
201
-		$this->addEditButton($visibleHover,$behavior,$callbackEdit);
200
+	public function addEditDeleteButtons($visibleHover=true, $behavior=[], $callbackEdit=null, $callbackDelete=null) {
201
+		$this->addEditButton($visibleHover, $behavior, $callbackEdit);
202 202
 		$index=$this->_instanceViewer->visiblePropertiesCount()-1;
203
-		$this->insertDeleteButtonIn($index,$visibleHover,$behavior,$callbackDelete);
203
+		$this->insertDeleteButtonIn($index, $visibleHover, $behavior, $callbackDelete);
204 204
 		return $this;
205 205
 	}
206 206
 
207
-	public function insertDeleteButtonIn($index,$visibleHover=true,$deleteBehavior=[],$callback=null){
207
+	public function insertDeleteButtonIn($index, $visibleHover=true, $deleteBehavior=[], $callback=null) {
208 208
 		$this->_deleteBehavior=$deleteBehavior;
209
-		return $this->insertDefaultButtonIn($index,"remove","_delete red basic",$visibleHover,$callback);
209
+		return $this->insertDefaultButtonIn($index, "remove", "_delete red basic", $visibleHover, $callback);
210 210
 	}
211 211
 
212
-	public function insertEditButtonIn($index,$visibleHover=true,$editBehavior=[],$callback=null){
212
+	public function insertEditButtonIn($index, $visibleHover=true, $editBehavior=[], $callback=null) {
213 213
 		$this->_editBehavior=$editBehavior;
214
-		return $this->insertDefaultButtonIn($index,"edit","_edit basic",$visibleHover,$callback);
214
+		return $this->insertDefaultButtonIn($index, "edit", "_edit basic", $visibleHover, $callback);
215 215
 	}
216 216
 }
Please login to merge, or discard this patch.
Braces   +12 added lines, -8 removed lines patch added patch discarded remove patch
@@ -63,15 +63,17 @@  discard block
 block discarded – undo
63 63
 	 */
64 64
 	private function getFieldButton($caption,$visibleHover=true){
65 65
 		$bt= new HtmlButton($this->cleanIdentifier($caption),$caption);
66
-		if($visibleHover)
67
-			$this->_visibleOver($bt);
66
+		if($visibleHover) {
67
+					$this->_visibleOver($bt);
68
+		}
68 69
 		return $bt;
69 70
 	}
70 71
 
71 72
 	private function getFieldButtons($buttons,$visibleHover=true){
72 73
 		$bts=new HtmlButtonGroups("",$buttons);
73
-		if($visibleHover)
74
-			$this->_visibleOver($bts);
74
+		if($visibleHover) {
75
+					$this->_visibleOver($bts);
76
+		}
75 77
 		return $bts;
76 78
 	}
77 79
 
@@ -88,8 +90,9 @@  discard block
 block discarded – undo
88 90
 		return $this->_fieldAs(function($id,$name,$value,$caption) use ($url,$responseElement,$cssStyle,$attributes){
89 91
 			$button=new HtmlButton($id,$value,$cssStyle);
90 92
 			$button->postOnClick($url,"$(event.target).closest('tr').find(':input').serialize()",$responseElement,$attributes["ajax"]);
91
-			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"])
92
-				$this->_visibleOver($button);
93
+			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"]) {
94
+							$this->_visibleOver($button);
95
+			}
93 96
 				return $button;
94 97
 		}, $index,$attributes);
95 98
 	}
@@ -160,8 +163,9 @@  discard block
 block discarded – undo
160 163
 	private function getDefaultButton($icon,$class=null,$visibleHover=true){
161 164
 		$bt=$this->getFieldButton("",$visibleHover);
162 165
 		$bt->asIcon($icon);
163
-		if(isset($class))
164
-			$bt->addClass($class);
166
+		if(isset($class)) {
167
+					$bt->addClass($class);
168
+		}
165 169
 		return $bt;
166 170
 	}
167 171
 
Please login to merge, or discard this patch.
Ajax/semantic/html/content/HtmlMenuItem.php 2 patches
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -9,25 +9,25 @@
 block discarded – undo
9 9
 class HtmlMenuItem extends HtmlSemDoubleElement {
10 10
 	use MenuItemTrait;
11 11
 	public function __construct($identifier, $content) {
12
-		parent::__construct($identifier,"div","item",$content);
12
+		parent::__construct($identifier, "div", "item", $content);
13 13
 	}
14 14
 
15
-	protected function initContent($content){
16
-		if(\is_array($content)){
17
-			if(JArray::isAssociative($content)===false){
15
+	protected function initContent($content) {
16
+		if (\is_array($content)) {
17
+			if (JArray::isAssociative($content)===false) {
18 18
 				$icon=@$content[0];
19 19
 				$title=@$content[1];
20
-			}else{
20
+			} else {
21 21
 				$icon=@$content["icon"];
22 22
 				$title=@$content["title"];
23 23
 			}
24
-			if(isset($icon)){
24
+			if (isset($icon)) {
25 25
 				$this->addIcon($icon);
26 26
 			}
27
-			if(isset($title)){
27
+			if (isset($title)) {
28 28
 				$this->setTitle($title);
29 29
 			}
30
-		}else{
30
+		} else {
31 31
 			$this->setContent($content);
32 32
 		}
33 33
 	}
Please login to merge, or discard this patch.
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 			if(JArray::isAssociative($content)===false){
18 18
 				$icon=@$content[0];
19 19
 				$title=@$content[1];
20
-			}else{
20
+			} else{
21 21
 				$icon=@$content["icon"];
22 22
 				$title=@$content["title"];
23 23
 			}
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
 			if(isset($title)){
28 28
 				$this->setTitle($title);
29 29
 			}
30
-		}else{
30
+		} else{
31 31
 			$this->setContent($content);
32 32
 		}
33 33
 	}
Please login to merge, or discard this patch.
Ajax/semantic/widgets/datatable/DataTable.php 2 patches
Spacing   +107 added lines, -107 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
  *
25 25
  */
26 26
 class DataTable extends Widget {
27
-	use TableTrait,DataTableFieldAsTrait,HasCheckboxesTrait,BaseTrait;
27
+	use TableTrait, DataTableFieldAsTrait, HasCheckboxesTrait, BaseTrait;
28 28
 	protected $_searchField;
29 29
 	protected $_urls;
30 30
 	protected $_pagination;
@@ -42,35 +42,35 @@  discard block
 block discarded – undo
42 42
 	protected $_colWidths;
43 43
 
44 44
 
45
-	public function __construct($identifier,$model,$modelInstance=NULL) {
46
-		parent::__construct($identifier, $model,$modelInstance);
47
-		$this->_init(new InstanceViewer($identifier), "table", new HtmlTable($identifier, 0,0), false);
45
+	public function __construct($identifier, $model, $modelInstance=NULL) {
46
+		parent::__construct($identifier, $model, $modelInstance);
47
+		$this->_init(new InstanceViewer($identifier), "table", new HtmlTable($identifier, 0, 0), false);
48 48
 		$this->_urls=[];
49
-		$this->_emptyMessage=new HtmlMessage("","nothing to display");
49
+		$this->_emptyMessage=new HtmlMessage("", "nothing to display");
50 50
 		$this->_emptyMessage->setIcon("info circle");
51 51
 	}
52 52
 
53
-	public function run(JsUtils $js){
54
-		if($this->_hasCheckboxes && isset($js)){
53
+	public function run(JsUtils $js) {
54
+		if ($this->_hasCheckboxes && isset($js)) {
55 55
 			$this->_runCheckboxes($js);
56 56
 		}
57
-		if($this->_visibleHover){
58
-			$js->execOn("mouseover", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'visible');",["preventDefault"=>false,"stopPropagation"=>true]);
59
-			$js->execOn("mouseout", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'hidden');",["preventDefault"=>false,"stopPropagation"=>true]);
57
+		if ($this->_visibleHover) {
58
+			$js->execOn("mouseover", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'visible');", ["preventDefault"=>false, "stopPropagation"=>true]);
59
+			$js->execOn("mouseout", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'hidden');", ["preventDefault"=>false, "stopPropagation"=>true]);
60 60
 		}
61
-		if(\is_array($this->_deleteBehavior))
62
-			$this->_generateBehavior("delete",$this->_deleteBehavior, $js);
63
-		if(\is_array($this->_editBehavior))
64
-			$this->_generateBehavior("edit",$this->_editBehavior,$js);
61
+		if (\is_array($this->_deleteBehavior))
62
+			$this->_generateBehavior("delete", $this->_deleteBehavior, $js);
63
+		if (\is_array($this->_editBehavior))
64
+			$this->_generateBehavior("edit", $this->_editBehavior, $js);
65 65
 		parent::run($js);
66 66
 	}
67 67
 
68 68
 
69 69
 
70
-	protected function _generateBehavior($op,$params,JsUtils $js){
71
-		if(isset($this->_urls[$op])){
72
-			$params=\array_merge($params,["attr"=>"data-ajax"]);
73
-			$js->ajaxOnClick("#".$this->identifier." ._".$op, $this->_urls[$op],$this->getTargetSelector($op),$params);
70
+	protected function _generateBehavior($op, $params, JsUtils $js) {
71
+		if (isset($this->_urls[$op])) {
72
+			$params=\array_merge($params, ["attr"=>"data-ajax"]);
73
+			$js->ajaxOnClick("#".$this->identifier." ._".$op, $this->_urls[$op], $this->getTargetSelector($op), $params);
74 74
 		}
75 75
 	}
76 76
 
@@ -83,164 +83,164 @@  discard block
 block discarded – undo
83 83
 	}
84 84
 
85 85
 
86
-	public function compile(JsUtils $js=NULL,&$view=NULL){
87
-		if(!$this->_generated){
86
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
87
+		if (!$this->_generated) {
88 88
 			$this->_instanceViewer->setInstance($this->_model);
89 89
 			$captions=$this->_instanceViewer->getCaptions();
90 90
 			$table=$this->content["table"];
91
-			if($this->_hasCheckboxes){
91
+			if ($this->_hasCheckboxes) {
92 92
 				$this->_generateMainCheckbox($captions);
93 93
 			}
94 94
 			$table->setRowCount(0, \sizeof($captions));
95
-			$this->_generateHeader($table,$captions);
95
+			$this->_generateHeader($table, $captions);
96 96
 
97
-			if(isset($this->_compileParts))
97
+			if (isset($this->_compileParts))
98 98
 				$table->setCompileParts($this->_compileParts);
99 99
 
100 100
 			$this->_generateContent($table);
101 101
 
102
-			$this->compileExtraElements($table, $captions,$js);
102
+			$this->compileExtraElements($table, $captions, $js);
103 103
 
104
-			$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE,"table",PositionInTable::AFTERTABLE]);
104
+			$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE, "table", PositionInTable::AFTERTABLE]);
105 105
 			$this->_compileForm();
106 106
 			$this->_applyStyleAttributes($table);
107 107
 			$this->_generated=true;
108 108
 		}
109
-		return parent::compile($js,$view);
109
+		return parent::compile($js, $view);
110 110
 	}
111 111
 
112
-	protected function compileExtraElements($table,$captions,JsUtils $js=NULL){
113
-		if(isset($this->_searchField) && isset($js) && isset($this->_urls["refresh"])){
114
-				$this->_searchField->postOn("change", $this->_urls["refresh"],"{'s':$(this).val()}","#".$this->identifier." tbody",["preventDefault"=>false,"jqueryDone"=>"replaceWith"]);
112
+	protected function compileExtraElements($table, $captions, JsUtils $js=NULL) {
113
+		if (isset($this->_searchField) && isset($js) && isset($this->_urls["refresh"])) {
114
+				$this->_searchField->postOn("change", $this->_urls["refresh"], "{'s':$(this).val()}", "#".$this->identifier." tbody", ["preventDefault"=>false, "jqueryDone"=>"replaceWith"]);
115 115
 		}
116
-		if($this->_hasCheckboxes && $table->hasPart("thead")){
116
+		if ($this->_hasCheckboxes && $table->hasPart("thead")) {
117 117
 			$table->getHeader()->getCell(0, 0)->addClass("no-sort");
118 118
 		}
119 119
 
120
-		if(isset($this->_toolbar)){
120
+		if (isset($this->_toolbar)) {
121 121
 			$this->_setToolbarPosition($table, $captions);
122 122
 		}
123
-		if(isset($this->_pagination) && $this->_pagination->getVisible()){
124
-			$this->_generatePagination($table,$js);
123
+		if (isset($this->_pagination) && $this->_pagination->getVisible()) {
124
+			$this->_generatePagination($table, $js);
125 125
 		}
126 126
 	}
127 127
 
128
-	protected function _applyStyleAttributes($table){
129
-		if(isset($this->_hiddenColumns))
128
+	protected function _applyStyleAttributes($table) {
129
+		if (isset($this->_hiddenColumns))
130 130
 			$this->_hideColumns();
131
-			if(isset($this->_colWidths)){
132
-				foreach ($this->_colWidths as $colIndex=>$width){
133
-					$table->setColWidth($colIndex,$width);
131
+			if (isset($this->_colWidths)) {
132
+				foreach ($this->_colWidths as $colIndex=>$width) {
133
+					$table->setColWidth($colIndex, $width);
134 134
 				}
135 135
 			}
136 136
 	}
137 137
 
138
-	protected function _hideColumns(){
139
-		foreach ($this->_hiddenColumns as $colIndex){
138
+	protected function _hideColumns() {
139
+		foreach ($this->_hiddenColumns as $colIndex) {
140 140
 			$this->_self->hideColumn($colIndex);
141 141
 		}
142 142
 		return $this;
143 143
 	}
144 144
 
145
-	protected function _generateHeader(HtmlTable $table,$captions){
145
+	protected function _generateHeader(HtmlTable $table, $captions) {
146 146
 		$table->setHeaderValues($captions);
147
-		if(isset($this->_sortable)){
147
+		if (isset($this->_sortable)) {
148 148
 			$table->setSortable($this->_sortable);
149 149
 		}
150 150
 	}
151 151
 
152 152
 
153 153
 
154
-	protected function _generateContent($table){
154
+	protected function _generateContent($table) {
155 155
 		$objects=$this->_modelInstance;
156
-		if(isset($this->_pagination)){
156
+		if (isset($this->_pagination)) {
157 157
 			$objects=$this->_pagination->getObjects($this->_modelInstance);
158 158
 		}
159 159
 			InstanceViewer::setIndex(0);
160 160
 			$table->fromDatabaseObjects($objects, function($instance) use($table){
161 161
 				return $this->_generateRow($instance, $table);
162 162
 			});
163
-		if($table->getRowCount()==0){
163
+		if ($table->getRowCount()==0) {
164 164
 			$result=$table->addRow();
165 165
 			$result->mergeRow();
166 166
 			$result->setValues([$this->_emptyMessage]);
167 167
 		}
168 168
 	}
169 169
 
170
-	protected function _generateRow($instance,&$table,$checkedClass=null){
170
+	protected function _generateRow($instance, &$table, $checkedClass=null) {
171 171
 		$this->_instanceViewer->setInstance($instance);
172 172
 		InstanceViewer::$index++;
173
-		$values= $this->_instanceViewer->getValues();
173
+		$values=$this->_instanceViewer->getValues();
174 174
 		$id=$this->_instanceViewer->getIdentifier();
175 175
 		$dataAjax=$id;
176 176
 		$id=$this->cleanIdentifier($id);
177
-		if($this->_hasCheckboxes){
178
-			$ck=new HtmlCheckbox("ck-".$this->identifier."-".$id,"");
177
+		if ($this->_hasCheckboxes) {
178
+			$ck=new HtmlCheckbox("ck-".$this->identifier."-".$id, "");
179 179
 			$checked=false;
180
-			if(isset($this->_checkedCallback)){
180
+			if (isset($this->_checkedCallback)) {
181 181
 				$func=$this->_checkedCallback;
182 182
 				$checked=$func($instance);
183 183
 			}
184 184
 			$ck->setChecked($checked);
185 185
 			$ck->setOnChange("event.stopPropagation();");
186 186
 			$field=$ck->getField();
187
-			$field->setProperty("value",$dataAjax);
187
+			$field->setProperty("value", $dataAjax);
188 188
 			$field->setProperty("name", "selection[]");
189
-			if(isset($checkedClass))
189
+			if (isset($checkedClass))
190 190
 				$field->setClass($checkedClass);
191 191
 			\array_unshift($values, $ck);
192 192
 		}
193 193
 		$result=$table->newRow();
194 194
 		$result->setIdentifier($this->identifier."-tr-".$id);
195
-		$result->setProperty("data-ajax",$dataAjax);
195
+		$result->setProperty("data-ajax", $dataAjax);
196 196
 		$result->setValues($values);
197
-		$result->addToProperty("class",$this->_rowClass);
197
+		$result->addToProperty("class", $this->_rowClass);
198 198
 		return $result;
199 199
 	}
200 200
 
201
-	protected function _generatePagination($table,$js=NULL){
202
-		if(isset($this->_toolbar)){
203
-			if($this->_toolbarPosition==PositionInTable::FOOTER)
201
+	protected function _generatePagination($table, $js=NULL) {
202
+		if (isset($this->_toolbar)) {
203
+			if ($this->_toolbarPosition==PositionInTable::FOOTER)
204 204
 				$this->_toolbar->setFloated("left");
205 205
 		}
206 206
 		$footer=$table->getFooter();
207 207
 		$footer->mergeCol();
208
-		$menu=new HtmlPaginationMenu("pagination-".$this->identifier,$this->_pagination->getPagesNumbers());
208
+		$menu=new HtmlPaginationMenu("pagination-".$this->identifier, $this->_pagination->getPagesNumbers());
209 209
 		$menu->floatRight();
210 210
 		$menu->setActiveItem($this->_pagination->getPage()-1);
211 211
 		$footer->addValues($menu);
212
-		$this->_associatePaginationBehavior($menu,$js);
212
+		$this->_associatePaginationBehavior($menu, $js);
213 213
 	}
214 214
 
215
-	protected function _associatePaginationBehavior(HtmlMenu $menu,JsUtils $js=NULL){
216
-		if(isset($this->_urls["refresh"])){
217
-			$menu->postOnClick($this->_urls["refresh"],"{'p':$(this).attr('data-page')}",$this->getRefreshSelector(),["preventDefault"=>false,"jqueryDone"=>"replaceWith","hasLoader"=>false]);
215
+	protected function _associatePaginationBehavior(HtmlMenu $menu, JsUtils $js=NULL) {
216
+		if (isset($this->_urls["refresh"])) {
217
+			$menu->postOnClick($this->_urls["refresh"], "{'p':$(this).attr('data-page')}", $this->getRefreshSelector(), ["preventDefault"=>false, "jqueryDone"=>"replaceWith", "hasLoader"=>false]);
218 218
 		}
219 219
 	}
220 220
 
221
-	protected function _getFieldName($index){
221
+	protected function _getFieldName($index) {
222 222
 		$fieldName=parent::_getFieldName($index);
223
-		if(\is_object($fieldName))
223
+		if (\is_object($fieldName))
224 224
 			$fieldName="field-".$index;
225 225
 		return $fieldName."[]";
226 226
 	}
227 227
 
228
-	protected function _getFieldCaption($index){
228
+	protected function _getFieldCaption($index) {
229 229
 		return null;
230 230
 	}
231 231
 
232
-	protected function _setToolbarPosition($table,$captions=NULL){
233
-		switch ($this->_toolbarPosition){
232
+	protected function _setToolbarPosition($table, $captions=NULL) {
233
+		switch ($this->_toolbarPosition) {
234 234
 			case PositionInTable::BEFORETABLE:
235 235
 			case PositionInTable::AFTERTABLE:
236
-				if(isset($this->_compileParts)===false){
236
+				if (isset($this->_compileParts)===false) {
237 237
 					$this->content[$this->_toolbarPosition]=$this->_toolbar;
238 238
 				}
239 239
 				break;
240 240
 			case PositionInTable::HEADER:
241 241
 			case PositionInTable::FOOTER:
242 242
 			case PositionInTable::BODY:
243
-				$this->addToolbarRow($this->_toolbarPosition,$table, $captions);
243
+				$this->addToolbarRow($this->_toolbarPosition, $table, $captions);
244 244
 				break;
245 245
 		}
246 246
 	}
@@ -252,16 +252,16 @@  discard block
 block discarded – undo
252 252
 	 * @param callable $callback function called after the field compilation
253 253
 	 * @return DataTable
254 254
 	 */
255
-	public function afterCompile($index,$callback){
256
-		$this->_instanceViewer->afterCompile($index,$callback);
255
+	public function afterCompile($index, $callback) {
256
+		$this->_instanceViewer->afterCompile($index, $callback);
257 257
 		return $this;
258 258
 	}
259 259
 
260
-	private function addToolbarRow($part,$table,$captions){
260
+	private function addToolbarRow($part, $table, $captions) {
261 261
 		$hasPart=$table->hasPart($part);
262
-		if($hasPart){
262
+		if ($hasPart) {
263 263
 			$row=$table->getPart($part)->addRow(\sizeof($captions));
264
-		}else{
264
+		} else {
265 265
 			$row=$table->getPart($part)->getRow(0);
266 266
 		}
267 267
 		$row->mergeCol();
@@ -273,7 +273,7 @@  discard block
 block discarded – undo
273 273
 	 * @see Widget::getHtmlComponent()
274 274
 	 * @return HtmlTable
275 275
 	 */
276
-	public function getHtmlComponent(){
276
+	public function getHtmlComponent() {
277 277
 		return $this->content["table"];
278 278
 	}
279 279
 
@@ -288,12 +288,12 @@  discard block
 block discarded – undo
288 288
 	 * @return DataTable
289 289
 	 */
290 290
 	public function setUrls($urls) {
291
-		if(\is_array($urls)){
292
-			$this->_urls["refresh"]=JArray::getValue($urls, "refresh",0);
293
-			$this->_urls["edit"]=JArray::getValue($urls, "edit",1);
294
-			$this->_urls["delete"]=JArray::getValue($urls, "delete",2);
295
-		}else{
296
-			$this->_urls=["refresh"=>$urls,"edit"=>$urls,"delete"=>$urls];
291
+		if (\is_array($urls)) {
292
+			$this->_urls["refresh"]=JArray::getValue($urls, "refresh", 0);
293
+			$this->_urls["edit"]=JArray::getValue($urls, "edit", 1);
294
+			$this->_urls["delete"]=JArray::getValue($urls, "delete", 2);
295
+		} else {
296
+			$this->_urls=["refresh"=>$urls, "edit"=>$urls, "delete"=>$urls];
297 297
 		}
298 298
 		return $this;
299 299
 	}
@@ -306,8 +306,8 @@  discard block
 block discarded – undo
306 306
 	 * @param number $pages_visibles The number of visible pages in the Pagination component
307 307
 	 * @return DataTable
308 308
 	 */
309
-	public function paginate($page,$total_rowcount,$items_per_page=10,$pages_visibles=null){
310
-		$this->_pagination=new Pagination($items_per_page,$pages_visibles,$page,$total_rowcount);
309
+	public function paginate($page, $total_rowcount, $items_per_page=10, $pages_visibles=null) {
310
+		$this->_pagination=new Pagination($items_per_page, $pages_visibles, $page, $total_rowcount);
311 311
 		return $this;
312 312
 	}
313 313
 
@@ -318,8 +318,8 @@  discard block
 block discarded – undo
318 318
 	 * @param number $pages_visibles The number of visible pages in the Pagination component
319 319
 	 * @return DataTable
320 320
 	 */
321
-	public function autoPaginate($page=1,$items_per_page=10,$pages_visibles=4){
322
-		$this->_pagination=new Pagination($items_per_page,$pages_visibles,$page);
321
+	public function autoPaginate($page=1, $items_per_page=10, $pages_visibles=4) {
322
+		$this->_pagination=new Pagination($items_per_page, $pages_visibles, $page);
323 323
 		return $this;
324 324
 	}
325 325
 
@@ -329,7 +329,7 @@  discard block
 block discarded – undo
329 329
 	 * @param array $compileParts
330 330
 	 * @return DataTable
331 331
 	 */
332
-	public function refresh($compileParts=["tbody"]){
332
+	public function refresh($compileParts=["tbody"]) {
333 333
 		$this->_compileParts=$compileParts;
334 334
 		return $this;
335 335
 	}
@@ -340,14 +340,14 @@  discard block
 block discarded – undo
340 340
 	 * @param string $position
341 341
 	 * @return \Ajax\common\html\HtmlDoubleElement
342 342
 	 */
343
-	public function addSearchInToolbar($position=Direction::RIGHT){
343
+	public function addSearchInToolbar($position=Direction::RIGHT) {
344 344
 		return $this->addInToolbar($this->getSearchField())->setPosition($position);
345 345
 	}
346 346
 
347
-	public function getSearchField(){
348
-		if(isset($this->_searchField)===false){
349
-			$this->_searchField=new HtmlInput("search-".$this->identifier,"search","","Search...");
350
-			$this->_searchField->addIcon("search",Direction::RIGHT);
347
+	public function getSearchField() {
348
+		if (isset($this->_searchField)===false) {
349
+			$this->_searchField=new HtmlInput("search-".$this->identifier, "search", "", "Search...");
350
+			$this->_searchField->addIcon("search", Direction::RIGHT);
351 351
 		}
352 352
 		return $this->_searchField;
353 353
 	}
@@ -367,7 +367,7 @@  discard block
 block discarded – undo
367 367
 	 * Returns a form corresponding to the Datatable
368 368
 	 * @return \Ajax\semantic\html\collections\form\HtmlForm
369 369
 	 */
370
-	public function asForm(){
370
+	public function asForm() {
371 371
 		return $this->getForm();
372 372
 	}
373 373
 
@@ -375,7 +375,7 @@  discard block
 block discarded – undo
375 375
 
376 376
 	protected function getTargetSelector($op) {
377 377
 		$result=$this->_targetSelector;
378
-		if(!isset($result[$op]))
378
+		if (!isset($result[$op]))
379 379
 			$result="#".$this->identifier;
380 380
 		return $result[$op];
381 381
 	}
@@ -386,15 +386,15 @@  discard block
 block discarded – undo
386 386
 	 * @return DataTable
387 387
 	 */
388 388
 	public function setTargetSelector($_targetSelector) {
389
-		if(!\is_array($_targetSelector)){
390
-			$_targetSelector=["edit"=>$_targetSelector,"delete"=>$_targetSelector];
389
+		if (!\is_array($_targetSelector)) {
390
+			$_targetSelector=["edit"=>$_targetSelector, "delete"=>$_targetSelector];
391 391
 		}
392 392
 		$this->_targetSelector=$_targetSelector;
393 393
 		return $this;
394 394
 	}
395 395
 
396 396
 	public function getRefreshSelector() {
397
-		if(isset($this->_refreshSelector))
397
+		if (isset($this->_refreshSelector))
398 398
 			return $this->_refreshSelector;
399 399
 		return "#".$this->identifier." tbody";
400 400
 	}
@@ -412,9 +412,9 @@  discard block
 block discarded – undo
412 412
 	 * {@inheritDoc}
413 413
 	 * @see \Ajax\common\Widget::show()
414 414
 	 */
415
-	public function show($modelInstance){
416
-		if(\is_array($modelInstance)){
417
-			if(isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0]))
415
+	public function show($modelInstance) {
416
+		if (\is_array($modelInstance)) {
417
+			if (isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0]))
418 418
 				$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
419 419
 		}
420 420
 		$this->_modelInstance=$modelInstance;
@@ -449,29 +449,29 @@  discard block
 block discarded – undo
449 449
 		return $this;
450 450
 	}
451 451
 
452
-	public function setActiveRowSelector($class="active",$event="click",$multiple=false){
453
-		$this->_self->setActiveRowSelector($class,$event,$multiple);
452
+	public function setActiveRowSelector($class="active", $event="click", $multiple=false) {
453
+		$this->_self->setActiveRowSelector($class, $event, $multiple);
454 454
 		return $this;
455 455
 	}
456 456
 
457
-	public function hideColumn($colIndex){
458
-		if(!\is_array($this->_hiddenColumns))
457
+	public function hideColumn($colIndex) {
458
+		if (!\is_array($this->_hiddenColumns))
459 459
 			$this->_hiddenColumns=[];
460 460
 		$this->_hiddenColumns[]=$colIndex;
461 461
 		return $this;
462 462
 	}
463 463
 
464
-	public function setColWidth($colIndex,$width){
464
+	public function setColWidth($colIndex, $width) {
465 465
 		$this->_colWidths[$colIndex]=$width;
466 466
 		return $this;
467 467
 	}
468 468
 	public function setColWidths($_colWidths) {
469
-		$this->_colWidths = $_colWidths;
469
+		$this->_colWidths=$_colWidths;
470 470
 		return $this;
471 471
 	}
472 472
 
473
-	public function setColAlignment($colIndex,$alignment){
474
-		$this->content["table"]->setColAlignment($colIndex,$alignment);
473
+	public function setColAlignment($colIndex, $alignment) {
474
+		$this->content["table"]->setColAlignment($colIndex, $alignment);
475 475
 		return $this;
476 476
 	}
477 477
 }
Please login to merge, or discard this patch.
Braces   +35 added lines, -24 removed lines patch added patch discarded remove patch
@@ -58,10 +58,12 @@  discard block
 block discarded – undo
58 58
 			$js->execOn("mouseover", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'visible');",["preventDefault"=>false,"stopPropagation"=>true]);
59 59
 			$js->execOn("mouseout", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'hidden');",["preventDefault"=>false,"stopPropagation"=>true]);
60 60
 		}
61
-		if(\is_array($this->_deleteBehavior))
62
-			$this->_generateBehavior("delete",$this->_deleteBehavior, $js);
63
-		if(\is_array($this->_editBehavior))
64
-			$this->_generateBehavior("edit",$this->_editBehavior,$js);
61
+		if(\is_array($this->_deleteBehavior)) {
62
+					$this->_generateBehavior("delete",$this->_deleteBehavior, $js);
63
+		}
64
+		if(\is_array($this->_editBehavior)) {
65
+					$this->_generateBehavior("edit",$this->_editBehavior,$js);
66
+		}
65 67
 		parent::run($js);
66 68
 	}
67 69
 
@@ -94,8 +96,9 @@  discard block
 block discarded – undo
94 96
 			$table->setRowCount(0, \sizeof($captions));
95 97
 			$this->_generateHeader($table,$captions);
96 98
 
97
-			if(isset($this->_compileParts))
98
-				$table->setCompileParts($this->_compileParts);
99
+			if(isset($this->_compileParts)) {
100
+							$table->setCompileParts($this->_compileParts);
101
+			}
99 102
 
100 103
 			$this->_generateContent($table);
101 104
 
@@ -126,8 +129,9 @@  discard block
 block discarded – undo
126 129
 	}
127 130
 
128 131
 	protected function _applyStyleAttributes($table){
129
-		if(isset($this->_hiddenColumns))
130
-			$this->_hideColumns();
132
+		if(isset($this->_hiddenColumns)) {
133
+					$this->_hideColumns();
134
+		}
131 135
 			if(isset($this->_colWidths)){
132 136
 				foreach ($this->_colWidths as $colIndex=>$width){
133 137
 					$table->setColWidth($colIndex,$width);
@@ -186,8 +190,9 @@  discard block
 block discarded – undo
186 190
 			$field=$ck->getField();
187 191
 			$field->setProperty("value",$dataAjax);
188 192
 			$field->setProperty("name", "selection[]");
189
-			if(isset($checkedClass))
190
-				$field->setClass($checkedClass);
193
+			if(isset($checkedClass)) {
194
+							$field->setClass($checkedClass);
195
+			}
191 196
 			\array_unshift($values, $ck);
192 197
 		}
193 198
 		$result=$table->newRow();
@@ -200,8 +205,9 @@  discard block
 block discarded – undo
200 205
 
201 206
 	protected function _generatePagination($table,$js=NULL){
202 207
 		if(isset($this->_toolbar)){
203
-			if($this->_toolbarPosition==PositionInTable::FOOTER)
204
-				$this->_toolbar->setFloated("left");
208
+			if($this->_toolbarPosition==PositionInTable::FOOTER) {
209
+							$this->_toolbar->setFloated("left");
210
+			}
205 211
 		}
206 212
 		$footer=$table->getFooter();
207 213
 		$footer->mergeCol();
@@ -220,8 +226,9 @@  discard block
 block discarded – undo
220 226
 
221 227
 	protected function _getFieldName($index){
222 228
 		$fieldName=parent::_getFieldName($index);
223
-		if(\is_object($fieldName))
224
-			$fieldName="field-".$index;
229
+		if(\is_object($fieldName)) {
230
+					$fieldName="field-".$index;
231
+		}
225 232
 		return $fieldName."[]";
226 233
 	}
227 234
 
@@ -261,7 +268,7 @@  discard block
 block discarded – undo
261 268
 		$hasPart=$table->hasPart($part);
262 269
 		if($hasPart){
263 270
 			$row=$table->getPart($part)->addRow(\sizeof($captions));
264
-		}else{
271
+		} else{
265 272
 			$row=$table->getPart($part)->getRow(0);
266 273
 		}
267 274
 		$row->mergeCol();
@@ -292,7 +299,7 @@  discard block
 block discarded – undo
292 299
 			$this->_urls["refresh"]=JArray::getValue($urls, "refresh",0);
293 300
 			$this->_urls["edit"]=JArray::getValue($urls, "edit",1);
294 301
 			$this->_urls["delete"]=JArray::getValue($urls, "delete",2);
295
-		}else{
302
+		} else{
296 303
 			$this->_urls=["refresh"=>$urls,"edit"=>$urls,"delete"=>$urls];
297 304
 		}
298 305
 		return $this;
@@ -375,8 +382,9 @@  discard block
 block discarded – undo
375 382
 
376 383
 	protected function getTargetSelector($op) {
377 384
 		$result=$this->_targetSelector;
378
-		if(!isset($result[$op]))
379
-			$result="#".$this->identifier;
385
+		if(!isset($result[$op])) {
386
+					$result="#".$this->identifier;
387
+		}
380 388
 		return $result[$op];
381 389
 	}
382 390
 
@@ -394,8 +402,9 @@  discard block
 block discarded – undo
394 402
 	}
395 403
 
396 404
 	public function getRefreshSelector() {
397
-		if(isset($this->_refreshSelector))
398
-			return $this->_refreshSelector;
405
+		if(isset($this->_refreshSelector)) {
406
+					return $this->_refreshSelector;
407
+		}
399 408
 		return "#".$this->identifier." tbody";
400 409
 	}
401 410
 
@@ -414,8 +423,9 @@  discard block
 block discarded – undo
414 423
 	 */
415 424
 	public function show($modelInstance){
416 425
 		if(\is_array($modelInstance)){
417
-			if(isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0]))
418
-				$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
426
+			if(isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0])) {
427
+							$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
428
+			}
419 429
 		}
420 430
 		$this->_modelInstance=$modelInstance;
421 431
 	}
@@ -455,8 +465,9 @@  discard block
 block discarded – undo
455 465
 	}
456 466
 
457 467
 	public function hideColumn($colIndex){
458
-		if(!\is_array($this->_hiddenColumns))
459
-			$this->_hiddenColumns=[];
468
+		if(!\is_array($this->_hiddenColumns)) {
469
+					$this->_hiddenColumns=[];
470
+		}
460 471
 		$this->_hiddenColumns[]=$colIndex;
461 472
 		return $this;
462 473
 	}
Please login to merge, or discard this patch.
Ajax/semantic/html/elements/HtmlIconGroups.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 	/**
25 25
 	 * @return HtmlIcon
26 26
 	 */
27
-	public function getItem($index){
27
+	public function getItem($index) {
28 28
 		return parent::getItem($index);
29 29
 	}
30 30
 
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
 			$icon=JArray::getValue($value, "icon", 0);
35 35
 			$size=JArray::getValue($value, "size", 1);
36 36
 		}
37
-		$iconO=new HtmlIcon("icon-" . $this->identifier, $icon);
37
+		$iconO=new HtmlIcon("icon-".$this->identifier, $icon);
38 38
 		if (isset($size)) {
39 39
 			$iconO->setSize($size);
40 40
 		}
@@ -42,15 +42,15 @@  discard block
 block discarded – undo
42 42
 	}
43 43
 
44 44
 	protected function createCondition($value) {
45
-		return ($value instanceof HtmlIcon) === false;
45
+		return ($value instanceof HtmlIcon)===false;
46 46
 	}
47 47
 
48 48
 	public function getIcon($index) {
49 49
 		return $this->content[$index];
50 50
 	}
51 51
 
52
-	public function run(JsUtils $js){
53
-		$result= parent::run($js);
52
+	public function run(JsUtils $js) {
53
+		$result=parent::run($js);
54 54
 		return $result->setItemSelector("i");
55 55
 	}
56 56
 
@@ -59,8 +59,8 @@  discard block
 block discarded – undo
59 59
 		return $this;
60 60
 	}
61 61
 
62
-	public static function corner($mainIcon,$cornerIcon,$size="huge"){
63
-		$icons=new HtmlIconGroups("icons",[$mainIcon,$cornerIcon],$size);
62
+	public static function corner($mainIcon, $cornerIcon, $size="huge") {
63
+		$icons=new HtmlIconGroups("icons", [$mainIcon, $cornerIcon], $size);
64 64
 		return $icons->toCorner(1);
65 65
 	}
66 66
 }
Please login to merge, or discard this patch.