phpMv / phpMv-UI

Ajax/semantic/widgets/base/InstanceViewer.php

Spacing +90 added lines, -90 removed lines

@@ -19,211 +19,211 @@ 
 
 	public static $index=0;
 
-	public function __construct($identifier,$instance=NULL,$captions=NULL){
+	public function __construct($identifier, $instance=NULL, $captions=NULL) {
 		$this->widgetIdentifier=$identifier;
 		$this->values=[];
 		$this->afterCompile=[];
-		if(isset($instance))
+		if (isset($instance))
 			$this->setInstance($instance);
 		$this->setCaptions($captions);
 		$this->captionCallback=NULL;
-		$this->defaultValueFunction=function($name,$value){return $value;};
+		$this->defaultValueFunction=function($name, $value) {return $value; };
 	}
 
-	public function moveFieldTo($from,$to){
-		if(JArray::moveElementTo($this->visibleProperties, $from, $to)){
+	public function moveFieldTo($from, $to) {
+		if (JArray::moveElementTo($this->visibleProperties, $from, $to)) {
 			return JArray::moveElementTo($this->values, $from, $to);
 		}
 		return false;
 	}
 
-	public function swapFields($index1,$index2){
-		if(JArray::swapElements($this->visibleProperties, $index1, $index2)){
+	public function swapFields($index1, $index2) {
+		if (JArray::swapElements($this->visibleProperties, $index1, $index2)) {
 			return JArray::swapElements($this->values, $index1, $index2);
 		}
 		return false;
 	}
 
-	public function removeField($index){
-		\array_splice($this->visibleProperties,$index,1);
-		\array_splice($this->values,$index,1);
-		\array_splice($this->captions,$index,1);
+	public function removeField($index) {
+		\array_splice($this->visibleProperties, $index, 1);
+		\array_splice($this->values, $index, 1);
+		\array_splice($this->captions, $index, 1);
 		return $this;
 	}
 
-	public function getValues(){
+	public function getValues() {
 		$values=[];
 		$index=0;
 		$count=$this->count();
-		while($index<$count){
+		while ($index<$count) {
 			$values[]=$this->getValue($index++);
 		}
 		return $values;
 	}
 
-	public function getIdentifier($index=NULL){
-		if(!isset($index))
+	public function getIdentifier($index=NULL) {
+		if (!isset($index))
 			$index=self::$index;
 		$value=$index;
-		if(isset($this->values["identifier"])){
-			if(\is_string($this->values["identifier"]))
+		if (isset($this->values["identifier"])) {
+			if (\is_string($this->values["identifier"]))
 				$value=JReflection::callMethod($this->instance, $this->values["identifier"], []);
 			else
-				$value=$this->values["identifier"]($index,$this->instance);
+				$value=$this->values["identifier"]($index, $this->instance);
 		}
 		return $value;
 	}
 
-	public function getValue($index){
+	public function getValue($index) {
 		$property=$this->properties[$index];
 		return $this->_getValue($property, $index);
 	}
 
-	protected function _beforeAddProperty($index,&$field){
+	protected function _beforeAddProperty($index, &$field) {
 
 	}
 
-	protected function _getDefaultValue($name,$value,$index){
+	protected function _getDefaultValue($name, $value, $index) {
 		$func=$this->defaultValueFunction;
-		return $func($name,$value,$index,$this->instance);
+		return $func($name, $value, $index, $this->instance);
 	}
 
-	protected function _getPropertyValue(\ReflectionProperty $property,$index){
+	protected function _getPropertyValue(\ReflectionProperty $property, $index) {
 		$property->setAccessible(true);
 		$value=$property->getValue($this->instance);
-		if(isset($this->values[$index])){
-			$value= $this->values[$index]($value,$this->instance,$index);
-		}else{
-			$value=$this->_getDefaultValue($property->getName(),$value, $index);
+		if (isset($this->values[$index])) {
+			$value=$this->values[$index]($value, $this->instance, $index);
+		} else {
+			$value=$this->_getDefaultValue($property->getName(), $value, $index);
 		}
 		return $value;
 	}
 
-	protected function _getValue($property,$index){
+	protected function _getValue($property, $index) {
 		$value=null;
-		if($property instanceof \ReflectionProperty){
+		if ($property instanceof \ReflectionProperty) {
 			$value=$this->_getPropertyValue($property, $index);
-		}else{
-			if(\is_callable($property))
+		} else {
+			if (\is_callable($property))
 				$value=$property($this->instance);
-			elseif(\is_array($property)){
-				$values=\array_map(function($v) use ($index){return $this->_getValue($v, $index);}, $property);
+			elseif (\is_array($property)) {
+				$values=\array_map(function($v) use ($index){return $this->_getValue($v, $index); }, $property);
 				$value=\implode("", $values);
-			}else{
-				if(isset($this->values[$index])){
-					$value= $this->values[$index]($property,$this->instance,$index);
-				}elseif(isset($this->instance->{$property})){
+			} else {
+				if (isset($this->values[$index])) {
+					$value=$this->values[$index]($property, $this->instance, $index);
+				}elseif (isset($this->instance->{$property})) {
 					$value=$this->instance->{$property};
 				}
 			}
 		}
-		if(isset($this->afterCompile[$index])){
-			if(\is_callable($this->afterCompile[$index])){
-				$this->afterCompile[$index]($value,$this->instance,$index);
+		if (isset($this->afterCompile[$index])) {
+			if (\is_callable($this->afterCompile[$index])) {
+				$this->afterCompile[$index]($value, $this->instance, $index);
 			}
 		}
 		return $value;
 	}
 
-	public function insertField($index,$field){
-		array_splice( $this->visibleProperties, $index, 0, $field );
+	public function insertField($index, $field) {
+		array_splice($this->visibleProperties, $index, 0, $field);
 		return $this;
 	}
 
-	public function insertInField($index,$field){
+	public function insertInField($index, $field) {
 		$vb=$this->visibleProperties;
-		if(isset($vb[$index])){
-			if(\is_array($vb[$index])){
+		if (isset($vb[$index])) {
+			if (\is_array($vb[$index])) {
 				$this->visibleProperties[$index][]=$field;
-			}else{
-				$this->visibleProperties[$index]=[$vb[$index],$field];
+			} else {
+				$this->visibleProperties[$index]=[$vb[$index], $field];
 			}
-		}else{
+		} else {
 			return $this->insertField($index, $field);
 		}
 		return $this;
 	}
 
-	public function addField($field){
+	public function addField($field) {
 		$this->visibleProperties[]=$field;
 		return $this;
 	}
 
-	public function count(){
+	public function count() {
 		return \sizeof($this->properties);
 	}
 
-	public function visiblePropertiesCount(){
+	public function visiblePropertiesCount() {
 		return \sizeof($this->visibleProperties);
 	}
 
-	public function getProperty($index){
+	public function getProperty($index) {
 		return $this->properties[$index];
 	}
 
-	public function getFieldName($index){
+	public function getFieldName($index) {
 		$property=$this->getProperty($index);
-		if($property instanceof \ReflectionProperty){
+		if ($property instanceof \ReflectionProperty) {
 			$result=$property->getName();
-		}elseif(\is_callable($property)){
+		}elseif (\is_callable($property)) {
 			$result=$this->visibleProperties[$index];
-		}else{
+		} else {
 			$result=$property;
 		}
 		return $result;
 	}
 
 
-	protected function showableProperty(\ReflectionProperty $rProperty){
-		return JString::startswith($rProperty->getName(),"_")===false;
+	protected function showableProperty(\ReflectionProperty $rProperty) {
+		return JString::startswith($rProperty->getName(), "_")===false;
 	}
 
 	public function setInstance($instance) {
-		if(\is_string($instance)){
+		if (\is_string($instance)) {
 			$instance=new $instance();
 		}
 		$this->instance=$instance;
 		$this->properties=[];
 		$this->reflect=new \ReflectionClass($instance);
-		if(\sizeof($this->visibleProperties)===0){
+		if (\sizeof($this->visibleProperties)===0) {
 			$this->properties=$this->getDefaultProperties();
-		}else{
-			foreach ($this->visibleProperties as $property){
+		} else {
+			foreach ($this->visibleProperties as $property) {
 				$this->setInstanceProperty($property);
 			}
 		}
 		return $this;
 	}
 
-	private function setInstanceProperty($property){
-		if(\is_callable($property)){
+	private function setInstanceProperty($property) {
+		if (\is_callable($property)) {
 			$this->properties[]=$property;
-		}elseif(\is_string($property)){
-			try{
+		}elseif (\is_string($property)) {
+			try {
 				$this->_beforeAddProperty(\sizeof($this->properties), $property);
 				$rProperty=$this->reflect->getProperty($property);
 				$this->properties[]=$rProperty;
-			}catch(\Exception $e){
+			}catch (\Exception $e) {
 				$this->_beforeAddProperty(\sizeof($this->properties), $property);
 				$this->properties[]=$property;
 			}
-		}elseif(\is_int($property)){
+		}elseif (\is_int($property)) {
 			$props=$this->getDefaultProperties();
-			if(isset($props[$property]))
+			if (isset($props[$property]))
 				$this->properties[]=$props[$property];
 				else
 					$this->properties[]=$property;
-		}else{
+		} else {
 			$this->properties[]=$property;
 		}
 	}
 
-	protected function getDefaultProperties(){
+	protected function getDefaultProperties() {
 		$result=[];
 		$properties=$this->reflect->getProperties();
-		foreach ($properties as $property){
+		foreach ($properties as $property) {
 			$showable=$this->showableProperty($property);
-			if($showable!==false){
+			if ($showable!==false) {
 				$result[]=$property;
 			}
 		}
@@ -235,12 +235,12 @@ 
 		return $this;
 	}
 
-	public function setValueFunction($index,$callback){
+	public function setValueFunction($index, $callback) {
 		$this->values[$index]=$callback;
 		return $this;
 	}
 
-	public function setIdentifierFunction($callback){
+	public function setIdentifierFunction($callback) {
 		$this->values["identifier"]=$callback;
 		return $this;
 	}
@@ -253,41 +253,41 @@ 
 		return $this->properties;
 	}
 
-	public function getCaption($index){
-		if(isset($this->captions[$index])){
+	public function getCaption($index) {
+		if (isset($this->captions[$index])) {
 			return $this->captions[$index];
 		}
-		if($this->properties[$index] instanceof \ReflectionProperty)
+		if ($this->properties[$index] instanceof \ReflectionProperty)
 			return $this->properties[$index]->getName();
-		elseif(\is_callable($this->properties[$index]))
+		elseif (\is_callable($this->properties[$index]))
 			return "";
 		else
 			return $this->properties[$index];
 	}
 
-	public function getCaptions(){
-		if(isset($this->captions)){
-			$captions= \array_values($this->captions);
-			for($i=\sizeof($captions);$i<$this->count();$i++){
+	public function getCaptions() {
+		if (isset($this->captions)) {
+			$captions=\array_values($this->captions);
+			for ($i=\sizeof($captions); $i<$this->count(); $i++) {
 				$captions[]="";
 			}
-		}else{
+		} else {
 			$captions=[];
 			$index=0;
 			$count=$this->count();
-			while($index<$count){
+			while ($index<$count) {
 				$captions[]=$this->getCaption($index++);
 			}
 		}
-		if(isset($this->captionCallback) && \is_callable($this->captionCallback)){
+		if (isset($this->captionCallback) && \is_callable($this->captionCallback)) {
 			$callback=$this->captionCallback;
-			$callback($captions,$this->instance);
+			$callback($captions, $this->instance);
 		}
 		return $captions;
 	}
 
-	public function setCaption($index,$caption){
-		if(isset($this->captions)===false)
+	public function setCaption($index, $caption) {
+		if (isset($this->captions)===false)
 			$this->captions=[];
 		$this->captions[$index]=$caption;
 		return $this;
@@ -305,7 +305,7 @@ 
 	 * @param callable $callback function called after the field compilation
 	 * @return \Ajax\semantic\widgets\datatable\InstanceViewer
 	 */
-	public function afterCompile($index,$callback){
+	public function afterCompile($index, $callback) {
 		$this->afterCompile[$index]=$callback;
 		return $this;
 	}

Braces +43 added lines, -37 removed lines

@@ -23,8 +23,9 @@ 
 		$this->widgetIdentifier=$identifier;
 		$this->values=[];
 		$this->afterCompile=[];
-		if(isset($instance))
-			$this->setInstance($instance);
+		if(isset($instance)) {
+					$this->setInstance($instance);
+		}
 		$this->setCaptions($captions);
 		$this->captionCallback=NULL;
 		$this->defaultValueFunction=function($name,$value){return $value;};
@@ -62,14 +63,16 @@ 
 	}
 
 	public function getIdentifier($index=NULL){
-		if(!isset($index))
-			$index=self::$index;
+		if(!isset($index)) {
+					$index=self::$index;
+		}
 		$value=$index;
 		if(isset($this->values["identifier"])){
-			if(\is_string($this->values["identifier"]))
-				$value=JReflection::callMethod($this->instance, $this->values["identifier"], []);
-			else
-				$value=$this->values["identifier"]($index,$this->instance);
+			if(\is_string($this->values["identifier"])) {
+							$value=JReflection::callMethod($this->instance, $this->values["identifier"], []);
+			} else {
+							$value=$this->values["identifier"]($index,$this->instance);
+			}
 		}
 		return $value;
 	}
@@ -93,7 +96,7 @@ 
 		$value=$property->getValue($this->instance);
 		if(isset($this->values[$index])){
 			$value= $this->values[$index]($value,$this->instance,$index);
-		}else{
+		} else{
 			$value=$this->_getDefaultValue($property->getName(),$value, $index);
 		}
 		return $value;
@@ -103,16 +106,16 @@ 
 		$value=null;
 		if($property instanceof \ReflectionProperty){
 			$value=$this->_getPropertyValue($property, $index);
-		}else{
-			if(\is_callable($property))
-				$value=$property($this->instance);
-			elseif(\is_array($property)){
+		} else{
+			if(\is_callable($property)) {
+							$value=$property($this->instance);
+			} elseif(\is_array($property)){
 				$values=\array_map(function($v) use ($index){return $this->_getValue($v, $index);}, $property);
 				$value=\implode("", $values);
-			}else{
+			} else{
 				if(isset($this->values[$index])){
 					$value= $this->values[$index]($property,$this->instance,$index);
-				}elseif(isset($this->instance->{$property})){
+				} elseif(isset($this->instance->{$property})){
 					$value=$this->instance->{$property};
 				}
 			}
@@ -135,10 +138,10 @@ 
 		if(isset($vb[$index])){
 			if(\is_array($vb[$index])){
 				$this->visibleProperties[$index][]=$field;
-			}else{
+			} else{
 				$this->visibleProperties[$index]=[$vb[$index],$field];
 			}
-		}else{
+		} else{
 			return $this->insertField($index, $field);
 		}
 		return $this;
@@ -165,9 +168,9 @@ 
 		$property=$this->getProperty($index);
 		if($property instanceof \ReflectionProperty){
 			$result=$property->getName();
-		}elseif(\is_callable($property)){
+		} elseif(\is_callable($property)){
 			$result=$this->visibleProperties[$index];
-		}else{
+		} else{
 			$result=$property;
 		}
 		return $result;
@@ -187,7 +190,7 @@ 
 		$this->reflect=new \ReflectionClass($instance);
 		if(\sizeof($this->visibleProperties)===0){
 			$this->properties=$this->getDefaultProperties();
-		}else{
+		} else{
 			foreach ($this->visibleProperties as $property){
 				$this->setInstanceProperty($property);
 			}
@@ -198,22 +201,23 @@ 
 	private function setInstanceProperty($property){
 		if(\is_callable($property)){
 			$this->properties[]=$property;
-		}elseif(\is_string($property)){
+		} elseif(\is_string($property)){
 			try{
 				$this->_beforeAddProperty(\sizeof($this->properties), $property);
 				$rProperty=$this->reflect->getProperty($property);
 				$this->properties[]=$rProperty;
-			}catch(\Exception $e){
+			} catch(\Exception $e){
 				$this->_beforeAddProperty(\sizeof($this->properties), $property);
 				$this->properties[]=$property;
 			}
-		}elseif(\is_int($property)){
+		} elseif(\is_int($property)){
 			$props=$this->getDefaultProperties();
-			if(isset($props[$property]))
-				$this->properties[]=$props[$property];
-				else
-					$this->properties[]=$property;
-		}else{
+			if(isset($props[$property])) {
+							$this->properties[]=$props[$property];
+			} else {
+									$this->properties[]=$property;
+				}
+		} else{
 			$this->properties[]=$property;
 		}
 	}
@@ -257,12 +261,13 @@ 
 		if(isset($this->captions[$index])){
 			return $this->captions[$index];
 		}
-		if($this->properties[$index] instanceof \ReflectionProperty)
-			return $this->properties[$index]->getName();
-		elseif(\is_callable($this->properties[$index]))
-			return "";
-		else
-			return $this->properties[$index];
+		if($this->properties[$index] instanceof \ReflectionProperty) {
+					return $this->properties[$index]->getName();
+		} elseif(\is_callable($this->properties[$index])) {
+					return "";
+		} else {
+					return $this->properties[$index];
+		}
 	}
 
 	public function getCaptions(){
@@ -271,7 +276,7 @@ 
 			for($i=\sizeof($captions);$i<$this->count();$i++){
 				$captions[]="";
 			}
-		}else{
+		} else{
 			$captions=[];
 			$index=0;
 			$count=$this->count();
@@ -287,8 +292,9 @@ 
 	}
 
 	public function setCaption($index,$caption){
-		if(isset($this->captions)===false)
-			$this->captions=[];
+		if(isset($this->captions)===false) {
+					$this->captions=[];
+		}
 		$this->captions[$index]=$caption;
 		return $this;
 	}

Ajax/semantic/widgets/datatable/HasCheckboxesTrait.php

Braces +9 added lines, -6 removed lines

@@ -50,8 +50,9 @@ 
 			if(isset($this->_checkedClass)){
 				$checkedMessageCall.="$(this).closest('tr').toggleClass('".$this->_checkedClass."',$(this).prop('checked'));";
 			}
-			if(isset($js))
-				$js->exec($this->_getCheckedMessageFunction(),true);
+			if(isset($js)) {
+							$js->exec($this->_getCheckedMessageFunction(),true);
+			}
 		}
 		return $checkedMessageCall;
 	}
@@ -59,8 +60,9 @@ 
 	protected function _generateMainCheckbox(&$captions){
 		$ck=new HtmlCheckbox("main-ck-".$this->identifier,"");
 		$checkedMessageCall="";
-		if($this->_hasCheckedMessage)
-			$checkedMessageCall="$('#{$this->identifier}').trigger('updateChecked');";
+		if($this->_hasCheckedMessage) {
+					$checkedMessageCall="$('#{$this->identifier}').trigger('updateChecked');";
+		}
 
 			$ck->setOnChecked($this->_setAllChecked("true").$checkedMessageCall);
 			$ck->setOnUnchecked($this->_setAllChecked("false").$checkedMessageCall);
@@ -108,8 +110,9 @@ 
 	 * @param callable $callback
 	 */
 	public function addCountCheckedInToolbar(array $checkedMessage=null,$callback=null){
-		if(isset($checkedMessage))
-			$this->_checkedMessage=$checkedMessage;
+		if(isset($checkedMessage)) {
+					$this->_checkedMessage=$checkedMessage;
+		}
 			$checkedMessage=$this->getCheckedMessage();
 			$this->_hasCheckboxes=true;
 			$this->_hasCheckedMessage=true;

Spacing +24 added lines, -24 removed lines

@@ -11,29 +11,29 @@ 
  * @property string identifier
  * @property array $_compileParts
  */
-trait HasCheckboxesTrait{
+trait HasCheckboxesTrait {
 	protected $_hasCheckboxes;
 	protected $_hasCheckedMessage=false;
 	protected $_checkedMessage;
 	protected $_checkedClass;
 
-	abstract public function addInToolbar($element,$callback=NULL);
+	abstract public function addInToolbar($element, $callback=NULL);
 
-	protected function _runCheckboxes(JsUtils $js){
+	protected function _runCheckboxes(JsUtils $js) {
 		$js->execOn("change", "#".$this->identifier." [name='selection[]']:not(._jsonArrayChecked)", $this->_getCheckedChange($js));
-		if(\sizeof($this->_compileParts)<3){
-			$js->trigger("#".$this->identifier." [name='selection[]']","change",true);
+		if (\sizeof($this->_compileParts)<3) {
+			$js->trigger("#".$this->identifier." [name='selection[]']", "change", true);
 		}
 	}
 
-	protected function _getCheckedChange(JsUtils $js=NULL){
+	protected function _getCheckedChange(JsUtils $js=NULL) {
 		$callback="var \$parentCheckbox=\$('#ck-main-ck-{$this->identifier}'),\$checkbox=\$('#{$this->identifier} [name=\"selection[]\"]'),allChecked=true,allUnchecked=true;
 				\$checkbox.each(function() {if($(this).prop('checked')){allUnchecked = false;}else{allChecked = false;}});
 				if(allChecked) {\$parentCheckbox.checkbox('set checked');}else if(allUnchecked){\$parentCheckbox.checkbox('set unchecked');}else{\$parentCheckbox.checkbox('set indeterminate');};".$this->_getCheckedMessageCall($js);
 		return $callback;
 	}
 
-	protected function _getCheckedMessageFunction(){
+	protected function _getCheckedMessageFunction() {
 		$msg=$this->getCheckedMessage();
 		$checkedMessageFunction="$('#{$this->identifier}').bind('updateChecked',function() {var msg='".$msg[0]."',count=\$('#{$this->identifier} [name=\"selection[]\"]:checked').length,all=\$('#{$this->identifier} [name=\"selection[]\"]').length;
 		if(count==1) msg='".$msg[1]."';
@@ -43,23 +43,23 @@ 
 		return $checkedMessageFunction;
 	}
 
-	protected function _getCheckedMessageCall(JsUtils $js=NULL){
+	protected function _getCheckedMessageCall(JsUtils $js=NULL) {
 		$checkedMessageCall="";
-		if($this->_hasCheckedMessage){
+		if ($this->_hasCheckedMessage) {
 			$checkedMessageCall="$('#{$this->identifier}').trigger('updateChecked');";
-			if(isset($this->_checkedClass)){
+			if (isset($this->_checkedClass)) {
 				$checkedMessageCall.="$(this).closest('tr').toggleClass('".$this->_checkedClass."',$(this).prop('checked'));";
 			}
-			if(isset($js))
-				$js->exec($this->_getCheckedMessageFunction(),true);
+			if (isset($js))
+				$js->exec($this->_getCheckedMessageFunction(), true);
 		}
 		return $checkedMessageCall;
 	}
 
-	protected function _generateMainCheckbox(&$captions){
-		$ck=new HtmlCheckbox("main-ck-".$this->identifier,"");
+	protected function _generateMainCheckbox(&$captions) {
+		$ck=new HtmlCheckbox("main-ck-".$this->identifier, "");
 		$checkedMessageCall="";
-		if($this->_hasCheckedMessage)
+		if ($this->_hasCheckedMessage)
 			$checkedMessageCall="$('#{$this->identifier}').trigger('updateChecked');";
 
 			$ck->setOnChecked($this->_setAllChecked("true").$checkedMessageCall);
@@ -67,9 +67,9 @@ 
 			\array_unshift($captions, $ck);
 	}
 
-	protected function _setAllChecked($checked){
+	protected function _setAllChecked($checked) {
 		$result="$('#".$this->identifier." [name=%quote%selection[]%quote%]:not(._jsonArrayChecked)').prop('checked',".$checked.");";
-		if(isset($this->_checkedClass)){
+		if (isset($this->_checkedClass)) {
 			$result.="$('#".$this->identifier." tr').toggleClass('".$this->_checkedClass."',".$checked.");";
 		}
 		return $result;
@@ -85,9 +85,9 @@ 
 	}
 
 	protected function getCheckedMessage() {
-		$result= $this->_checkedMessage;
-		if(!isset($result)){
-			$result=[0=>"none selected",1=>"one item selected","other"=>"{count} items selected"];
+		$result=$this->_checkedMessage;
+		if (!isset($result)) {
+			$result=[0=>"none selected", 1=>"one item selected", "other"=>"{count} items selected"];
 		}
 		return $result;
 	}
@@ -107,14 +107,14 @@ 
 	 * @param array $checkedMessage
 	 * @param callable $callback
 	 */
-	public function addCountCheckedInToolbar(array $checkedMessage=null,$callback=null){
-		if(isset($checkedMessage))
+	public function addCountCheckedInToolbar(array $checkedMessage=null, $callback=null) {
+		if (isset($checkedMessage))
 			$this->_checkedMessage=$checkedMessage;
 			$checkedMessage=$this->getCheckedMessage();
 			$this->_hasCheckboxes=true;
 			$this->_hasCheckedMessage=true;
-			$element=new HtmlLabel("checked-count-".$this->identifier,$checkedMessage[0]);
-			$this->addInToolbar($element,$callback);
+			$element=new HtmlLabel("checked-count-".$this->identifier, $checkedMessage[0]);
+			$this->addInToolbar($element, $callback);
 	}
 
 	public function setCheckedClass($_checkedClass) {

Ajax/semantic/widgets/datatable/Pagination.php

Spacing +19 added lines, -19 removed lines

@@ -9,7 +9,7 @@ 
 	private $pages_visibles;
 	private $row_count;
 
-	public function __construct($items_per_page=10,$pages_visibles=null,$page=1,$row_count=null){
+	public function __construct($items_per_page=10, $pages_visibles=null, $page=1, $row_count=null) {
 		$this->items_per_page=$items_per_page;
 		$this->row_count=$row_count;
 		$this->page=$page;
@@ -17,29 +17,29 @@ 
 		$this->visible=true;
 	}
 
-	public function getObjects($objects){
+	public function getObjects($objects) {
 		$auto=(!isset($this->row_count));
 		$os=$objects;
-		if(!\is_array($os)){
+		if (!\is_array($os)) {
 			$os=[];
-			foreach ($objects as $o){
+			foreach ($objects as $o) {
 				$os[]=$o;
 			}
 		}
-		$this->page_count = 0;
-		$row_count=($auto)?\sizeof($os):$this->row_count;
-		if (0 === $row_count) {
+		$this->page_count=0;
+		$row_count=($auto) ? \sizeof($os) : $this->row_count;
+		if (0===$row_count) {
 			$this->visible=false;
 		} else {
 			$this->visible=true;
-			$this->page_count = (int)ceil($row_count / $this->items_per_page);
-			if($this->page > $this->page_count+1) {
-				$this->page = 1;
+			$this->page_count=(int)ceil($row_count/$this->items_per_page);
+			if ($this->page>$this->page_count+1) {
+				$this->page=1;
 			}
 		}
-		if($auto){
-			$offset = ($this->page - 1) * $this->items_per_page;
-			return array_slice($os, $offset,$this->items_per_page);
+		if ($auto) {
+			$offset=($this->page-1)*$this->items_per_page;
+			return array_slice($os, $offset, $this->items_per_page);
 		}
 		return $os;
 	}
@@ -75,22 +75,22 @@ 
 		return $this->page_count;
 	}
 
-	public function getPagesNumbers(){
-		$middle= (int)ceil(($this->pages_visibles-1)/ 2);
+	public function getPagesNumbers() {
+		$middle=(int)ceil(($this->pages_visibles-1)/2);
 		$first=$this->page-$middle;
-		if($first<1){
+		if ($first<1) {
 			$first=1;
 		}
 		$last=$first+$this->pages_visibles-1;
-		if($last>$this->page_count){
+		if ($last>$this->page_count) {
 			$last=$this->page_count;
 		}
 		return \range($first, $last);
 	}
 
 	public function setPagesVisibles($pages_visibles) {
-		if(!isset($pages_visibles))
-			$pages_visibles=(int)ceil($this->row_count / $this->items_per_page)+1;
+		if (!isset($pages_visibles))
+			$pages_visibles=(int)ceil($this->row_count/$this->items_per_page)+1;
 		$this->pages_visibles=$pages_visibles;
 		return $this;
 	}

Braces +3 added lines, -2 removed lines

@@ -89,8 +89,9 @@
 	}
 
 	public function setPagesVisibles($pages_visibles) {
-		if(!isset($pages_visibles))
-			$pages_visibles=(int)ceil($this->row_count / $this->items_per_page)+1;
+		if(!isset($pages_visibles)) {
+					$pages_visibles=(int)ceil($this->row_count / $this->items_per_page)+1;
+		}
 		$this->pages_visibles=$pages_visibles;
 		return $this;
 	}

Ajax/semantic/widgets/datatable/JsonDataTable.php

Braces +3 added lines, -2 removed lines

@@ -48,8 +48,9 @@
 			$('#pagination-{$id} ._firstPage').attr('data-page',Math.max(1,page-1));
 			var lastPage=$('#pagination-{$id} ._lastPage');lastPage.attr('data-page',Math.min(lastPage.attr('data-max'),page+1));";
 		}
-		if(isset($this->_urls["refresh"]))
-			$this->jsonArrayOnClick($menu, $this->_urls["refresh"],"post","{'p':$(this).attr('data-page')}",$callback);
+		if(isset($this->_urls["refresh"])) {
+					$this->jsonArrayOnClick($menu, $this->_urls["refresh"],"post","{'p':$(this).attr('data-page')}",$callback);
+		}
 	}
 
 	/**

Spacing +22 added lines, -22 removed lines

@@ -16,17 +16,17 @@ 
 		parent::__construct($identifier, $model, $modelInstance);
 	}
 
-	protected function _generateContent($table){
+	protected function _generateContent($table) {
 		$this->_addRowModel($table);
 		$this->_rowClass="_json";
 		parent::_generateContent($table);
 	}
 
-	protected function _addRowModel($table){
+	protected function _addRowModel($table) {
 		$object=JReflection::jsonObject($this->_model);
-		$row=$this->_generateRow($object, $table,"_jsonArrayChecked");
+		$row=$this->_generateRow($object, $table, "_jsonArrayChecked");
 		$row->setClass($this->_modelClass);
-		$row->addToProperty("style","display:none;");
+		$row->addToProperty("style", "display:none;");
 		$table->getBody()->_addRow($row);
 	}
 
@@ -34,22 +34,22 @@ 
 	 * {@inheritDoc}
 	 * @see DataTable::_associatePaginationBehavior()
 	 */
-	protected function _associatePaginationBehavior(HtmlMenu $menu,JsUtils $js=NULL){
+	protected function _associatePaginationBehavior(HtmlMenu $menu, JsUtils $js=NULL) {
 		$callback=null;
-		if(isset($js)){
+		if (isset($js)) {
 			$id=$this->identifier;
 			$offset=$js->scriptCount();
 			$this->run($js);
 			$callback=$js->getScript($offset);
-			$callback.=$js->trigger("#".$id." [name='selection[]']","change",false)."$('#".$id." tbody .ui.checkbox').checkbox();".$js->execOn("change", "#".$id." [name='selection[]']", $this->_getCheckedChange($js));
+			$callback.=$js->trigger("#".$id." [name='selection[]']", "change", false)."$('#".$id." tbody .ui.checkbox').checkbox();".$js->execOn("change", "#".$id." [name='selection[]']", $this->_getCheckedChange($js));
 			$callback.=";var page=parseInt($(self).attr('data-page'));
 			$('#pagination-{$id} .item').removeClass('active');
 			$('#pagination-{$id} [data-page='+page+']:not(.no-active)').addClass('active');
 			$('#pagination-{$id} ._firstPage').attr('data-page',Math.max(1,page-1));
 			var lastPage=$('#pagination-{$id} ._lastPage');lastPage.attr('data-page',Math.min(lastPage.attr('data-max'),page+1));";
 		}
-		if(isset($this->_urls["refresh"]))
-			$this->jsonArrayOnClick($menu, $this->_urls["refresh"],"post","{'p':$(this).attr('data-page')}",$callback);
+		if (isset($this->_urls["refresh"]))
+			$this->jsonArrayOnClick($menu, $this->_urls["refresh"], "post", "{'p':$(this).attr('data-page')}", $callback);
 	}
 
 	/**
@@ -60,29 +60,29 @@ 
 	 * @param callable $jsCallback
 	 * @return AjaxCall
 	 */
-	public function jsJsonArray($url, $method="get", $params="{}", $jsCallback=NULL,$parameters=[]){
-		$parameters=\array_merge($parameters,["modelSelector"=>"#".$this->_identifier." tr.".$this->_modelClass,"url"=>$url,"method"=>$method,"params"=>$params,"jsCallback"=>$jsCallback]);
+	public function jsJsonArray($url, $method="get", $params="{}", $jsCallback=NULL, $parameters=[]) {
+		$parameters=\array_merge($parameters, ["modelSelector"=>"#".$this->_identifier." tr.".$this->_modelClass, "url"=>$url, "method"=>$method, "params"=>$params, "jsCallback"=>$jsCallback]);
 		return new AjaxCall("jsonArray", $parameters);
 	}
 
-	public function jsClear(){
+	public function jsClear() {
 		return "$('#{$this->identifier} tbody').find('._json').remove();";
 	}
 
-	public function clearOn(BaseHtml $element,$event, $stopPropagation=false, $preventDefault=false){
-		return $element->addEvent($event, $this->jsClear(),$stopPropagation,$preventDefault);
+	public function clearOn(BaseHtml $element, $event, $stopPropagation=false, $preventDefault=false) {
+		return $element->addEvent($event, $this->jsClear(), $stopPropagation, $preventDefault);
 	}
 
-	public function clearOnClick(BaseHtml $element,$stopPropagation=false, $preventDefault=false){
-		return $this->clearOn($element, "click",$stopPropagation,$preventDefault);
+	public function clearOnClick(BaseHtml $element, $stopPropagation=false, $preventDefault=false) {
+		return $this->clearOn($element, "click", $stopPropagation, $preventDefault);
 	}
 
-	public function jsonArrayOn(BaseHtml $element,$event,$url, $method="get", $params="{}", $jsCallback=NULL,$parameters=[]){
-		return $element->_addEvent($event, $this->jsJsonArray($url,$method,$params,$jsCallback,$parameters));
+	public function jsonArrayOn(BaseHtml $element, $event, $url, $method="get", $params="{}", $jsCallback=NULL, $parameters=[]) {
+		return $element->_addEvent($event, $this->jsJsonArray($url, $method, $params, $jsCallback, $parameters));
 	}
 
-	public function jsonArrayOnClick(BaseHtml $element,$url, $method="get", $params="{}", $jsCallback=NULL,$parameters=[]){
-		return $this->jsonArrayOn($element, "click", $url,$method,$params,$jsCallback,$parameters);
+	public function jsonArrayOnClick(BaseHtml $element, $url, $method="get", $params="{}", $jsCallback=NULL, $parameters=[]) {
+		return $this->jsonArrayOn($element, "click", $url, $method, $params, $jsCallback, $parameters);
 	}
 
 	/**
@@ -93,7 +93,7 @@ 
 	 * @param number $pages_visibles The number of visible pages in the Pagination component
 	 * @return DataTable
 	 */
-	public function paginate($page,$total_rowcount,$items_per_page=10,$pages_visibles=null){
-		return parent::paginate($page, $total_rowcount,$items_per_page,null);
+	public function paginate($page, $total_rowcount, $items_per_page=10, $pages_visibles=null) {
+		return parent::paginate($page, $total_rowcount, $items_per_page, null);
 	}
 }
\ No newline at end of file

Ajax/semantic/traits/SemanticWidgetsTrait.php

Spacing +14 added lines, -14 removed lines

@@ -17,8 +17,8 @@ 
 	 * @param array $instances
 	 * @return DataTable
 	 */
-	public function dataTable($identifier,$model, $instances){
-		return $this->addHtmlComponent(new DataTable($identifier,$model,$instances));
+	public function dataTable($identifier, $model, $instances) {
+		return $this->addHtmlComponent(new DataTable($identifier, $model, $instances));
 	}
 
 	/**
@@ -27,8 +27,8 @@ 
 	 * @param array $instances
 	 * @return JsonDataTable
 	 */
-	public function jsonDataTable($identifier,$model, $instances){
-		return $this->addHtmlComponent(new JsonDataTable($identifier,$model,$instances));
+	public function jsonDataTable($identifier, $model, $instances) {
+		return $this->addHtmlComponent(new JsonDataTable($identifier, $model, $instances));
 	}
 
 	/**
@@ -36,8 +36,8 @@ 
 	 * @param object $instance
 	 * @return DataElement
 	 */
-	public function dataElement($identifier, $instance){
-		return $this->addHtmlComponent(new DataElement($identifier,$instance));
+	public function dataElement($identifier, $instance) {
+		return $this->addHtmlComponent(new DataElement($identifier, $instance));
 	}
 
 	/**
@@ -45,19 +45,19 @@ 
 	 * @param object $instance
 	 * @return DataForm
 	 */
-	public function dataForm($identifier, $instance){
-		return $this->addHtmlComponent(new DataForm($identifier,$instance));
+	public function dataForm($identifier, $instance) {
+		return $this->addHtmlComponent(new DataForm($identifier, $instance));
 	}
 
-	public function defaultLogin($identifier,$instance=null){
-		return $this->addHtmlComponent(FormLogin::regular($identifier,$instance));
+	public function defaultLogin($identifier, $instance=null) {
+		return $this->addHtmlComponent(FormLogin::regular($identifier, $instance));
 	}
 
-	public function smallLogin($identifier,$instance=null){
-		return $this->addHtmlComponent(FormLogin::small($identifier,$instance));
+	public function smallLogin($identifier, $instance=null) {
+		return $this->addHtmlComponent(FormLogin::small($identifier, $instance));
 	}
 
-	public function segmentedLogin($identifier,$instance=null){
-		return $this->addHtmlComponent(FormLogin::attachedSegment($identifier,$instance));
+	public function segmentedLogin($identifier, $instance=null) {
+		return $this->addHtmlComponent(FormLogin::attachedSegment($identifier, $instance));
 	}
 }
\ No newline at end of file

Ajax/semantic/widgets/dataform/DataForm.php

Doc Comments +11 added lines

@@ -67,6 +67,9 @@ 
 		}
 	}
 
+	/**
+	 * @param HtmlForm $form
+	 */
 	protected function _generateFields($form,$values,$headers,$sepFirst,$sepLast,$wrappers){
 		$wrapper=null;
 		$fields=\array_slice($values, $sepFirst+1,$sepLast-$sepFirst);
@@ -132,12 +135,20 @@ 
 		$this->content[$this->_toolbarPosition]=$this->_toolbar;
 	}
 
+	/**
+	 * @param integer $index
+	 * @param string $title
+	 */
 	public function addDividerBefore($index,$title){
 		$index=$this->_getIndex($index);
 		$this->_instanceViewer->addHeaderDividerBefore($index, $title);
 		return $this;
 	}
 
+	/**
+	 * @param string $index
+	 * @param string $contentAfter
+	 */
 	public function addWrapper($index,$contentBefore,$contentAfter=null){
 		$index=$this->_getIndex($index);
 		$this->_instanceViewer->addWrapper($index, $contentBefore,$contentAfter);

Spacing +32 added lines, -32 removed lines

@@ -19,80 +19,80 @@ 
 class DataForm extends Widget {
 
 	public function __construct($identifier, $modelInstance=NULL) {
-		parent::__construct($identifier, null,$modelInstance);
+		parent::__construct($identifier, null, $modelInstance);
 		$this->_form=new HtmlForm($identifier);
 		$this->_init(new FormInstanceViewer($identifier), "form", $this->_form, true);
 	}
 
-	protected function _getFieldIdentifier($prefix,$name=""){
+	protected function _getFieldIdentifier($prefix, $name="") {
 		return $this->identifier."-{$name}-".$this->_instanceViewer->getIdentifier();
 	}
 
-	public function compile(JsUtils $js=NULL,&$view=NULL){
-		if(!$this->_generated){
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
+		if (!$this->_generated) {
 			$this->_instanceViewer->setInstance($this->_modelInstance);
 
 			$form=$this->content["form"];
 			$this->_generateContent($form);
 
-			if(isset($this->_toolbar)){
+			if (isset($this->_toolbar)) {
 				$this->_setToolbarPosition($form);
 			}
-			$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE,"form",PositionInTable::AFTERTABLE]);
+			$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE, "form", PositionInTable::AFTERTABLE]);
 			$this->_generated=true;
 		}
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 
 	/**
 	 * @param HtmlForm $form
 	 */
-	protected function _generateContent($form){
-		$values= $this->_instanceViewer->getValues();
+	protected function _generateContent($form) {
+		$values=$this->_instanceViewer->getValues();
 		$count=$this->_instanceViewer->count();
 		$separators=$this->_instanceViewer->getSeparators();
 		$headers=$this->_instanceViewer->getHeaders();
 		$wrappers=$this->_instanceViewer->getWrappers();
 		\sort($separators);
 		$size=\sizeof($separators);
-		if($size===1){
-			foreach ($values as $v){
+		if ($size===1) {
+			foreach ($values as $v) {
 				$form->addField($v);
 			}
-		}else{
+		} else {
 			$separators[]=$count;
-			for($i=0;$i<$size;$i++){
+			for ($i=0; $i<$size; $i++) {
 				$this->_generateFields($form, $values, $headers, $separators[$i], $separators[$i+1], $wrappers);
 			}
 		}
 	}
 
-	protected function _generateFields($form,$values,$headers,$sepFirst,$sepLast,$wrappers){
+	protected function _generateFields($form, $values, $headers, $sepFirst, $sepLast, $wrappers) {
 		$wrapper=null;
-		$fields=\array_slice($values, $sepFirst+1,$sepLast-$sepFirst);
-		if(isset($headers[$sepFirst+1]))
-			$form->addHeader($headers[$sepFirst+1],4,true);
-			if(isset($wrappers[$sepFirst+1])){
+		$fields=\array_slice($values, $sepFirst+1, $sepLast-$sepFirst);
+		if (isset($headers[$sepFirst+1]))
+			$form->addHeader($headers[$sepFirst+1], 4, true);
+			if (isset($wrappers[$sepFirst+1])) {
 				$wrapper=$wrappers[$sepFirst+1];
 			}
-			if(\sizeof($fields)===1){
+			if (\sizeof($fields)===1) {
 				$added=$form->addField($fields[0]);
-			}elseif(\sizeof($fields)>1){
+			}elseif (\sizeof($fields)>1) {
 				$added=$form->addFields($fields);
 			}
-			if(isset($wrapper))
-				$added->wrap($wrapper[0],$wrapper[1]);
+			if (isset($wrapper))
+				$added->wrap($wrapper[0], $wrapper[1]);
 	}
 
 	/**
 	 * {@inheritDoc}
 	 * @see \Ajax\common\Widget::getForm()
 	 */
-	public function getForm(){
+	public function getForm() {
 		return $this->content["form"];
 	}
 
-	public function addSeparatorAfter($fieldNum){
+	public function addSeparatorAfter($fieldNum) {
 		$fieldNum=$this->_getIndex($fieldNum);
 		$this->_instanceViewer->addSeparatorAfter($fieldNum);
 		return $this;
@@ -107,12 +107,12 @@ 
 		return $this;
 	}
 
-	public function fieldAsReset($index,$cssStyle=NULL,$attributes=NULL){
-		return $this->_fieldAs(function($id,$name,$value,$caption) use ($cssStyle){
-			$button=new HtmlButton($id,$value,$cssStyle);
+	public function fieldAsReset($index, $cssStyle=NULL, $attributes=NULL) {
+		return $this->_fieldAs(function($id, $name, $value, $caption) use ($cssStyle){
+			$button=new HtmlButton($id, $value, $cssStyle);
 			$button->setProperty("type", "reset");
 			return $button;
-		}, $index,$attributes);
+		}, $index, $attributes);
 	}
 
 	/**
@@ -131,19 +131,19 @@ 
 		$this->content[$this->_toolbarPosition]=$this->_toolbar;
 	}
 
-	public function addDividerBefore($index,$title){
+	public function addDividerBefore($index, $title) {
 		$index=$this->_getIndex($index);
 		$this->_instanceViewer->addHeaderDividerBefore($index, $title);
 		return $this;
 	}
 
-	public function addWrapper($index,$contentBefore,$contentAfter=null){
+	public function addWrapper($index, $contentBefore, $contentAfter=null) {
 		$index=$this->_getIndex($index);
-		$this->_instanceViewer->addWrapper($index, $contentBefore,$contentAfter);
+		$this->_instanceViewer->addWrapper($index, $contentBefore, $contentAfter);
 		return $this;
 	}
 
-	public function run(JsUtils $js){
+	public function run(JsUtils $js) {
 		return parent::run($js);
 	}
 }
\ No newline at end of file

Braces +8 added lines, -6 removed lines

@@ -59,7 +59,7 @@ 
 			foreach ($values as $v){
 				$form->addField($v);
 			}
-		}else{
+		} else{
 			$separators[]=$count;
 			for($i=0;$i<$size;$i++){
 				$this->_generateFields($form, $values, $headers, $separators[$i], $separators[$i+1], $wrappers);
@@ -70,18 +70,20 @@ 
 	protected function _generateFields($form,$values,$headers,$sepFirst,$sepLast,$wrappers){
 		$wrapper=null;
 		$fields=\array_slice($values, $sepFirst+1,$sepLast-$sepFirst);
-		if(isset($headers[$sepFirst+1]))
-			$form->addHeader($headers[$sepFirst+1],4,true);
+		if(isset($headers[$sepFirst+1])) {
+					$form->addHeader($headers[$sepFirst+1],4,true);
+		}
 			if(isset($wrappers[$sepFirst+1])){
 				$wrapper=$wrappers[$sepFirst+1];
 			}
 			if(\sizeof($fields)===1){
 				$added=$form->addField($fields[0]);
-			}elseif(\sizeof($fields)>1){
+			} elseif(\sizeof($fields)>1){
 				$added=$form->addFields($fields);
 			}
-			if(isset($wrapper))
-				$added->wrap($wrapper[0],$wrapper[1]);
+			if(isset($wrapper)) {
+							$added->wrap($wrapper[0],$wrapper[1]);
+			}
 	}
 
 	/**

Ajax/service/JReflection.php

Braces +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 					if($class->getProperty($property)!==null){
 						\call_user_func_array([$object,$name],[$value]);
 					}
-				}catch(\Exception $e){
+				} catch(\Exception $e){
 					//Nothing to do
 				}
 			}

Spacing +12 added lines, -12 removed lines

@@ -3,25 +3,25 @@ 
 namespace Ajax\service;
 
 class JReflection {
-	public static function shortClassName($object){
-		$classNameWithNamespace = get_class($object);
+	public static function shortClassName($object) {
+		$classNameWithNamespace=get_class($object);
 		return substr($classNameWithNamespace, strrpos($classNameWithNamespace, '\\')+1);
 	}
 
-	public static function jsonObject($classname){
+	public static function jsonObject($classname) {
 		$object=new $classname();
-		$class = new \ReflectionClass($classname);
+		$class=new \ReflectionClass($classname);
 		$methods=$class->getMethods(\ReflectionMethod::IS_PUBLIC);
-		foreach ($methods as $method){
+		foreach ($methods as $method) {
 			$name=$method->getName();
-			if(JString::startswith($name, "set")){
+			if (JString::startswith($name, "set")) {
 				$property=\lcfirst(JString::replaceAtFirst($name, "set", ""));
 				$value="__".$property."__";
-				try{
-					if($class->getProperty($property)!==null){
-						\call_user_func_array([$object,$name],[$value]);
+				try {
+					if ($class->getProperty($property)!==null) {
+						\call_user_func_array([$object, $name], [$value]);
 					}
-				}catch(\Exception $e){
+				}catch (\Exception $e) {
 					//Nothing to do
 				}
 			}
@@ -29,7 +29,7 @@ 
 		return $object;
 	}
 
-	public static function callMethod($object,$callback,array $values){
-		return \call_user_func_array([$object,$callback],$values);
+	public static function callMethod($object, $callback, array $values) {
+		return \call_user_func_array([$object, $callback], $values);
 	}
 }
\ No newline at end of file

Ajax/semantic/html/elements/html5/HtmlLink.php

Spacing +4 added lines, -4 removed lines

@@ -10,10 +10,10 @@ 
 class HtmlLink extends HtmlSemDoubleElement {
 	use HtmlLinkTrait;
 
-	public function __construct($identifier, $href="#", $content="Link",$target=NULL) {
+	public function __construct($identifier, $href="#", $content="Link", $target=NULL) {
 		parent::__construct($identifier, "a", "");
 		$this->setHref($href);
-		if(isset($target))
+		if (isset($target))
 			$this->setTarget($target);
 		$this->content=$content;
 	}
@@ -23,13 +23,13 @@ 
 	 * @see \Ajax\bootstrap\html\base\HtmlSingleElement::run()
 	 */
 	public function run(JsUtils $js) {
-		$this->_bsComponent=$js->semantic()->generic("#" . $this->identifier);
+		$this->_bsComponent=$js->semantic()->generic("#".$this->identifier);
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}
 
 	public function addIcon($icon, $before=true) {
-		return $this->addContent(new HtmlIcon("icon-" . $this->identifier, $icon), $before);
+		return $this->addContent(new HtmlIcon("icon-".$this->identifier, $icon), $before);
 	}
 
 	public static function icon($identifier, $icon, $href="#", $label=NULL) {

Braces +3 added lines, -2 removed lines

@@ -13,8 +13,9 @@
 	public function __construct($identifier, $href="#", $content="Link",$target=NULL) {
 		parent::__construct($identifier, "a", "");
 		$this->setHref($href);
-		if(isset($target))
-			$this->setTarget($target);
+		if(isset($target)) {
+					$this->setTarget($target);
+		}
 		$this->content=$content;
 	}
 

Ajax/semantic/html/elements/HtmlIcon.php

Spacing +9 added lines, -9 removed lines

@@ -68,7 +68,7 @@ 
 	 * @return \Ajax\semantic\html\HtmlIcon
 	 */
 	public function setFlipped($sens="horizontally") {
-		return $this->addToProperty("class", "flipped " . $sens);
+		return $this->addToProperty("class", "flipped ".$sens);
 	}
 
 	/**
@@ -77,7 +77,7 @@ 
 	 * @return \Ajax\semantic\html\HtmlIcon
 	 */
 	public function setRotated($sens="clockwise") {
-		return $this->addToProperty("class", "rotated " . $sens);
+		return $this->addToProperty("class", "rotated ".$sens);
 	}
 
 	/**
@@ -86,12 +86,12 @@ 
 	 * @param string target
 	 * @return HtmlIcon
 	 */
-	public function asLink($href=NULL,$target=NULL) {
+	public function asLink($href=NULL, $target=NULL) {
 		if (isset($href)) {
 			$_target="";
-			if(isset($target))
+			if (isset($target))
 				$_target="target='{$target}'";
-			$this->wrap("<a href='" . $href . "' {$_target}>", "</a>");
+			$this->wrap("<a href='".$href."' {$_target}>", "</a>");
 		}
 		return $this->addToProperty("class", "link");
 	}
@@ -107,9 +107,9 @@ 
 	 */
 	public function setBordered($inverted=false) {
 		$invertedStr="";
-		if ($inverted !== false)
+		if ($inverted!==false)
 			$invertedStr=" inverted";
-		return $this->addToProperty("class", "bordered" . $invertedStr);
+		return $this->addToProperty("class", "bordered".$invertedStr);
 	}
 
 	/**
@@ -121,11 +121,11 @@ 
 	}
 
 	public function addLabel($label, $before=false, $icon=NULL) {
-		if($before)
+		if ($before)
 			$this->wrap($label);
 		else
 			$this->wrap("", $label);
-		if(isset($icon))
+		if (isset($icon))
 			$this->addToIcon($icon);
 		return $this;
 	}

Braces +14 added lines, -10 removed lines

@@ -89,8 +89,9 @@ 
 	public function asLink($href=NULL,$target=NULL) {
 		if (isset($href)) {
 			$_target="";
-			if(isset($target))
-				$_target="target='{$target}'";
+			if(isset($target)) {
+							$_target="target='{$target}'";
+			}
 			$this->wrap("<a href='" . $href . "' {$_target}>", "</a>");
 		}
 		return $this->addToProperty("class", "link");
@@ -107,8 +108,9 @@ 
 	 */
 	public function setBordered($inverted=false) {
 		$invertedStr="";
-		if ($inverted !== false)
-			$invertedStr=" inverted";
+		if ($inverted !== false) {
+					$invertedStr=" inverted";
+		}
 		return $this->addToProperty("class", "bordered" . $invertedStr);
 	}
 
@@ -121,12 +123,14 @@ 
 	}
 
 	public function addLabel($label, $before=false, $icon=NULL) {
-		if($before)
-			$this->wrap($label);
-		else
-			$this->wrap("", $label);
-		if(isset($icon))
-			$this->addToIcon($icon);
+		if($before) {
+					$this->wrap($label);
+		} else {
+					$this->wrap("", $label);
+		}
+		if(isset($icon)) {
+					$this->addToIcon($icon);
+		}
 		return $this;
 	}