phpMv / phpMv-UI

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/html/collections/menus/HtmlPaginationMenu.php

Spacing +7 added lines, -7 removed lines

@@ -7,21 +7,21 @@
 
 use Ajax\semantic\html\elements\HtmlIcon;
 
-class HtmlPaginationMenu extends HtmlMenu{
+class HtmlPaginationMenu extends HtmlMenu {
 	private $_page;
-	public function __construct( $identifier, $items=array() ){
-		parent::__construct( $identifier,$items);
+	public function __construct($identifier, $items=array()) {
+		parent::__construct($identifier, $items);
 	}
 	/**
 	 * {@inheritDoc}
 	 * @see \Ajax\common\html\BaseHtml::compile()
 	 */
-	public function compile(JsUtils $js=NULL,&$view=NULL){
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
 		$max=\sizeof($this->content);
-		$this->insertItem(new HtmlIcon("", "left chevron"))->setProperty("data-page", \max([1,$this->_page-1]))->addToProperty("class","_firstPage no-active");
-		$this->addItem(new HtmlIcon("", "right chevron"))->setProperty("data-page", \min([$max,$this->_page+1]))->setProperty("data-max", $max)->addToProperty("class","_lastPage no-active");
+		$this->insertItem(new HtmlIcon("", "left chevron"))->setProperty("data-page", \max([1, $this->_page-1]))->addToProperty("class", "_firstPage no-active");
+		$this->addItem(new HtmlIcon("", "right chevron"))->setProperty("data-page", \min([$max, $this->_page+1]))->setProperty("data-max", $max)->addToProperty("class", "_lastPage no-active");
 		$this->asPagination();
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 
 	public function setActiveItem($index) {

Ajax/semantic/widgets/datatable/HasCheckboxesTrait.php

Braces +9 added lines, -6 removed lines

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

Spacing +24 added lines, -24 removed lines

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

Ajax/semantic/widgets/datatable/Pagination.php

Braces +3 added lines, -2 removed lines

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

Spacing +19 added lines, -19 removed lines

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

Ajax/semantic/widgets/datatable/JsonDataTable.php

Braces +3 added lines, -2 removed lines

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

Spacing +24 added lines, -24 removed lines

@@ -23,24 +23,24 @@ 
 		$this->_rowClass="_json";
 	}
 
-	protected function _generateContent($table){
+	protected function _generateContent($table) {
 		$this->_addRowModel($table);
 		parent::_generateContent($table);
 	}
 
-	protected function _addRowModel($table){
+	protected function _addRowModel($table) {
 		$row=$this->_createRow($table, $this->_modelClass);
-		$row->addToProperty("style","display:none;");
+		$row->addToProperty("style", "display:none;");
 		$table->getBody()->_addRow($row);
 	}
 
-	protected function _createRow($table,$rowClass){
+	protected function _createRow($table, $rowClass) {
 		$object=JReflection::jsonObject($this->_model);
-		if(isset($this->_rowModelCallback)){
+		if (isset($this->_rowModelCallback)) {
 			$callback=$this->_rowModelCallback;
 			$callback($object);
 		}
-		$row=$this->_generateRow($object, $table,"_jsonArrayChecked");
+		$row=$this->_generateRow($object, $table, "_jsonArrayChecked");
 		$row->setClass($rowClass);
 		return $row;
 	}
@@ -49,22 +49,22 @@ 
 	 * {@inheritDoc}
 	 * @see DataTable::_associatePaginationBehavior()
 	 */
-	protected function _associatePaginationBehavior(HtmlMenu $menu,JsUtils $js=NULL){
+	protected function _associatePaginationBehavior(HtmlMenu $menu, JsUtils $js=NULL) {
 		$callback=null;
-		if(isset($js)){
+		if (isset($js)) {
 			$id=$this->identifier;
 			$offset=$js->scriptCount();
 			$this->run($js);
 			$callback=$js->getScript($offset);
-			$callback.=$js->trigger("#".$id." [name='selection[]']","change",false)."$('#".$id." tbody .ui.checkbox').checkbox();".$js->execOn("change", "#".$id." [name='selection[]']", $this->_getCheckedChange($js));
+			$callback.=$js->trigger("#".$id." [name='selection[]']", "change", false)."$('#".$id." tbody .ui.checkbox').checkbox();".$js->execOn("change", "#".$id." [name='selection[]']", $this->_getCheckedChange($js));
 			$callback.=";var page=parseInt($(self).attr('data-page'));
 			$('#pagination-{$id} .item').removeClass('active');
 			$('#pagination-{$id} [data-page='+page+']:not(.no-active)').addClass('active');
 			$('#pagination-{$id} ._firstPage').attr('data-page',Math.max(1,page-1));
 			var lastPage=$('#pagination-{$id} ._lastPage');lastPage.attr('data-page',Math.min(lastPage.attr('data-max'),page+1));";
 		}
-		if(isset($this->_urls["refresh"]))
-			$this->jsonArrayOnClick($menu, $this->_urls["refresh"],"post","{'p':$(this).attr('data-page')}",$callback);
+		if (isset($this->_urls["refresh"]))
+			$this->jsonArrayOnClick($menu, $this->_urls["refresh"], "post", "{'p':$(this).attr('data-page')}", $callback);
 	}
 
 	/**
@@ -75,29 +75,29 @@ 
 	 * @param string $jsCallback
 	 * @return AjaxCall
 	 */
-	public function jsJsonArray($url, $method="get", $params="{}", $jsCallback=NULL,$parameters=[]){
-		$parameters=\array_merge($parameters,["modelSelector"=>"#".$this->_identifier." tr.".$this->_modelClass,"url"=>$url,"method"=>$method,"params"=>$params,"jsCallback"=>$jsCallback]);
+	public function jsJsonArray($url, $method="get", $params="{}", $jsCallback=NULL, $parameters=[]) {
+		$parameters=\array_merge($parameters, ["modelSelector"=>"#".$this->_identifier." tr.".$this->_modelClass, "url"=>$url, "method"=>$method, "params"=>$params, "jsCallback"=>$jsCallback]);
 		return new AjaxCall("jsonArray", $parameters);
 	}
 
-	public function jsClear(){
+	public function jsClear() {
 		return "$('#{$this->identifier} tbody').find('._json').remove();";
 	}
 
-	public function clearOn(BaseHtml $element,$event, $stopPropagation=false, $preventDefault=false){
-		return $element->addEvent($event, $this->jsClear(),$stopPropagation,$preventDefault);
+	public function clearOn(BaseHtml $element, $event, $stopPropagation=false, $preventDefault=false) {
+		return $element->addEvent($event, $this->jsClear(), $stopPropagation, $preventDefault);
 	}
 
-	public function clearOnClick(BaseHtml $element,$stopPropagation=false, $preventDefault=false){
-		return $this->clearOn($element, "click",$stopPropagation,$preventDefault);
+	public function clearOnClick(BaseHtml $element, $stopPropagation=false, $preventDefault=false) {
+		return $this->clearOn($element, "click", $stopPropagation, $preventDefault);
 	}
 
-	public function jsonArrayOn(BaseHtml $element,$event,$url, $method="get", $params="{}", $jsCallback=NULL,$parameters=[]){
-		return $element->_addEvent($event, $this->jsJsonArray($url,$method,$params,$jsCallback,$parameters));
+	public function jsonArrayOn(BaseHtml $element, $event, $url, $method="get", $params="{}", $jsCallback=NULL, $parameters=[]) {
+		return $element->_addEvent($event, $this->jsJsonArray($url, $method, $params, $jsCallback, $parameters));
 	}
 
-	public function jsonArrayOnClick(BaseHtml $element,$url, $method="get", $params="{}", $jsCallback=NULL,$parameters=[]){
-		return $this->jsonArrayOn($element, "click", $url,$method,$params,$jsCallback,$parameters);
+	public function jsonArrayOnClick(BaseHtml $element, $url, $method="get", $params="{}", $jsCallback=NULL, $parameters=[]) {
+		return $this->jsonArrayOn($element, "click", $url, $method, $params, $jsCallback, $parameters);
 	}
 
 	/**
@@ -108,8 +108,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){
-		return parent::paginate($page, $total_rowcount,$items_per_page,null);
+	public function paginate($page, $total_rowcount, $items_per_page=10, $pages_visibles=null) {
+		return parent::paginate($page, $total_rowcount, $items_per_page, null);
 	}
 
 	public function setRowModelCallback($_rowModelCallback) {