phpMv / phpMv-UI

Ajax/semantic/widgets/base/InstanceViewer.php

Braces +28 added lines, -23 removed lines

@@ -35,8 +35,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) {
@@ -86,14 +87,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;
 	}
@@ -124,9 +127,8 @@ 
 		} elseif (\is_callable($property) && \array_search($property, ['system','date']) === false){
 			try{
 				$value = $property($this->instance);
-			}catch(\Error $e){}
-		}
-		elseif (\is_array($property)) {
+			} catch(\Error $e){}
+		} elseif (\is_array($property)) {
 			$values = \array_map(function ($v) use ($index) {
 				return $this->_getValue($v, $index);
 			}, $property);
@@ -280,10 +282,11 @@ 
 			}
 		} elseif (\is_int($property)) {
 			$props = $this->getDefaultProperties();
-			if (isset($props[$property]))
-				$this->properties[] = $props[$property];
-			else
-				$this->properties[] = $property;
+			if (isset($props[$property])) {
+							$this->properties[] = $props[$property];
+			} else {
+							$this->properties[] = $property;
+			}
 		} else {
 			$this->properties[] = $property;
 		}
@@ -328,12 +331,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() {
@@ -366,8 +370,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;
 	}

Spacing +110 added lines, -110 removed lines

@@ -29,17 +29,17 @@ 
 
 	protected $groupByFields;
 
-	public static $index = 0;
+	public static $index=0;
 
-	public function __construct($identifier, $instance = NULL, $captions = NULL) {
-		$this->widgetIdentifier = $identifier;
-		$this->values = [];
-		$this->afterCompile = [];
+	public function __construct($identifier, $instance=NULL, $captions=NULL) {
+		$this->widgetIdentifier=$identifier;
+		$this->values=[];
+		$this->afterCompile=[];
 		if (isset($instance))
 			$this->setInstance($instance);
 		$this->setCaptions($captions);
-		$this->captionCallback = NULL;
-		$this->defaultValueFunction = function ($name, $value) {
+		$this->captionCallback=NULL;
+		$this->defaultValueFunction=function($name, $value) {
 			return $value;
 		};
 	}
@@ -66,47 +66,47 @@ 
 	}
 
 	public function getValues() {
-		$values = [];
-		$index = 0;
-		$count = $this->count();
-		$hasGroupby = is_array($this->groupByFields);
-		if (! $hasGroupby) {
-			while ($index < $count) {
-				$values[] = $this->getValue($index ++);
+		$values=[];
+		$index=0;
+		$count=$this->count();
+		$hasGroupby=is_array($this->groupByFields);
+		if (!$hasGroupby) {
+			while ($index<$count) {
+				$values[]=$this->getValue($index++);
 			}
 		} else {
-			while ($index < $count) {
-				if (array_search($index, $this->groupByFields) === false) {
-					$values[] = $this->getValue($index);
+			while ($index<$count) {
+				if (array_search($index, $this->groupByFields)===false) {
+					$values[]=$this->getValue($index);
 				}
-				$index ++;
+				$index++;
 			}
 		}
 		return $values;
 	}
 
-	public function getIdentifier($index = NULL) {
-		if (! isset($index))
-			$index = self::$index;
-		$value = $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"]))
-				$value = JReflection::callMethod($this->instance, $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) {
-		$property = $this->properties[$index];
+		$property=$this->properties[$index];
 		return $this->_getValue($property, $index);
 	}
 
 	protected function _beforeAddProperty($index, &$field) {}
 
 	protected function _getDefaultValue($name, $value, $index) {
-		$func = $this->defaultValueFunction;
+		$func=$this->defaultValueFunction;
 		return $func($name, $value, $index, $this->instance);
 	}
 
@@ -116,27 +116,27 @@ 
 	}
 
 	protected function _getValue($property, $index) {
-		$value = null;
-		$propertyName = $property;
+		$value=null;
+		$propertyName=$property;
 		if ($property instanceof \ReflectionProperty) {
-			$value = $this->_getPropertyValue($property);
-			$propertyName = $property->getName();
-		} elseif (\is_callable($property) && \array_search($property, ['system','date']) === false){
-			try{
-				$value = $property($this->instance);
-			}catch(\Error $e){}
+			$value=$this->_getPropertyValue($property);
+			$propertyName=$property->getName();
+		} elseif (\is_callable($property) && \array_search($property, ['system', 'date'])===false) {
+			try {
+				$value=$property($this->instance);
+			}catch (\Error $e) {}
 		}
 		elseif (\is_array($property)) {
-			$values = \array_map(function ($v) use ($index) {
+			$values=\array_map(function($v) use ($index) {
 				return $this->_getValue($v, $index);
 			}, $property);
-			$value = \implode('', $values);
+			$value=\implode('', $values);
 		} elseif (\is_string($property)) {
-			$value = '';
+			$value='';
 			if (isset($this->instance->{$property})) {
-				$value = $this->instance->{$property};
-			} elseif (\method_exists($this->instance, $getter = JReflection::getterName($property))) {
-				$value = JReflection::callMethod($this->instance, $getter, []);
+				$value=$this->instance->{$property};
+			} elseif (\method_exists($this->instance, $getter=JReflection::getterName($property))) {
+				$value=JReflection::callMethod($this->instance, $getter, []);
 			}
 		}
 
@@ -145,9 +145,9 @@ 
 
 	protected function _postGetValue($index, $propertyName, $value) {
 		if (isset($this->values[$index])) {
-			$value = $this->values[$index]($value, $this->instance, $index, self::$index);
+			$value=$this->values[$index]($value, $this->instance, $index, self::$index);
 		} else {
-			$value = $this->_getDefaultValue($propertyName, $value, $index);
+			$value=$this->_getDefaultValue($propertyName, $value, $index);
 		}
 		if (isset($this->afterCompile[$index])) {
 			if (\is_callable($this->afterCompile[$index])) {
@@ -157,7 +157,7 @@ 
 		return $value;
 	}
 
-	public function insertField($index, $field, $key = null) {
+	public function insertField($index, $field, $key=null) {
 		if (isset($key)) {
 			array_splice($this->visibleProperties, $index, 0, [
 				$key => $field
@@ -171,29 +171,29 @@ 
 	public function sortColumnContent($index, $array) {
 		if (isset($this->visibleProperties[$index])) {
 			if (is_array($this->visibleProperties[$index])) {
-				$this->visibleProperties[$index] = JArray::sortAssociative($this->visibleProperties[$index], $array);
+				$this->visibleProperties[$index]=JArray::sortAssociative($this->visibleProperties[$index], $array);
 			}
 		}
 		return $this;
 	}
 
-	public function insertInField($index, $field, $key = null) {
-		$vb = $this->visibleProperties;
+	public function insertInField($index, $field, $key=null) {
+		$vb=$this->visibleProperties;
 		if (isset($vb[$index])) {
 			if (isset($key)) {
 				if (\is_array($vb[$index])) {
-					$this->visibleProperties[$index][$key] = $field;
+					$this->visibleProperties[$index][$key]=$field;
 				} else {
-					$this->visibleProperties[$index] = [
+					$this->visibleProperties[$index]=[
 						$vb[$index],
 						$key => $field
 					];
 				}
 			} else {
 				if (\is_array($vb[$index])) {
-					$this->visibleProperties[$index][] = $field;
+					$this->visibleProperties[$index][]=$field;
 				} else {
-					$this->visibleProperties[$index] = [
+					$this->visibleProperties[$index]=[
 						$vb[$index],
 						$field
 					];
@@ -205,19 +205,19 @@ 
 		return $this;
 	}
 
-	public function addField($field, $key = null) {
+	public function addField($field, $key=null) {
 		if (isset($key)) {
-			$this->visibleProperties[] = [
+			$this->visibleProperties[]=[
 				$key => $field
 			];
 		} else {
-			$this->visibleProperties[] = $field;
+			$this->visibleProperties[]=$field;
 		}
 		return $this;
 	}
 
 	public function addFields($fields) {
-		$this->visibleProperties = \array_merge($this->visibleProperties, $fields);
+		$this->visibleProperties=\array_merge($this->visibleProperties, $fields);
 		return $this;
 	}
 
@@ -234,30 +234,30 @@ 
 	}
 
 	public function getFieldName($index) {
-		$property = $this->getProperty($index);
+		$property=$this->getProperty($index);
 		if ($property instanceof \ReflectionProperty) {
-			$result = $property->getName();
+			$result=$property->getName();
 		} elseif (\is_callable($property)) {
-			$result = $this->visibleProperties[$index];
+			$result=$this->visibleProperties[$index];
 		} else {
-			$result = $property;
+			$result=$property;
 		}
 		return $result;
 	}
 
 	protected function showableProperty(\ReflectionProperty $rProperty) {
-		return JString::startswith($rProperty->getName(), "_") === false;
+		return JString::startswith($rProperty->getName(), "_")===false;
 	}
 
 	public function setInstance($instance) {
 		if (\is_string($instance)) {
-			$instance = new $instance();
+			$instance=new $instance();
 		}
-		$this->instance = $instance;
-		$this->properties = [];
-		$this->reflect = new \ReflectionClass($instance);
-		if (JArray::count($this->visibleProperties) === 0) {
-			$this->properties = $this->getDefaultProperties();
+		$this->instance=$instance;
+		$this->properties=[];
+		$this->reflect=new \ReflectionClass($instance);
+		if (JArray::count($this->visibleProperties)===0) {
+			$this->properties=$this->getDefaultProperties();
 		} else {
 			foreach ($this->visibleProperties as $property) {
 				$this->setInstanceProperty($property);
@@ -268,56 +268,56 @@ 
 
 	private function setInstanceProperty($property) {
 		if (\is_callable($property)) {
-			$this->properties[] = $property;
+			$this->properties[]=$property;
 		} elseif (\is_string($property)) {
 			try {
 				$this->_beforeAddProperty(\sizeof($this->properties), $property);
-				$rProperty = $this->reflect->getProperty($property);
-				$this->properties[] = $rProperty;
-			} catch (\Exception $e) {
+				$rProperty=$this->reflect->getProperty($property);
+				$this->properties[]=$rProperty;
+			}catch (\Exception $e) {
 				$this->_beforeAddProperty(\sizeof($this->properties), $property);
-				$this->properties[] = $property;
+				$this->properties[]=$property;
 			}
 		} elseif (\is_int($property)) {
-			$props = $this->getDefaultProperties();
+			$props=$this->getDefaultProperties();
 			if (isset($props[$property]))
-				$this->properties[] = $props[$property];
+				$this->properties[]=$props[$property];
 			else
-				$this->properties[] = $property;
+				$this->properties[]=$property;
 		} else {
-			$this->properties[] = $property;
+			$this->properties[]=$property;
 		}
 	}
 
 	protected function getDefaultProperties() {
-		$result = [];
-		$properties = $this->reflect->getProperties();
+		$result=[];
+		$properties=$this->reflect->getProperties();
 		foreach ($properties as $property) {
-			$showable = $this->showableProperty($property);
-			if ($showable !== false) {
-				$result[] = $property;
+			$showable=$this->showableProperty($property);
+			if ($showable!==false) {
+				$result[]=$property;
 			}
 		}
 		return $result;
 	}
 
 	public function setVisibleProperties($visibleProperties) {
-		$this->visibleProperties = $visibleProperties;
+		$this->visibleProperties=$visibleProperties;
 		return $this;
 	}
 
 	public function setValueFunction($index, $callback) {
-		$this->values[$index] = $callback;
+		$this->values[$index]=$callback;
 		return $this;
 	}
 
 	public function setIdentifierFunction($callback) {
-		$this->values["identifier"] = $callback;
+		$this->values["identifier"]=$callback;
 		return $this;
 	}
 
 	public static function setIndex($index) {
-		self::$index = $index;
+		self::$index=$index;
 	}
 
 	public function getProperties() {
@@ -337,43 +337,43 @@ 
 	}
 
 	public function getCaptions() {
-		$hasGroupby = is_array($this->groupByFields);
-		$count = $this->count();
-		$moreAdded = false;
+		$hasGroupby=is_array($this->groupByFields);
+		$count=$this->count();
+		$moreAdded=false;
 		if (isset($this->captions)) {
-			$captions = \array_values($this->captions);
-			$gbSize = $hasGroupby ? sizeof($this->groupByFields) : 0;
-			$captionsSize = \sizeof($captions);
-			for ($i = $captionsSize; $i < $count - $gbSize; $i ++) {
-				$captions[] = "";
-				$moreAdded = true;
+			$captions=\array_values($this->captions);
+			$gbSize=$hasGroupby ? sizeof($this->groupByFields) : 0;
+			$captionsSize=\sizeof($captions);
+			for ($i=$captionsSize; $i<$count-$gbSize; $i++) {
+				$captions[]="";
+				$moreAdded=true;
 			}
 		} else {
-			$captions = [];
-			$index = 0;
-			while ($index < $count) {
-				$captions[] = $this->getCaption($index ++);
+			$captions=[];
+			$index=0;
+			while ($index<$count) {
+				$captions[]=$this->getCaption($index++);
 			}
 		}
-		if ($hasGroupby && sizeof($captions) >= $count && ! $moreAdded) {
-			$captions = JArray::removeByKeys($captions, $this->groupByFields);
+		if ($hasGroupby && sizeof($captions)>=$count && !$moreAdded) {
+			$captions=JArray::removeByKeys($captions, $this->groupByFields);
 		}
 		if (isset($this->captionCallback) && \is_callable($this->captionCallback)) {
-			$callback = $this->captionCallback;
+			$callback=$this->captionCallback;
 			$callback($captions, $this->instance);
 		}
 		return $captions;
 	}
 
 	public function setCaption($index, $caption) {
-		if (isset($this->captions) === false)
-			$this->captions = [];
-		$this->captions[$index] = $caption;
+		if (isset($this->captions)===false)
+			$this->captions=[];
+		$this->captions[$index]=$caption;
 		return $this;
 	}
 
 	public function setCaptions($captions) {
-		$this->captions = $captions;
+		$this->captions=$captions;
 		return $this;
 	}
 
@@ -388,7 +388,7 @@ 
 	 * @return InstanceViewer
 	 */
 	public function afterCompile($index, $callback) {
-		$this->afterCompile[$index] = $callback;
+		$this->afterCompile[$index]=$callback;
 		return $this;
 	}
 
@@ -400,7 +400,7 @@ 
 	 * @return \Ajax\semantic\widgets\base\InstanceViewer
 	 */
 	public function setCaptionCallback($captionCallback) {
-		$this->captionCallback = $captionCallback;
+		$this->captionCallback=$captionCallback;
 		return $this;
 	}
 
@@ -412,7 +412,7 @@ 
 	 * @return \Ajax\semantic\widgets\base\InstanceViewer
 	 */
 	public function setDefaultValueFunction($defaultValueFunction) {
-		$this->defaultValueFunction = $defaultValueFunction;
+		$this->defaultValueFunction=$defaultValueFunction;
 		return $this;
 	}
 
@@ -421,8 +421,8 @@ 
 	}
 
 	public function getSimpleProperties() {
-		return array_filter($this->visibleProperties, function ($item) {
-			return ! (is_array($item) || is_object($item));
+		return array_filter($this->visibleProperties, function($item) {
+			return !(is_array($item) || is_object($item));
 		});
 	}
 
@@ -447,10 +447,10 @@ 
 	 * @param mixed $groupByFields
 	 */
 	public function setGroupByFields($groupByFields) {
-		$this->groupByFields = $groupByFields;
+		$this->groupByFields=$groupByFields;
 	}
 	
-	public function addGroupBy($groupByField){
+	public function addGroupBy($groupByField) {
 		$this->groupByFields[]=$groupByField;
 	}
 

Ajax/semantic/html/modules/HtmlTab.php

Spacing +45 added lines, -45 removed lines

@@ -18,19 +18,19 @@ 
  */
 class HtmlTab extends HtmlSemCollection {
 
-	protected $params = [];
+	protected $params=[];
 
-	protected $_activated = false;
+	protected $_activated=false;
 
-	public function __construct($identifier, $tabs = array()) {
+	public function __construct($identifier, $tabs=array()) {
 		parent::__construct($identifier, "div", "");
-		$menu = new HtmlMenu("menu" . $this->identifier);
+		$menu=new HtmlMenu("menu".$this->identifier);
 		$menu->asTab(false)->setAttachment(NULL, Side::TOP);
-		$this->content["menu"] = $menu;
+		$this->content["menu"]=$menu;
 		$this->addItems($tabs);
 	}
 	
-	public function getMenu(){
+	public function getMenu() {
 		return $this->content['menu'];
 	}
 
@@ -41,18 +41,18 @@ 
 	 * @return HtmlSegment
 	 */
 	protected function createItem($value) {
-		$count = $this->count();
-		$title = $value;
-		$content = NULL;
+		$count=$this->count();
+		$title=$value;
+		$content=NULL;
 		if (\is_array($value)) {
-			$title = @$value[0];
-			$content = @$value[1];
+			$title=@$value[0];
+			$content=@$value[1];
 		}
-		$menuItem = $this->content["menu"]->addItem($title);
+		$menuItem=$this->content["menu"]->addItem($title);
 		$menuItem->addToProperty("data-tab", $menuItem->getIdentifier());
 		$menuItem->removeProperty("href");
-		$result = $this->createSegment($count, $content, $menuItem->getIdentifier());
-		$result->menuTab = $menuItem;
+		$result=$this->createSegment($count, $content, $menuItem->getIdentifier());
+		$result->menuTab=$menuItem;
 		return $result;
 	}
 
@@ -64,7 +64,7 @@ 
 	 * @return \Ajax\semantic\html\elements\HtmlSegment
 	 */
 	private function createSegment($count, $content, $datatab) {
-		$segment = new HtmlSegment("item-" . $this->identifier . "-" . $count, $content);
+		$segment=new HtmlSegment("item-".$this->identifier."-".$count, $content);
 		$segment->setAttachment(NULL, Side::BOTTOM)
 			->addToProperty("class", "tab")
 			->addToProperty("data-tab", $datatab);
@@ -81,10 +81,10 @@ 
 	 * @return \Ajax\semantic\html\modules\HtmlTab
 	 */
 	public function setTabContent($index, $content) {
-		$menu = $this->content["menu"];
-		if ($index < $menu->count()) {
-			if (isset($this->content[$index]) === false) {
-				$this->content[$index] = $this->createSegment($index, $content, $menu->getItem($index)
+		$menu=$this->content["menu"];
+		if ($index<$menu->count()) {
+			if (isset($this->content[$index])===false) {
+				$this->content[$index]=$this->createSegment($index, $content, $menu->getItem($index)
 					->getIdentifier());
 			} else
 				$this->content[$index]->setContent($content);
@@ -99,8 +99,8 @@ 
 	 * @return \Ajax\semantic\html\modules\HtmlTab
 	 */
 	public function setTabsContent($contents) {
-		$size = \sizeof($contents);
-		for ($i = 0; $i < $size; $i ++) {
+		$size=\sizeof($contents);
+		for ($i=0; $i<$size; $i++) {
 			$this->setTabContent($i, $contents[$i]);
 		}
 		return $this;
@@ -113,11 +113,11 @@ 
 	 * @return \Ajax\semantic\html\modules\HtmlTab
 	 */
 	public function activate($index) {
-		$item = $this->content["menu"]->getItem($index);
-		if ($item != null) {
+		$item=$this->content["menu"]->getItem($index);
+		if ($item!=null) {
 			$item->setActive(true);
 			$this->content[$index]->setActive(true);
-			$this->_activated = true;
+			$this->_activated=true;
 		}
 		return $this;
 	}
@@ -151,9 +151,9 @@ 
 	 * @param array $params
 	 * @return \Ajax\semantic\html\elements\HtmlSegment
 	 */
-	public function forwardTab($index, JsUtils $js, $title, $initialController, $controller, $action, $params = array()) {
+	public function forwardTab($index, JsUtils $js, $title, $initialController, $controller, $action, $params=array()) {
 		if (\array_key_exists($index, $this->content)) {
-			$this->content[$index] = $js->forward($initialController, $controller, $action, $params);
+			$this->content[$index]=$js->forward($initialController, $controller, $action, $params);
 			return $this->content[$index];
 		}
 
@@ -174,9 +174,9 @@ 
 	 *        	The parameters to pass to the view
 	 * @return \Ajax\semantic\html\elements\HtmlSegment
 	 */
-	public function renderViewTab($index, JsUtils $js, $title, $initialController, $viewName, $params = array()) {
+	public function renderViewTab($index, JsUtils $js, $title, $initialController, $viewName, $params=array()) {
 		if (\array_key_exists($index, $this->content)) {
-			$this->content[$index] = $js->renderContent($initialController, $viewName, $params);
+			$this->content[$index]=$js->renderContent($initialController, $viewName, $params);
 			return $this->content[$index];
 		}
 		return $this->addAndRenderViewTab($js, $title, $initialController, $viewName, $params);
@@ -196,10 +196,10 @@ 
 	 * @param array $params
 	 * @return \Ajax\semantic\html\elements\HtmlSegment
 	 */
-	public function addAndForwardTab(JsUtils $js, $title, $initialController, $controller, $action, $params = array()) {
+	public function addAndForwardTab(JsUtils $js, $title, $initialController, $controller, $action, $params=array()) {
 		\ob_start();
 		$js->forward($initialController, $controller, $action, $params);
-		$content = \ob_get_clean();
+		$content=\ob_get_clean();
 		return $this->addTab($title, $content);
 	}
 
@@ -215,11 +215,11 @@ 
 	 *        	The parameters to pass to the view
 	 * @return \Ajax\semantic\html\elements\HtmlSegment
 	 */
-	public function addAndRenderViewTab(JsUtils $js, $title, $initialController, $viewName, $params = array()) {
+	public function addAndRenderViewTab(JsUtils $js, $title, $initialController, $viewName, $params=array()) {
 		return $this->addTab($title, $js->renderContent($initialController, $viewName, $params));
 	}
 
-	public function setPointing($value = Direction::NONE) {
+	public function setPointing($value=Direction::NONE) {
 		return $this->content["menu"]->setPointing($value);
 	}
 
@@ -254,25 +254,25 @@ 
 	 * @return \Ajax\semantic\html\modules\HtmlTab
 	 */
 	public function setMenu($menu) {
-		$contentSize = \sizeof($this->content);
-		for ($i = 0; $i < $contentSize; $i ++) {
-			if ($menu->getItem($i) !== NULL) {
+		$contentSize=\sizeof($this->content);
+		for ($i=0; $i<$contentSize; $i++) {
+			if ($menu->getItem($i)!==NULL) {
 				if (isset($this->content[$i])) {
 					$menu->getItem($i)->addToProperty("data-tab", $this->content[$i]->getProperty("data-tab"));
 				}
 			}
 		}
-		$menuSize = $menu->count();
-		for ($i = 0; $i < $menuSize; $i ++) {
+		$menuSize=$menu->count();
+		for ($i=0; $i<$menuSize; $i++) {
 			$menu->getItem($i)->removeProperty("href");
-			if (isset($this->content[$i]) === false) {
-				$this->content[$i] = $this->createSegment($i, "New content", $menu->getItem($i)
+			if (isset($this->content[$i])===false) {
+				$this->content[$i]=$this->createSegment($i, "New content", $menu->getItem($i)
 					->getIdentifier());
 			}
 			$menu->getItem($i)->addToProperty("data-tab", $this->content[$i]->getProperty("data-tab"));
 		}
 
-		$this->content["menu"] = $menu;
+		$this->content["menu"]=$menu;
 		return $this;
 	}
 
@@ -281,19 +281,19 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if (isset($this->_bsComponent) === false)
-			$this->_bsComponent = $js->semantic()->tab("#" . $this->identifier . " .item", $this->params);
+		if (isset($this->_bsComponent)===false)
+			$this->_bsComponent=$js->semantic()->tab("#".$this->identifier." .item", $this->params);
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}
 
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
-		if (! $this->_activated && $this->content["menu"]->count() > 0 && \sizeof($this->content) > 1)
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
+		if (!$this->_activated && $this->content["menu"]->count()>0 && \sizeof($this->content)>1)
 			$this->activate(0);
 		return parent::compile($js, $view);
 	}
 	
-	public function setInverted($recursive=true){
+	public function setInverted($recursive=true) {
 		parent::setInverted($recursive);
 		$this->content['menu']->addClass('inverted');
 	}

Braces +9 added lines, -6 removed lines

@@ -86,8 +86,9 @@ 
 			if (isset($this->content[$index]) === false) {
 				$this->content[$index] = $this->createSegment($index, $content, $menu->getItem($index)
 					->getIdentifier());
-			} else
-				$this->content[$index]->setContent($content);
+			} else {
+							$this->content[$index]->setContent($content);
+			}
 		}
 		return $this;
 	}
@@ -281,15 +282,17 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if (isset($this->_bsComponent) === false)
-			$this->_bsComponent = $js->semantic()->tab("#" . $this->identifier . " .item", $this->params);
+		if (isset($this->_bsComponent) === false) {
+					$this->_bsComponent = $js->semantic()->tab("#" . $this->identifier . " .item", $this->params);
+		}
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}
 
 	public function compile(JsUtils $js = NULL, &$view = NULL) {
-		if (! $this->_activated && $this->content["menu"]->count() > 0 && \sizeof($this->content) > 1)
-			$this->activate(0);
+		if (! $this->_activated && $this->content["menu"]->count() > 0 && \sizeof($this->content) > 1) {
+					$this->activate(0);
+		}
 		return parent::compile($js, $view);
 	}
 	

Ajax/common/Widget.php

Spacing +75 added lines, -75 removed lines

@@ -20,7 +20,7 @@ 
 use Ajax\common\html\traits\BaseHooksTrait;
 
 abstract class Widget extends HtmlDoubleElement {
-	use FieldAsTrait,FormTrait,BaseHooksTrait;
+	use FieldAsTrait, FormTrait, BaseHooksTrait;
 
 	/**
 	 *
@@ -66,24 +66,24 @@ 
 
 	protected $_hasRules;
 
-	public function __construct($identifier, $model, $modelInstance = NULL) {
+	public function __construct($identifier, $model, $modelInstance=NULL) {
 		parent::__construct($identifier);
-		$this->_template = "%wrapContentBefore%%content%%wrapContentAfter%";
+		$this->_template="%wrapContentBefore%%content%%wrapContentAfter%";
 		$this->setModel($model);
 		if (isset($modelInstance)) {
 			$this->show($modelInstance);
 		}
-		$this->_generated = false;
+		$this->_generated=false;
 	}
 
 	protected function _init($instanceViewer, $contentKey, $content, $edition) {
-		$this->_instanceViewer = $instanceViewer;
-		$this->content = [
+		$this->_instanceViewer=$instanceViewer;
+		$this->content=[
 			$contentKey => $content
 		];
-		$this->_self = $content;
-		$this->_toolbarPosition = PositionInTable::BEFORETABLE;
-		$this->_edition = $edition;
+		$this->_self=$content;
+		$this->_toolbarPosition=PositionInTable::BEFORETABLE;
+		$this->_edition=$edition;
 	}
 
 	/**
@@ -92,16 +92,16 @@ 
 	 * @return int|string|boolean
 	 */
 	protected function _getIndex($fieldName) {
-		$index = $fieldName;
+		$index=$fieldName;
 		if (\is_string($fieldName)) {
-			$fields = $this->_instanceViewer->getVisibleProperties();
-			$index = \array_search($fieldName, $fields);
+			$fields=$this->_instanceViewer->getVisibleProperties();
+			$index=\array_search($fieldName, $fields);
 		}
 		return $index;
 	}
 
-	protected function _getFieldIdentifier($prefix, $name = "") {
-		return $this->identifier . "-{$prefix}-" . $this->_instanceViewer->getIdentifier();
+	protected function _getFieldIdentifier($prefix, $name="") {
+		return $this->identifier."-{$prefix}-".$this->_instanceViewer->getIdentifier();
 	}
 
 	protected function _getFieldName($index) {
@@ -112,13 +112,13 @@ 
 		return $this->_instanceViewer->getCaption($index);
 	}
 
-	abstract protected function _setToolbarPosition($table, $captions = NULL);
+	abstract protected function _setToolbarPosition($table, $captions=NULL);
 
 	public function show($modelInstance) {
 		if (\is_array($modelInstance)) {
-			$modelInstance = \json_decode(\json_encode($modelInstance), FALSE);
+			$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
 		}
-		$this->_modelInstance = $modelInstance;
+		$this->_modelInstance=$modelInstance;
 	}
 
 	public function getModel() {
@@ -126,7 +126,7 @@ 
 	}
 
 	public function setModel($_model) {
-		$this->_model = $_model;
+		$this->_model=$_model;
 		return $this;
 	}
 
@@ -135,13 +135,13 @@ 
 	}
 
 	public function setInstanceViewer($_instanceViewer) {
-		$this->_instanceViewer = $_instanceViewer;
+		$this->_instanceViewer=$_instanceViewer;
 		return $this;
 	}
 
 	abstract public function getHtmlComponent();
 
-	public function setAttached($value = true) {
+	public function setAttached($value=true) {
 		return $this->getHtmlComponent()->setAttached($value);
 	}
 
@@ -156,8 +156,8 @@ 
 	 * @return Widget
 	 */
 	public function afterCompile($index, $callback) {
-		$index = $this->_getIndex($index);
-		if ($index !== false) {
+		$index=$this->_getIndex($index);
+		if ($index!==false) {
 			$this->_instanceViewer->afterCompile($index, $callback);
 		}
 		return $this;
@@ -182,7 +182,7 @@ 
 		return $this;
 	}
 
-	public function addField($field, $key = null) {
+	public function addField($field, $key=null) {
 		$this->_instanceViewer->addField($field, $key);
 		return $this;
 	}
@@ -196,10 +196,10 @@ 
 		return $this->_instanceViewer->visiblePropertiesCount();
 	}
 
-	public function addMessage($attributes = NULL, $fieldName = "message") {
+	public function addMessage($attributes=NULL, $fieldName="message") {
 		$this->_instanceViewer->addField($fieldName);
-		$count = $this->_instanceViewer->visiblePropertiesCount();
-		return $this->fieldAsMessage($count - 1, $attributes);
+		$count=$this->_instanceViewer->visiblePropertiesCount();
+		return $this->fieldAsMessage($count-1, $attributes);
 	}
 
 	public function addErrorMessage() {
@@ -208,15 +208,15 @@ 
 		], "message");
 	}
 
-	public function insertField($index, $field, $key = null) {
-		$index = $this->_getIndex($index);
+	public function insertField($index, $field, $key=null) {
+		$index=$this->_getIndex($index);
 		$this->_instanceViewer->insertField($index, $field, $key);
 		return $this;
 	}
 
-	public function insertInField($index, $field, $key = null) {
-		$index = $this->_getIndex($index);
-		if ($index !== false) {
+	public function insertInField($index, $field, $key=null) {
+		$index=$this->_getIndex($index);
+		if ($index!==false) {
 			$this->_instanceViewer->insertInField($index, $field, $key);
 		}
 		return $this;
@@ -232,8 +232,8 @@ 
 	 * @return Widget
 	 */
 	public function setValueFunction($index, $callback) {
-		$index = $this->_getIndex($index);
-		if ($index !== false) {
+		$index=$this->_getIndex($index);
+		if ($index!==false) {
 			$this->_instanceViewer->setValueFunction($index, $callback);
 		}
 		return $this;
@@ -249,8 +249,8 @@ 
 	 * @return \Ajax\semantic\html\collections\menus\HtmlMenu
 	 */
 	public function getToolbar() {
-		if (isset($this->_toolbar) === false) {
-			$this->_toolbar = new HtmlMenu("toolbar-" . $this->identifier);
+		if (isset($this->_toolbar)===false) {
+			$this->_toolbar=new HtmlMenu("toolbar-".$this->identifier);
 		}
 		return $this->_toolbar;
 	}
@@ -263,11 +263,11 @@ 
 	 *        	function to call on $element
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addInToolbar($element, $callback = NULL) {
-		$tb = $this->getToolbar();
+	public function addInToolbar($element, $callback=NULL) {
+		$tb=$this->getToolbar();
 		if ($element instanceof BaseWidget) {
-			if ($element->getIdentifier() === "") {
-				$element->setIdentifier("tb-item-" . $this->identifier . "-" . $tb->count());
+			if ($element->getIdentifier()==="") {
+				$element->setIdentifier("tb-item-".$this->identifier."-".$tb->count());
 			}
 		}
 		if (isset($callback)) {
@@ -286,8 +286,8 @@ 
 	 *        	function($element)
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addItemInToolbar($caption, $icon = NULL, $callback = NULL) {
-		$result = $this->addInToolbar($caption, $callback);
+	public function addItemInToolbar($caption, $icon=NULL, $callback=NULL) {
+		$result=$this->addInToolbar($caption, $callback);
 		if (isset($icon) && method_exists($result, "addIcon"))
 			$result->addIcon($icon);
 		return $result;
@@ -300,7 +300,7 @@ 
 	 *        	function($element)
 	 * @return \Ajax\common\Widget
 	 */
-	public function addItemsInToolbar(array $items, $callback = NULL) {
+	public function addItemsInToolbar(array $items, $callback=NULL) {
 		if (JArray::isAssociative($items)) {
 			foreach ($items as $icon => $item) {
 				$this->addItemInToolbar($item, $icon, $callback);
@@ -321,10 +321,10 @@ 
 	 *        	function($element)
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addDropdownInToolbar($value, $items, $callback = NULL) {
-		$dd = $value;
+	public function addDropdownInToolbar($value, $items, $callback=NULL) {
+		$dd=$value;
 		if (\is_string($value)) {
-			$dd = new HtmlDropdown("dropdown-" . $this->identifier . "-" . $value, $value, $items);
+			$dd=new HtmlDropdown("dropdown-".$this->identifier."-".$value, $value, $items);
 		}
 		return $this->addInToolbar($dd, $callback);
 	}
@@ -337,8 +337,8 @@ 
 	 *        	function($element)
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addButtonInToolbar($caption, $cssStyle = null, $callback = NULL) {
-		$bt = new HtmlButton("bt-" . $caption, $caption, $cssStyle);
+	public function addButtonInToolbar($caption, $cssStyle=null, $callback=NULL) {
+		$bt=new HtmlButton("bt-".$caption, $caption, $cssStyle);
 		return $this->addInToolbar($bt, $callback);
 	}
 
@@ -350,8 +350,8 @@ 
 	 *        	function($element)
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addButtonsInToolbar(array $captions, $asIcon = false, $callback = NULL) {
-		$bts = new HtmlButtonGroups("", $captions, $asIcon);
+	public function addButtonsInToolbar(array $captions, $asIcon=false, $callback=NULL) {
+		$bts=new HtmlButtonGroups("", $captions, $asIcon);
 		return $this->addInToolbar($bts, $callback);
 	}
 
@@ -363,14 +363,14 @@ 
 	 * @param boolean $labeled
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addLabelledIconButtonInToolbar($caption, $icon, $before = true, $labeled = false) {
-		$bt = new HtmlButton("", $caption);
+	public function addLabelledIconButtonInToolbar($caption, $icon, $before=true, $labeled=false) {
+		$bt=new HtmlButton("", $caption);
 		$bt->addIcon($icon, $before, $labeled);
 		return $this->addInToolbar($bt);
 	}
 
-	public function addSubmitInToolbar($identifier, $value, $cssStyle = NULL, $url = NULL, $responseElement = NULL, $parameters = NULL) {
-		$button = new HtmlButton($identifier, $value, $cssStyle);
+	public function addSubmitInToolbar($identifier, $value, $cssStyle=NULL, $url=NULL, $responseElement=NULL, $parameters=NULL) {
+		$button=new HtmlButton($identifier, $value, $cssStyle);
 		$this->_buttonAsSubmit($button, "click", $url, $responseElement, $parameters);
 		return $this->addInToolbar($button);
 	}
@@ -393,8 +393,8 @@ 
 	 * @param boolean $_edition
 	 * @return \Ajax\common\Widget
 	 */
-	public function setEdition($_edition = true) {
-		$this->_edition = $_edition;
+	public function setEdition($_edition=true) {
+		$this->_edition=$_edition;
 		return $this;
 	}
 
@@ -423,8 +423,8 @@ 
 	 * @param string|boolean $disable
 	 * @return string
 	 */
-	public function jsDisabled($disable = true) {
-		return "$('#" . $this->identifier . " .ui.input,#" . $this->identifier . " .ui.dropdown,#" . $this->identifier . " .ui.checkbox').toggleClass('disabled'," . $disable . ");";
+	public function jsDisabled($disable=true) {
+		return "$('#".$this->identifier." .ui.input,#".$this->identifier." .ui.dropdown,#".$this->identifier." .ui.checkbox').toggleClass('disabled',".$disable.");";
 	}
 
 	/**
@@ -434,8 +434,8 @@ 
 	 *        	function($element)
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addEditButtonInToolbar($caption, $callback = NULL) {
-		$bt = new HtmlButton($this->identifier . "-editBtn", $caption);
+	public function addEditButtonInToolbar($caption, $callback=NULL) {
+		$bt=new HtmlButton($this->identifier."-editBtn", $caption);
 		$bt->setToggle();
 		$bt->setActive($this->_edition);
 		$bt->onClick($this->jsDisabled(Javascript::prep_value("!$(event.target).hasClass('active')")));
@@ -443,18 +443,18 @@ 
 	}
 
 	public function setToolbar(HtmlMenu $_toolbar) {
-		$this->_toolbar = $_toolbar;
+		$this->_toolbar=$_toolbar;
 		return $this;
 	}
 
 	public function setToolbarPosition($_toolbarPosition) {
-		$this->_toolbarPosition = $_toolbarPosition;
+		$this->_toolbarPosition=$_toolbarPosition;
 		return $this;
 	}
 
 	public function getForm() {
-		if (! isset($this->_form)) {
-			$this->_form = new HtmlForm("frm-" . $this->identifier);
+		if (!isset($this->_form)) {
+			$this->_form=new HtmlForm("frm-".$this->identifier);
 			$this->setEdition(true);
 		}
 		return $this->_form;
@@ -468,7 +468,7 @@ 
 	}
 
 	protected function runForm(JsUtils $js) {
-		$fields = $this->getContentInstances(HtmlFormField::class);
+		$fields=$this->getContentInstances(HtmlFormField::class);
 		foreach ($fields as $field) {
 			$this->_form->addField($field);
 		}
@@ -477,10 +477,10 @@ 
 
 	protected function _compileForm() {
 		if (isset($this->_form)) {
-			$noValidate = "";
-			if (\sizeof($this->_form->getValidationParams()) > 0)
-				$noValidate = "novalidate";
-				$this->wrapContent("<form class='".$this->_form->getProperty('class')."' id='frm-" . $this->identifier . "' name='frm-" . $this->identifier . "' " . $noValidate . ">", "</form>");
+			$noValidate="";
+			if (\sizeof($this->_form->getValidationParams())>0)
+				$noValidate="novalidate";
+				$this->wrapContent("<form class='".$this->_form->getProperty('class')."' id='frm-".$this->identifier."' name='frm-".$this->identifier."' ".$noValidate.">", "</form>");
 		}
 	}
 
@@ -502,21 +502,21 @@ 
 	}
 
 	public function swapFields($index1, $index2) {
-		$index1 = $this->_getIndex($index1);
-		$index2 = $this->_getIndex($index2);
+		$index1=$this->_getIndex($index1);
+		$index2=$this->_getIndex($index2);
 		return $this->_instanceViewer->swapFields($index1, $index2);
 	}
 
 	public function removeField($index) {
-		$index = $this->_getIndex($index);
-		if ($index !== false) {
+		$index=$this->_getIndex($index);
+		if ($index!==false) {
 			$this->_instanceViewer->removeField($index);
 		}
 		return $this;
 	}
 
-	public function asModal($header = null) {
-		$modal = new HtmlModal("modal-" . $this->identifier, $header);
+	public function asModal($header=null) {
+		$modal=new HtmlModal("modal-".$this->identifier, $header);
 		$modal->setContent($this);
 		if (isset($this->_form)) {
 			$this->_form->onSuccess($modal->jsHide());
@@ -524,7 +524,7 @@ 
 		return $modal;
 	}
 
-	public function addToProperty($name, $value, $separator = " ") {
+	public function addToProperty($name, $value, $separator=" ") {
 		return $this->getHtmlComponent()->addToProperty($name, $value, $separator);
 	}
 

Braces +6 added lines, -4 removed lines

@@ -288,8 +288,9 @@ 
 	 */
 	public function addItemInToolbar($caption, $icon = NULL, $callback = NULL) {
 		$result = $this->addInToolbar($caption, $callback);
-		if (isset($icon) && method_exists($result, "addIcon"))
-			$result->addIcon($icon);
+		if (isset($icon) && method_exists($result, "addIcon")) {
+					$result->addIcon($icon);
+		}
 		return $result;
 	}
 
@@ -478,8 +479,9 @@ 
 	protected function _compileForm() {
 		if (isset($this->_form)) {
 			$noValidate = "";
-			if (\sizeof($this->_form->getValidationParams()) > 0)
-				$noValidate = "novalidate";
+			if (\sizeof($this->_form->getValidationParams()) > 0) {
+							$noValidate = "novalidate";
+			}
 				$this->wrapContent("<form class='".$this->_form->getProperty('class')."' id='frm-" . $this->identifier . "' name='frm-" . $this->identifier . "' " . $noValidate . ">", "</form>");
 		}
 	}

Ajax/semantic/components/validation/AjaxRule.php

Spacing +4 added lines, -4 removed lines

@@ -16,9 +16,9 @@ 
 
 	private $ajaxCall;
 
-	public function __construct($type, $url, $params, $jsCallback = null, $method = 'post', $parameters = [], $prompt = NULL, $value = NULL) {
+	public function __construct($type, $url, $params, $jsCallback=null, $method='post', $parameters=[], $prompt=NULL, $value=NULL) {
 		parent::__construct($type, $prompt, $value);
-		$parameters = \array_merge([
+		$parameters=\array_merge([
 			'async' => false,
 			'url' => $url,
 			'params' => $params,
@@ -29,10 +29,10 @@ 
 			'preventDefault' => false,
 			'responseElement' => null
 		], $parameters);
-		$this->ajaxCall = new AjaxCall($method, $parameters);
+		$this->ajaxCall=new AjaxCall($method, $parameters);
 	}
 
 	public function compile(JsUtils $js) {
-		$js->exec(Rule::custom($this->getType(), "function(value,ruleValue){var result=true;" . $this->ajaxCall->compile($js) . "return result;}"), true);
+		$js->exec(Rule::custom($this->getType(), "function(value,ruleValue){var result=true;".$this->ajaxCall->compile($js)."return result;}"), true);
 	}
 }
\ No newline at end of file

Ajax/semantic/components/validation/CustomRule.php

Spacing +2 added lines, -2 removed lines

@@ -15,9 +15,9 @@
 
 	protected $jsFunction;
 
-	public function __construct($type, $jsFunction, $prompt = NULL, $value = NULL) {
+	public function __construct($type, $jsFunction, $prompt=NULL, $value=NULL) {
 		parent::__construct($type, $prompt, $value);
-		$this->jsFunction = $jsFunction;
+		$this->jsFunction=$jsFunction;
 	}
 
 	public function compile(JsUtils $js) {

Ajax/semantic/components/validation/FieldValidation.php

Spacing +16 added lines, -16 removed lines

@@ -8,7 +8,7 @@ 
  * @version 1.001
  * Generates a JSON field validator
  */
-class FieldValidation implements \JsonSerializable{
+class FieldValidation implements \JsonSerializable {
 	/**
 	 * @var string
 	 */
@@ -31,7 +31,7 @@ 
 
 	protected $optional;
 
-	public function __construct($identifier){
+	public function __construct($identifier) {
 		$this->identifier=$identifier;
 		$this->rules=[];
 	}
@@ -55,31 +55,31 @@ 
 	 * @param string $value
 	 * @return Rule
 	 */
-	public function addRule($type,$prompt=NULL,$value=NULL){
-		if($type instanceof  Rule) {
-			$rule = $type;
-			if($type instanceof CustomRule){
+	public function addRule($type, $prompt=NULL, $value=NULL) {
+		if ($type instanceof  Rule) {
+			$rule=$type;
+			if ($type instanceof CustomRule) {
 				$this->customRules[]=$type;
 				$this->hasCustomRules=true;
 			}
-		}elseif(\is_array($type)){
+		}elseif (\is_array($type)) {
 			$value=JArray::getValue($type, "value", 2);
 			$prompt=JArray::getValue($type, "prompt", 1);
 			$type=JArray::getValue($type, "type", 0);
-			$rule=new Rule($type,$prompt,$value);
-		}else {
-			$rule = new Rule($type, $prompt, $value);
+			$rule=new Rule($type, $prompt, $value);
+		} else {
+			$rule=new Rule($type, $prompt, $value);
 		}
 		$this->rules[]=$rule;
 		return $rule;
 	}
 
-	public function jsonSerialize(){
-		$result=["identifier"=>$this->identifier,"rules"=>$this->rules];
-		if($this->optional){
+	public function jsonSerialize() {
+		$result=["identifier"=>$this->identifier, "rules"=>$this->rules];
+		if ($this->optional) {
 			$result["optional"]=true;
 		}
-		if(isset($this->depends)){
+		if (isset($this->depends)) {
 			$result["depends"]=$this->depends;
 		}
 		return $result;
@@ -95,8 +95,8 @@ 
 		return $this;
 	}
 
-	public function compile(JsUtils $js){
-		if($this->hasCustomRules) {
+	public function compile(JsUtils $js) {
+		if ($this->hasCustomRules) {
 			foreach ($this->customRules as $rule) {
 				$rule->compile($js);
 			}

Braces +2 added lines, -2 removed lines

@@ -62,12 +62,12 @@
 				$this->customRules[]=$type;
 				$this->hasCustomRules=true;
 			}
-		}elseif(\is_array($type)){
+		} elseif(\is_array($type)){
 			$value=JArray::getValue($type, "value", 2);
 			$prompt=JArray::getValue($type, "prompt", 1);
 			$type=JArray::getValue($type, "type", 0);
 			$rule=new Rule($type,$prompt,$value);
-		}else {
+		} else {
 			$rule = new Rule($type, $prompt, $value);
 		}
 		$this->rules[]=$rule;

Ajax/semantic/html/elements/HtmlRail.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
  */
 class HtmlRail extends HtmlSemDoubleElement {
 
-	public function __construct($identifier, $content = NULL) {
+	public function __construct($identifier, $content=NULL) {
 		parent::__construct($identifier, 'div', 'ui rail', $content);
 	}
 

Ajax/semantic/traits/SemanticHtmlElementsTrait.php

Spacing +16 added lines, -16 removed lines

@@ -45,7 +45,7 @@ 
 	 * @param string $onClick
 	 * @return HtmlButton
 	 */
-	public function htmlButton($identifier, $value = null, $cssStyle = null, $onClick = null) {
+	public function htmlButton($identifier, $value=null, $cssStyle=null, $onClick=null) {
 		return $this->addHtmlComponent(new HtmlButton($identifier, $value, $cssStyle, $onClick));
 	}
 
@@ -59,7 +59,7 @@ 
 	 * @param boolean $asIcons
 	 * @return HtmlButtonGroups
 	 */
-	public function htmlButtonGroups($identifier, $elements = array(), $asIcons = false) {
+	public function htmlButtonGroups($identifier, $elements=array(), $asIcons=false) {
 		return $this->addHtmlComponent(new HtmlButtonGroups($identifier, $elements, $asIcons));
 	}
 
@@ -72,7 +72,7 @@ 
 	 * @param string $content
 	 * @return HtmlContainer
 	 */
-	public function htmlContainer($identifier, $content = "") {
+	public function htmlContainer($identifier, $content="") {
 		return $this->addHtmlComponent(new HtmlContainer($identifier, $content));
 	}
 
@@ -85,7 +85,7 @@ 
 	 * @param string $content
 	 * @return HtmlDivider
 	 */
-	public function htmlDivider($identifier, $content = "", $tagName = "div") {
+	public function htmlDivider($identifier, $content="", $tagName="div") {
 		return $this->addHtmlComponent(new HtmlDivider($identifier, $content, $tagName));
 	}
 
@@ -100,7 +100,7 @@ 
 	 * @param string $type
 	 * @return HtmlHeader
 	 */
-	public function htmlHeader($identifier, $niveau = 1, $content = NULL, $type = "page") {
+	public function htmlHeader($identifier, $niveau=1, $content=NULL, $type="page") {
 		return $this->addHtmlComponent(new HtmlHeader($identifier, $niveau, $content, $type));
 	}
 
@@ -128,7 +128,7 @@ 
 	 * @param string $size
 	 * @return HtmlImage
 	 */
-	public function htmlImage($identifier, $src = "", $alt = "", $size = NULL) {
+	public function htmlImage($identifier, $src="", $alt="", $size=NULL) {
 		return $this->addHtmlComponent(new HtmlImage($identifier, $src, $alt, $size));
 	}
 
@@ -142,7 +142,7 @@ 
 	 * @param string $size
 	 * @return HtmlIconGroups
 	 */
-	public function htmlIconGroups($identifier, $icons = array(), $size = "") {
+	public function htmlIconGroups($identifier, $icons=array(), $size="") {
 		return $this->addHtmlComponent(new HtmlIconGroups($identifier, $icons, $size));
 	}
 
@@ -157,7 +157,7 @@ 
 	 * @param string $placeholder
 	 * @return HtmlInput
 	 */
-	public function htmlInput($identifier, $type = "text", $value = "", $placeholder = "") {
+	public function htmlInput($identifier, $type="text", $value="", $placeholder="") {
 		return $this->addHtmlComponent(new HtmlInput($identifier, $type, $value, $placeholder));
 	}
 
@@ -171,7 +171,7 @@ 
 	 * @param string $tagName
 	 * @return HtmlLabel
 	 */
-	public function htmlLabel($identifier, $content = "", $icon = NULL, $tagName = "div") {
+	public function htmlLabel($identifier, $content="", $icon=NULL, $tagName="div") {
 		return $this->addHtmlComponent(new HtmlLabel($identifier, $content, $icon, $tagName));
 	}
 
@@ -182,7 +182,7 @@ 
 	 * @param array $attributes
 	 * @return HtmlLabelGroups
 	 */
-	public function htmlLabelGroups($identifier, $labels = array(), $attributes = array()) {
+	public function htmlLabelGroups($identifier, $labels=array(), $attributes=array()) {
 		return $this->addHtmlComponent(new HtmlLabelGroups($identifier, $labels, $attributes));
 	}
 
@@ -193,7 +193,7 @@ 
 	 * @param array $items
 	 * @return HtmlList
 	 */
-	public function htmlList($identifier, $items = array()) {
+	public function htmlList($identifier, $items=array()) {
 		return $this->addHtmlComponent(new HtmlList($identifier, $items));
 	}
 
@@ -204,7 +204,7 @@ 
 	 * @param string $content
 	 * @return HtmlSegment
 	 */
-	public function htmlSegment($identifier, $content = "") {
+	public function htmlSegment($identifier, $content="") {
 		return $this->addHtmlComponent(new HtmlSegment($identifier, $content));
 	}
 
@@ -216,7 +216,7 @@ 
 	 *        	the segments
 	 * @return HtmlSegmentGroups
 	 */
-	public function htmlSegmentGroups($identifier, $items = array()) {
+	public function htmlSegmentGroups($identifier, $items=array()) {
 		return $this->addHtmlComponent(new HtmlSegmentGroups($identifier, $items));
 	}
 
@@ -230,7 +230,7 @@ 
 	 * @param Direction|string $attributeType
 	 * @return HtmlReveal
 	 */
-	public function htmlReveal($identifier, $visibleContent, $hiddenContent, $type = RevealType::FADE, $attributeType = NULL) {
+	public function htmlReveal($identifier, $visibleContent, $hiddenContent, $type=RevealType::FADE, $attributeType=NULL) {
 		return $this->addHtmlComponent(new HtmlReveal($identifier, $visibleContent, $hiddenContent, $type, $attributeType));
 	}
 
@@ -241,7 +241,7 @@ 
 	 * @param array $steps
 	 * @return HtmlStep
 	 */
-	public function htmlStep($identifier, $steps = array()) {
+	public function htmlStep($identifier, $steps=array()) {
 		return $this->addHtmlComponent(new HtmlStep($identifier, $steps));
 	}
 
@@ -275,7 +275,7 @@ 
 	 * @param mixed $content
 	 * @return HtmlRail
 	 */
-	public function htmlRail($identifier, $content = null) {
+	public function htmlRail($identifier, $content=null) {
 		return $this->addHtmlComponent(new HtmlRail($identifier, $content));
 	}
 }

Ajax/common/html/HtmlDoubleElement.php

Spacing +36 added lines, -36 removed lines

@@ -13,19 +13,19 @@ 
 	 */
 	protected $content;
 
-	protected $wrapContentBefore = "";
+	protected $wrapContentBefore="";
 
-	protected $wrapContentAfter = "";
+	protected $wrapContentAfter="";
 
 	protected $_editableContent;
 
-	public function __construct($identifier, $tagName = "p") {
+	public function __construct($identifier, $tagName="p") {
 		parent::__construct($identifier, $tagName);
-		$this->_template = '<%tagName% id="%identifier%" %properties%>%wrapContentBefore%%content%%wrapContentAfter%</%tagName%>';
+		$this->_template='<%tagName% id="%identifier%" %properties%>%wrapContentBefore%%content%%wrapContentAfter%</%tagName%>';
 	}
 
 	public function setContent($content) {
-		$this->content = $content;
+		$this->content=$content;
 		return $this;
 	}
 
@@ -33,19 +33,19 @@ 
 		return $this->content;
 	}
 
-	public function addContent($content, $before = false) {
-		if (! \is_array($this->content)) {
+	public function addContent($content, $before=false) {
+		if (!\is_array($this->content)) {
 			if (isset($this->content))
-				$this->content = array(
+				$this->content=array(
 					$this->content
 				);
 			else
-				$this->content = array();
+				$this->content=array();
 		}
 		if ($before)
 			\array_unshift($this->content, $content);
 		else
-			$this->content[] = $content;
+			$this->content[]=$content;
 		return $this;
 	}
 
@@ -68,9 +68,9 @@ 
 
 	public function setValue($value) {}
 
-	public function wrapContent($before, $after = "") {
-		$this->wrapContentBefore .= $before;
-		$this->wrapContentAfter = $after . $this->wrapContentAfter;
+	public function wrapContent($before, $after="") {
+		$this->wrapContentBefore.=$before;
+		$this->wrapContentAfter=$after.$this->wrapContentAfter;
 		return $this;
 	}
 
@@ -89,14 +89,14 @@ 
 	}
 
 	protected function _getContentInstances($class, $content) {
-		$instances = [];
+		$instances=[];
 		if ($content instanceof $class) {
-			$instances[] = $content;
+			$instances[]=$content;
 		} elseif ($content instanceof HtmlDoubleElement) {
-			$instances = \array_merge($instances, $content->getContentInstances($class));
+			$instances=\array_merge($instances, $content->getContentInstances($class));
 		} elseif (\is_array($content)) {
 			foreach ($content as $element) {
-				$instances = \array_merge($instances, $this->_getContentInstances($class, $element));
+				$instances=\array_merge($instances, $this->_getContentInstances($class, $element));
 			}
 		}
 		return $instances;
@@ -107,7 +107,7 @@ 
 	 *
 	 * @return HtmlDoubleElement
 	 */
-	public function asLink($href = NULL, $target = NULL) {
+	public function asLink($href=NULL, $target=NULL) {
 		if (isset($href))
 			$this->setProperty("href", $href);
 		if (isset($target))
@@ -122,28 +122,28 @@ 
 		return \strip_tags($this->content);
 	}
 
-	public function asEditable(HtmlFormField $field, $asForm = false, $setValueProperty = "val()") {
-		$idF = $field->getIdentifier();
-		$idE = $idF;
+	public function asEditable(HtmlFormField $field, $asForm=false, $setValueProperty="val()") {
+		$idF=$field->getIdentifier();
+		$idE=$idF;
 		if ($asForm) {
-			$frm = new HtmlForm("frm-" . $field->getIdentifier());
+			$frm=new HtmlForm("frm-".$field->getIdentifier());
 			$frm->setProperty("onsubmit", "return false;");
-			$fields = $frm->addFields();
-			$idE = $frm->getIdentifier();
+			$fields=$frm->addFields();
+			$idE=$frm->getIdentifier();
 			$fields->addItem($field);
-			$fields->addButtonIcon("bt-okay", "check", "green mini", "\$('#" . $idE . "').trigger('validate',{value: $('#'+idF+' input').val()});");
-			$fields->addButtonIcon("bt-cancel", "close", "mini", "\$('#" . $idE . "').trigger('endEdit');");
-			$this->_editableContent = $frm;
-			$keypress = "";
-			$focusOut = "";
+			$fields->addButtonIcon("bt-okay", "check", "green mini", "\$('#".$idE."').trigger('validate',{value: $('#'+idF+' input').val()});");
+			$fields->addButtonIcon("bt-cancel", "close", "mini", "\$('#".$idE."').trigger('endEdit');");
+			$this->_editableContent=$frm;
+			$keypress="";
+			$focusOut="";
 		} else {
-			$focusOut = "if(e.relatedTarget==null)elm.trigger('endEdit');";
-			$this->_editableContent = $field;
-			$keypress = "$('#" . $idF . "').keyup(function(e){if(e.which == 13) {\$('#" . $idE . "').trigger('validate',{value: $('#'+idF+' input').val()});}if(e.keyCode===27) {\$('#" . $idE . "').trigger('endEdit');}});";
+			$focusOut="if(e.relatedTarget==null)elm.trigger('endEdit');";
+			$this->_editableContent=$field;
+			$keypress="$('#".$idF."').keyup(function(e){if(e.which == 13) {\$('#".$idE."').trigger('validate',{value: $('#'+idF+' input').val()});}if(e.keyCode===27) {\$('#".$idE."').trigger('endEdit');}});";
 		}
 		$this->_editableContent->setProperty("style", "display:none;");
-		$this->onCreate("let idF='" . $idF . "';let idE='" . $idE . "';let elm=$('#'+idE);let self=$('#" . $this->getIdentifier() . "');" . $keypress . "elm.on('validate',function(){self.html($('#'+idE+' input')." . $setValueProperty . ");elm.trigger('endEdit');});elm.on('endEdit',function(){self.show();$(this).hide();});elm.focusout(function(e){" . $focusOut . "});");
-		$this->onClick("let self=$(this);self.hide();" . $field->setJsContent("self.html()") . ";$('#" . $idF . " input').trigger('change');elm.show();$('#'+idE+' input').focus();");
+		$this->onCreate("let idF='".$idF."';let idE='".$idE."';let elm=$('#'+idE);let self=$('#".$this->getIdentifier()."');".$keypress."elm.on('validate',function(){self.html($('#'+idE+' input').".$setValueProperty.");elm.trigger('endEdit');});elm.on('endEdit',function(){self.show();$(this).hide();});elm.focusout(function(e){".$focusOut."});");
+		$this->onClick("let self=$(this);self.hide();".$field->setJsContent("self.html()").";$('#".$idF." input').trigger('change');elm.show();$('#'+idE+' input').focus();");
 	}
 
 	/**
@@ -151,8 +151,8 @@ 
 	 * {@inheritdoc}
 	 * @see \Ajax\common\html\BaseHtml::compile_once()
 	 */
-	protected function compile_once(\Ajax\JsUtils $js = NULL, &$view = NULL) {
-		if (! $this->_compiled && isset($this->_editableContent)) {
+	protected function compile_once(\Ajax\JsUtils $js=NULL, &$view=NULL) {
+		if (!$this->_compiled && isset($this->_editableContent)) {
 			$this->wrap("", $this->_editableContent);
 		}
 		parent::compile_once($js, $view);

Braces +16 added lines, -12 removed lines

@@ -35,17 +35,19 @@ 
 
 	public function addContent($content, $before = false) {
 		if (! \is_array($this->content)) {
-			if (isset($this->content))
-				$this->content = array(
+			if (isset($this->content)) {
+							$this->content = array(
 					$this->content
 				);
-			else
-				$this->content = array();
+			} else {
+							$this->content = array();
+			}
+		}
+		if ($before) {
+					\array_unshift($this->content, $content);
+		} else {
+					$this->content[] = $content;
 		}
-		if ($before)
-			\array_unshift($this->content, $content);
-		else
-			$this->content[] = $content;
 		return $this;
 	}
 
@@ -108,10 +110,12 @@ 
 	 * @return HtmlDoubleElement
 	 */
 	public function asLink($href = NULL, $target = NULL) {
-		if (isset($href))
-			$this->setProperty("href", $href);
-		if (isset($target))
-			$this->setProperty("target", $target);
+		if (isset($href)) {
+					$this->setProperty("href", $href);
+		}
+		if (isset($target)) {
+					$this->setProperty("target", $target);
+		}
 		return $this->setTagName("a");
 	}