jcheron / phalcon-jquery

Ajax/semantic/components/Popup.php

Spacing +4 added lines, -4 removed lines

@@ -30,7 +30,7 @@ 
 		return $this->setParam("onShow", "%function(){".$jsCode."}%");
 	}
 
-	public function setExclusive($value){
+	public function setExclusive($value) {
 		return $this->setParam("exclusive", $value);
 	}
 
@@ -39,15 +39,15 @@ 
 	 * @param string $popup the css selector of the popup
 	 * @return \Ajax\semantic\components\Popup
 	 */
-	public function setPopup($popup){
+	public function setPopup($popup) {
 		return $this->setParam("popup", $popup);
 	}
 
-	public function setInline($value){
+	public function setInline($value) {
 		return $this->setParam("inline", $value);
 	}
 
-	public function setPosition($value){
+	public function setPosition($value) {
 		return $this->setParam("position", $value);
 	}
 	//TODO other events implementation

Ajax/semantic/html/collections/HtmlGrid.php

Doc Comments +7 added lines, -2 removed lines

@@ -21,6 +21,11 @@ 
 	private $_createCols;
 	private $_colSizing=true;
 	private $_implicitRows=false;
+
+	/**
+	 * @param string $identifier
+	 * @param integer $numCols
+	 */
 	public function __construct( $identifier,$numRows=1,$numCols=NULL,$createCols=true,$implicitRows=false){
 		parent::__construct( $identifier, "div","ui grid");
 		$this->_implicitRows=$implicitRows;
@@ -95,7 +100,7 @@ 
 	/**
 	 * return the row at $index
 	 * @param int $index
-	 * @return \Ajax\semantic\html\collections\HtmlGridRow
+	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
 	public function getRow($index){
 		return $this->getItem($index);
@@ -211,7 +216,7 @@ 
 
 	/**
 	 * stretch the row contents to take up the entire column height
-	 * @return \Ajax\semantic\html\content\HtmlGridRow
+	 * @return HtmlGrid
 	 */
 	public function setStretched(){
 		return $this->addToProperty("class", "stretched");

Spacing +55 added lines, -55 removed lines

@@ -16,31 +16,31 @@ 
  * @author jc
  * @version 1.001
  */
-class HtmlGrid extends HtmlSemCollection{
+class HtmlGrid extends HtmlSemCollection {
 	use TextAlignmentTrait;
 	private $_createCols;
 	private $_colSizing=true;
 	private $_implicitRows=false;
-	public function __construct( $identifier,$numRows=1,$numCols=NULL,$createCols=true,$implicitRows=false){
-		parent::__construct( $identifier, "div","ui grid");
+	public function __construct($identifier, $numRows=1, $numCols=NULL, $createCols=true, $implicitRows=false) {
+		parent::__construct($identifier, "div", "ui grid");
 		$this->_implicitRows=$implicitRows;
 		$this->_createCols=$createCols;
-		if(isset($numCols)){
+		if (isset($numCols)) {
 			//if($this->_createCols){
 				$this->_colSizing=false;
 			//}
 			$this->setWide($numCols);
 		}
-		$this->setRowsCount($numRows,$numCols);
+		$this->setRowsCount($numRows, $numCols);
 	}
 
-	public function setWide($wide){
+	public function setWide($wide) {
 		$wide=Wide::getConstants()["W".$wide];
 		$this->addToPropertyCtrl("class", $wide, Wide::getConstants());
-		return $this->addToPropertyCtrl("class","column",array("column"));
+		return $this->addToPropertyCtrl("class", "column", array("column"));
 	}
 
-	public function addRow($colsCount=NULL){
+	public function addRow($colsCount=NULL) {
 		return $this->addItem($colsCount);
 	}
 
@@ -50,41 +50,41 @@ 
 	 * @param int $colsCount
 	 * @return \Ajax\semantic\html\collections\HtmlGrid
 	 */
-	public function setRowsCount($rowsCount,$colsCount=NULL){
+	public function setRowsCount($rowsCount, $colsCount=NULL) {
 		$count=$this->count();
-		if($rowsCount<2){
-			for($i=$count;$i<$colsCount;$i++){
+		if ($rowsCount<2) {
+			for ($i=$count; $i<$colsCount; $i++) {
 				$this->addItem(new HtmlGridCol("col-".$this->identifier."-".$i));
 			}
-		}else{
-			if($this->hasOnlyCols($count)){
+		}else {
+			if ($this->hasOnlyCols($count)) {
 				$tmpContent=$this->content;
 				$item=$this->addItem($colsCount);
 				$item->setContent($tmpContent);
 				$this->content=array();
 				$count=1;
 			}
-			for($i=$count;$i<$rowsCount;$i++){
+			for ($i=$count; $i<$rowsCount; $i++) {
 				$this->addItem($colsCount);
 			}
 		}
 		return $this;
 	}
 
-	protected function hasOnlyCols($count){
+	protected function hasOnlyCols($count) {
 		return $count>0 && $this->content[0] instanceof HtmlGridCol;
 	}
 
-	public function setColsCount($numCols,$toCreate=true){
+	public function setColsCount($numCols, $toCreate=true) {
 		$this->setWide($numCols);
-		if($toCreate==true){
+		if ($toCreate==true) {
 			$count=$this->count();
-			if($count==0 || $this->hasOnlyCols($count)){
-				for($i=$count;$i<$numCols;$i++){
+			if ($count==0 || $this->hasOnlyCols($count)) {
+				for ($i=$count; $i<$numCols; $i++) {
 					$this->addItem(new HtmlGridCol("col-".$this->identifier."-".$i));
 				}
-			}else{
-				for($i=0;$i<$count;$i++){
+			}else {
+				for ($i=0; $i<$count; $i++) {
 					$this->getItem($i)->setNumCols($numCols);
 				}
 			}
@@ -97,7 +97,7 @@ 
 	 * @param int $index
 	 * @return \Ajax\semantic\html\collections\HtmlGridRow
 	 */
-	public function getRow($index){
+	public function getRow($index) {
 		return $this->getItem($index);
 	}
 
@@ -106,11 +106,11 @@ 
 	 * @param int $col
 	 * @return \Ajax\semantic\html\collections\HtmlGridCol
 	 */
-	public function getCell($row,$col){
-		if($row<2 && $this->hasOnlyCols($this->count()))
+	public function getCell($row, $col) {
+		if ($row<2 && $this->hasOnlyCols($this->count()))
 			return $this->getItem($col);
 		$row=$this->getItem($row);
-		if(isset($row)){
+		if (isset($row)) {
 			$col=$row->getItem($col);
 		}
 		return $col;
@@ -121,9 +121,9 @@ 
 	 * @param boolean $vertically
 	 * @return \Ajax\semantic\html\collections\HtmlGrid
 	 */
-	public function setDivided($vertically=false){
-		$value=($vertically===true)?"vertically divided":"divided";
-		return $this->addToPropertyCtrl("class", $value,array("divided"));
+	public function setDivided($vertically=false) {
+		$value=($vertically===true) ? "vertically divided" : "divided";
+		return $this->addToPropertyCtrl("class", $value, array("divided"));
 	}
 
 	/**
@@ -131,16 +131,16 @@ 
 	 * @param boolean $internally true for internal cells
 	 * @return \Ajax\semantic\html\collections\HtmlGrid
 	 */
-	public function setCelled($internally=false){
-		$value=($internally===true)?"internally celled":"celled";
-		return $this->addToPropertyCtrl("class", $value,array("celled","internally celled"));
+	public function setCelled($internally=false) {
+		$value=($internally===true) ? "internally celled" : "celled";
+		return $this->addToPropertyCtrl("class", $value, array("celled", "internally celled"));
 	}
 
 	/**
 	 * automatically resize all elements to split the available width evenly
 	 * @return \Ajax\semantic\html\collections\HtmlGrid
 	 */
-	public function setEqualWidth(){
+	public function setEqualWidth() {
 		return $this->addToProperty("class", "equal width");
 	}
 
@@ -149,9 +149,9 @@ 
 	 * @param string $value
 	 * @return \Ajax\semantic\html\collections\HtmlGrid
 	 */
-	public function setPadded($value=NULL){
-		if(isset($value))
-			$this->addToPropertyCtrl("class", $value,array("vertically","horizontally"));
+	public function setPadded($value=NULL) {
+		if (isset($value))
+			$this->addToPropertyCtrl("class", $value, array("vertically", "horizontally"));
 		return $this->addToProperty("class", "padded");
 	}
 
@@ -159,49 +159,49 @@ 
 	 * @param boolean $very
 	 * @return \Ajax\semantic\html\collections\HtmlGrid
 	 */
-	public function setRelaxed($very=false){
-		$value=($very===true)?"very relaxed":"relaxed";
-		return $this->addToPropertyCtrl("class", $value,array("relaxed","very relaxed"));
+	public function setRelaxed($very=false) {
+		$value=($very===true) ? "very relaxed" : "relaxed";
+		return $this->addToPropertyCtrl("class", $value, array("relaxed", "very relaxed"));
 	}
 
-	public function setVerticalAlignment($value=VerticalAlignment::MIDDLE){
-		return $this->addToPropertyCtrl("class", $value." aligned",VerticalAlignment::getConstantValues("aligned"));
+	public function setVerticalAlignment($value=VerticalAlignment::MIDDLE) {
+		return $this->addToPropertyCtrl("class", $value." aligned", VerticalAlignment::getConstantValues("aligned"));
 	}
 
 	/**
 	 * {@inheritDoc}
 	 * @see \Ajax\common\html\HtmlCollection::createItem()
 	 */
-	protected function createItem($value){
-		if($this->_createCols===false)
+	protected function createItem($value) {
+		if ($this->_createCols===false)
 			$value=null;
-		$item=new HtmlGridRow($this->identifier."-row-".($this->count()+1),$value,$this->_colSizing,$this->_implicitRows);
+		$item=new HtmlGridRow($this->identifier."-row-".($this->count()+1), $value, $this->_colSizing, $this->_implicitRows);
 		return $item;
 	}
 
-	public function setValues($values){
+	public function setValues($values) {
 		$count=$this->count();
-		if($this->_createCols===false){
-			for($i=$count;$i<\sizeof($values);$i++){
+		if ($this->_createCols===false) {
+			for ($i=$count; $i<\sizeof($values); $i++) {
 				$colSize=\sizeof($values[$i]);
-				$this->addItem(new HtmlGridRow($this->identifier."-row-".($this->count()+1),$colSize,$this->_colSizing,$this->_implicitRows));
+				$this->addItem(new HtmlGridRow($this->identifier."-row-".($this->count()+1), $colSize, $this->_colSizing, $this->_implicitRows));
 			}
 		}
-		$count=\min(array($this->count(),\sizeof($values)));
-		for($i=0;$i<$count;$i++){
-			$this->content[$i]->setValues($values[$i],$this->_createCols===false);
+		$count=\min(array($this->count(), \sizeof($values)));
+		for ($i=0; $i<$count; $i++) {
+			$this->content[$i]->setValues($values[$i], $this->_createCols===false);
 		}
 	}
 
-	public function rowCount(){
+	public function rowCount() {
 		return $this->count();
 	}
 
-	public function colCount(){
+	public function colCount() {
 		$result=0;
 		$count=$this->count();
-		if($count>0){
-			if($this->content[0] instanceof HtmlGridCol)
+		if ($count>0) {
+			if ($this->content[0] instanceof HtmlGridCol)
 				$result=$count;
 			else
 				$result=$this->content[0]->count();
@@ -213,7 +213,7 @@ 
 	 * stretch the row contents to take up the entire column height
 	 * @return \Ajax\semantic\html\content\HtmlGridRow
 	 */
-	public function setStretched(){
+	public function setStretched() {
 		return $this->addToProperty("class", "stretched");
 	}
 

Braces +16 added lines, -12 removed lines

@@ -56,7 +56,7 @@ 
 			for($i=$count;$i<$colsCount;$i++){
 				$this->addItem(new HtmlGridCol("col-".$this->identifier."-".$i));
 			}
-		}else{
+		} else{
 			if($this->hasOnlyCols($count)){
 				$tmpContent=$this->content;
 				$item=$this->addItem($colsCount);
@@ -83,7 +83,7 @@ 
 				for($i=$count;$i<$numCols;$i++){
 					$this->addItem(new HtmlGridCol("col-".$this->identifier."-".$i));
 				}
-			}else{
+			} else{
 				for($i=0;$i<$count;$i++){
 					$this->getItem($i)->setNumCols($numCols);
 				}
@@ -107,8 +107,9 @@ 
 	 * @return \Ajax\semantic\html\collections\HtmlGridCol
 	 */
 	public function getCell($row,$col){
-		if($row<2 && $this->hasOnlyCols($this->count()))
-			return $this->getItem($col);
+		if($row<2 && $this->hasOnlyCols($this->count())) {
+					return $this->getItem($col);
+		}
 		$row=$this->getItem($row);
 		if(isset($row)){
 			$col=$row->getItem($col);
@@ -150,8 +151,9 @@ 
 	 * @return \Ajax\semantic\html\collections\HtmlGrid
 	 */
 	public function setPadded($value=NULL){
-		if(isset($value))
-			$this->addToPropertyCtrl("class", $value,array("vertically","horizontally"));
+		if(isset($value)) {
+					$this->addToPropertyCtrl("class", $value,array("vertically","horizontally"));
+		}
 		return $this->addToProperty("class", "padded");
 	}
 
@@ -173,8 +175,9 @@ 
 	 * @see \Ajax\common\html\HtmlCollection::createItem()
 	 */
 	protected function createItem($value){
-		if($this->_createCols===false)
-			$value=null;
+		if($this->_createCols===false) {
+					$value=null;
+		}
 		$item=new HtmlGridRow($this->identifier."-row-".($this->count()+1),$value,$this->_colSizing,$this->_implicitRows);
 		return $item;
 	}
@@ -201,10 +204,11 @@ 
 		$result=0;
 		$count=$this->count();
 		if($count>0){
-			if($this->content[0] instanceof HtmlGridCol)
-				$result=$count;
-			else
-				$result=$this->content[0]->count();
+			if($this->content[0] instanceof HtmlGridCol) {
+							$result=$count;
+			} else {
+							$result=$this->content[0]->count();
+			}
 		}
 		return $result;
 	}

Ajax/semantic/html/modules/HtmlPopup.php

Doc Comments +7 added lines

@@ -12,6 +12,10 @@ 
 class HtmlPopup extends HtmlSemDoubleElement {
 	private $params;
 	private $_container;
+
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct(BaseHtml $container,$identifier, $content="") {
 		parent::__construct($identifier, "div");
 		$this->_container=$container;
@@ -72,6 +76,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $position
+	 */
 	public function setPosition($position){
 		$this->params["position"]=$position;
 		return $this;

Spacing +15 added lines, -15 removed lines

@@ -12,7 +12,7 @@ 
 class HtmlPopup extends HtmlSemDoubleElement {
 	private $params;
 	private $_container;
-	public function __construct(BaseHtml $container,$identifier, $content="") {
+	public function __construct(BaseHtml $container, $identifier, $content="") {
 		parent::__construct($identifier, "div");
 		$this->_container=$container;
 		$this->setClass("ui popup");
@@ -20,21 +20,21 @@ 
 		$this->params=array("on"=>"hover");
 	}
 
-	public function addList($items=array(),$header=NULL){
-		if(!$this->content instanceof HtmlGrid){
-			$this->content=new HtmlGrid("grid-".$this->identifier,0);
+	public function addList($items=array(), $header=NULL) {
+		if (!$this->content instanceof HtmlGrid) {
+			$this->content=new HtmlGrid("grid-".$this->identifier, 0);
 		}
 		$grid=$this->content;
 
 		$colCount=$grid->colCount();
 		$grid->setNumCols(++$colCount);
 
-		$list=new HtmlList("",$items);
+		$list=new HtmlList("", $items);
 		$list->asLink();
-		if(isset($header)){
-			$list->addHeader(4,$header);
+		if (isset($header)) {
+			$list->addHeader(4, $header);
 		}
-		$grid->getCell(0,$colCount-1)->setContent($list);
+		$grid->getCell(0, $colCount-1)->setContent($list);
 		$grid->setDivided()->setRelaxed(true);
 		return $list;
 	}
@@ -42,14 +42,14 @@ 
 	/**
 	 * A popup can have no maximum width and continue to flow to fit its content
 	 */
-	public function setFlowing(){
+	public function setFlowing() {
 		return $this->addToProperty("class", "flowing");
 	}
 
 	/**
 	 * A popup can provide more basic formatting
 	 */
-	public function setBasic(){
+	public function setBasic() {
 		return $this->addToProperty("class", "basic");
 	}
 
@@ -57,22 +57,22 @@ 
 	 * {@inheritDoc}
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::run()
 	 */
-	public function run(JsUtils $js){
+	public function run(JsUtils $js) {
 		$this->params["popup"]="#".$this->identifier;
-		$js->semantic()->popup("#".$this->_container->getIdentifier(),$this->params);
+		$js->semantic()->popup("#".$this->_container->getIdentifier(), $this->params);
 	}
 
-	public function setOn($event="click"){
+	public function setOn($event="click") {
 		$this->params["on"]=$event;
 		return $this;
 	}
 
-	public function setInline($value=true){
+	public function setInline($value=true) {
 		$this->params["inline"]=$value;
 		return $this;
 	}
 
-	public function setPosition($position){
+	public function setPosition($position) {
 		$this->params["position"]=$position;
 		return $this;
 	}