phpMv / phpMv-UI

Ajax/semantic/widgets/dataelement/DeInstanceViewer.php

Spacing +3 added lines, -3 removed lines

@@ -12,10 +12,10 @@
 		parent::__construct($identifier, $instance, $captions);
 	}
 
-	public function getValue($index){
+	public function getValue($index) {
 		$result=parent::getValue($index);
-		if($result instanceof HtmlFormField){
-			$lbl=new HtmlSemDoubleElement("lbl-".$this->widgetIdentifier."-".$index,"label","",$this->getCaption($index));
+		if ($result instanceof HtmlFormField) {
+			$lbl=new HtmlSemDoubleElement("lbl-".$this->widgetIdentifier."-".$index, "label", "", $this->getCaption($index));
 			$lbl->setProperty("for", $result->getDataField()->getIdentifier());
 			$this->captions[$index]=$lbl;
 		}

Ajax/semantic/widgets/business/BusinessForm.php

Doc Comments +1 added lines

@@ -15,6 +15,7 @@
 	 * {@inheritdoc}
 	 *
 	 * @see \Ajax\semantic\widgets\dataform\DataForm::__construct()
+	 * @param string $identifier
 	 */
 	public function __construct($identifier,$modelInstance=null,$fieldsOrder,$fieldsDefinition,$fields=[],$captions=[],$separators=[]) {
 		if(!isset($modelInstance)){

Spacing +13 added lines, -13 removed lines

@@ -16,17 +16,17 @@ 
 	 *
 	 * @see \Ajax\semantic\widgets\dataform\DataForm::__construct()
 	 */
-	public function __construct($identifier,$modelInstance=null,$fieldsOrder,$fieldsDefinition,$fields=[],$captions=[],$separators=[]) {
-		if(!isset($modelInstance)){
+	public function __construct($identifier, $modelInstance=null, $fieldsOrder, $fieldsDefinition, $fields=[], $captions=[], $separators=[]) {
+		if (!isset($modelInstance)) {
 			$modelInstance=$this->getDefaultModelInstance();
 		}
-		parent::__construct($identifier,$modelInstance);
-		$this->_initForm($fieldsOrder, $fieldsDefinition,$fields,$captions,$separators);
+		parent::__construct($identifier, $modelInstance);
+		$this->_initForm($fieldsOrder, $fieldsDefinition, $fields, $captions, $separators);
 	}
 
 	abstract protected function getDefaultModelInstance();
 
-	protected function _initForm($fieldsOrder,$fieldsDefinition,$fields=[],$captions=[],$separators=[]){
+	protected function _initForm($fieldsOrder, $fieldsDefinition, $fields=[], $captions=[], $separators=[]) {
 		$this->_fieldsOrder=$fieldsOrder;
 		$this->setFields($fields);
 		$this->setSeparators($separators);
@@ -34,26 +34,26 @@ 
 		$this->setCaptions($captions);
 	}
 
-	protected function _getIndex($fieldName){
+	protected function _getIndex($fieldName) {
 		$index=$fieldName;
-		if(\is_string($fieldName)){
+		if (\is_string($fieldName)) {
 			$index=\array_search($fieldName, $this->_fieldsOrder);
 		}
 		return $index;
 	}
-	protected function _fieldAs($elementCallback,&$index,$attributes=NULL,$prefix=null){
+	protected function _fieldAs($elementCallback, &$index, $attributes=NULL, $prefix=null) {
 		$index=$this->_getIndex($index);
-		return parent::_fieldAs($elementCallback, $index,$attributes,$prefix);
+		return parent::_fieldAs($elementCallback, $index, $attributes, $prefix);
 	}
 
 
-	public function removeField($fieldName){
+	public function removeField($fieldName) {
 		parent::removeField($fieldName);
-		\array_splice($this->_fieldsOrder,$this->_getIndex($fieldName),1);
+		\array_splice($this->_fieldsOrder, $this->_getIndex($fieldName), 1);
 		return $this;
 	}
 
-	public function compile(JsUtils $js=NULL,&$view=NULL){
-		return parent::compile($js,$view);
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
+		return parent::compile($js, $view);
 	}
 }

Ajax/semantic/widgets/dataform/FormInstanceViewer.php

Braces +6 added lines, -4 removed lines

@@ -47,15 +47,17 @@
 
 
 	public function addSeparatorAfter($fieldNum){
-		if(\array_search($fieldNum, $this->separators)===false)
-			$this->separators[]=$fieldNum;
+		if(\array_search($fieldNum, $this->separators)===false) {
+					$this->separators[]=$fieldNum;
+		}
 		return $this;
 	}
 
 	public function addHeaderDividerBefore($fieldNum,$header){
 		$this->headers[$fieldNum]=$header;
-		if($fieldNum>0)
-			$this->addSeparatorAfter($fieldNum-1);
+		if($fieldNum>0) {
+					$this->addSeparatorAfter($fieldNum-1);
+		}
 		return $this;
 	}
 

Spacing +23 added lines, -23 removed lines

@@ -11,58 +11,58 @@ 
 	protected $headers;
 	protected $wrappers;
 
-	public function __construct($identifier,$instance=NULL, $captions=NULL) {
-		parent::__construct($identifier,$instance, $captions);
+	public function __construct($identifier, $instance=NULL, $captions=NULL) {
+		parent::__construct($identifier, $instance, $captions);
 		$this->separators=[-1];
 		$this->headers=[];
 		$this->wrappers=[];
-		$this->defaultValueFunction=function($name,$value,$index){
+		$this->defaultValueFunction=function($name, $value, $index) {
 			$caption=$this->getCaption($index);
-			$input=new HtmlFormInput($this->widgetIdentifier."-".$name,$caption,"text",$value);
+			$input=new HtmlFormInput($this->widgetIdentifier."-".$name, $caption, "text", $value);
 			$input->setName($name);
 			return $input;
 		};
 	}
 
-	protected function _beforeAddProperty($index,&$field){
-		if(JString::endswith($field, "\n")===true){
+	protected function _beforeAddProperty($index, &$field) {
+		if (JString::endswith($field, "\n")===true) {
 			$this->addSeparatorAfter($index);
 		}
-		if($index>1 && JString::startswith($field, "\n")===true){
+		if ($index>1 && JString::startswith($field, "\n")===true) {
 			$this->addSeparatorAfter($index-1);
 		}
 		$field=\str_replace("\n", "", $field);
-		if(($header=$this->hasHeader($field))!==false){
+		if (($header=$this->hasHeader($field))!==false) {
 			$this->addHeaderDividerBefore($index, $header);
 		}
 	}
 
-	protected function hasHeader(&$field){
-		$matches=[];$result=false;
-		if(\preg_match('/\{(.*?)\}/s', $field, $matches)===1){
+	protected function hasHeader(&$field) {
+		$matches=[]; $result=false;
+		if (\preg_match('/\{(.*?)\}/s', $field, $matches)===1) {
 			$result=$matches[1];
-			$field=\str_replace("{".$result."}","", $field);
+			$field=\str_replace("{".$result."}", "", $field);
 		}
 		return $result;
 	}
 
 
 
-	public function addSeparatorAfter($fieldNum){
-		if(\array_search($fieldNum, $this->separators)===false)
+	public function addSeparatorAfter($fieldNum) {
+		if (\array_search($fieldNum, $this->separators)===false)
 			$this->separators[]=$fieldNum;
 		return $this;
 	}
 
-	public function addHeaderDividerBefore($fieldNum,$header){
+	public function addHeaderDividerBefore($fieldNum, $header) {
 		$this->headers[$fieldNum]=$header;
-		if($fieldNum>0)
+		if ($fieldNum>0)
 			$this->addSeparatorAfter($fieldNum-1);
 		return $this;
 	}
 
-	public function addWrapper($fieldNum,$contentBefore,$contentAfter=null){
-		$this->wrappers[$fieldNum]=[$contentBefore,$contentAfter];
+	public function addWrapper($fieldNum, $contentBefore, $contentAfter=null) {
+		$this->wrappers[$fieldNum]=[$contentBefore, $contentAfter];
 			return $this;
 	}
 
@@ -70,16 +70,16 @@ 
 		return $this->separators;
 	}
 
-	public function removeSeparator($index){
-		\array_splice($this->separators,$index,1);
+	public function removeSeparator($index) {
+		\array_splice($this->separators, $index, 1);
 	}
 
-	public function removeField($index){
+	public function removeField($index) {
 		parent::removeField($index);
 		$pos=\array_search($index, $this->separators);
-		if($pos!==false){
+		if ($pos!==false) {
 			$sepCount=\sizeof($this->separators);
-			for($i=$pos+1;$i<$sepCount;$i++){
+			for ($i=$pos+1; $i<$sepCount; $i++) {
 				$this->separators[$i]--;
 			}
 			\array_splice($this->separators, $pos, 1);

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

Spacing +37 added lines, -37 removed lines

@@ -14,34 +14,34 @@ 
 	use FieldTrait;
 	protected $_container;
 	protected $_validation;
-	public function __construct($identifier, $field,$label=NULL) {
-		parent::__construct($identifier, "div","field");
+	public function __construct($identifier, $field, $label=NULL) {
+		parent::__construct($identifier, "div", "field");
 		$this->content=array();
-		$this->_states=[State::ERROR,State::DISABLED];
-		if(isset($label) && $label!=="")
+		$this->_states=[State::ERROR, State::DISABLED];
+		if (isset($label) && $label!=="")
 			$this->setLabel($label);
 		$this->setField($field);
 		$this->_validation=NULL;
 	}
 
-	public function addPointingLabel($label,$pointing=Direction::NONE){
-		$labelO=new HtmlLabel("",$label);
+	public function addPointingLabel($label, $pointing=Direction::NONE) {
+		$labelO=new HtmlLabel("", $label);
 		$labelO->setPointing($pointing);
-		$this->addContent($labelO,$pointing==="below" || $pointing==="right");
+		$this->addContent($labelO, $pointing==="below" || $pointing==="right");
 		return $labelO;
 	}
 
-	public function setLabel($label){
+	public function setLabel($label) {
 		$labelO=$label;
-		if(\is_string($label)){
-			$labelO=new HtmlSemDoubleElement("","label","");
+		if (\is_string($label)) {
+			$labelO=new HtmlSemDoubleElement("", "label", "");
 			$labelO->setContent($label);
-			$labelO->setProperty("for", \str_replace("field-", "",$this->identifier));
+			$labelO->setProperty("for", \str_replace("field-", "", $this->identifier));
 		}
 		$this->content["label"]=$labelO;
 	}
 
-	public function setField($field){
+	public function setField($field) {
 		$this->content["field"]=$field;
 	}
 
@@ -49,8 +49,8 @@ 
 	 * Returns the label or null
 	 * @return mixed
 	 */
-	public function getLabel(){
-		if(\array_key_exists("label", $this->content))
+	public function getLabel() {
+		if (\array_key_exists("label", $this->content))
 			return $this->content["label"];
 	}
 
@@ -58,7 +58,7 @@ 
 	 * Return the field
 	 * @return mixed
 	 */
-	public function getField(){
+	public function getField() {
 		return $this->content["field"];
 	}
 
@@ -66,14 +66,14 @@ 
 	 * Return the field with data
 	 * @return mixed
 	 */
-	public function getDataField(){
+	public function getDataField() {
 		return $this->content["field"];
 	}
 
 	/**
 	 * puts the label before or behind
 	 */
-	public function swapLabel(){
+	public function swapLabel() {
 		$label=$this->getLabel();
 		unset($this->content["label"]);
 		$this->content["label"]=$label;
@@ -84,31 +84,31 @@ 
 	 * @param int $width
 	 * @return \Ajax\semantic\html\collections\form\HtmlFormField
 	 */
-	public function setWidth($width){
-		if(\is_int($width)){
+	public function setWidth($width) {
+		if (\is_int($width)) {
 			$width=Wide::getConstants()["W".$width];
 		}
 		$this->addToPropertyCtrl("class", $width, Wide::getConstants());
-		if(isset($this->_container)){
+		if (isset($this->_container)) {
 			$this->_container->setEqualWidth(false);
 		}
-		return $this->addToPropertyCtrl("class", "wide",array("wide"));
+		return $this->addToPropertyCtrl("class", "wide", array("wide"));
 	}
 
 	/**
 	 * Field displays an error state
 	 * @return \Ajax\semantic\html\collections\form\HtmlFormField
 	 */
-	public function setError(){
+	public function setError() {
 		return $this->addToProperty("class", "error");
 	}
 
-	public function setInline(){
+	public function setInline() {
 		return $this->addToProperty("class", "inline");
 	}
 
-	public function jsState($state){
-		return $this->jsDoJquery("addClass",$state);
+	public function jsState($state) {
+		return $this->jsDoJquery("addClass", $state);
 	}
 
 	public function setContainer($_container) {
@@ -116,39 +116,39 @@ 
 		return $this;
 	}
 
-	public function setReadonly(){
+	public function setReadonly() {
 		$this->getField()->setProperty("readonly", "");
 	}
 
-	public function addRule($type,$prompt=NULL,$value=NULL){
+	public function addRule($type, $prompt=NULL, $value=NULL) {
 		$field=$this->getDataField();
-		if(isset($field)){
-			if(!isset($this->_validation)){
+		if (isset($field)) {
+			if (!isset($this->_validation)) {
 				$this->_validation=new FieldValidation($field->getIdentifier());
 			}
-			if($type==="empty"){
-				$this->addToProperty("class","required");
+			if ($type==="empty") {
+				$this->addToProperty("class", "required");
 			}
-			$this->_validation->addRule($type,$prompt,$value);
+			$this->_validation->addRule($type, $prompt, $value);
 		}
 		return $this;
 	}
 
-	public function addRules(array $rules){
-		foreach ($rules as $rule){
+	public function addRules(array $rules) {
+		foreach ($rules as $rule) {
 			$this->addRule($rule);
 		}
 		return $this;
 	}
 
-	public function setRules(array $rules){
+	public function setRules(array $rules) {
 		$this->_validation=null;
 		return $this->addRules($rules);
 	}
 
-	public function addIcon($icon,$direction=Direction::LEFT){
+	public function addIcon($icon, $direction=Direction::LEFT) {
 		$field=$this->getField();
-		return $field->addIcon($icon,$direction);
+		return $field->addIcon($icon, $direction);
 	}
 
 	public function getValidation() {

Braces +6 added lines, -4 removed lines

@@ -18,8 +18,9 @@ 
 		parent::__construct($identifier, "div","field");
 		$this->content=array();
 		$this->_states=[State::ERROR,State::DISABLED];
-		if(isset($label) && $label!=="")
-			$this->setLabel($label);
+		if(isset($label) && $label!=="") {
+					$this->setLabel($label);
+		}
 		$this->setField($field);
 		$this->_validation=NULL;
 	}
@@ -50,8 +51,9 @@ 
 	 * @return mixed
 	 */
 	public function getLabel(){
-		if(\array_key_exists("label", $this->content))
-			return $this->content["label"];
+		if(\array_key_exists("label", $this->content)) {
+					return $this->content["label"];
+		}
 	}
 
 	/**

Ajax/semantic/html/modules/HtmlSearch.php

Spacing +11 added lines, -11 removed lines

@@ -8,12 +8,12 @@ 
 use Ajax\semantic\html\base\constants\Direction;
 
 class HtmlSearch extends HtmlSemDoubleElement {
-	private $_elements=array ();
-	private $_searchFields=array ("title" );
+	private $_elements=array();
+	private $_searchFields=array("title");
 	private $_local=false;
 
 	public function __construct($identifier, $placeholder=NULL, $icon=NULL) {
-		parent::__construct("search-" . $identifier, "div", "ui search", array ());
+		parent::__construct("search-".$identifier, "div", "ui search", array());
 		$this->_identifier=$identifier;
 		$this->createField($placeholder, $icon);
 		$this->createResult();
@@ -33,7 +33,7 @@ 
 	}
 
 	private function createResult() {
-		$this->content["result"]=new HtmlSemDoubleElement("results-" . $this->identifier, "div", "results");
+		$this->content["result"]=new HtmlSemDoubleElement("results-".$this->identifier, "div", "results");
 		return $this->content["result"];
 	}
 
@@ -50,7 +50,7 @@ 
 	}
 
 	public function setUrl($url) {
-		$this->_params["apiSettings"]="%{url: %quote%" . $url . "%quote%}%";
+		$this->_params["apiSettings"]="%{url: %quote%".$url."%quote%}%";
 		return $this;
 	}
 
@@ -73,16 +73,16 @@ 
 	}
 
 	public function run(JsUtils $js) {
-		$this->_params["onSelect"]='%function(result,response){$(%quote%#' . $this->identifier . '%quote%).trigger(%quote%onSelect%quote%, {%quote%result%quote%: result, %quote%response%quote%:response} );}%';
+		$this->_params["onSelect"]='%function(result,response){$(%quote%#'.$this->identifier.'%quote%).trigger(%quote%onSelect%quote%, {%quote%result%quote%: result, %quote%response%quote%:response} );}%';
 		$searchFields=\json_encode($this->_searchFields);
 		$searchFields=str_ireplace("\"", "%quote%", $searchFields);
-		$this->_params["searchFields"]="%" . $searchFields . "%";
-		if ($this->_local === true) {
+		$this->_params["searchFields"]="%".$searchFields."%";
+		if ($this->_local===true) {
 			$this->_params["source"]="%content%";
-			$this->addEvent("beforeExecute", "var content=" . $this->resultsToJson() . ";");
+			$this->addEvent("beforeExecute", "var content=".$this->resultsToJson().";");
 		}
-		if (isset($this->_bsComponent) === false) {
-			$this->_bsComponent=$js->semantic()->search("#" . $this->identifier, $this->_params);
+		if (isset($this->_bsComponent)===false) {
+			$this->_bsComponent=$js->semantic()->search("#".$this->identifier, $this->_params);
 		}
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;

Braces +6 added lines, -4 removed lines

@@ -22,10 +22,12 @@
 
 	private function createField($placeholder=NULL, $icon=NULL) {
 		$field=new HtmlInput($this->identifier);
-		if (isset($placeholder))
-			$field->setPlaceholder($placeholder);
-		if (isset($icon))
-			$field->addIcon($icon, Direction::RIGHT);
+		if (isset($placeholder)) {
+					$field->setPlaceholder($placeholder);
+		}
+		if (isset($icon)) {
+					$field->addIcon($icon, Direction::RIGHT);
+		}
 		$field->getDataField()->setClass("prompt");
 		$this->content["field"]=$field;
 		return $field;

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

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 class HtmlFormCheckbox extends HtmlFormField {
 	use CheckboxTrait;
 	public function __construct($identifier, $label=NULL, $value=NULL, $type=NULL) {
-		parent::__construct("field-".$identifier, new HtmlCheckbox($identifier,$label,$value,$type));
+		parent::__construct("field-".$identifier, new HtmlCheckbox($identifier, $label, $value, $type));
 		$this->_identifier=$identifier;
 	}
 

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

Braces +3 added lines, -2 removed lines

@@ -10,8 +10,9 @@
 	use TextFieldsTrait;
 
 	public function __construct($identifier, $label=NULL,$value=NULL,$placeholder=NULL,$rows=NULL) {
-		if(!isset($placeholder))
-			$placeholder=$label;
+		if(!isset($placeholder)) {
+					$placeholder=$label;
+		}
 		parent::__construct("field-".$identifier, new HtmlTextarea($identifier,$value,$placeholder,$rows), $label);
 		$this->_identifier=$identifier;
 	}

Spacing +6 added lines, -6 removed lines

@@ -9,10 +9,10 @@ 
 class HtmlFormTextarea extends HtmlFormField {
 	use TextFieldsTrait;
 
-	public function __construct($identifier, $label=NULL,$value=NULL,$placeholder=NULL,$rows=NULL) {
-		if(!isset($placeholder))
+	public function __construct($identifier, $label=NULL, $value=NULL, $placeholder=NULL, $rows=NULL) {
+		if (!isset($placeholder))
 			$placeholder=$label;
-		parent::__construct("field-".$identifier, new HtmlTextarea($identifier,$value,$placeholder,$rows), $label);
+		parent::__construct("field-".$identifier, new HtmlTextarea($identifier, $value, $placeholder, $rows), $label);
 		$this->_identifier=$identifier;
 	}
 
@@ -20,7 +20,7 @@ 
 	 * Defines the textarea row count
 	 * @param int $count
 	 */
-	public function setRows($count){
+	public function setRows($count) {
 		$this->getField()->setRows($count);
 	}
 
@@ -28,7 +28,7 @@ 
 		return $this->content["field"];
 	}
 
-	public function setName($name){
-		$this->getDataField()->setProperty("name",$name);
+	public function setName($name) {
+		$this->getDataField()->setProperty("name", $name);
 	}
 }

documentation/filters/traits.php

Spacing +10 added lines, -10 removed lines

@@ -7,22 +7,22 @@
  */
 
 // Get the input
-$source = file_get_contents($argv[1]);
+$source=file_get_contents($argv[1]);
 
 // make traits to classes
-$regexp = '#trait([\s]+[\S]+[\s]*){#';
-$replace = 'class$1{';
-$source = preg_replace($regexp, $replace, $source);
+$regexp='#trait([\s]+[\S]+[\s]*){#';
+$replace='class$1{';
+$source=preg_replace($regexp, $replace, $source);
 
 // use traits by extending them (classes that not extending a class)
-$regexp = '#class([\s]+[\S]+[\s]*)(implements[\s]+[\S]+[\s]*)?{[\s]+use([^;]+);#';
-$replace = 'class$1 extends $3 $2 {';
-$source = preg_replace($regexp, $replace, $source);
+$regexp='#class([\s]+[\S]+[\s]*)(implements[\s]+[\S]+[\s]*)?{[\s]+use([^;]+);#';
+$replace='class$1 extends $3 $2 {';
+$source=preg_replace($regexp, $replace, $source);
 
 // use traits by extending them (classes that already extending a class)
-$regexp = '#class([\s]+[\S]+[\s]+extends[\s]+[\S]+[\s]*)(implements[\s]+[\S]+[\s]*)?{[\s]+use([^;]+);#';
-$replace = 'class$1, $3 $2{';
-$source = preg_replace($regexp, $replace, $source);
+$regexp='#class([\s]+[\S]+[\s]+extends[\s]+[\S]+[\s]*)(implements[\s]+[\S]+[\s]*)?{[\s]+use([^;]+);#';
+$replace='class$1, $3 $2{';
+$source=preg_replace($regexp, $replace, $source);
 
 // Output
 echo $source;

Ajax/semantic/widgets/datatable/DataTable.php

Doc Comments +16 added lines, -4 removed lines

@@ -63,6 +63,9 @@ 
 
 
 
+	/**
+	 * @param string $op
+	 */
 	protected function _generateBehavior($op,$params,JsUtils $js){
 		if(isset($this->_urls[$op])){
 			$params=\array_merge($params,["attr"=>"data-ajax"]);
@@ -139,6 +142,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $checkedClass
+	 */
 	protected function _generateRow($instance,&$table,$checkedClass=null){
 		$this->_instanceViewer->setInstance($instance);
 		InstanceViewer::$index++;
@@ -215,6 +221,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param PositionInTable $part
+	 */
 	private function addToolbarRow($part,$table,$captions){
 		$hasPart=$table->hasPart($part);
 		if($hasPart){
@@ -254,7 +263,7 @@ 
 	 * Paginates the DataTable element with a Semantic HtmlPaginationMenu component
 	 * @param number $page the active page number
 	 * @param number $total_rowcount the total number of items
-	 * @param number $items_per_page The number of items per page
+	 * @param integer $items_per_page The number of items per page
 	 * @param number $pages_visibles The number of visible pages in the Pagination component
 	 * @return DataTable
 	 */
@@ -265,9 +274,9 @@ 
 
 	/**
 	 * Auto Paginates the DataTable element with a Semantic HtmlPaginationMenu component
-	 * @param number $page the active page number
-	 * @param number $items_per_page The number of items per page
-	 * @param number $pages_visibles The number of visible pages in the Pagination component
+	 * @param integer $page the active page number
+	 * @param integer $items_per_page The number of items per page
+	 * @param integer $pages_visibles The number of visible pages in the Pagination component
 	 * @return DataTable
 	 */
 	public function autoPaginate($page=1,$items_per_page=10,$pages_visibles=4){
@@ -333,6 +342,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @return string
+	 */
 	public function getRefreshSelector() {
 		if(isset($this->_refreshSelector))
 			return $this->_refreshSelector;

Braces +38 added lines, -26 removed lines

@@ -57,10 +57,12 @@ 
 			$js->execOn("mouseover", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'visible');",["preventDefault"=>false,"stopPropagation"=>true]);
 			$js->execOn("mouseout", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'hidden');",["preventDefault"=>false,"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);
+		}
 		return parent::run($js);
 	}
 
@@ -93,12 +95,14 @@ 
 			$table->setRowCount(0, \sizeof($captions));
 			$this->_generateHeader($table,$captions);
 
-			if(isset($this->_compileParts))
-				$table->setCompileParts($this->_compileParts);
+			if(isset($this->_compileParts)) {
+							$table->setCompileParts($this->_compileParts);
+			}
 
 			if(isset($this->_searchField) && isset($js)){
-				if(isset($this->_urls["refresh"]))
-					$this->_searchField->postOn("change", $this->_urls["refresh"],"{'s':$(this).val()}","#".$this->identifier." tbody",["preventDefault"=>false,"jqueryDone"=>"replaceWith"]);
+				if(isset($this->_urls["refresh"])) {
+									$this->_searchField->postOn("change", $this->_urls["refresh"],"{'s':$(this).val()}","#".$this->identifier." tbody",["preventDefault"=>false,"jqueryDone"=>"replaceWith"]);
+				}
 			}
 
 			$this->_generateContent($table);
@@ -123,8 +127,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);
@@ -176,8 +181,9 @@ 
 			$field=$ck->getField();
 			$field->setProperty("value",$id);
 			$field->setProperty("name", "selection[]");
-			if(isset($checkedClass))
-				$field->setClass($checkedClass);
+			if(isset($checkedClass)) {
+							$field->setClass($checkedClass);
+			}
 			\array_unshift($values, $ck);
 		}
 		$result=$table->newRow();
@@ -190,8 +196,9 @@ 
 
 	protected function _generatePagination($table,$js=NULL){
 		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();
@@ -210,8 +217,9 @@ 
 
 	protected function _getFieldName($index){
 		$fieldName=parent::_getFieldName($index);
-		if(\is_object($fieldName))
-			$fieldName="field-".$index;
+		if(\is_object($fieldName)) {
+					$fieldName="field-".$index;
+		}
 		return $fieldName."[]";
 	}
 
@@ -251,7 +259,7 @@ 
 		$hasPart=$table->hasPart($part);
 		if($hasPart){
 			$row=$table->getPart($part)->addRow(\sizeof($captions));
-		}else{
+		} else{
 			$row=$table->getPart($part)->getRow(0);
 		}
 		$row->mergeCol();
@@ -276,7 +284,7 @@ 
 			$this->_urls["refresh"]=JArray::getValue($urls, "refresh",0);
 			$this->_urls["edit"]=JArray::getValue($urls, "edit",1);
 			$this->_urls["delete"]=JArray::getValue($urls, "delete",2);
-		}else{
+		} else{
 			$this->_urls=["refresh"=>$urls,"edit"=>$urls,"delete"=>$urls];
 		}
 		return $this;
@@ -350,8 +358,9 @@ 
 
 	protected function getTargetSelector() {
 		$result=$this->_targetSelector;
-		if(!isset($result))
-			$result="#".$this->identifier;
+		if(!isset($result)) {
+					$result="#".$this->identifier;
+		}
 		return $result;
 	}
 
@@ -366,8 +375,9 @@ 
 	}
 
 	public function getRefreshSelector() {
-		if(isset($this->_refreshSelector))
-			return $this->_refreshSelector;
+		if(isset($this->_refreshSelector)) {
+					return $this->_refreshSelector;
+		}
 		return "#".$this->identifier." tbody";
 	}
 
@@ -386,8 +396,9 @@ 
 	 */
 	public function show($modelInstance){
 		if(\is_array($modelInstance)){
-			if(\is_array(array_values($modelInstance)[0]))
-				$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
+			if(\is_array(array_values($modelInstance)[0])) {
+							$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
+			}
 		}
 		$this->_modelInstance=$modelInstance;
 	}
@@ -427,8 +438,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 +101 added lines, -101 removed lines

@@ -24,7 +24,7 @@ 
  *
  */
 class DataTable extends Widget {
-	use TableTrait,DataTableFieldAsTrait,HasCheckboxesTrait,BaseTrait;
+	use TableTrait, DataTableFieldAsTrait, HasCheckboxesTrait, BaseTrait;
 	protected $_searchField;
 	protected $_urls;
 	protected $_pagination;
@@ -42,35 +42,35 @@ 
 	protected $_colWidths;
 
 
-	public function __construct($identifier,$model,$modelInstance=NULL) {
-		parent::__construct($identifier, $model,$modelInstance);
-		$this->_init(new InstanceViewer($identifier), "table", new HtmlTable($identifier, 0,0), false);
+	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->_emptyMessage=new HtmlMessage("", "nothing to display");
 		$this->_emptyMessage->setIcon("info circle");
 	}
 
-	public function run(JsUtils $js){
-		if($this->_hasCheckboxes && isset($js)){
+	public function run(JsUtils $js) {
+		if ($this->_hasCheckboxes && isset($js)) {
 			$this->_runCheckboxes($js);
 		}
-		if($this->_visibleHover){
-			$js->execOn("mouseover", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'visible');",["preventDefault"=>false,"stopPropagation"=>true]);
-			$js->execOn("mouseout", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'hidden');",["preventDefault"=>false,"stopPropagation"=>true]);
+		if ($this->_visibleHover) {
+			$js->execOn("mouseover", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'visible');", ["preventDefault"=>false, "stopPropagation"=>true]);
+			$js->execOn("mouseout", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'hidden');", ["preventDefault"=>false, "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);
 		return parent::run($js);
 	}
 
 
 
-	protected function _generateBehavior($op,$params,JsUtils $js){
-		if(isset($this->_urls[$op])){
-			$params=\array_merge($params,["attr"=>"data-ajax"]);
-			$js->getOnClick("#".$this->identifier." ._".$op, $this->_urls[$op],$this->getTargetSelector(),$params);
+	protected function _generateBehavior($op, $params, JsUtils $js) {
+		if (isset($this->_urls[$op])) {
+			$params=\array_merge($params, ["attr"=>"data-ajax"]);
+			$js->getOnClick("#".$this->identifier." ._".$op, $this->_urls[$op], $this->getTargetSelector(), $params);
 		}
 	}
 
@@ -83,154 +83,154 @@ 
 	}
 
 
-	public function compile(JsUtils $js=NULL,&$view=NULL){
-		if(!$this->_generated){
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
+		if (!$this->_generated) {
 			$this->_instanceViewer->setInstance($this->_model);
 			$captions=$this->_instanceViewer->getCaptions();
 			$table=$this->content["table"];
-			if($this->_hasCheckboxes){
+			if ($this->_hasCheckboxes) {
 				$this->_generateMainCheckbox($captions);
 			}
 			$table->setRowCount(0, \sizeof($captions));
-			$this->_generateHeader($table,$captions);
+			$this->_generateHeader($table, $captions);
 
-			if(isset($this->_compileParts))
+			if (isset($this->_compileParts))
 				$table->setCompileParts($this->_compileParts);
 
-			if(isset($this->_searchField) && isset($js)){
-				if(isset($this->_urls["refresh"]))
-					$this->_searchField->postOn("change", $this->_urls["refresh"],"{'s':$(this).val()}","#".$this->identifier." tbody",["preventDefault"=>false,"jqueryDone"=>"replaceWith"]);
+			if (isset($this->_searchField) && isset($js)) {
+				if (isset($this->_urls["refresh"]))
+					$this->_searchField->postOn("change", $this->_urls["refresh"], "{'s':$(this).val()}", "#".$this->identifier." tbody", ["preventDefault"=>false, "jqueryDone"=>"replaceWith"]);
 			}
 
 			$this->_generateContent($table);
 
-			if($this->_hasCheckboxes && $table->hasPart("thead")){
+			if ($this->_hasCheckboxes && $table->hasPart("thead")) {
 					$table->getHeader()->getCell(0, 0)->addClass("no-sort");
 			}
 
-			if(isset($this->_toolbar)){
+			if (isset($this->_toolbar)) {
 				$this->_setToolbarPosition($table, $captions);
 			}
-			if(isset($this->_pagination) && $this->_pagination->getVisible()){
-				$this->_generatePagination($table,$js);
+			if (isset($this->_pagination) && $this->_pagination->getVisible()) {
+				$this->_generatePagination($table, $js);
 			}
 
-			$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE,"table",PositionInTable::AFTERTABLE]);
+			$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE, "table", PositionInTable::AFTERTABLE]);
 			$this->_compileForm();
 			$this->_applyStyleAttributes($table);
 			$this->_generated=true;
 		}
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 
-	protected function _applyStyleAttributes($table){
-		if(isset($this->_hiddenColumns))
+	protected function _applyStyleAttributes($table) {
+		if (isset($this->_hiddenColumns))
 			$this->_hideColumns();
-			if(isset($this->_colWidths)){
-				foreach ($this->_colWidths as $colIndex=>$width){
-					$table->setColWidth($colIndex,$width);
+			if (isset($this->_colWidths)) {
+				foreach ($this->_colWidths as $colIndex=>$width) {
+					$table->setColWidth($colIndex, $width);
 				}
 			}
 	}
 
-	protected function _hideColumns(){
-		foreach ($this->_hiddenColumns as $colIndex){
+	protected function _hideColumns() {
+		foreach ($this->_hiddenColumns as $colIndex) {
 			$this->_self->hideColumn($colIndex);
 		}
 		return $this;
 	}
 
-	protected function _generateHeader(HtmlTable $table,$captions){
+	protected function _generateHeader(HtmlTable $table, $captions) {
 		$table->setHeaderValues($captions);
-		if(isset($this->_sortable)){
+		if (isset($this->_sortable)) {
 			$table->setSortable($this->_sortable);
 		}
 	}
 
 
 
-	protected function _generateContent($table){
+	protected function _generateContent($table) {
 		$objects=$this->_modelInstance;
-		if(isset($this->_pagination)){
+		if (isset($this->_pagination)) {
 			$objects=$this->_pagination->getObjects($this->_modelInstance);
 		}
 			InstanceViewer::setIndex(0);
 			$table->fromDatabaseObjects($objects, function($instance) use($table){
 				return $this->_generateRow($instance, $table);
 			});
-		if($table->getRowCount()==0){
+		if ($table->getRowCount()==0) {
 			$result=$table->addRow();
 			$result->mergeRow();
 			$result->setValues([$this->_emptyMessage]);
 		}
 	}
 
-	protected function _generateRow($instance,&$table,$checkedClass=null){
+	protected function _generateRow($instance, &$table, $checkedClass=null) {
 		$this->_instanceViewer->setInstance($instance);
 		InstanceViewer::$index++;
-		$values= $this->_instanceViewer->getValues();
+		$values=$this->_instanceViewer->getValues();
 		$id=$this->_instanceViewer->getIdentifier();
-		if($this->_hasCheckboxes){
-			$ck=new HtmlCheckbox("ck-".$this->identifier."-".$id,"");
+		if ($this->_hasCheckboxes) {
+			$ck=new HtmlCheckbox("ck-".$this->identifier."-".$id, "");
 			$ck->setOnChange("event.stopPropagation();");
 			$field=$ck->getField();
-			$field->setProperty("value",$id);
+			$field->setProperty("value", $id);
 			$field->setProperty("name", "selection[]");
-			if(isset($checkedClass))
+			if (isset($checkedClass))
 				$field->setClass($checkedClass);
 			\array_unshift($values, $ck);
 		}
 		$result=$table->newRow();
 		$result->setIdentifier($this->identifier."-tr-".$id);
-		$result->setProperty("data-ajax",$id);
+		$result->setProperty("data-ajax", $id);
 		$result->setValues($values);
-		$result->addToProperty("class",$this->_rowClass);
+		$result->addToProperty("class", $this->_rowClass);
 		return $result;
 	}
 
-	protected function _generatePagination($table,$js=NULL){
-		if(isset($this->_toolbar)){
-			if($this->_toolbarPosition==PositionInTable::FOOTER)
+	protected function _generatePagination($table, $js=NULL) {
+		if (isset($this->_toolbar)) {
+			if ($this->_toolbarPosition==PositionInTable::FOOTER)
 				$this->_toolbar->setFloated("left");
 		}
 		$footer=$table->getFooter();
 		$footer->mergeCol();
-		$menu=new HtmlPaginationMenu("pagination-".$this->identifier,$this->_pagination->getPagesNumbers());
+		$menu=new HtmlPaginationMenu("pagination-".$this->identifier, $this->_pagination->getPagesNumbers());
 		$menu->floatRight();
 		$menu->setActiveItem($this->_pagination->getPage()-1);
 		$footer->addValues($menu);
-		$this->_associatePaginationBehavior($menu,$js);
+		$this->_associatePaginationBehavior($menu, $js);
 	}
 
-	protected function _associatePaginationBehavior(HtmlMenu $menu,JsUtils $js=NULL){
-		if(isset($this->_urls["refresh"])){
-			$menu->postOnClick($this->_urls["refresh"],"{'p':$(this).attr('data-page')}",$this->getRefreshSelector(),["preventDefault"=>false,"jqueryDone"=>"replaceWith"]);
+	protected function _associatePaginationBehavior(HtmlMenu $menu, JsUtils $js=NULL) {
+		if (isset($this->_urls["refresh"])) {
+			$menu->postOnClick($this->_urls["refresh"], "{'p':$(this).attr('data-page')}", $this->getRefreshSelector(), ["preventDefault"=>false, "jqueryDone"=>"replaceWith"]);
 		}
 	}
 
-	protected function _getFieldName($index){
+	protected function _getFieldName($index) {
 		$fieldName=parent::_getFieldName($index);
-		if(\is_object($fieldName))
+		if (\is_object($fieldName))
 			$fieldName="field-".$index;
 		return $fieldName."[]";
 	}
 
-	protected function _getFieldCaption($index){
+	protected function _getFieldCaption($index) {
 		return null;
 	}
 
-	protected function _setToolbarPosition($table,$captions=NULL){
-		switch ($this->_toolbarPosition){
+	protected function _setToolbarPosition($table, $captions=NULL) {
+		switch ($this->_toolbarPosition) {
 			case PositionInTable::BEFORETABLE:
 			case PositionInTable::AFTERTABLE:
-				if(isset($this->_compileParts)===false){
+				if (isset($this->_compileParts)===false) {
 					$this->content[$this->_toolbarPosition]=$this->_toolbar;
 				}
 				break;
 			case PositionInTable::HEADER:
 			case PositionInTable::FOOTER:
 			case PositionInTable::BODY:
-				$this->addToolbarRow($this->_toolbarPosition,$table, $captions);
+				$this->addToolbarRow($this->_toolbarPosition, $table, $captions);
 				break;
 		}
 	}
@@ -242,16 +242,16 @@ 
 	 * @param callable $callback function called after the field compilation
 	 * @return DataTable
 	 */
-	public function afterCompile($index,$callback){
-		$this->_instanceViewer->afterCompile($index,$callback);
+	public function afterCompile($index, $callback) {
+		$this->_instanceViewer->afterCompile($index, $callback);
 		return $this;
 	}
 
-	private function addToolbarRow($part,$table,$captions){
+	private function addToolbarRow($part, $table, $captions) {
 		$hasPart=$table->hasPart($part);
-		if($hasPart){
+		if ($hasPart) {
 			$row=$table->getPart($part)->addRow(\sizeof($captions));
-		}else{
+		} else {
 			$row=$table->getPart($part)->getRow(0);
 		}
 		$row->mergeCol();
@@ -263,7 +263,7 @@ 
 	 * @see Widget::getHtmlComponent()
 	 * @return HtmlTable
 	 */
-	public function getHtmlComponent(){
+	public function getHtmlComponent() {
 		return $this->content["table"];
 	}
 
@@ -277,12 +277,12 @@ 
 	 * @return DataTable
 	 */
 	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);
-		}else{
-			$this->_urls=["refresh"=>$urls,"edit"=>$urls,"delete"=>$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);
+		} else {
+			$this->_urls=["refresh"=>$urls, "edit"=>$urls, "delete"=>$urls];
 		}
 		return $this;
 	}
@@ -295,8 +295,8 @@ 
 	 * @param number $pages_visibles The number of visible pages in the Pagination component
 	 * @return DataTable
 	 */
-	public function paginate($page,$total_rowcount,$items_per_page=10,$pages_visibles=null){
-		$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;
 	}
 
@@ -307,8 +307,8 @@ 
 	 * @param number $pages_visibles 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;
 	}
 
@@ -318,20 +318,20 @@ 
 	 * @param array $compileParts
 	 * @return DataTable
 	 */
-	public function refresh($compileParts=["tbody"]){
+	public function refresh($compileParts=["tbody"]) {
 		$this->_compileParts=$compileParts;
 		return $this;
 	}
 
 
-	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...");
-			$this->_searchField->addIcon("search",Direction::RIGHT);
+	public function getSearchField() {
+		if (isset($this->_searchField)===false) {
+			$this->_searchField=new HtmlInput("search-".$this->identifier, "search", "", "Search...");
+			$this->_searchField->addIcon("search", Direction::RIGHT);
 		}
 		return $this->_searchField;
 	}
@@ -347,7 +347,7 @@ 
 		return $this;
 	}
 
-	public function asForm(){
+	public function asForm() {
 		return $this->getForm();
 	}
 
@@ -355,7 +355,7 @@ 
 
 	protected function getTargetSelector() {
 		$result=$this->_targetSelector;
-		if(!isset($result))
+		if (!isset($result))
 			$result="#".$this->identifier;
 		return $result;
 	}
@@ -371,7 +371,7 @@ 
 	}
 
 	public function getRefreshSelector() {
-		if(isset($this->_refreshSelector))
+		if (isset($this->_refreshSelector))
 			return $this->_refreshSelector;
 		return "#".$this->identifier." tbody";
 	}
@@ -389,9 +389,9 @@ 
 	 * {@inheritDoc}
 	 * @see \Ajax\common\Widget::show()
 	 */
-	public function show($modelInstance){
-		if(\is_array($modelInstance)){
-			if(\is_array(array_values($modelInstance)[0]))
+	public function show($modelInstance) {
+		if (\is_array($modelInstance)) {
+			if (\is_array(array_values($modelInstance)[0]))
 				$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
 		}
 		$this->_modelInstance=$modelInstance;
@@ -426,24 +426,24 @@ 
 		return $this;
 	}
 
-	public function setActiveRowSelector($class="active",$event="click",$multiple=false){
-		$this->_self->setActiveRowSelector($class,$event,$multiple);
+	public function setActiveRowSelector($class="active", $event="click", $multiple=false) {
+		$this->_self->setActiveRowSelector($class, $event, $multiple);
 		return $this;
 	}
 
-	public function hideColumn($colIndex){
-		if(!\is_array($this->_hiddenColumns))
+	public function hideColumn($colIndex) {
+		if (!\is_array($this->_hiddenColumns))
 			$this->_hiddenColumns=[];
 		$this->_hiddenColumns[]=$colIndex;
 		return $this;
 	}
 
-	public function setColWidth($colIndex,$width){
+	public function setColWidth($colIndex, $width) {
 		$this->_colWidths[$colIndex]=$width;
 		return $this;
 	}
 	public function setColWidths($_colWidths) {
-		$this->_colWidths = $_colWidths;
+		$this->_colWidths=$_colWidths;
 		return $this;
 	}
 }