phpMv / phpMv-UI

Ajax/semantic/html/elements/HtmlIconGroups.php

Doc Comments +3 added lines

@@ -13,6 +13,9 @@
  */
 class HtmlIconGroups extends HtmlSemCollection {
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $icons=array(), $size="") {
 		parent::__construct($identifier, "i", "icons");
 		$this->addItems($icons);

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
 			$icon=JArray::getValue($value, "icon", 0);
 			$size=JArray::getValue($value, "size", 1);
 		}
-		$iconO=new HtmlIcon("icon-" . $this->identifier, $icon);
+		$iconO=new HtmlIcon("icon-".$this->identifier, $icon);
 		if (isset($size)) {
 			$iconO->setSize($size);
 		}
@@ -33,7 +33,7 @@ 
 	}
 
 	protected function createCondition($value) {
-		return ($value instanceof HtmlIcon) === false;
+		return ($value instanceof HtmlIcon)===false;
 	}
 
 	public function getIcon($index) {

Ajax/semantic/html/elements/HtmlLabel.php

Doc Comments -1 removed lines

@@ -20,7 +20,6 @@
 
 	/**
 	 *
-	 * @param string $side
 	 * @return \Ajax\semantic\html\elements\HtmlLabel
 	 */
 	public function setPointing($value=Direction::NONE) {

Indentation +6 added lines, -6 removed lines

@@ -91,12 +91,12 @@
 	}
 
 	/**
-	*  Adds an image
-	* @param string $src
-	* @param string $alt
-	* @param boolean $before
-	* @return \Ajax\semantic\html\elements\html5\HtmlImg
-	*/
+	 *  Adds an image
+	 * @param string $src
+	 * @param string $alt
+	 * @param boolean $before
+	 * @return \Ajax\semantic\html\elements\html5\HtmlImg
+	 */
 	public function addImage($src, $alt="", $before=true) {
 		$img=new HtmlImg("image-" . $this->identifier, $src, $alt);
 		$img->setClass("");

Spacing +18 added lines, -18 removed lines

@@ -24,10 +24,10 @@ 
 	 * @return \Ajax\semantic\html\elements\HtmlLabel
 	 */
 	public function setPointing($value=Direction::NONE) {
-		if($value==="left" || $value==="right")
+		if ($value==="left" || $value==="right")
 			return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
 		else
-			return $this->addToPropertyCtrl("class", "pointing ".$value, Direction::getConstantValues("pointing",true));
+			return $this->addToPropertyCtrl("class", "pointing ".$value, Direction::getConstantValues("pointing", true));
 	}
 
 	/**
@@ -36,15 +36,15 @@ 
 	 * @return \Ajax\semantic\html\elements\HtmlLabel
 	 */
 	public function toCorner($side="left") {
-		return $this->addToPropertyCtrl("class", $side . " corner", array ("right corner","left corner" ));
+		return $this->addToPropertyCtrl("class", $side." corner", array("right corner", "left corner"));
 	}
 
-	public function setHorizontal(){
-		return $this->addToPropertyCtrl("class", "hozizontal",array("horizontal"));
+	public function setHorizontal() {
+		return $this->addToPropertyCtrl("class", "hozizontal", array("horizontal"));
 	}
 
-	public function setFloating(){
-		return $this->addToPropertyCtrl("class", "floating",array("floating"));
+	public function setFloating() {
+		return $this->addToPropertyCtrl("class", "floating", array("floating"));
 	}
 
 	/**
@@ -55,9 +55,9 @@ 
 		return $this->addToProperty("class", "tag");
 	}
 
-	public function setEmpty(){
+	public function setEmpty() {
 		$this->content=NULL;
-		return $this->addToPropertyCtrl("class", "empty",array("empty"));
+		return $this->addToPropertyCtrl("class", "empty", array("empty"));
 	}
 
 	public function setBasic() {
@@ -73,7 +73,7 @@ 
 	 */
 	public function addEmphasisImage($src, $alt="", $before=true) {
 		$this->addToProperty("class", "image");
-		return $this->addImage($src,$alt,$before);
+		return $this->addImage($src, $alt, $before);
 	}
 
 	/**
@@ -84,7 +84,7 @@ 
 	 * @return \Ajax\semantic\html\elements\html5\HtmlImg
 	 */
 	public function addAvatarImage($src, $alt="", $before=true) {
-		$img=$this->addImage($src,$alt,$before);
+		$img=$this->addImage($src, $alt, $before);
 		$img->setClass("ui image");
 		$img->asAvatar();
 		return $img;
@@ -98,7 +98,7 @@ 
 	* @return \Ajax\semantic\html\elements\html5\HtmlImg
 	*/
 	public function addImage($src, $alt="", $before=true) {
-		$img=new HtmlImg("image-" . $this->identifier, $src, $alt);
+		$img=new HtmlImg("image-".$this->identifier, $src, $alt);
 		$img->setClass("");
 		$this->addContent($img, $before);
 		return $img;
@@ -110,21 +110,21 @@ 
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
 	public function addDetail($detail) {
-		$div=new HtmlSemDoubleElement("detail-" . $this->identifier, $this->tagName,"detail");
+		$div=new HtmlSemDoubleElement("detail-".$this->identifier, $this->tagName, "detail");
 		$div->setContent($detail);
 		$this->addContent($div);
 		return $div;
 	}
 
 	public function asRibbon($direction=Direction::NONE) {
-		return $this->addToPropertyCtrl("class", $direction." ribbon", array ("ribbon","right ribbon","left ribbon" ));
+		return $this->addToPropertyCtrl("class", $direction." ribbon", array("ribbon", "right ribbon", "left ribbon"));
 	}
 
-	public function setAttached($side=Side::TOP,$direction=Direction::NONE){
-		if($direction!==Direction::NONE)
-			return $this->addToPropertyCtrl("class", $side." ".$direction." attached",Side::getConstantValues($direction." attached"));
+	public function setAttached($side=Side::TOP, $direction=Direction::NONE) {
+		if ($direction!==Direction::NONE)
+			return $this->addToPropertyCtrl("class", $side." ".$direction." attached", Side::getConstantValues($direction." attached"));
 		else
-			return $this->addToPropertyCtrl("class", $side." attached",Side::getConstantValues("attached"));
+			return $this->addToPropertyCtrl("class", $side." attached", Side::getConstantValues("attached"));
 	}
 
 	public static function ribbon($identifier, $caption) {

Braces +13 added lines, -10 removed lines

@@ -14,8 +14,9 @@ 
 	public function __construct($identifier, $caption="", $icon=NULL, $tagName="div") {
 		parent::__construct($identifier, $tagName, "ui label");
 		$this->content=$caption;
-		if (isset($icon)===true)
-			$this->addIcon($icon);
+		if (isset($icon)===true) {
+					$this->addIcon($icon);
+		}
 	}
 
 	/**
@@ -24,10 +25,11 @@ 
 	 * @return \Ajax\semantic\html\elements\HtmlLabel
 	 */
 	public function setPointing($value=Direction::NONE) {
-		if($value==="left" || $value==="right")
-			return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
-		else
-			return $this->addToPropertyCtrl("class", "pointing ".$value, Direction::getConstantValues("pointing",true));
+		if($value==="left" || $value==="right") {
+					return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
+		} else {
+					return $this->addToPropertyCtrl("class", "pointing ".$value, Direction::getConstantValues("pointing",true));
+		}
 	}
 
 	/**
@@ -121,10 +123,11 @@ 
 	}
 
 	public function setAttached($side=Side::TOP,$direction=Direction::NONE){
-		if($direction!==Direction::NONE)
-			return $this->addToPropertyCtrl("class", $side." ".$direction." attached",Side::getConstantValues($direction." attached"));
-		else
-			return $this->addToPropertyCtrl("class", $side." attached",Side::getConstantValues("attached"));
+		if($direction!==Direction::NONE) {
+					return $this->addToPropertyCtrl("class", $side." ".$direction." attached",Side::getConstantValues($direction." attached"));
+		} else {
+					return $this->addToPropertyCtrl("class", $side." attached",Side::getConstantValues("attached"));
+		}
 	}
 
 	public static function ribbon($identifier, $caption) {

Ajax/semantic/html/elements/HtmlList.php

Doc Comments +6 added lines

@@ -10,6 +10,9 @@ 
 class HtmlList extends HtmlSemCollection {
 	protected $_hasCheckedList;
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $items=array()) {
 		parent::__construct($identifier, "div", "ui list");
 		$this->addItems($items);
@@ -22,6 +25,9 @@ 
 		return $item;
 	}
 
+	/**
+	 * @param integer $niveau
+	 */
 	public function addHeader($niveau, $content) {
 		$header=new HtmlHeader("header-" . $this->identifier, $niveau, $content, "page");
 		$this->wrap($header);

Spacing +14 added lines, -14 removed lines

@@ -18,17 +18,17 @@ 
 
 	protected function createItem($value) {
 		$count=$this->count();
-		$item=new HtmlListItem("item-" . $this->identifier . "-" . $count, $value);
+		$item=new HtmlListItem("item-".$this->identifier."-".$count, $value);
 		return $item;
 	}
 
 	public function addHeader($niveau, $content) {
-		$header=new HtmlHeader("header-" . $this->identifier, $niveau, $content, "page");
+		$header=new HtmlHeader("header-".$this->identifier, $niveau, $content, "page");
 		$this->wrap($header);
 		return $header;
 	}
 
-	public function getItemPart($index,$partName="header"){
+	public function getItemPart($index, $partName="header") {
 		return $this->getItem($index)->getPart($partName);
 	}
 
@@ -37,7 +37,7 @@ 
 	}
 
 	public function asLink() {
-		$this->addToPropertyCtrl("class", "link", array ("link" ));
+		$this->addToPropertyCtrl("class", "link", array("link"));
 		return $this->contentAs("a");
 	}
 
@@ -60,9 +60,9 @@ 
 	}
 
 	public function run(JsUtils $js) {
-		if ($this->_hasCheckedList === true) {
-			$jsCode=include dirname(__FILE__) . '/../../components/jsTemplates/tplCheckedList.php';
-			$jsCode=\str_replace("%identifier%", "#" . $this->identifier, $jsCode);
+		if ($this->_hasCheckedList===true) {
+			$jsCode=include dirname(__FILE__).'/../../components/jsTemplates/tplCheckedList.php';
+			$jsCode=\str_replace("%identifier%", "#".$this->identifier, $jsCode);
 			$this->executeOnRun($jsCode);
 		}
 		return parent::run($js);
@@ -86,24 +86,24 @@ 
 
 	public function addCheckedList($items=array(), $masterItem=NULL, $values=array()) {
 		$count=$this->count();
-		$identifier=$this->identifier . "-" . $count;
+		$identifier=$this->identifier."-".$count;
 		if (isset($masterItem)) {
-			$masterO=new HtmlFormCheckbox("master-" . $identifier, $masterItem);
+			$masterO=new HtmlFormCheckbox("master-".$identifier, $masterItem);
 			$masterO->getField()->addToProperty("class", "master");
 			$masterO->setClass("item");
 			$this->addItem($masterO);
 		}
-		$fields=array ();
+		$fields=array();
 		$i=0;
-		foreach ( $items as $val => $caption ) {
-			$itemO=new HtmlFormCheckbox($identifier . "-" . $i++, $caption, $val, "child");
-			if (\array_search($val, $values) !== false) {
+		foreach ($items as $val => $caption) {
+			$itemO=new HtmlFormCheckbox($identifier."-".$i++, $caption, $val, "child");
+			if (\array_search($val, $values)!==false) {
 				$itemO->getField()->getField()->setProperty("checked", "");
 			}
 			$itemO->setClass("item");
 			$fields[]=$itemO;
 		}
-		if (isset($masterO) === true) {
+		if (isset($masterO)===true) {
 			$list=new HtmlList("", $fields);
 			$list->setClass("list");
 			$masterO->addContent($list);

Ajax/semantic/html/elements/HtmlStep.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ajax\semantic\html\base\HtmlSemCollection;
 use Ajax\semantic\html\content\HtmlStepItem;
 use Ajax\JsUtils;
-
 use Ajax\common\html\HtmlDoubleElement;
 use Ajax\semantic\html\base\constants\Side;
 

Spacing +23 added lines, -23 removed lines

@@ -9,11 +9,11 @@ 
 use Ajax\common\html\HtmlDoubleElement;
 use Ajax\semantic\html\base\constants\Side;
 
-class HtmlStep extends HtmlSemCollection{
+class HtmlStep extends HtmlSemCollection {
 	protected $_activeStep;
 
-	public function __construct( $identifier,$steps=array()){
-		parent::__construct( $identifier,"div", "ui steps");
+	public function __construct($identifier, $steps=array()) {
+		parent::__construct($identifier, "div", "ui steps");
 		$this->addItems($steps);
 	}
 
@@ -23,7 +23,7 @@ 
 	 * @see \Ajax\common\html\html5\HtmlCollection::createItem()
 	 */
 	protected function createItem($value) {
-		$itemO=new HtmlStepItem("item-".\sizeof($this->content),$value);
+		$itemO=new HtmlStepItem("item-".\sizeof($this->content), $value);
 		return $itemO;
 	}
 
@@ -31,50 +31,50 @@ 
 	 * @param string|array $step
 	 * @return HtmlStepItem
 	 */
-	public function addStep($step){
+	public function addStep($step) {
 		return $this->addItem($step);
 	}
 
-	public function setOrdered(){
+	public function setOrdered() {
 		return $this->addToProperty("class", "ordered");
 	}
 
-	public function isOrdered(){
+	public function isOrdered() {
 		return $this->propertyContains("class", "ordered");
 	}
 
-	public function setVertical(){
+	public function setVertical() {
 		return $this->addToProperty("class", "vertical");
 	}
 
-	protected function defineActiveStep(){
+	protected function defineActiveStep() {
 		$activestep=$this->_activeStep;
 		$count=$this->count();
-		if($this->isOrdered()){
-			for($i=0;$i<$count;$i++){
+		if ($this->isOrdered()) {
+			for ($i=0; $i<$count; $i++) {
 				$step=$this->content[$i];
 				$step->removeStatus();
-				if($i<$activestep)
+				if ($i<$activestep)
 					$step->setCompleted();
 				elseif ($i===$activestep)
 					$step->setActive();
 				else
 					$step->setDisabled();
 			}
-		}else{
-			foreach ($this->content as $step){
+		} else {
+			foreach ($this->content as $step) {
 				$step->removeStatus();
 			}
-			if($activestep<$count)
+			if ($activestep<$count)
 				$this->content[$activestep]->setActive();
 		}
 		return $this;
 	}
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		if(isset($this->_activeStep)===true && \is_numeric($this->_activeStep))
+		if (isset($this->_activeStep)===true && \is_numeric($this->_activeStep))
 			$this->defineActiveStep();
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 
 	public function setActiveStep($_activeStep) {
@@ -82,15 +82,15 @@ 
 		return $this;
 	}
 
-	public function setAttached($side="",HtmlDoubleElement $toElement=NULL){
-		if(isset($toElement)){
-			$toElement->addToPropertyCtrl("class", "attached",array("attached"));
+	public function setAttached($side="", HtmlDoubleElement $toElement=NULL) {
+		if (isset($toElement)) {
+			$toElement->addToPropertyCtrl("class", "attached", array("attached"));
 		}
-		return $this->addToPropertyCtrl("class", $side." attached",Side::getConstantValues("attached"));
+		return $this->addToPropertyCtrl("class", $side." attached", Side::getConstantValues("attached"));
 	}
 
-	public function asLink(){
-		foreach ($this->content as $step){
+	public function asLink() {
+		foreach ($this->content as $step) {
 			$step->asLink();
 		}
 		return $this;

Braces +14 added lines, -11 removed lines

@@ -54,26 +54,29 @@
 			for($i=0;$i<$count;$i++){
 				$step=$this->content[$i];
 				$step->removeStatus();
-				if($i<$activestep)
-					$step->setCompleted();
-				elseif ($i===$activestep)
-					$step->setActive();
-				else
-					$step->setDisabled();
+				if($i<$activestep) {
+									$step->setCompleted();
+				} elseif ($i===$activestep) {
+									$step->setActive();
+				} else {
+									$step->setDisabled();
+				}
 			}
-		}else{
+		} else{
 			foreach ($this->content as $step){
 				$step->removeStatus();
 			}
-			if($activestep<$count)
-				$this->content[$activestep]->setActive();
+			if($activestep<$count) {
+							$this->content[$activestep]->setActive();
+			}
 		}
 		return $this;
 	}
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		if(isset($this->_activeStep)===true && \is_numeric($this->_activeStep))
-			$this->defineActiveStep();
+		if(isset($this->_activeStep)===true && \is_numeric($this->_activeStep)) {
+					$this->defineActiveStep();
+		}
 		return parent::compile($js,$view);
 	}
 

Ajax/semantic/html/modules/HtmlAccordion.php

Doc Comments +6 added lines

@@ -10,6 +10,9 @@ 
 
 	protected $params=array();
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct( $identifier, $tagName="div", $baseClass="ui"){
 		parent::__construct( $identifier, "div", "ui accordion");
 	}
@@ -29,6 +32,9 @@ 
 		return ($value instanceof HtmlAccordionItem)===false;
 	}
 
+	/**
+	 * @param string $title
+	 */
 	public function addPanel($title,$content){
 		return $this->addItem([$title,$content]);
 	}

Spacing +19 added lines, -19 removed lines

@@ -6,31 +6,31 @@ 
 use Ajax\semantic\html\content\HtmlAccordionItem;
 use Ajax\JsUtils;
 
-class HtmlAccordion extends HtmlSemCollection{
+class HtmlAccordion extends HtmlSemCollection {
 
 	protected $params=array();
 
-	public function __construct( $identifier, $tagName="div", $baseClass="ui"){
-		parent::__construct( $identifier, "div", "ui accordion");
+	public function __construct($identifier, $tagName="div", $baseClass="ui") {
+		parent::__construct($identifier, "div", "ui accordion");
 	}
 
 
-	protected function createItem($value){
+	protected function createItem($value) {
 		$count=$this->count();
 		$title=$value;
 		$content=NULL;
-		if(\is_array($value)){
-			$title=@$value[0];$content=@$value[1];
+		if (\is_array($value)) {
+			$title=@$value[0]; $content=@$value[1];
 		}
-		return new HtmlAccordionItem("item-".$this->identifier."-".$count, $title,$content);
+		return new HtmlAccordionItem("item-".$this->identifier."-".$count, $title, $content);
 	}
 
-	protected function createCondition($value){
+	protected function createCondition($value) {
 		return ($value instanceof HtmlAccordionItem)===false;
 	}
 
-	public function addPanel($title,$content){
-		return $this->addItem([$title,$content]);
+	public function addPanel($title, $content) {
+		return $this->addItem([$title, $content]);
 	}
 
 	/**
@@ -42,8 +42,8 @@ 
 	 * @param string $action a Phalcon action
 	 * @param array $params
 	 */
-	public function forwardPanel(JsUtils $js,$title,$initialController,$controller,$action,$params=array()){
-		return $this->addPanel($title, $js->forward($initialController, $controller, $action,$params));
+	public function forwardPanel(JsUtils $js, $title, $initialController, $controller, $action, $params=array()) {
+		return $this->addPanel($title, $js->forward($initialController, $controller, $action, $params));
 	}
 
 	/**
@@ -54,30 +54,30 @@ 
 	 * @param string $viewName
 	 * @param $params The parameters to pass to the view
 	 */
-	public function renderViewPanel(JsUtils $js,$title,$initialController, $viewName, $params=array()) {
-		return $this->addPanel($title, $js->renderContent($initialController, $viewName,$params));
+	public function renderViewPanel(JsUtils $js, $title, $initialController, $viewName, $params=array()) {
+		return $this->addPanel($title, $js->renderContent($initialController, $viewName, $params));
 	}
 	/*
 	 * (non-PHPdoc)
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if(isset($this->_bsComponent)===false)
-			$this->_bsComponent=$js->semantic()->accordion("#".$this->identifier,$this->params);
+		if (isset($this->_bsComponent)===false)
+			$this->_bsComponent=$js->semantic()->accordion("#".$this->identifier, $this->params);
 			$this->addEventsOnRun($js);
 			return $this->_bsComponent;
 	}
 
-	public function setStyled(){
+	public function setStyled() {
 		return $this->addToProperty("class", "styled");
 	}
 
-	public function setActive($index){
+	public function setActive($index) {
 		$this->getItem($index)->setActive(true);
 		return $this;
 	}
 
-	public function setExclusive($value){
+	public function setExclusive($value) {
 		$this->params["exclusive"]=$value;
 	}
 }
\ No newline at end of file

Braces +3 added lines, -2 removed lines

@@ -62,8 +62,9 @@
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if(isset($this->_bsComponent)===false)
-			$this->_bsComponent=$js->semantic()->accordion("#".$this->identifier,$this->params);
+		if(isset($this->_bsComponent)===false) {
+					$this->_bsComponent=$js->semantic()->accordion("#".$this->identifier,$this->params);
+		}
 			$this->addEventsOnRun($js);
 			return $this->_bsComponent;
 	}

Ajax/semantic/html/modules/HtmlDropdown.php

Doc Comments +7 added lines, -1 removed lines

@@ -21,6 +21,9 @@ 
 	protected $_params=array("action"=>"nothing","on"=>"hover");
 	protected $input;
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $value="", $items=array()) {
 		parent::__construct($identifier, "div");
 		$this->_template=include dirname(__FILE__).'/../templates/tplDropdown.php';
@@ -47,7 +50,7 @@ 
 	/**
 	 * Insert an item at a position
 	 * @param mixed $item
-	 * @param number $position
+	 * @param integer $position
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem|unknown
 	 */
 	public function insertItem($item,$position=0){
@@ -205,6 +208,9 @@ 
 		return $this->addToPropertyCtrl("class", "compact", array("compact"));
 	}
 
+	/**
+	 * @param string $action
+	 */
 	public function setAction($action){
 		$this->_params["action"]=$action;
 	}

Spacing +60 added lines, -60 removed lines

@@ -17,31 +17,31 @@ 
 	}
 	protected $mClass="menu";
 	protected $mTagName="div";
-	protected $items=array ();
-	protected $_params=array("action"=>"nothing","on"=>"hover");
+	protected $items=array();
+	protected $_params=array("action"=>"nothing", "on"=>"hover");
 	protected $input;
 
 	public function __construct($identifier, $value="", $items=array()) {
 		parent::__construct($identifier, "div");
 		$this->_template=include dirname(__FILE__).'/../templates/tplDropdown.php';
 		$this->setProperty("class", "ui dropdown");
-		$content=new HtmlSemDoubleElement("text-".$this->identifier,"div");
+		$content=new HtmlSemDoubleElement("text-".$this->identifier, "div");
 		$content->setClass("text");
 		$content->setContent($value);
-		$content->wrap("",new HtmlIcon("", "dropdown"));
+		$content->wrap("", new HtmlIcon("", "dropdown"));
 		$this->content=array($content);
 		$this->tagName="div";
 		$this->addItems($items);
 	}
 
-	public function addItem($item,$value=NULL,$image=NULL){
-		$itemO=$this->beforeAddItem($item,$value,$image);
+	public function addItem($item, $value=NULL, $image=NULL) {
+		$itemO=$this->beforeAddItem($item, $value, $image);
 		$this->items[]=$itemO;
 		return $itemO;
 	}
 
-	public function addIcon($icon,$before=true,$labeled=false){
-		$this->addIconP($icon,$before,$labeled);
+	public function addIcon($icon, $before=true, $labeled=false) {
+		$this->addIconP($icon, $before, $labeled);
 		return $this->getElementById("text-".$this->identifier, $this->content)->setWrapAfter("");
 	}
 	/**
@@ -50,25 +50,25 @@ 
 	 * @param number $position
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem|unknown
 	 */
-	public function insertItem($item,$position=0){
+	public function insertItem($item, $position=0) {
 		$itemO=$this->beforeAddItem($item);
-		 $start = array_slice($this->items, 0, $position);
-		 $end = array_slice($this->items, $position);
-		 $start[] = $item;
+		 $start=array_slice($this->items, 0, $position);
+		 $end=array_slice($this->items, $position);
+		 $start[]=$item;
 		 $this->items=array_merge($start, $end);
 		 return $itemO;
 	}
 
-	protected function beforeAddItem($item,$value=NULL,$image=NULL){
+	protected function beforeAddItem($item, $value=NULL, $image=NULL) {
 		$itemO=$item;
-		if(\is_array($item)){
+		if (\is_array($item)) {
 			$value=JArray::getValue($item, "value", 1);
 			$image=JArray::getValue($item, "image", 2);
 			$item=JArray::getValue($item, "item", 0);
 		}
-		if(!$item instanceof HtmlDropdownItem){
-			$itemO=new HtmlDropdownItem("dd-item-".$this->identifier."-".\sizeof($this->items),$item,$value,$image);
-		}elseif($itemO instanceof HtmlDropdownItem){
+		if (!$item instanceof HtmlDropdownItem) {
+			$itemO=new HtmlDropdownItem("dd-item-".$this->identifier."-".\sizeof($this->items), $item, $value, $image);
+		}elseif ($itemO instanceof HtmlDropdownItem) {
 			$this->addToProperty("class", "vertical");
 		}
 		return $itemO;
@@ -81,76 +81,76 @@ 
 		$this->addItem($function($object));
 	}
 
-	public function addInput($name){
-		if(!isset($name))
+	public function addInput($name) {
+		if (!isset($name))
 			$name="input-".$this->identifier;
 		$this->setAction("activate");
-		$this->input=new HtmlInput($name,"hidden");
+		$this->input=new HtmlInput($name, "hidden");
 	}
 
-	public function addItems($items){
-		if(JArray::isAssociative($items)){
-			foreach ($items as $k=>$v){
+	public function addItems($items) {
+		if (JArray::isAssociative($items)) {
+			foreach ($items as $k=>$v) {
 				$this->addItem($v)->setData($k);
 			}
-		}else{
-			foreach ($items as $item){
+		} else {
+			foreach ($items as $item) {
 				$this->addItem($item);
 			}
 		}
 	}
 
-	public function count(){
+	public function count() {
 		return \sizeof($this->items);
 	}
 	/**
 	 * @param boolean $dropdown
 	 */
-	public function asDropdown($dropdown){
-		if($dropdown===false){
+	public function asDropdown($dropdown) {
+		if ($dropdown===false) {
 			$this->_template=include dirname(__FILE__).'/../templates/tplDropdownMenu.php';
 			$dropdown="menu";
-		}else{
+		} else {
 			$dropdown="dropdown";
 			$this->mClass="menu";
 		}
-		return $this->addToPropertyCtrl("class", $dropdown,array("menu","dropdown"));
+		return $this->addToPropertyCtrl("class", $dropdown, array("menu", "dropdown"));
 	}
 
-	public function setVertical(){
-		return $this->addToPropertyCtrl("class", "vertical",array("vertical"));
+	public function setVertical() {
+		return $this->addToPropertyCtrl("class", "vertical", array("vertical"));
 	}
 
-	public function setSimple(){
-		return $this->addToPropertyCtrl("class", "simple",array("simple"));
+	public function setSimple() {
+		return $this->addToPropertyCtrl("class", "simple", array("simple"));
 	}
 
-	public function asButton($floating=false){
-		if($floating)
+	public function asButton($floating=false) {
+		if ($floating)
 			$this->addToProperty("class", "floating");
 		return $this->addToProperty("class", "button");
 	}
 
-	public function asSelect($name=NULL,$multiple=false,$selection=true){
-		if(isset($name))
+	public function asSelect($name=NULL, $multiple=false, $selection=true) {
+		if (isset($name))
 			$this->addInput($name);
-		if($multiple)
+		if ($multiple)
 			$this->addToProperty("class", "multiple");
 		if ($selection)
-			$this->addToPropertyCtrl("class", "selection",array("selection"));
+			$this->addToPropertyCtrl("class", "selection", array("selection"));
 		return $this;
 	}
 
-	public function asSearch($name=NULL,$multiple=false,$selection=false){
-		$this->asSelect($name,$multiple,$selection);
+	public function asSearch($name=NULL, $multiple=false, $selection=false) {
+		$this->asSelect($name, $multiple, $selection);
 		return $this->addToProperty("class", "search");
 	}
 
-	public function setSelect($name=NULL,$multiple=false){
-		if(!isset($name))
+	public function setSelect($name=NULL, $multiple=false) {
+		if (!isset($name))
 			$name="select-".$this->identifier;
 		$this->input=null;
-		if($multiple){
+		if ($multiple) {
 			$this->setProperty("multiple", true);
 			$this->addToProperty("class", "multiple");
 		}
@@ -158,32 +158,32 @@ 
 		$this->tagName="select";
 		$this->setProperty("name", $name);
 		$this->content=null;
-		foreach ($this->items as $item){
+		foreach ($this->items as $item) {
 			$item->asOption();
 		}
 		return $this;
 	}
 
-	public function asSubmenu($pointing=NULL){
+	public function asSubmenu($pointing=NULL) {
 		$this->setClass("ui dropdown link item");
-		if(isset($pointing)){
+		if (isset($pointing)) {
 			$this->setPointing($pointing);
 		}
 		return $this;
 	}
 
-	public function setPointing($value=Direction::NONE){
-		return $this->addToPropertyCtrl("class", $value." pointing",Direction::getConstantValues("pointing"));
+	public function setPointing($value=Direction::NONE) {
+		return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
 	}
 
-	public function setValue($value){
-		if(isset($this->input)){
+	public function setValue($value) {
+		if (isset($this->input)) {
 			$this->input->setProperty("value", $value);
-		}else{
+		} else {
 			$this->setProperty("value", $value);
 		}
 		$textElement=$this->getElementById("text-".$this->identifier, $this->content);
-		if(isset($textElement))
+		if (isset($textElement))
 			$textElement->setContent($value);
 		return $this;
 	}
@@ -193,23 +193,23 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if($this->propertyContains("class", "simple")===false){
-			if(isset($this->_bsComponent)===false)
-				$this->_bsComponent=$js->semantic()->dropdown("#".$this->identifier,$this->_params);
+		if ($this->propertyContains("class", "simple")===false) {
+			if (isset($this->_bsComponent)===false)
+				$this->_bsComponent=$js->semantic()->dropdown("#".$this->identifier, $this->_params);
 			$this->addEventsOnRun($js);
 			return $this->_bsComponent;
 		}
 	}
 
-	public function setCompact(){
+	public function setCompact() {
 		return $this->addToPropertyCtrl("class", "compact", array("compact"));
 	}
 
-	public function setAction($action){
+	public function setAction($action) {
 		$this->_params["action"]=$action;
 	}
 
-	public function setFullTextSearch($value){
+	public function setFullTextSearch($value) {
 		$this->_params["fullTextSearch"]=$value;
 	}
 }
\ No newline at end of file

Braces +28 added lines, -20 removed lines

@@ -68,7 +68,7 @@ 
 		}
 		if(!$item instanceof HtmlDropdownItem){
 			$itemO=new HtmlDropdownItem("dd-item-".$this->identifier."-".\sizeof($this->items),$item,$value,$image);
-		}elseif($itemO instanceof HtmlDropdownItem){
+		} elseif($itemO instanceof HtmlDropdownItem){
 			$this->addToProperty("class", "vertical");
 		}
 		return $itemO;
@@ -82,8 +82,9 @@ 
 	}
 
 	public function addInput($name){
-		if(!isset($name))
-			$name="input-".$this->identifier;
+		if(!isset($name)) {
+					$name="input-".$this->identifier;
+		}
 		$this->setAction("activate");
 		$this->input=new HtmlInput($name,"hidden");
 	}
@@ -93,7 +94,7 @@ 
 			foreach ($items as $k=>$v){
 				$this->addItem($v)->setData($k);
 			}
-		}else{
+		} else{
 			foreach ($items as $item){
 				$this->addItem($item);
 			}
@@ -110,7 +111,7 @@ 
 		if($dropdown===false){
 			$this->_template=include dirname(__FILE__).'/../templates/tplDropdownMenu.php';
 			$dropdown="menu";
-		}else{
+		} else{
 			$dropdown="dropdown";
 			$this->mClass="menu";
 		}
@@ -126,18 +127,22 @@ 
 	}
 
 	public function asButton($floating=false){
-		if($floating)
-			$this->addToProperty("class", "floating");
+		if($floating) {
+					$this->addToProperty("class", "floating");
+		}
 		return $this->addToProperty("class", "button");
 	}
 
 	public function asSelect($name=NULL,$multiple=false,$selection=true){
-		if(isset($name))
-			$this->addInput($name);
-		if($multiple)
-			$this->addToProperty("class", "multiple");
-		if ($selection)
-			$this->addToPropertyCtrl("class", "selection",array("selection"));
+		if(isset($name)) {
+					$this->addInput($name);
+		}
+		if($multiple) {
+					$this->addToProperty("class", "multiple");
+		}
+		if ($selection) {
+					$this->addToPropertyCtrl("class", "selection",array("selection"));
+		}
 		return $this;
 	}
 
@@ -147,8 +152,9 @@ 
 	}
 
 	public function setSelect($name=NULL,$multiple=false){
-		if(!isset($name))
-			$name="select-".$this->identifier;
+		if(!isset($name)) {
+					$name="select-".$this->identifier;
+		}
 		$this->input=null;
 		if($multiple){
 			$this->setProperty("multiple", true);
@@ -179,12 +185,13 @@ 
 	public function setValue($value){
 		if(isset($this->input)){
 			$this->input->setProperty("value", $value);
-		}else{
+		} else{
 			$this->setProperty("value", $value);
 		}
 		$textElement=$this->getElementById("text-".$this->identifier, $this->content);
-		if(isset($textElement))
-			$textElement->setContent($value);
+		if(isset($textElement)) {
+					$textElement->setContent($value);
+		}
 		return $this;
 	}
 
@@ -194,8 +201,9 @@ 
 	 */
 	public function run(JsUtils $js) {
 		if($this->propertyContains("class", "simple")===false){
-			if(isset($this->_bsComponent)===false)
-				$this->_bsComponent=$js->semantic()->dropdown("#".$this->identifier,$this->_params);
+			if(isset($this->_bsComponent)===false) {
+							$this->_bsComponent=$js->semantic()->dropdown("#".$this->identifier,$this->_params);
+			}
 			$this->addEventsOnRun($js);
 			return $this->_bsComponent;
 		}

Ajax/semantic/html/modules/HtmlModal.php

Doc Comments +9 added lines, -1 removed lines

@@ -14,6 +14,9 @@ 
 	protected $_params=array();
 	protected $_paramParts=array();
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $header="", $content="", $actions=array()) {
 		parent::__construct($identifier, "div","ui modal");
 		if(isset($header)){
@@ -27,6 +30,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $value
+	 */
 	public function setHeader($value) {
 		$this->content["header"]=new HtmlSemDoubleElement("header-" . $this->identifier, "a", "header", $value);
 		return $this;
@@ -103,6 +109,9 @@ 
 		return $this->addElementInPart(new HtmlSemDoubleElement($part."-" . $this->identifier, "div", $uiClass, $content), $part);
 	}
 
+	/**
+	 * @param BaseHtml $element
+	 */
 	private function addElementInPart($element,$part) {
 		$this->content[$part]->addContent($element);
 		return $element;
@@ -145,7 +154,6 @@ 
 	/**
 	 * render the content of $controller::$action and set the response to the modal content
 	 * @param JsUtils $js
-	 * @param string $title The panel title
 	 * @param Controller $initialControllerInstance
 	 * @param string $controllerName the controller name
 	 * @param string $actionName the action name

Spacing +51 added lines, -51 removed lines

@@ -15,120 +15,120 @@ 
 	protected $_paramParts=array();
 
 	public function __construct($identifier, $header="", $content="", $actions=array()) {
-		parent::__construct($identifier, "div","ui modal");
-		if(isset($header)){
+		parent::__construct($identifier, "div", "ui modal");
+		if (isset($header)) {
 			$this->setHeader($header);
 		}
-		if(isset($content)){
+		if (isset($content)) {
 			$this->setContent($content);
 		}
-		if(isset($actions)){
+		if (isset($actions)) {
 			$this->setActions($actions);
 		}
 	}
 
 	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;
 	}
 
 	public function setActions($actions) {
-		$this->content["actions"]=new HtmlSemDoubleElement("content-" . $this->identifier, "div", "actions");
-		if(\is_array($actions)){
-			foreach ($actions as $action){
+		$this->content["actions"]=new HtmlSemDoubleElement("content-".$this->identifier, "div", "actions");
+		if (\is_array($actions)) {
+			foreach ($actions as $action) {
 				$this->addAction($action);
 			}
 		}
-		else{
+		else {
 			$this->addAction($actions);
 		}
 		return $this;
 	}
 
-	public function addAction($action){
-		if(!$action instanceof BaseHtml){
+	public function addAction($action) {
+		if (!$action instanceof BaseHtml) {
 			$class="";
-			if(\array_search($action, ["Okay","Yes"])!==false){
+			if (\array_search($action, ["Okay", "Yes"])!==false) {
 				$class="approve";
 			}
-			if(\array_search($action, ["Cancel","No"])!==false){
+			if (\array_search($action, ["Cancel", "No"])!==false) {
 				$class="cancel";
 			}
-			$action=new HtmlButton("action-".$this->identifier,$action);
-			if($class!=="")
+			$action=new HtmlButton("action-".$this->identifier, $action);
+			if ($class!=="")
 				$action->addToProperty("class", $class);
 		}
 		return $this->addElementInPart($action, "actions");
 	}
 
-	public function addContent($content,$before=false){
-		$this->content["content"]->addContent($content,$before);
+	public function addContent($content, $before=false) {
+		$this->content["content"]->addContent($content, $before);
 		return $this;
 	}
 
-	public function addImageContent($image,$description=NULL){
+	public function addImageContent($image, $description=NULL) {
 		$content=$this->content["content"];
-		if(isset($description)){
-			$description=new HtmlSemDoubleElement("description-".$this->identifier,"div","description",$description);
-			$content->addContent($description,true);
+		if (isset($description)) {
+			$description=new HtmlSemDoubleElement("description-".$this->identifier, "div", "description", $description);
+			$content->addContent($description, true);
 		}
-		if($image!==""){
-			$img=new HtmlImage("image-".$this->identifier,$image,"","medium");
-			$content->addContent($img,true);
-			$content->addToProperty("class","image");
+		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){
+	public function addIconContent($icon, $description=NULL) {
 		$content=$this->content["content"];
-		if(isset($description)){
-			$description=new HtmlSemDoubleElement("description-".$this->identifier,"div","description",$description);
-			$content->addContent($description,true);
+		if (isset($description)) {
+			$description=new HtmlSemDoubleElement("description-".$this->identifier, "div", "description", $description);
+			$content->addContent($description, true);
 		}
-		if($icon!==""){
-			$img=new HtmlIcon("image-".$this->identifier,$icon);
-			$content->addContent($img,true);
-			$content->addToProperty("class","image");
+		if ($icon!=="") {
+			$img=new HtmlIcon("image-".$this->identifier, $icon);
+			$content->addContent($img, true);
+			$content->addToProperty("class", "image");
 		}
 		return $this;
 	}
 
-	private function addContentInPart($content,$uiClass,$part) {
-		return $this->addElementInPart(new HtmlSemDoubleElement($part."-" . $this->identifier, "div", $uiClass, $content), $part);
+	private function addContentInPart($content, $uiClass, $part) {
+		return $this->addElementInPart(new HtmlSemDoubleElement($part."-".$this->identifier, "div", $uiClass, $content), $part);
 	}
 
-	private function addElementInPart($element,$part) {
+	private function addElementInPart($element, $part) {
 		$this->content[$part]->addContent($element);
 		return $element;
 	}
 
-	public function showDimmer($value){
-		$value=$value?"show":"hide";
+	public function showDimmer($value) {
+		$value=$value ? "show" : "hide";
 		$this->_paramParts[]=["'".$value." dimmer'"];
 		return $this;
 	}
 
-	public function setInverted(){
+	public function setInverted() {
 		$this->_params["inverted"]=true;
 		return $this;
 	}
 
-	public function setBasic(){
+	public function setBasic() {
 		return $this->addToProperty("class", "basic");
 	}
 
-	public function setActive(){
+	public function setActive() {
 		return $this->addToProperty("class", "active");
 	}
 
-	public function setTransition($value){
-		$this->_paramParts[]=["'setting'","'transition'","'".$value."'"];
+	public function setTransition($value) {
+		$this->_paramParts[]=["'setting'", "'transition'", "'".$value."'"];
 	}
 
 	/**
@@ -138,8 +138,8 @@ 
 	 * @param string $viewName
 	 * @param $params The parameters to pass to the view
 	 */
-	public function renderView(JsUtils $js,$initialController,$viewName, $params=array()) {
-		return $this->setContent($js->renderContent($initialController, $viewName,$params));
+	public function renderView(JsUtils $js, $initialController, $viewName, $params=array()) {
+		return $this->setContent($js->renderContent($initialController, $viewName, $params));
 	}
 
 	/**
@@ -150,8 +150,8 @@ 
 	 * @param string $controllerName the controller name
 	 * @param string $actionName the action name
 	 */
-	public function forward(JsUtils $js,$initialControllerInstance,$controllerName,$actionName,$params=NULL){
-		return $this->setContent($js->forward($initialControllerInstance, $controllerName, $actionName,$params));
+	public function forward(JsUtils $js, $initialControllerInstance, $controllerName, $actionName, $params=NULL) {
+		return $this->setContent($js->forward($initialControllerInstance, $controllerName, $actionName, $params));
 	}
 
 	/**
@@ -161,7 +161,7 @@ 
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		$this->content=JArray::sortAssociative($this->content, ["header","content","actions" ]);
+		$this->content=JArray::sortAssociative($this->content, ["header", "content", "actions"]);
 		return parent::compile($js, $view);
 	}
 	/*
@@ -169,8 +169,8 @@ 
 	 * @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;
 	}

Braces +7 added lines, -6 removed lines

@@ -43,8 +43,7 @@ 
 			foreach ($actions as $action){
 				$this->addAction($action);
 			}
-		}
-		else{
+		} else{
 			$this->addAction($actions);
 		}
 		return $this;
@@ -60,8 +59,9 @@ 
 				$class="cancel";
 			}
 			$action=new HtmlButton("action-".$this->identifier,$action);
-			if($class!=="")
-				$action->addToProperty("class", $class);
+			if($class!=="") {
+							$action->addToProperty("class", $class);
+			}
 		}
 		return $this->addElementInPart($action, "actions");
 	}
@@ -169,8 +169,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/modules/HtmlProgress.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ajax\semantic\html\base\HtmlSemDoubleElement;
 use Ajax\JsUtils;
 use Ajax\service\JArray;
-
 use Ajax\semantic\html\base\constants\State;
 
 class HtmlProgress extends HtmlSemDoubleElement {

Spacing +15 added lines, -15 removed lines

@@ -9,26 +9,26 @@ 
 use Ajax\semantic\html\base\constants\State;
 
 class HtmlProgress extends HtmlSemDoubleElement {
-	private $_params=array ();
+	private $_params=array();
 
 	public function __construct($identifier, $value=NULL, $label=NULL, $attributes=array()) {
 		parent::__construct($identifier, "div", "ui progress");
-		if (isset($value) === true)
+		if (isset($value)===true)
 			$this->setProperty("data-percent", $value);
 		$this->createBar();
-		if (isset($label) === true)
+		if (isset($label)===true)
 			$this->setLabel($label);
-		$this->_states=[ State::SUCCESS,State::WARNING,State::ERROR,State::ACTIVE,State::DISABLED ];
+		$this->_states=[State::SUCCESS, State::WARNING, State::ERROR, State::ACTIVE, State::DISABLED];
 		$this->addToProperty("class", $attributes);
 	}
 
 	public function setLabel($label) {
-		$this->content["label"]=new HtmlSemDoubleElement("lbl-" . $this->identifier, "div", "label", $label);
+		$this->content["label"]=new HtmlSemDoubleElement("lbl-".$this->identifier, "div", "label", $label);
 		return $this;
 	}
 
 	private function createBar() {
-		$bar=new HtmlSemDoubleElement("bar-" . $this->identifier, "div", "bar", new HtmlSemDoubleElement("progress-" . $this->identifier, "div", "progress"));
+		$bar=new HtmlSemDoubleElement("bar-".$this->identifier, "div", "bar", new HtmlSemDoubleElement("progress-".$this->identifier, "div", "progress"));
 		$this->content["bar"]=$bar;
 		return $this;
 	}
@@ -68,20 +68,20 @@ 
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		$this->content=JArray::sortAssociative($this->content, [ "bar","label" ]);
+		$this->content=JArray::sortAssociative($this->content, ["bar", "label"]);
 		return parent::compile($js, $view);
 	}
 
 	public function jsSetValue($value) {
-		return '$("#' . $this->identifier . '").progress({value:' . $value . '});';
+		return '$("#'.$this->identifier.'").progress({value:'.$value.'});';
 	}
 
 	public function jsIncValue() {
-		return '$("#' . $this->identifier . '").progress("increment");';
+		return '$("#'.$this->identifier.'").progress("increment");';
 	}
 
 	public function jsDecValue() {
-		return '$("#' . $this->identifier . '").progress("decrement");';
+		return '$("#'.$this->identifier.'").progress("decrement");';
 	}
 
 	/**
@@ -95,7 +95,7 @@ 
 	 * @return HtmlProgress
 	 */
 	public function setTextValues($active=false, $error=false, $success=false, $warning=false, $percent="{percent}%", $ratio="{value} of {total}") {
-		if (\is_array($active) == true) {
+		if (\is_array($active)==true) {
 			$array=$active;
 			$active=JArray::getDefaultValue($array, "active", false);
 			$success=JArray::getDefaultValue($array, "success", $success);
@@ -103,12 +103,12 @@ 
 			$percent=JArray::getDefaultValue($array, "percent", $percent);
 			$ratio=JArray::getDefaultValue($array, "ratio", $ratio);
 		}
-		$this->_params["text"]="%{active  : " . \var_export($active, true) . ",error: " . \var_export($error, true) . ",success : " . \var_export($success, true) . ",warning : " . \var_export($warning, true) . ",percent : " . \var_export($percent, true) . ",ratio   : " . \var_export($ratio, true) . "}%";
+		$this->_params["text"]="%{active  : ".\var_export($active, true).",error: ".\var_export($error, true).",success : ".\var_export($success, true).",warning : ".\var_export($warning, true).",percent : ".\var_export($percent, true).",ratio   : ".\var_export($ratio, true)."}%";
 		return $this;
 	}
 
 	public function onChange($jsCode) {
-		return $this->_params["onChange"]="%function(percent, value, total){" . $jsCode . "}%";
+		return $this->_params["onChange"]="%function(percent, value, total){".$jsCode."}%";
 	}
 
 	/*
@@ -116,8 +116,8 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if (isset($this->_bsComponent) === false)
-			$this->_bsComponent=$js->semantic()->progress("#" . $this->identifier, $this->_params);
+		if (isset($this->_bsComponent)===false)
+			$this->_bsComponent=$js->semantic()->progress("#".$this->identifier, $this->_params);
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}

Braces +9 added lines, -6 removed lines

@@ -13,11 +13,13 @@ 
 
 	public function __construct($identifier, $value=NULL, $label=NULL, $attributes=array()) {
 		parent::__construct($identifier, "div", "ui progress");
-		if (isset($value) === true)
-			$this->setProperty("data-percent", $value);
+		if (isset($value) === true) {
+					$this->setProperty("data-percent", $value);
+		}
 		$this->createBar();
-		if (isset($label) === true)
-			$this->setLabel($label);
+		if (isset($label) === true) {
+					$this->setLabel($label);
+		}
 		$this->_states=[ State::SUCCESS,State::WARNING,State::ERROR,State::ACTIVE,State::DISABLED ];
 		$this->addToProperty("class", $attributes);
 	}
@@ -116,8 +118,9 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if (isset($this->_bsComponent) === false)
-			$this->_bsComponent=$js->semantic()->progress("#" . $this->identifier, $this->_params);
+		if (isset($this->_bsComponent) === false) {
+					$this->_bsComponent=$js->semantic()->progress("#" . $this->identifier, $this->_params);
+		}
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}

Ajax/semantic/traits/SemanticComponentsTrait.php

Doc Comments +18 added lines

@@ -59,6 +59,9 @@ 
 		return $this->addComponent(new Accordion($this->js), $attachTo, $params);
 	}
 
+	/**
+	 * @param string $attachTo
+	 */
 	public function sticky($attachTo=NULL, $params=NULL) {
 		return $this->addComponent(new Sticky($this->js), $attachTo, $params);
 	}
@@ -67,22 +70,37 @@ 
 		return $this->addComponent(new Checkbox($this->js), $attachTo, $params);
 	}
 
+	/**
+	 * @param string $attachTo
+	 */
 	public function rating($attachTo=NULL, $params=NULL) {
 		return $this->addComponent(new Rating($this->js), $attachTo, $params);
 	}
 
+	/**
+	 * @param string $attachTo
+	 */
 	public function progress($attachTo=NULL, $params=NULL) {
 		return $this->addComponent(new Progress($this->js), $attachTo, $params);
 	}
 
+	/**
+	 * @param string $attachTo
+	 */
 	public function search($attachTo=NULL, $params=NULL) {
 		return $this->addComponent(new Search($this->js), $attachTo, $params);
 	}
 
+	/**
+	 * @param string $attachTo
+	 */
 	public function dimmer($attachTo=NULL, $params=NULL) {
 		return $this->addComponent(new Dimmer($this->js), $attachTo, $params);
 	}
 
+	/**
+	 * @param string $attachTo
+	 */
 	public function modal($attachTo=NULL, $params=NULL,$paramsParts=NULL) {
 		$result= $this->addComponent(new Modal($this->js), $attachTo, $params);
 		$result->setParamParts($paramsParts);

Spacing +2 added lines, -2 removed lines

@@ -83,8 +83,8 @@
 		return $this->addComponent(new Dimmer($this->js), $attachTo, $params);
 	}
 
-	public function modal($attachTo=NULL, $params=NULL,$paramsParts=NULL) {
-		$result= $this->addComponent(new Modal($this->js), $attachTo, $params);
+	public function modal($attachTo=NULL, $params=NULL, $paramsParts=NULL) {
+		$result=$this->addComponent(new Modal($this->js), $attachTo, $params);
 		$result->setParamParts($paramsParts);
 		return $result;
 	}