phpMv / phpMv-UI

Ajax/semantic/html/collections/form/HtmlFormDropdown.php

Spacing +15 added lines, -15 removed lines

@@ -7,20 +7,20 @@ 
 
 class HtmlFormDropdown extends HtmlFormField {
 
-	public function __construct($identifier, $items = array(), $label = NULL, $value = "", $multiple = false, $associative = true) {
-		parent::__construct("field-" . $identifier, (new HtmlDropdown("dropdown-" . $identifier, $value, $items, $associative))->asSelect($identifier, $multiple), $label);
-		$this->_identifier = $identifier;
+	public function __construct($identifier, $items=array(), $label=NULL, $value="", $multiple=false, $associative=true) {
+		parent::__construct("field-".$identifier, (new HtmlDropdown("dropdown-".$identifier, $value, $items, $associative))->asSelect($identifier, $multiple), $label);
+		$this->_identifier=$identifier;
 	}
 
 	public function setItems($items) {
 		return $this->getField()->setItems($items);
 	}
 
-	public function addItem($item, $value = NULL, $image = NULL) {
+	public function addItem($item, $value=NULL, $image=NULL) {
 		return $this->getField()->addItem($item, $value, $image);
 	}
 
-	public static function multipleDropdown($identifier, $items = array(), $label = NULL, $value = "", $associative = true) {
+	public static function multipleDropdown($identifier, $items=array(), $label=NULL, $value="", $associative=true) {
 		return new HtmlFormDropdown($identifier, $items, $label, $value, true, $associative);
 	}
 
@@ -32,7 +32,7 @@ 
 		return $this->getField()->getInput();
 	}
 
-	public function asSelect($name = NULL, $multiple = false, $selection = true) {
+	public function asSelect($name=NULL, $multiple=false, $selection=true) {
 		$this->getField()->asSelect($name, $multiple, $selection);
 		return $this;
 	}
@@ -42,11 +42,11 @@ 
 	 * @param boolean $floating
 	 * @return HtmlDropdown
 	 */
-	public function asButton($floating = false) {
-		$field = $this->content["field"];
-		$label = $this->content["label"];
+	public function asButton($floating=false) {
+		$field=$this->content["field"];
+		$label=$this->content["label"];
 		$field->addContent($label);
-		$this->content = [
+		$this->content=[
 			"field" => $field
 		];
 		$this->content["field"]->asButton($floating);
@@ -61,15 +61,15 @@ 
 	 * @param boolean $labeled
 	 * @return mixed|HtmlButton
 	 */
-	public function addAction($action, $direction = Direction::RIGHT, $icon = NULL, $labeled = false) {
+	public function addAction($action, $direction=Direction::RIGHT, $icon=NULL, $labeled=false) {
 		$this->getField()->setStyle('display:inline-block; width: auto');
-		$actionO = $action;
-		if (\is_object($action) === false) {
-			$actionO = new HtmlButton("action-" . $this->identifier, $action);
+		$actionO=$action;
+		if (\is_object($action)===false) {
+			$actionO=new HtmlButton("action-".$this->identifier, $action);
 			if (isset($icon))
 				$actionO->addIcon($icon, true, $labeled);
 		}
-		$this->addContent($actionO, \strstr($direction, Direction::LEFT) !== false);
+		$this->addContent($actionO, \strstr($direction, Direction::LEFT)!==false);
 		return $actionO;
 	}
 }

Braces +3 added lines, -2 removed lines

@@ -66,8 +66,9 @@
 		$actionO = $action;
 		if (\is_object($action) === false) {
 			$actionO = new HtmlButton("action-" . $this->identifier, $action);
-			if (isset($icon))
-				$actionO->addIcon($icon, true, $labeled);
+			if (isset($icon)) {
+							$actionO->addIcon($icon, true, $labeled);
+			}
 		}
 		$this->addContent($actionO, \strstr($direction, Direction::LEFT) !== false);
 		return $actionO;

Ajax/semantic/html/base/HtmlSemNavElement.php

Spacing +11 added lines, -11 removed lines

@@ -29,12 +29,12 @@ 
 	 *
 	 * @var string|array
 	 */
-	protected $_contentSeparator = "";
+	protected $_contentSeparator="";
 
 	public function __construct($identifier, $tagName, $baseClass) {
 		parent::__construct($identifier, $tagName, $baseClass);
-		$this->root = "";
-		$this->attr = "data-ajax";
+		$this->root="";
+		$this->attr="data-ajax";
 	}
 
 	/**
@@ -54,23 +54,23 @@ 
 		return JArray::implode($this->_contentSeparator, $this->content);
 	}
 
-	public function setContentDivider($divider, $index = NULL) {
-		$divider = "<div class='divider'> {$divider} </div>";
+	public function setContentDivider($divider, $index=NULL) {
+		$divider="<div class='divider'> {$divider} </div>";
 		return $this->setDivider($divider, $index);
 	}
 
-	public function setIconContentDivider($iconContentDivider, $index = NULL) {
-		$contentDivider = "<i class='" . $iconContentDivider . " icon divider'></i>";
+	public function setIconContentDivider($iconContentDivider, $index=NULL) {
+		$contentDivider="<i class='".$iconContentDivider." icon divider'></i>";
 		return $this->setDivider($contentDivider, $index);
 	}
 
 	protected function setDivider($divider, $index) {
 		if (isset($index)) {
-			if (! \is_array($this->_contentSeparator))
-				$this->_contentSeparator = array_fill(0, $this->count() - 1, $this->_contentSeparator);
-			$this->_contentSeparator[$index] = $divider;
+			if (!\is_array($this->_contentSeparator))
+				$this->_contentSeparator=array_fill(0, $this->count()-1, $this->_contentSeparator);
+			$this->_contentSeparator[$index]=$divider;
 		} else {
-			$this->_contentSeparator = $divider;
+			$this->_contentSeparator=$divider;
 		}
 		return $this;
 	}

Braces +3 added lines, -2 removed lines

@@ -66,8 +66,9 @@
 
 	protected function setDivider($divider, $index) {
 		if (isset($index)) {
-			if (! \is_array($this->_contentSeparator))
-				$this->_contentSeparator = array_fill(0, $this->count() - 1, $this->_contentSeparator);
+			if (! \is_array($this->_contentSeparator)) {
+							$this->_contentSeparator = array_fill(0, $this->count() - 1, $this->_contentSeparator);
+			}
 			$this->_contentSeparator[$index] = $divider;
 		} else {
 			$this->_contentSeparator = $divider;

Ajax/semantic/html/modules/HtmlModal.php

Spacing +43 added lines, -43 removed lines

@@ -16,11 +16,11 @@ 
  */
 class HtmlModal extends HtmlSemDoubleElement {
 
-	protected $_params = [];
+	protected $_params=[];
 
-	protected $_paramParts = [];
+	protected $_paramParts=[];
 
-	public function __construct($identifier, $header = '', $content = '', $actions = null) {
+	public function __construct($identifier, $header='', $content='', $actions=null) {
 		parent::__construct($identifier, 'div', 'ui modal');
 		if (isset($header)) {
 			$this->setHeader($header);
@@ -34,12 +34,12 @@ 
 	}
 
 	public function setHeader($value) {
-		$this->content['header'] = new HtmlSemDoubleElement('header-' . $this->identifier, 'a', 'header', $value);
+		$this->content['header']=new HtmlSemDoubleElement('header-'.$this->identifier, 'a', 'header', $value);
 		return $this;
 	}
 
 	public function setContent($value) {
-		$this->content['content'] = new HtmlSemDoubleElement('content-' . $this->identifier, 'div', 'content', $value);
+		$this->content['content']=new HtmlSemDoubleElement('content-'.$this->identifier, 'div', 'content', $value);
 		return $this;
 	}
 
@@ -50,14 +50,14 @@ 
 	 * @return HtmlButton[]
 	 */
 	public function setActions($actions): array {
-		$this->content['actions'] = new HtmlSemDoubleElement('content-' . $this->identifier, 'div', 'actions');
-		$r = [];
+		$this->content['actions']=new HtmlSemDoubleElement('content-'.$this->identifier, 'div', 'actions');
+		$r=[];
 		if (\is_array($actions)) {
 			foreach ($actions as $action) {
-				$r[] = $this->addAction($action);
+				$r[]=$this->addAction($action);
 			}
 		} else {
-			$r[] = $this->addAction($actions);
+			$r[]=$this->addAction($actions);
 		}
 		return $r;
 	}
@@ -68,24 +68,24 @@ 
 	 * @return HtmlButton
 	 */
 	public function addAction($action) {
-		if (! $action instanceof BaseHtml) {
-			$class = '';
+		if (!$action instanceof BaseHtml) {
+			$class='';
 			if (\array_search($action, [
 				'Okay',
 				'Yes',
 				'Validate'
-			]) !== false) {
-				$class = 'approve';
+			])!==false) {
+				$class='approve';
 			}
 			if (\array_search($action, [
 				'Close',
 				'Cancel',
 				'No'
-			]) !== false) {
-				$class = 'cancel';
+			])!==false) {
+				$class='cancel';
 			}
-			$action = new HtmlButton('action-' . $this->identifier . '-' . JArray::count($this->content['actions']->getContent()), $action);
-			if ($class !== '')
+			$action=new HtmlButton('action-'.$this->identifier.'-'.JArray::count($this->content['actions']->getContent()), $action);
+			if ($class!=='')
 				$action->addToProperty('class', $class);
 		}
 		return $this->addElementInPart($action, 'actions');
@@ -100,33 +100,33 @@ 
 		return $this->content['actions']->getContent()[$index];
 	}
 
-	public function addContent($content, $before = false) {
+	public function addContent($content, $before=false) {
 		$this->content['content']->addContent($content, $before);
 		return $this;
 	}
 
-	public function addImageContent($image, $description = NULL) {
-		$content = $this->content['content'];
+	public function addImageContent($image, $description=NULL) {
+		$content=$this->content['content'];
 		if (isset($description)) {
-			$description = new HtmlSemDoubleElement('description-' . $this->identifier, 'div', 'description', $description);
+			$description=new HtmlSemDoubleElement('description-'.$this->identifier, 'div', 'description', $description);
 			$content->addContent($description, true);
 		}
-		if ($image !== '') {
-			$img = new HtmlImage('image-' . $this->identifier, $image, '', 'medium');
+		if ($image!=='') {
+			$img=new HtmlImage('image-'.$this->identifier, $image, '', 'medium');
 			$content->addContent($img, true);
 			$content->addToProperty('class', 'image');
 		}
 		return $this;
 	}
 
-	public function addIconContent($icon, $description = NULL) {
-		$content = $this->content['content'];
+	public function addIconContent($icon, $description=NULL) {
+		$content=$this->content['content'];
 		if (isset($description)) {
-			$description = new HtmlSemDoubleElement('description-' . $this->identifier, 'div', 'description', $description);
+			$description=new HtmlSemDoubleElement('description-'.$this->identifier, 'div', 'description', $description);
 			$content->addContent($description, true);
 		}
-		if ($icon !== '') {
-			$img = new HtmlIcon('image-' . $this->identifier, $icon);
+		if ($icon!=='') {
+			$img=new HtmlIcon('image-'.$this->identifier, $icon);
 			$content->addContent($img, true);
 			$content->addToProperty('class', 'image');
 		}
@@ -139,15 +139,15 @@ 
 	}
 
 	public function showDimmer($value) {
-		$value = $value ? 'show' : 'hide';
-		$this->_paramParts[] = [
-			"'" . $value . " dimmer'"
+		$value=$value ? 'show' : 'hide';
+		$this->_paramParts[]=[
+			"'".$value." dimmer'"
 		];
 		return $this;
 	}
 
-	public function setInverted($recursive = true) {
-		$this->_params['inverted'] = true;
+	public function setInverted($recursive=true) {
+		$this->_params['inverted']=true;
 		return $this;
 	}
 
@@ -156,10 +156,10 @@ 
 	}
 
 	public function setTransition($value) {
-		$this->_paramParts[] = [
+		$this->_paramParts[]=[
 			"'setting'",
 			"'transition'",
-			"'" . $value . "'"
+			"'".$value."'"
 		];
 	}
 
@@ -172,7 +172,7 @@ 
 	 * @param array $params
 	 *        	The parameters to pass to the view
 	 */
-	public function renderView(JsUtils $js, $initialController, $viewName, $params = array()) {
+	public function renderView(JsUtils $js, $initialController, $viewName, $params=array()) {
 		return $this->setContent($js->renderContent($initialController, $viewName, $params));
 	}
 
@@ -187,7 +187,7 @@ 
 	 *        	the action name
 	 * @param array $params
 	 */
-	public function forward(JsUtils $js, $initialControllerInstance, $controllerName, $actionName, $params = NULL) {
+	public function forward(JsUtils $js, $initialControllerInstance, $controllerName, $actionName, $params=NULL) {
 		return $this->setContent($js->forward($initialControllerInstance, $controllerName, $actionName, $params));
 	}
 
@@ -197,8 +197,8 @@ 
 	 *
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
-		$this->content = JArray::sortAssociative($this->content, [
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
+		$this->content=JArray::sortAssociative($this->content, [
 			'header',
 			'content',
 			'actions'
@@ -214,14 +214,14 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if (isset($this->_bsComponent) === false)
-			$this->_bsComponent = $js->semantic()->modal('#' . $this->identifier, $this->_params, $this->_paramParts);
+		if (isset($this->_bsComponent)===false)
+			$this->_bsComponent=$js->semantic()->modal('#'.$this->identifier, $this->_params, $this->_paramParts);
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}
 
 	public function jsDo($behavior) {
-		return "$('#" . $this->identifier . "').modal('" . $behavior . "');";
+		return "$('#".$this->identifier."').modal('".$behavior."');";
 	}
 
 	public function jsHide() {
@@ -229,6 +229,6 @@ 
 	}
 
 	public function onHidden($js) {
-		$this->_params['onHidden'] = $js;
+		$this->_params['onHidden']=$js;
 	}
 }

Braces +6 added lines, -4 removed lines

@@ -85,8 +85,9 @@ 
 				$class = 'cancel';
 			}
 			$action = new HtmlButton('action-' . $this->identifier . '-' . JArray::count($this->content['actions']->getContent()), $action);
-			if ($class !== '')
-				$action->addToProperty('class', $class);
+			if ($class !== '') {
+							$action->addToProperty('class', $class);
+			}
 		}
 		return $this->addElementInPart($action, 'actions');
 	}
@@ -214,8 +215,9 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if (isset($this->_bsComponent) === false)
-			$this->_bsComponent = $js->semantic()->modal('#' . $this->identifier, $this->_params, $this->_paramParts);
+		if (isset($this->_bsComponent) === false) {
+					$this->_bsComponent = $js->semantic()->modal('#' . $this->identifier, $this->_params, $this->_paramParts);
+		}
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}

Ajax/semantic/html/collections/HtmlBreadcrumb.php

Spacing +40 added lines, -40 removed lines

@@ -20,7 +20,7 @@ 
 	 *
 	 * @var integer the start index for href generation
 	 */
-	protected $startIndex = 0;
+	protected $startIndex=0;
 
 	/**
 	 *
@@ -32,7 +32,7 @@ 
 	 *
 	 * @var boolean if set to true, the path of the elements is absolute
 	 */
-	protected $absolutePaths = false;
+	protected $absolutePaths=false;
 
 	/**
 	 *
@@ -49,16 +49,16 @@ 
 	 * @param callable $hrefFunction
 	 *        	the function who generates the href elements. default : function($e){return $e->getContent()}
 	 */
-	public function __construct($identifier, $items = array(), $autoActive = true, $startIndex = 0, $hrefFunction = NULL) {
+	public function __construct($identifier, $items=array(), $autoActive=true, $startIndex=0, $hrefFunction=NULL) {
 		parent::__construct($identifier, "div", "ui breadcrumb");
-		$this->startIndex = $startIndex;
-		$this->autoActive = $autoActive;
-		$this->_contentSeparator = "<div class='divider'> / </div>";
-		$this->_hrefFunction = function (HtmlSemDoubleElement $e) {
+		$this->startIndex=$startIndex;
+		$this->autoActive=$autoActive;
+		$this->_contentSeparator="<div class='divider'> / </div>";
+		$this->_hrefFunction=function(HtmlSemDoubleElement $e) {
 			return $e->getContent();
 		};
 		if (isset($hrefFunction)) {
-			$this->_hrefFunction = $hrefFunction;
+			$this->_hrefFunction=$hrefFunction;
 		}
 		$this->addItems($items);
 	}
@@ -88,11 +88,11 @@ 
 	 *
 	 * @param int $index
 	 */
-	public function setActive($index = null) {
-		if (! isset($index)) {
-			$index = sizeof($this->content) - 1;
+	public function setActive($index=null) {
+		if (!isset($index)) {
+			$index=sizeof($this->content)-1;
 		}
-		$activeItem = $this->content[$index];
+		$activeItem=$this->content[$index];
 		$activeItem->addToProperty("class", "active");
 		$activeItem->setTagName("div");
 	}
@@ -105,8 +105,8 @@ 
 	 *        	the request dispatcher
 	 * @return HtmlBreadcrumb
 	 */
-	public function fromDispatcher(JsUtils $js, $dispatcher, $startIndex = 0) {
-		$this->startIndex = $startIndex;
+	public function fromDispatcher(JsUtils $js, $dispatcher, $startIndex=0) {
+		$this->startIndex=$startIndex;
 		return $this->addItems($js->fromDispatcher($dispatcher));
 	}
 
@@ -117,16 +117,16 @@ 
 	 * @param string $separator
 	 * @return string
 	 */
-	public function getHref($index = null, $separator = "/") {
-		if (! isset($index)) {
-			$index = sizeof($this->content);
+	public function getHref($index=null, $separator="/") {
+		if (!isset($index)) {
+			$index=sizeof($this->content);
 		}
-		if ($this->absolutePaths === true) {
+		if ($this->absolutePaths===true) {
 			return $this->_hrefFunction($this->content[$index]);
 		} else {
-			return $this->root . implode($separator, array_slice(array_map(function ($e) {
+			return $this->root.implode($separator, array_slice(array_map(function($e) {
 				return $this->_hrefFunction($e);
-			}, $this->content), $this->startIndex, $index + 1));
+			}, $this->content), $this->startIndex, $index+1));
 		}
 	}
 
@@ -138,17 +138,17 @@ 
 	 * @return HtmlBreadcrumb
 	 */
 	public function setHrefFunction($_hrefFunction) {
-		$this->_hrefFunction = $_hrefFunction;
+		$this->_hrefFunction=$_hrefFunction;
 		return $this;
 	}
 
 	public function setStartIndex($startIndex) {
-		$this->startIndex = $startIndex;
+		$this->startIndex=$startIndex;
 		return $this;
 	}
 
 	public function setAutoActive($autoActive) {
-		$this->autoActive = $autoActive;
+		$this->autoActive=$autoActive;
 		return $this;
 	}
 
@@ -156,13 +156,13 @@ 
 	 * (non-PHPdoc)
 	 * @see \Ajax\bootstrap\html\BaseHtml::compile()
 	 */
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
 		if ($this->autoActive) {
 			$this->setActive();
 		}
-		$count = $this->count();
-		for ($i = 1; $i < $count; $i ++) {
-			$this->content[$i]->wrap($this->getContentDivider($i - 1));
+		$count=$this->count();
+		for ($i=1; $i<$count; $i++) {
+			$this->content[$i]->wrap($this->getContentDivider($i-1));
 		}
 		return parent::compile($js, $view);
 	}
@@ -171,16 +171,16 @@ 
 	 * (non-PHPdoc)
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::on()
 	 */
-	public function on($event, $jsCode, $stopPropagation = false, $preventDefault = false) {
+	public function on($event, $jsCode, $stopPropagation=false, $preventDefault=false) {
 		foreach ($this->content as $element) {
 			$element->on($event, $jsCode, $stopPropagation, $preventDefault);
 		}
 		return $this;
 	}
 
-	public function _ajaxOn($operation, $event, $url, $responseElement = "", $parameters = array()) {
+	public function _ajaxOn($operation, $event, $url, $responseElement="", $parameters=array()) {
 		foreach ($this->content as $element) {
-			if ($element->getProperty($this->attr) != NULL) {
+			if ($element->getProperty($this->attr)!=NULL) {
 				$element->_ajaxOn($operation, $event, $url, $responseElement, $parameters);
 			}
 		}
@@ -194,10 +194,10 @@ 
 	 * @see \Ajax\common\html\HtmlCollection::createItem()
 	 */
 	protected function createItem($value) {
-		$count = $this->count();
-		$itemO = new HtmlSemDoubleElement("item-" . $this->identifier . "-" . $count, "a", "section");
+		$count=$this->count();
+		$itemO=new HtmlSemDoubleElement("item-".$this->identifier."-".$count, "a", "section");
 		if (\is_array($value)) {
-			@list ($text, $href) = $value;
+			@list ($text, $href)=$value;
 			$itemO->setContent($text);
 			$itemO->setProperty('href', $href);
 		} else {
@@ -207,16 +207,16 @@ 
 	}
 
 	public function addIconAt($icon, $index) {
-		$item = $this->getItem($index);
+		$item=$this->getItem($index);
 		if (isset($item)) {
-			$icon = new HtmlIcon("icon-" . $this->identifier, $icon);
+			$icon=new HtmlIcon("icon-".$this->identifier, $icon);
 			$item->wrapContent($icon);
 		}
 	}
 
 	public function addItem($item) {
-		$count = $this->count();
-		$itemO = parent::addItem($item);
+		$count=$this->count();
+		$itemO=parent::addItem($item);
 		$this->addToPropertyCtrl("class", "section", array(
 			"section"
 		));
@@ -229,9 +229,9 @@ 
 	}
 
 	public function setAbsolutePaths($absolutePaths) {
-		$this->absolutePaths = $absolutePaths;
-		$size = \sizeof($this->content);
-		for ($i = 0; $i < $size; $i ++) {
+		$this->absolutePaths=$absolutePaths;
+		$size=\sizeof($this->content);
+		for ($i=0; $i<$size; $i++) {
 			$this->content[$i]->setProperty($this->attr, $this->getHref($i));
 		}
 		return $this;

Ajax/semantic/widgets/datatable/DataTableFieldAsTrait.php

Spacing +68 added lines, -68 removed lines

@@ -14,15 +14,15 @@ 
  * @property boolean $_visibleHover
  * @property InstanceViewer $_instanceViewer
  */
-trait DataTableFieldAsTrait{
-	protected $_buttons=["display","edit","delete"];
+trait DataTableFieldAsTrait {
+	protected $_buttons=["display", "edit", "delete"];
 	protected $_buttonsColumn;
-	abstract public function addField($field,$key=null);
-	abstract public function insertField($index,$field,$key=null);
-	abstract public function insertInField($index,$field,$key=null);
-	abstract public function fieldAs($index,$type,$attributes=NULL);
+	abstract public function addField($field, $key=null);
+	abstract public function insertField($index, $field, $key=null);
+	abstract public function insertInField($index, $field, $key=null);
+	abstract public function fieldAs($index, $type, $attributes=NULL);
 	abstract protected function cleanIdentifier($id);
-	abstract protected function _fieldAs($elementCallback,&$index,$attributes=NULL,$prefix=null);
+	abstract protected function _fieldAs($elementCallback, &$index, $attributes=NULL, $prefix=null);
 	
 	/**
 	 * @param string $caption
@@ -30,8 +30,8 @@ 
 	 * @param boolean $visibleHover
 	 * @return callable
 	 */
-	private function getFieldButtonCallable($caption,$visibleHover=true,$callback=null){
-		return $this->getCallable("getFieldButton",[$caption,$visibleHover],$callback);
+	private function getFieldButtonCallable($caption, $visibleHover=true, $callback=null) {
+		return $this->getCallable("getFieldButton", [$caption, $visibleHover], $callback);
 	}
 
 	/**
@@ -40,20 +40,20 @@ 
 	 * @param callable $callback
 	 * @return callable
 	 */
-	private function getCallable($thisCallback,$parameters,$callback=null){
-		$result=function($instance) use($thisCallback,$parameters,$callback){
-			$object=call_user_func_array(array($this,$thisCallback), $parameters);
-			if(isset($callback)){
-				if(\is_callable($callback)){
-					$callback($object,$instance,$this->_instanceViewer->count()+1);
+	private function getCallable($thisCallback, $parameters, $callback=null) {
+		$result=function($instance) use($thisCallback, $parameters, $callback){
+			$object=call_user_func_array(array($this, $thisCallback), $parameters);
+			if (isset($callback)) {
+				if (\is_callable($callback)) {
+					$callback($object, $instance, $this->_instanceViewer->count()+1);
 				}
 			}
-			if($object instanceof HtmlSemDoubleElement){
+			if ($object instanceof HtmlSemDoubleElement) {
 				$id=$this->_instanceViewer->getIdentifier();
-				$object->setProperty("data-ajax",$id);
-				if($object->propertyContains("class","visibleover")){
+				$object->setProperty("data-ajax", $id);
+				if ($object->propertyContains("class", "visibleover")) {
 					$this->_visibleHover=true;
-					$object->setProperty("style","visibility:hidden;");
+					$object->setProperty("style", "visibility:hidden;");
 				}
 			}
 			return $object;
@@ -65,17 +65,17 @@ 
 	 * @param string $caption
 	 * @return HtmlButton
 	 */
-	private function getFieldButton($caption,$visibleHover=true,$icon=null){
+	private function getFieldButton($caption, $visibleHover=true, $icon=null) {
 		$id=$this->_instanceViewer->getIdentifier();
-		$bt= new HtmlButton($this->cleanIdentifier($icon??$caption).'-'.$id,$caption);
-		if($visibleHover)
+		$bt=new HtmlButton($this->cleanIdentifier($icon ?? $caption).'-'.$id, $caption);
+		if ($visibleHover)
 			$this->_visibleOver($bt);
 		return $bt;
 	}
 
-	private function getFieldButtons($buttons,$visibleHover=true){
-		$bts=new HtmlButtonGroups("",$buttons);
-		if($visibleHover)
+	private function getFieldButtons($buttons, $visibleHover=true) {
+		$bts=new HtmlButtonGroups("", $buttons);
+		if ($visibleHover)
 			$this->_visibleOver($bts);
 		return $bts;
 	}
@@ -89,19 +89,19 @@ 
 	 * @param array $attributes associative array (<b>ajax</b> key is for ajax post)
 	 * @return DataTable
 	 */
-	public function fieldAsSubmit($index,$cssStyle=NULL,$url=NULL,$responseElement=NULL,$attributes=NULL){
-		return $this->_fieldAs(function($id,$name,$value,$caption) use ($url,$responseElement,$cssStyle,$attributes){
-			$button=new HtmlButton($id,$value,$cssStyle);
-			$button->postOnClick($url,"$(event.target).closest('tr').find(':input').serialize()",$responseElement,$attributes["ajax"]);
-			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"])
+	public function fieldAsSubmit($index, $cssStyle=NULL, $url=NULL, $responseElement=NULL, $attributes=NULL) {
+		return $this->_fieldAs(function($id, $name, $value, $caption) use ($url, $responseElement, $cssStyle, $attributes){
+			$button=new HtmlButton($id, $value, $cssStyle);
+			$button->postOnClick($url, "$(event.target).closest('tr').find(':input').serialize()", $responseElement, $attributes["ajax"]);
+			if (!isset($attributes["visibleHover"]) || $attributes["visibleHover"])
 				$this->_visibleOver($button);
 				return $button;
-		}, $index,$attributes);
+		}, $index, $attributes);
 	}
 
-	protected function _visibleOver(BaseHtml $element){
+	protected function _visibleOver(BaseHtml $element) {
 		$this->_visibleHover=true;
-		return $element->addToProperty("class", "visibleover")->setProperty("style","visibility:hidden;");
+		return $element->addToProperty("class", "visibleover")->setProperty("style", "visibility:hidden;");
 	}
 
 	/**
@@ -111,8 +111,8 @@ 
 	 * @param boolean $visibleHover
 	 * @return DataTable
 	 */
-	public function addFieldButton($caption,$visibleHover=true,$callback=null){
-		$this->addField($this->getCallable("getFieldButton",[$caption,$visibleHover],$callback));
+	public function addFieldButton($caption, $visibleHover=true, $callback=null) {
+		$this->addField($this->getCallable("getFieldButton", [$caption, $visibleHover], $callback));
 		return $this;
 	}
 
@@ -123,8 +123,8 @@ 
 	 * @param boolean $visibleHover
 	 * @return DataTable
 	 */
-	public function addFieldButtons($buttons,$visibleHover=true,$callback=null){
-		$this->addField($this->getCallable("getFieldButtons",[$buttons,$visibleHover],$callback));
+	public function addFieldButtons($buttons, $visibleHover=true, $callback=null) {
+		$this->addField($this->getCallable("getFieldButtons", [$buttons, $visibleHover], $callback));
 		return $this;
 	}
 
@@ -135,8 +135,8 @@ 
 	 * @param callable $callback
 	 * @return DataTable
 	 */
-	public function insertFieldButton($index,$caption,$visibleHover=true,$callback=null){
-		$this->insertField($index, $this->getFieldButtonCallable($caption,$visibleHover,$callback));
+	public function insertFieldButton($index, $caption, $visibleHover=true, $callback=null) {
+		$this->insertField($index, $this->getFieldButtonCallable($caption, $visibleHover, $callback));
 		return $this;
 	}
 
@@ -147,25 +147,25 @@ 
 	 * @param callable $callback
 	 * @return DataTable
 	 */
-	public function insertInFieldButton($index,$caption,$visibleHover=true,$callback=null,$key=null){
-		$this->insertInField($index, $this->getFieldButtonCallable($caption,$visibleHover,$callback),$key);
+	public function insertInFieldButton($index, $caption, $visibleHover=true, $callback=null, $key=null) {
+		$this->insertInField($index, $this->getFieldButtonCallable($caption, $visibleHover, $callback), $key);
 		return $this;
 	}
 
-	public function addDefaultButton($icon,$class=null,$visibleHover=true,$callback=null,$key=null){
-		$this->addField($this->getCallable("getDefaultButton",[$icon,$class,$visibleHover],$callback),$key);
+	public function addDefaultButton($icon, $class=null, $visibleHover=true, $callback=null, $key=null) {
+		$this->addField($this->getCallable("getDefaultButton", [$icon, $class, $visibleHover], $callback), $key);
 		return $this;
 	}
 
-	public function insertDefaultButtonIn($index,$icon,$class=null,$visibleHover=true,$callback=null,$key=null){
-		$this->insertInField($index,$this->getCallable("getDefaultButton",[$icon,$class,$visibleHover],$callback),$key);
+	public function insertDefaultButtonIn($index, $icon, $class=null, $visibleHover=true, $callback=null, $key=null) {
+		$this->insertInField($index, $this->getCallable("getDefaultButton", [$icon, $class, $visibleHover], $callback), $key);
 		return $this;
 	}
 
-	private function getDefaultButton($icon,$class=null,$visibleHover=true){
-		$bt=$this->getFieldButton("",$visibleHover,$icon);
+	private function getDefaultButton($icon, $class=null, $visibleHover=true) {
+		$bt=$this->getFieldButton("", $visibleHover, $icon);
 		$bt->asIcon($icon);
-		if(isset($class))
+		if (isset($class))
 			$bt->addClass($class);
 		return $bt;
 	}
@@ -177,9 +177,9 @@ 
 	 * @param callable $callback this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
 	 * @return DataTable
 	 */
-	public function addDeleteButton($visibleHover=true,$deleteBehavior=[],$callback=null){
+	public function addDeleteButton($visibleHover=true, $deleteBehavior=[], $callback=null) {
 		$this->_deleteBehavior=$deleteBehavior;
-		return $this->addDefaultButton("remove","_delete red basic",$visibleHover,$callback,"delete");
+		return $this->addDefaultButton("remove", "_delete red basic", $visibleHover, $callback, "delete");
 	}
 
 	/**
@@ -189,9 +189,9 @@ 
 	 * @param callable $callback this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
 	 * @return DataTable
 	 */
-	public function addEditButton($visibleHover=true,$editBehavior=[],$callback=null){
+	public function addEditButton($visibleHover=true, $editBehavior=[], $callback=null) {
 		$this->_editBehavior=$editBehavior;
-		return $this->addDefaultButton("edit","_edit basic",$visibleHover,$callback,"edit");
+		return $this->addDefaultButton("edit", "_edit basic", $visibleHover, $callback, "edit");
 	}
 	
 	/**
@@ -201,9 +201,9 @@ 
 	 * @param callable $callback this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
 	 * @return DataTable
 	 */
-	public function addDisplayButton($visibleHover=true,$displayBehavior=[],$callback=null){
+	public function addDisplayButton($visibleHover=true, $displayBehavior=[], $callback=null) {
 		$this->_displayBehavior=$displayBehavior;
-		return $this->addDefaultButton("eye","_display basic",$visibleHover,$callback,"display");
+		return $this->addDefaultButton("eye", "_display basic", $visibleHover, $callback, "display");
 	}
 
 	/**
@@ -214,10 +214,10 @@ 
 	 * @param callable $callbackDelete this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
 	 * @return DataTable
 	 */
-	public function addEditDeleteButtons($visibleHover=true,$behavior=[],$callbackEdit=null,$callbackDelete=null){
-		$this->addEditButton($visibleHover,$behavior,$callbackEdit);
+	public function addEditDeleteButtons($visibleHover=true, $behavior=[], $callbackEdit=null, $callbackDelete=null) {
+		$this->addEditButton($visibleHover, $behavior, $callbackEdit);
 		$index=$this->_instanceViewer->visiblePropertiesCount()-1;
-		$this->insertDeleteButtonIn($index,$visibleHover,$behavior,$callbackDelete);
+		$this->insertDeleteButtonIn($index, $visibleHover, $behavior, $callbackDelete);
 		return $this;
 	}
 	
@@ -230,28 +230,28 @@ 
 	 * @param callable $callbackDelete this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
 	 * @return DataTable
 	 */
-	public function addAllButtons($visibleHover=true,$behavior=[],$callbackDisplay=null,$callbackEdit=null,$callbackDelete=null){
-		$this->addDisplayButton($visibleHover,$behavior,$callbackDisplay);
+	public function addAllButtons($visibleHover=true, $behavior=[], $callbackDisplay=null, $callbackEdit=null, $callbackDelete=null) {
+		$this->addDisplayButton($visibleHover, $behavior, $callbackDisplay);
 		$index=$this->_instanceViewer->visiblePropertiesCount()-1;
 		$this->_buttonsColumn=$index;
-		$this->insertEditButtonIn($index,$visibleHover,$behavior,$callbackEdit);
-		$this->insertDeleteButtonIn($index,$visibleHover,$behavior,$callbackDelete);
+		$this->insertEditButtonIn($index, $visibleHover, $behavior, $callbackEdit);
+		$this->insertDeleteButtonIn($index, $visibleHover, $behavior, $callbackDelete);
 		return $this;
 	}
 
-	public function insertDeleteButtonIn($index,$visibleHover=true,$deleteBehavior=[],$callback=null){
+	public function insertDeleteButtonIn($index, $visibleHover=true, $deleteBehavior=[], $callback=null) {
 		$this->_deleteBehavior=$deleteBehavior;
-		return $this->insertDefaultButtonIn($index,"remove","_delete red basic",$visibleHover,$callback,"delete");
+		return $this->insertDefaultButtonIn($index, "remove", "_delete red basic", $visibleHover, $callback, "delete");
 	}
 
-	public function insertEditButtonIn($index,$visibleHover=true,$editBehavior=[],$callback=null){
+	public function insertEditButtonIn($index, $visibleHover=true, $editBehavior=[], $callback=null) {
 		$this->_editBehavior=$editBehavior;
-		return $this->insertDefaultButtonIn($index,"edit","_edit basic",$visibleHover,$callback,"edit");
+		return $this->insertDefaultButtonIn($index, "edit", "_edit basic", $visibleHover, $callback, "edit");
 	}
 	
-	public function insertDisplayButtonIn($index,$visibleHover=true,$displayBehavior=[],$callback=null){
+	public function insertDisplayButtonIn($index, $visibleHover=true, $displayBehavior=[], $callback=null) {
 		$this->_displayBehavior=$displayBehavior;
-		return $this->insertDefaultButtonIn($index,"eye","_display basic",$visibleHover,$callback,"display");
+		return $this->insertDefaultButtonIn($index, "eye", "_display basic", $visibleHover, $callback, "display");
 	}
 
 	/**
@@ -265,7 +265,7 @@ 
 	 * @param array  $_buttons
 	 */
 	public function setButtons($_buttons) {
-		$this->_buttons = $_buttons;
+		$this->_buttons=$_buttons;
 		return $this;
 	}
 

Braces +12 added lines, -8 removed lines

@@ -68,15 +68,17 @@ 
 	private function getFieldButton($caption,$visibleHover=true,$icon=null){
 		$id=$this->_instanceViewer->getIdentifier();
 		$bt= new HtmlButton($this->cleanIdentifier($icon??$caption).'-'.$id,$caption);
-		if($visibleHover)
-			$this->_visibleOver($bt);
+		if($visibleHover) {
+					$this->_visibleOver($bt);
+		}
 		return $bt;
 	}
 
 	private function getFieldButtons($buttons,$visibleHover=true){
 		$bts=new HtmlButtonGroups("",$buttons);
-		if($visibleHover)
-			$this->_visibleOver($bts);
+		if($visibleHover) {
+					$this->_visibleOver($bts);
+		}
 		return $bts;
 	}
 
@@ -93,8 +95,9 @@ 
 		return $this->_fieldAs(function($id,$name,$value,$caption) use ($url,$responseElement,$cssStyle,$attributes){
 			$button=new HtmlButton($id,$value,$cssStyle);
 			$button->postOnClick($url,"$(event.target).closest('tr').find(':input').serialize()",$responseElement,$attributes["ajax"]);
-			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"])
-				$this->_visibleOver($button);
+			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"]) {
+							$this->_visibleOver($button);
+			}
 				return $button;
 		}, $index,$attributes);
 	}
@@ -165,8 +168,9 @@ 
 	private function getDefaultButton($icon,$class=null,$visibleHover=true){
 		$bt=$this->getFieldButton("",$visibleHover,$icon);
 		$bt->asIcon($icon);
-		if(isset($class))
-			$bt->addClass($class);
+		if(isset($class)) {
+					$bt->addClass($class);
+		}
 		return $bt;
 	}
 

Ajax/semantic/widgets/base/InstanceViewer.php

Spacing +112 added lines, -112 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,37 +234,37 @@ 
 	}
 
 	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) && \class_exists($instance)) {
-			$instance = new $instance();
+			$instance=new $instance();
 		}
-		$this->instance = $instance;
-		$this->properties = [];
-		try{
-			$this->reflect = new \ReflectionClass($instance);
-			if (JArray::count($this->visibleProperties) === 0) {
-				$this->properties = $this->getDefaultProperties();
+		$this->instance=$instance;
+		$this->properties=[];
+		try {
+			$this->reflect=new \ReflectionClass($instance);
+			if (JArray::count($this->visibleProperties)===0) {
+				$this->properties=$this->getDefaultProperties();
 			} else {
 				foreach ($this->visibleProperties as $property) {
 					$this->setInstanceProperty($property);
 				}
 			}
-		}catch (\Throwable $th){
+		}catch (\Throwable $th) {
 			
 		}
 		return $this;
@@ -272,56 +272,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() {
@@ -341,43 +341,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;
 	}
 
@@ -392,7 +392,7 @@ 
 	 * @return InstanceViewer
 	 */
 	public function afterCompile($index, $callback) {
-		$this->afterCompile[$index] = $callback;
+		$this->afterCompile[$index]=$callback;
 		return $this;
 	}
 
@@ -404,7 +404,7 @@ 
 	 * @return \Ajax\semantic\widgets\base\InstanceViewer
 	 */
 	public function setCaptionCallback($captionCallback) {
-		$this->captionCallback = $captionCallback;
+		$this->captionCallback=$captionCallback;
 		return $this;
 	}
 
@@ -416,7 +416,7 @@ 
 	 * @return \Ajax\semantic\widgets\base\InstanceViewer
 	 */
 	public function setDefaultValueFunction($defaultValueFunction) {
-		$this->defaultValueFunction = $defaultValueFunction;
+		$this->defaultValueFunction=$defaultValueFunction;
 		return $this;
 	}
 
@@ -425,8 +425,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));
 		});
 	}
 
@@ -451,10 +451,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;
 	}
 

Braces +29 added lines, -24 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);
@@ -264,7 +266,7 @@ 
 					$this->setInstanceProperty($property);
 				}
 			}
-		}catch (\Throwable $th){
+		} catch (\Throwable $th){
 			
 		}
 		return $this;
@@ -284,10 +286,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;
 		}
@@ -332,12 +335,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() {
@@ -370,8 +374,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/common/BaseEnum.php

Braces +3 added lines, -3 removed lines

@@ -27,9 +27,9 @@
 	}
 
 	public static function getConstantValues($postFix = "", $prefixBefore = false) {
-		if ($postFix == "")
-			return \array_values(self::getConstants());
-		else {
+		if ($postFix == "") {
+					return \array_values(self::getConstants());
+		} else {
 			if ($prefixBefore === false) {
 				return \array_map(function ($elem) use ($postFix) {
 					return $elem . " " . $postFix;

Spacing +27 added lines, -27 removed lines

@@ -10,65 +10,65 @@
  */
 abstract class BaseEnum {
 
-	private static $constCacheArray = NULL;
+	private static $constCacheArray=NULL;
 
-	private static $picked = [];
+	private static $picked=[];
 
 	public static function getConstants() {
-		if (self::$constCacheArray == NULL) {
-			self::$constCacheArray = [];
+		if (self::$constCacheArray==NULL) {
+			self::$constCacheArray=[];
 		}
-		$calledClass = get_called_class();
-		if (! \array_key_exists($calledClass, self::$constCacheArray)) {
-			$reflect = new \ReflectionClass($calledClass);
-			self::$constCacheArray[$calledClass] = $reflect->getConstants();
+		$calledClass=get_called_class();
+		if (!\array_key_exists($calledClass, self::$constCacheArray)) {
+			$reflect=new \ReflectionClass($calledClass);
+			self::$constCacheArray[$calledClass]=$reflect->getConstants();
 		}
 		return self::$constCacheArray[$calledClass];
 	}
 
-	public static function getConstantValues($postFix = "", $prefixBefore = false) {
-		if ($postFix == "")
+	public static function getConstantValues($postFix="", $prefixBefore=false) {
+		if ($postFix=="")
 			return \array_values(self::getConstants());
 		else {
-			if ($prefixBefore === false) {
-				return \array_map(function ($elem) use ($postFix) {
-					return $elem . " " . $postFix;
+			if ($prefixBefore===false) {
+				return \array_map(function($elem) use ($postFix) {
+					return $elem." ".$postFix;
 				}, \array_values(self::getConstants()));
 			} else {
-				return \array_map(function ($elem) use ($postFix) {
-					return $postFix . " " . $elem;
+				return \array_map(function($elem) use ($postFix) {
+					return $postFix." ".$elem;
 				}, \array_values(self::getConstants()));
 			}
 		}
 	}
 
-	public static function isValidName($name, $strict = false) {
-		$constants = self::getConstants();
+	public static function isValidName($name, $strict=false) {
+		$constants=self::getConstants();
 
 		if ($strict) {
 			return \array_key_exists($name, $constants);
 		}
 
-		$keys = \array_map('strtolower', array_keys($constants));
+		$keys=\array_map('strtolower', array_keys($constants));
 		return \in_array(\strtolower($name), $keys);
 	}
 
 	public static function isValidValue($value) {
-		$values = \array_values(self::getConstants());
+		$values=\array_values(self::getConstants());
 		return \in_array($value, $values, true);
 	}
 
-	public static function getRandomValue(bool $unique = false) {
-		$values = self::getConstantValues();
-		$count = \count($values);
-		$calledClass = \get_called_class();
-		if ($unique && $count > count(self::$picked[$calledClass] ?? [])) {
+	public static function getRandomValue(bool $unique=false) {
+		$values=self::getConstantValues();
+		$count=\count($values);
+		$calledClass=\get_called_class();
+		if ($unique && $count>count(self::$picked[$calledClass] ?? [])) {
 			do {
-				$newVal = $values[\rand(0, $count - 1)];
+				$newVal=$values[\rand(0, $count-1)];
 			} while (isset(self::$picked[$calledClass][$newVal]));
-			self::$picked[$calledClass][$newVal] = true;
+			self::$picked[$calledClass][$newVal]=true;
 			return $newVal;
 		}
-		return $values[\rand(0, $count - 1)];
+		return $values[\rand(0, $count-1)];
 	}
 }

Ajax/semantic/html/base/constants/icons/Animals.php

Spacing +1 added lines, -1 removed lines

@@ -5,5 +5,5 @@
 
 abstract class Animals extends BaseEnum {
 
-	const CAT = 'cat', CROW = 'crow', DOG = 'dog', DOVE = 'dove', DRAGON = 'dragon', FEATHER = 'feather', FEATHER_ALTERNATE = 'feather alternate', FISH = 'fish', FOG = 'frog', HIPPO = 'hippo', HORSE = 'horse', HORSE_HEAD = 'horse head', KIWI_BIRD = 'kiwi bird', OTTER = 'otter', PAW = 'paw', SPIDER = 'spider';
+	const CAT='cat', CROW='crow', DOG='dog', DOVE='dove', DRAGON='dragon', FEATHER='feather', FEATHER_ALTERNATE='feather alternate', FISH='fish', FOG='frog', HIPPO='hippo', HORSE='horse', HORSE_HEAD='horse head', KIWI_BIRD='kiwi bird', OTTER='otter', PAW='paw', SPIDER='spider';
 }

Ajax/semantic/widgets/datatable/DataTable.php

Braces +33 added lines, -22 removed lines

@@ -89,10 +89,12 @@ 
 					"stopPropagation" => true
 				]);
 			}
-			if (\is_array($this->_deleteBehavior))
-				$this->_generateBehavior("delete", $this->_deleteBehavior, $js);
-			if (\is_array($this->_editBehavior))
-				$this->_generateBehavior("edit", $this->_editBehavior, $js);
+			if (\is_array($this->_deleteBehavior)) {
+							$this->_generateBehavior("delete", $this->_deleteBehavior, $js);
+			}
+			if (\is_array($this->_editBehavior)) {
+							$this->_generateBehavior("edit", $this->_editBehavior, $js);
+			}
 			if (\is_array($this->_displayBehavior)) {
 				$this->_generateBehavior("display", $this->_displayBehavior, $js);
 			}
@@ -150,8 +152,9 @@ 
 			$table->setRowCount(0, \count($captions));
 			$this->_generateHeader($table, $captions);
 
-			if (isset($this->_compileParts))
-				$table->setCompileParts($this->_compileParts);
+			if (isset($this->_compileParts)) {
+							$table->setCompileParts($this->_compileParts);
+			}
 
 			$this->_generateContent($table);
 
@@ -189,8 +192,9 @@ 
 	}
 
 	protected function _applyStyleAttributes($table) {
-		if (isset($this->_hiddenColumns))
-			$this->_hideColumns();
+		if (isset($this->_hiddenColumns)) {
+					$this->_hideColumns();
+		}
 		if (isset($this->_colWidths)) {
 			foreach ($this->_colWidths as $colIndex => $width) {
 				$table->setColWidth($colIndex, $width);
@@ -303,8 +307,9 @@ 
 			$field = $ck->getField();
 			$field->setProperty("value", $dataAjax);
 			$field->setProperty("name", "selection[]");
-			if (isset($checkedClass))
-				$field->setClass($checkedClass);
+			if (isset($checkedClass)) {
+							$field->setClass($checkedClass);
+			}
 			\array_unshift($values, $ck);
 		}
 		$result = $table->newRow();
@@ -321,8 +326,9 @@ 
 
 	protected function _generatePagination($table) {
 		if (isset($this->_toolbar)) {
-			if ($this->_toolbarPosition == PositionInTable::FOOTER)
-				$this->_toolbar->setFloated("left");
+			if ($this->_toolbarPosition == PositionInTable::FOOTER) {
+							$this->_toolbar->setFloated("left");
+			}
 		}
 		$footer = $table->getFooter();
 		$footer->mergeCol();
@@ -364,8 +370,9 @@ 
 
 	protected function _getFieldName($index) {
 		$fieldName = parent::_getFieldName($index);
-		if (\is_object($fieldName))
-			$fieldName = "field-" . $index;
+		if (\is_object($fieldName)) {
+					$fieldName = "field-" . $index;
+		}
 		if ($this->_namePrefix != null) {
 			$fieldName = $this->_namePrefix . '.' . $fieldName;
 		}
@@ -557,8 +564,9 @@ 
 
 	protected function getTargetSelector($op) {
 		$result = $this->_targetSelector;
-		if (! isset($result[$op]))
-			$result = "#" . $this->identifier;
+		if (! isset($result[$op])) {
+					$result = "#" . $this->identifier;
+		}
 		return $result[$op];
 	}
 
@@ -581,8 +589,9 @@ 
 	}
 
 	public function getRefreshSelector() {
-		if (isset($this->_refreshSelector))
-			return $this->_refreshSelector;
+		if (isset($this->_refreshSelector)) {
+					return $this->_refreshSelector;
+		}
 		return "#" . $this->identifier . " tbody";
 	}
 
@@ -603,8 +612,9 @@ 
 	 */
 	public function show($modelInstance) {
 		if (\is_array($modelInstance)) {
-			if (isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0]))
-				$modelInstance = \json_decode(\json_encode($modelInstance), FALSE);
+			if (isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0])) {
+							$modelInstance = \json_decode(\json_encode($modelInstance), FALSE);
+			}
 		}
 		$this->_modelInstance = $modelInstance;
 	}
@@ -654,8 +664,9 @@ 
 	}
 
 	public function hideColumn($colIndex) {
-		if (! \is_array($this->_hiddenColumns))
-			$this->_hiddenColumns = [];
+		if (! \is_array($this->_hiddenColumns)) {
+					$this->_hiddenColumns = [];
+		}
 		$this->_hiddenColumns[] = $colIndex;
 		return $this;
 	}

Spacing +128 added lines, -128 removed lines

@@ -25,7 +25,7 @@ 
  *
  */
 class DataTable extends Widget {
-	use TableTrait,DataTableFieldAsTrait,HasCheckboxesTrait,BaseTrait;
+	use TableTrait, DataTableFieldAsTrait, HasCheckboxesTrait, BaseTrait;
 
 	protected $_searchField;
 
@@ -41,7 +41,7 @@ 
 
 	protected $_displayBehavior;
 
-	protected $_visibleHover = false;
+	protected $_visibleHover=false;
 
 	protected $_targetSelector;
 
@@ -51,7 +51,7 @@ 
 
 	protected $_json;
 
-	protected $_rowClass = "_element";
+	protected $_rowClass="_element";
 
 	protected $_sortable;
 
@@ -65,26 +65,26 @@ 
 
 	protected $_namePrefix;
 
-	public function __construct($identifier, $model, $modelInstance = NULL) {
+	public function __construct($identifier, $model, $modelInstance=NULL) {
 		parent::__construct($identifier, $model, $modelInstance);
 		$this->_init(new InstanceViewer($identifier), "table", new HtmlTable($identifier, 0, 0), false);
-		$this->_urls = [];
-		$this->_emptyMessage = new HtmlMessage("", "nothing to display");
+		$this->_urls=[];
+		$this->_emptyMessage=new HtmlMessage("", "nothing to display");
 		$this->_emptyMessage->setIcon("info circle");
 	}
 
 	public function run(JsUtils $js) {
-		if ($this->_runned !== true) {
-			$offset = $js->scriptCount();
+		if ($this->_runned!==true) {
+			$offset=$js->scriptCount();
 			if ($this->_hasCheckboxes && isset($js)) {
 				$this->_runCheckboxes($js);
 			}
 			if ($this->_visibleHover) {
-				$js->execOn("mouseover", "#" . $this->identifier . " tr", "$(event.currentTarget).closest('tr').find('.visibleover').css('visibility', 'visible');", [
+				$js->execOn("mouseover", "#".$this->identifier." tr", "$(event.currentTarget).closest('tr').find('.visibleover').css('visibility', 'visible');", [
 					"preventDefault" => false,
 					"stopPropagation" => true
 				]);
-				$js->execOn("mouseout", "#" . $this->identifier . " tr", "$(event.currentTarget).closest('tr').find('.visibleover').css('visibility', 'hidden');$(event.currentTarget).trigger('visibleoverOut');", [
+				$js->execOn("mouseout", "#".$this->identifier." tr", "$(event.currentTarget).closest('tr').find('.visibleover').css('visibility', 'hidden');$(event.currentTarget).trigger('visibleoverOut');", [
 					"preventDefault" => false,
 					"stopPropagation" => true
 				]);
@@ -101,16 +101,16 @@ 
 				$this->_associatePaginationBehavior($js, $offset);
 			}
 			$this->_associateSearchFieldBehavior($js, $offset);
-			$this->_runned = true;
+			$this->_runned=true;
 		}
 	}
 
 	protected function _generateBehavior($op, $params, JsUtils $js) {
 		if (isset($this->_urls[$op])) {
-			$params = \array_merge($params, [
+			$params=\array_merge($params, [
 				"attr" => "data-ajax"
 			]);
-			$js->ajaxOnClick("#" . $this->identifier . " ._" . $op, $this->_urls[$op], $this->getTargetSelector($op), $params);
+			$js->ajaxOnClick("#".$this->identifier." ._".$op, $this->_urls[$op], $this->getTargetSelector($op), $params);
 		}
 	}
 
@@ -129,21 +129,21 @@ 
 	}
 
 	public function refreshTD($fieldName, $jquery, $view) {
-		$index = $this->_getIndex($fieldName);
+		$index=$this->_getIndex($fieldName);
 		$this->compile($jquery, $view);
 		return $this->refreshTR()
 			->getTable()
 			->getCell(0, $index);
 	}
 
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
-		if (! $this->_generated) {
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
+		if (!$this->_generated) {
 			if (isset($this->_buttonsColumn)) {
 				$this->_instanceViewer->sortColumnContent($this->_buttonsColumn, $this->_buttons);
 			}
 			$this->_instanceViewer->setInstance($this->_model);
-			$captions = $this->_instanceViewer->getCaptions();
-			$table = $this->content["table"];
+			$captions=$this->_instanceViewer->getCaptions();
+			$table=$this->content["table"];
 			if ($this->_hasCheckboxes) {
 				$this->_generateMainCheckbox($captions);
 			}
@@ -158,17 +158,17 @@ 
 			$this->compileExtraElements($table, $captions);
 			$this->_compileSearchFieldBehavior($js);
 
-			$this->content = JArray::sortAssociative($this->content, [
+			$this->content=JArray::sortAssociative($this->content, [
 				PositionInTable::BEFORETABLE,
 				"table",
 				PositionInTable::AFTERTABLE
 			]);
-			if ($this->_caption != null) {
+			if ($this->_caption!=null) {
 				$this->wrap("<div class='field'><label>{$this->_caption}</label>", "</div>");
 			}
 			$this->_compileForm();
 			$this->_applyStyleAttributes($table);
-			$this->_generated = true;
+			$this->_generated=true;
 		}
 		return parent::compile($js, $view);
 	}
@@ -206,9 +206,9 @@ 
 	}
 
 	protected function _generateHeader(HtmlTable $table, $captions) {
-		$gbFields = $this->_instanceViewer->getGroupByFields();
+		$gbFields=$this->_instanceViewer->getGroupByFields();
 		if (\is_array($gbFields)) {
-			$captions = \array_values(JArray::removeByKeys($captions, $gbFields));
+			$captions=\array_values(JArray::removeByKeys($captions, $gbFields));
 		}
 		$table->setHeaderValues($captions);
 		if (isset($this->_sortable)) {
@@ -217,22 +217,22 @@ 
 	}
 
 	protected function _generateContent($table) {
-		$objects = $this->_modelInstance;
+		$objects=$this->_modelInstance;
 		if (isset($this->_pagination)) {
-			$objects = $this->_pagination->getObjects($this->_modelInstance);
+			$objects=$this->_pagination->getObjects($this->_modelInstance);
 		}
 		InstanceViewer::setIndex(0);
-		$fields = $this->_instanceViewer->getSimpleProperties();
-		$groupByFields = $this->_instanceViewer->getGroupByFields();
-		if (! \is_array($groupByFields)) {
-			$table->fromDatabaseObjects($objects, function ($instance) use ($table, $fields) {
+		$fields=$this->_instanceViewer->getSimpleProperties();
+		$groupByFields=$this->_instanceViewer->getGroupByFields();
+		if (!\is_array($groupByFields)) {
+			$table->fromDatabaseObjects($objects, function($instance) use ($table, $fields) {
 				return $this->_generateRow($instance, $fields, $table);
 			});
 		} else {
-			$diffFields = \array_values(JArray::removeByKeys($fields, $groupByFields));
-			$activeValues = \array_combine($groupByFields, \array_fill(0, \count($groupByFields), null));
-			$uuids = [];
-			$table->fromDatabaseObjects($objects, function ($instance) use ($table, $fields, &$activeValues, $groupByFields, &$uuids, $diffFields) {
+			$diffFields=\array_values(JArray::removeByKeys($fields, $groupByFields));
+			$activeValues=\array_combine($groupByFields, \array_fill(0, \count($groupByFields), null));
+			$uuids=[];
+			$table->fromDatabaseObjects($objects, function($instance) use ($table, $fields, &$activeValues, $groupByFields, &$uuids, $diffFields) {
 				$this->_instanceViewer->setInstance($instance);
 				foreach ($groupByFields as $index => $gbField) {
 					$this->_generateGroupByRow($index, $gbField, $table, $fields, $activeValues, $uuids);
@@ -240,8 +240,8 @@ 
 				return $this->_generateRow($instance, $diffFields, $table, null, $uuids);
 			});
 		}
-		if ($table->getRowCount() == 0) {
-			$result = $table->addRow();
+		if ($table->getRowCount()==0) {
+			$result=$table->addRow();
 			$result->mergeCol();
 			$result->setValues([
 				$this->_emptyMessage
@@ -250,20 +250,20 @@ 
 	}
 
 	protected function _generateGroupByRow($index, $gbField, $table, $fields, &$activeValues, &$uuids) {
-		$newValue = $this->_instanceViewer->getValue($gbField);
-		if ($this->getElementContent($activeValues[$gbField]) !== $this->getElementContent($newValue)) {
-			if ($index == 0) {
-				$uuids = [];
+		$newValue=$this->_instanceViewer->getValue($gbField);
+		if ($this->getElementContent($activeValues[$gbField])!==$this->getElementContent($newValue)) {
+			if ($index==0) {
+				$uuids=[];
 			}
-			$uuid = \uniqid("grp");
-			$uuids[$gbField] = $uuid;
-			$id = $this->_instanceViewer->getIdentifier();
-			$result = $table->addMergeRow(\count($fields) + 1, $newValue);
-			$result->setIdentifier($this->identifier . "-tr-gb-" . $id);
+			$uuid=\uniqid("grp");
+			$uuids[$gbField]=$uuid;
+			$id=$this->_instanceViewer->getIdentifier();
+			$result=$table->addMergeRow(\count($fields)+1, $newValue);
+			$result->setIdentifier($this->identifier."-tr-gb-".$id);
 			$result->setProperty("data-ajax", $id);
 			$result->setProperty("data-group", $uuid);
 			$result->addToProperty("class", $this->_rowClass);
-			$activeValues[$gbField] = $newValue;
+			$activeValues[$gbField]=$newValue;
 		}
 	}
 
@@ -275,9 +275,9 @@ 
 	}
 
 	public function getFieldValue($index) {
-		$index = $this->_getIndex($index);
+		$index=$this->_getIndex($index);
 		if (\is_numeric($index)) {
-			$values = $this->_instanceViewer->getValues();
+			$values=$this->_instanceViewer->getValues();
 			if (isset($values[$index])) {
 				return $values[$index];
 			}
@@ -285,30 +285,30 @@ 
 		return null;
 	}
 
-	protected function _generateRow($instance, $fields, &$table, $checkedClass = null, $uuids = null) {
+	protected function _generateRow($instance, $fields, &$table, $checkedClass=null, $uuids=null) {
 		$this->_instanceViewer->setInstance($instance);
-		InstanceViewer::$index ++;
-		$values = $this->_instanceViewer->getValues();
-		$id = $this->_instanceViewer->getIdentifier();
-		$dataAjax = $id;
-		$id = $this->cleanIdentifier($id);
+		InstanceViewer::$index++;
+		$values=$this->_instanceViewer->getValues();
+		$id=$this->_instanceViewer->getIdentifier();
+		$dataAjax=$id;
+		$id=$this->cleanIdentifier($id);
 		if ($this->_hasCheckboxes) {
-			$ck = new HtmlCheckbox("ck-" . $this->identifier . "-" . $id, "");
-			$checked = false;
+			$ck=new HtmlCheckbox("ck-".$this->identifier."-".$id, "");
+			$checked=false;
 			if (isset($this->_checkedCallback)) {
-				$func = $this->_checkedCallback;
-				$checked = $func($instance);
+				$func=$this->_checkedCallback;
+				$checked=$func($instance);
 			}
 			$ck->setChecked($checked);
-			$field = $ck->getField();
+			$field=$ck->getField();
 			$field->setProperty("value", $dataAjax);
 			$field->setProperty("name", "selection[]");
 			if (isset($checkedClass))
 				$field->setClass($checkedClass);
 			\array_unshift($values, $ck);
 		}
-		$result = $table->newRow();
-		$result->setIdentifier($this->identifier . "-tr-" . $id);
+		$result=$table->newRow();
+		$result->setIdentifier($this->identifier."-tr-".$id);
 		$result->setProperty("data-ajax", $dataAjax);
 		$result->setValues($values);
 		$result->addToProperty("class", $this->_rowClass);
@@ -321,67 +321,67 @@ 
 
 	protected function _generatePagination($table) {
 		if (isset($this->_toolbar)) {
-			if ($this->_toolbarPosition == PositionInTable::FOOTER)
+			if ($this->_toolbarPosition==PositionInTable::FOOTER)
 				$this->_toolbar->setFloated("left");
 		}
-		$footer = $table->getFooter();
+		$footer=$table->getFooter();
 		$footer->mergeCol();
-		$this->_paginationToolbar = $this->_pagination->generateMenu($this->identifier);
+		$this->_paginationToolbar=$this->_pagination->generateMenu($this->identifier);
 		$footer->addValues($this->_paginationToolbar);
 	}
 
-	protected function _associatePaginationBehavior(JsUtils $js = NULL, $offset = null) {
+	protected function _associatePaginationBehavior(JsUtils $js=NULL, $offset=null) {
 		if (isset($this->_urls["refresh"])) {
-			$menu = $this->_pagination->getMenu();
+			$menu=$this->_pagination->getMenu();
 			if (isset($menu) && isset($js)) {
-				$js->postOnClick("#" . $menu->getIdentifier() . " .item", $this->_urls["refresh"], "{'p':$(this).attr('data-page'),'_model':'" . JString::doubleBackSlashes($this->_model) . "'}", $this->getRefreshSelector(), [
+				$js->postOnClick("#".$menu->getIdentifier()." .item", $this->_urls["refresh"], "{'p':$(this).attr('data-page'),'_model':'".JString::doubleBackSlashes($this->_model)."'}", $this->getRefreshSelector(), [
 					"preventDefault" => false,
 					"jqueryDone" => "replaceWith",
 					"hasLoader" => false,
-					"jsCallback" => '$("#' . $this->identifier . '").trigger("pageChange");$("#' . $this->identifier . '").trigger("activeRowChange");'
+					"jsCallback" => '$("#'.$this->identifier.'").trigger("pageChange");$("#'.$this->identifier.'").trigger("activeRowChange");'
 				]);
-				$page = $_POST["p"] ?? null;
+				$page=$_POST["p"] ?? null;
 				if (isset($page)) {
-					$activeClass = $this->getActiveRowClass();
-					$js->execAtLast('$("#' . $this->getIdentifier() . ' .pagination").children("a.item").removeClass("' . $activeClass . '");$("#' . $this->getIdentifier() . ' .pagination").children("a.item[data-page=' . $page . ']:not(.no-active)").addClass("' . $activeClass . '");');
+					$activeClass=$this->getActiveRowClass();
+					$js->execAtLast('$("#'.$this->getIdentifier().' .pagination").children("a.item").removeClass("'.$activeClass.'");$("#'.$this->getIdentifier().' .pagination").children("a.item[data-page='.$page.']:not(.no-active)").addClass("'.$activeClass.'");');
 				}
 			}
 		}
 	}
 
-	protected function _compileSearchFieldBehavior(JsUtils $js = NULL) {
+	protected function _compileSearchFieldBehavior(JsUtils $js=NULL) {
 		if (isset($this->_searchField) && isset($js) && isset($this->_urls["refresh"])) {
-			$this->_searchField->postOn("change", $this->_urls["refresh"], "{'s':$(self).val(),'_model':'" . JString::doubleBackSlashes($this->_model) . "'}", "#" . $this->identifier . " tbody", [
+			$this->_searchField->postOn("change", $this->_urls["refresh"], "{'s':$(self).val(),'_model':'".JString::doubleBackSlashes($this->_model)."'}", "#".$this->identifier." tbody", [
 				"preventDefault" => false,
 				"jqueryDone" => "replaceWith",
 				"hasLoader" => "internal",
-				"jsCallback" => '$("#' . $this->identifier . '").trigger("searchTerminate",[$(self).val()]);'
+				"jsCallback" => '$("#'.$this->identifier.'").trigger("searchTerminate",[$(self).val()]);'
 			]);
 		}
 	}
 
-	protected function _associateSearchFieldBehavior(JsUtils $js = NULL, $offset = null) {}
+	protected function _associateSearchFieldBehavior(JsUtils $js=NULL, $offset=null) {}
 
 	protected function _getFieldName($index) {
-		$fieldName = parent::_getFieldName($index);
+		$fieldName=parent::_getFieldName($index);
 		if (\is_object($fieldName))
-			$fieldName = "field-" . $index;
-		if ($this->_namePrefix != null) {
-			$fieldName = $this->_namePrefix . '.' . $fieldName;
+			$fieldName="field-".$index;
+		if ($this->_namePrefix!=null) {
+			$fieldName=$this->_namePrefix.'.'.$fieldName;
 		}
-		return $fieldName . "[]";
+		return $fieldName."[]";
 	}
 
 	protected function _getFieldCaption($index) {
 		return null;
 	}
 
-	protected function applyToolbarPosition(string $position, $table, $captions = NULL) {
+	protected function applyToolbarPosition(string $position, $table, $captions=NULL) {
 		switch ($position) {
 			case PositionInTable::BEFORETABLE:
 			case PositionInTable::AFTERTABLE:
-				if (isset($this->_compileParts) === false) {
-					$this->content[$position] = $this->_toolbar;
+				if (isset($this->_compileParts)===false) {
+					$this->content[$position]=$this->_toolbar;
 				}
 				break;
 			case PositionInTable::HEADER:
@@ -392,7 +392,7 @@ 
 		}
 	}
 
-	protected function _setToolbarPosition($table, $captions = NULL) {
+	protected function _setToolbarPosition($table, $captions=NULL) {
 		if (\is_array($this->_toolbarPosition)) {
 			foreach ($this->_toolbarPosition as $tbp) {
 				$this->applyToolbarPosition($tbp, $table, $captions);
@@ -418,11 +418,11 @@ 
 	}
 
 	private function addToolbarRow($part, $table, $captions) {
-		$hasPart = $table->hasPart($part);
+		$hasPart=$table->hasPart($part);
 		if ($hasPart) {
-			$row = $table->getPart($part)->addRow(\count($captions));
+			$row=$table->getPart($part)->addRow(\count($captions));
 		} else {
-			$row = $table->getPart($part)->getRow(0);
+			$row=$table->getPart($part)->getRow(0);
 		}
 		$row->mergeCol();
 		$row->setValues([
@@ -454,12 +454,12 @@ 
 	 */
 	public function setUrls($urls) {
 		if (\is_array($urls)) {
-			$this->_urls["refresh"] = JArray::getValue($urls, "refresh", 0);
-			$this->_urls["edit"] = JArray::getValue($urls, "edit", 1);
-			$this->_urls["delete"] = JArray::getValue($urls, "delete", 2);
-			$this->_urls["display"] = JArray::getValue($urls, "display", 3);
+			$this->_urls["refresh"]=JArray::getValue($urls, "refresh", 0);
+			$this->_urls["edit"]=JArray::getValue($urls, "edit", 1);
+			$this->_urls["delete"]=JArray::getValue($urls, "delete", 2);
+			$this->_urls["display"]=JArray::getValue($urls, "display", 3);
 		} else {
-			$this->_urls = [
+			$this->_urls=[
 				"refresh" => $urls,
 				"edit" => $urls,
 				"delete" => $urls,
@@ -482,8 +482,8 @@ 
 	 *        	The number of visible pages in the Pagination component
 	 * @return DataTable
 	 */
-	public function paginate($page, $total_rowcount, $items_per_page = 10, $pages_visibles = null) {
-		$this->_pagination = new Pagination($items_per_page, $pages_visibles, $page, $total_rowcount);
+	public function paginate($page, $total_rowcount, $items_per_page=10, $pages_visibles=null) {
+		$this->_pagination=new Pagination($items_per_page, $pages_visibles, $page, $total_rowcount);
 		return $this;
 	}
 
@@ -498,8 +498,8 @@ 
 	 *        	The number of visible pages in the Pagination component
 	 * @return DataTable
 	 */
-	public function autoPaginate($page = 1, $items_per_page = 10, $pages_visibles = 4) {
-		$this->_pagination = new Pagination($items_per_page, $pages_visibles, $page);
+	public function autoPaginate($page=1, $items_per_page=10, $pages_visibles=4) {
+		$this->_pagination=new Pagination($items_per_page, $pages_visibles, $page);
 		return $this;
 	}
 
@@ -508,10 +508,10 @@ 
 	 * @param array $compileParts
 	 * @return DataTable
 	 */
-	public function refresh($compileParts = [
+	public function refresh($compileParts=[
 		'tbody'
 	]) {
-		$this->_compileParts = $compileParts;
+		$this->_compileParts=$compileParts;
 		return $this;
 	}
 
@@ -521,14 +521,14 @@ 
 	 * @param string $position
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addSearchInToolbar($position = Direction::RIGHT) {
+	public function addSearchInToolbar($position=Direction::RIGHT) {
 		return $this->addInToolbar($this->getSearchField())
 			->setPosition($position);
 	}
 
 	public function getSearchField() {
-		if (isset($this->_searchField) === false) {
-			$this->_searchField = new HtmlInput("search-" . $this->identifier, "search", "", "Search...");
+		if (isset($this->_searchField)===false) {
+			$this->_searchField=new HtmlInput("search-".$this->identifier, "search", "", "Search...");
 			$this->_searchField->addIcon("search", Direction::RIGHT);
 		}
 		return $this->_searchField;
@@ -556,9 +556,9 @@ 
 	}
 
 	protected function getTargetSelector($op) {
-		$result = $this->_targetSelector;
-		if (! isset($result[$op]))
-			$result = "#" . $this->identifier;
+		$result=$this->_targetSelector;
+		if (!isset($result[$op]))
+			$result="#".$this->identifier;
 		return $result[$op];
 	}
 
@@ -570,20 +570,20 @@ 
 	 * @return DataTable
 	 */
 	public function setTargetSelector($_targetSelector) {
-		if (! \is_array($_targetSelector)) {
-			$_targetSelector = [
+		if (!\is_array($_targetSelector)) {
+			$_targetSelector=[
 				"edit" => $_targetSelector,
 				"delete" => $_targetSelector
 			];
 		}
-		$this->_targetSelector = $_targetSelector;
+		$this->_targetSelector=$_targetSelector;
 		return $this;
 	}
 
 	public function getRefreshSelector() {
 		if (isset($this->_refreshSelector))
 			return $this->_refreshSelector;
-		return "#" . $this->identifier . " tbody";
+		return "#".$this->identifier." tbody";
 	}
 
 	/**
@@ -592,7 +592,7 @@ 
 	 * @return DataTable
 	 */
 	public function setRefreshSelector($_refreshSelector) {
-		$this->_refreshSelector = $_refreshSelector;
+		$this->_refreshSelector=$_refreshSelector;
 		return $this;
 	}
 
@@ -604,9 +604,9 @@ 
 	public function show($modelInstance) {
 		if (\is_array($modelInstance)) {
 			if (isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0]))
-				$modelInstance = \json_decode(\json_encode($modelInstance), FALSE);
+				$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
 		}
-		$this->_modelInstance = $modelInstance;
+		$this->_modelInstance=$modelInstance;
 	}
 
 	public function getRowClass() {
@@ -620,7 +620,7 @@ 
 	 * @return DataTable
 	 */
 	public function setRowClass($_rowClass) {
-		$this->_rowClass = $_rowClass;
+		$this->_rowClass=$_rowClass;
 		return $this;
 	}
 
@@ -631,16 +631,16 @@ 
 	 * @return DataTable
 	 */
 	public function setEmptyMessage($_emptyMessage) {
-		$this->_emptyMessage = $_emptyMessage;
+		$this->_emptyMessage=$_emptyMessage;
 		return $this;
 	}
 
-	public function setSortable($colIndex = NULL) {
-		$this->_sortable = $colIndex;
+	public function setSortable($colIndex=NULL) {
+		$this->_sortable=$colIndex;
 		return $this;
 	}
 
-	public function setActiveRowSelector($class = "active", $event = "click", $multiple = false) {
+	public function setActiveRowSelector($class="active", $event="click", $multiple=false) {
 		$this->_self->setActiveRowSelector($class, $event, $multiple);
 		return $this;
 	}
@@ -654,19 +654,19 @@ 
 	}
 
 	public function hideColumn($colIndex) {
-		if (! \is_array($this->_hiddenColumns))
-			$this->_hiddenColumns = [];
-		$this->_hiddenColumns[] = $colIndex;
+		if (!\is_array($this->_hiddenColumns))
+			$this->_hiddenColumns=[];
+		$this->_hiddenColumns[]=$colIndex;
 		return $this;
 	}
 
 	public function setColWidth($colIndex, $width) {
-		$this->_colWidths[$colIndex] = $width;
+		$this->_colWidths[$colIndex]=$width;
 		return $this;
 	}
 
 	public function setColWidths($_colWidths) {
-		$this->_colWidths = $_colWidths;
+		$this->_colWidths=$_colWidths;
 		return $this;
 	}
 
@@ -675,7 +675,7 @@ 
 		return $this;
 	}
 
-	public function trigger($event, $params = "[]") {
+	public function trigger($event, $params="[]") {
 		return $this->getHtmlComponent()->trigger($event, $params);
 	}
 
@@ -713,7 +713,7 @@ 
 	 * @param mixed $_displayBehavior
 	 */
 	public function setDisplayBehavior($_displayBehavior) {
-		$this->_displayBehavior = $_displayBehavior;
+		$this->_displayBehavior=$_displayBehavior;
 	}
 
 	/**
@@ -733,8 +733,8 @@ 
 	}
 
 	public function addGroupBy($index) {
-		$index = $this->_getIndex($index);
-		if ($index !== false) {
+		$index=$this->_getIndex($index);
+		if ($index!==false) {
 			$this->_instanceViewer->addGroupBy($index);
 		}
 	}
@@ -744,7 +744,7 @@ 
 	 * @param boolean $_visibleHover
 	 */
 	public function setVisibleHover($_visibleHover) {
-		$this->_visibleHover = $_visibleHover;
+		$this->_visibleHover=$_visibleHover;
 	}
 
 	/**
@@ -755,7 +755,7 @@ 
 		return $this->_paginationToolbar;
 	}
 
-	public function setInverted($recursive = true) {
+	public function setInverted($recursive=true) {
 		$this->getHtmlComponent()->setInverted($recursive);
 		if ($this->_emptyMessage instanceof HtmlSemDoubleElement) {
 			$this->_emptyMessage->setInverted($recursive);
@@ -767,7 +767,7 @@ 
 	}
 
 	public function setFormCaption($caption) {
-		$this->_caption = $caption;
+		$this->_caption=$caption;
 	}
 
 	/**
@@ -783,6 +783,6 @@ 
 	 * @param mixed $namePrefix
 	 */
 	public function setNamePrefix($namePrefix): void {
-		$this->_namePrefix = $namePrefix;
+		$this->_namePrefix=$namePrefix;
 	}
 }