Ajax\semantic\html\content\HtmlGridRow

Complexity

Total Complexity

17

Size/Duplication

Total Lines	93
Duplicated Lines	7.53 %

Coupling/Cohesion

Components	3
Dependencies	5

Importance

Changes	5
Bugs	0	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	17	4
A	setWidth()	7	7	2
A	getCol()	0	3	1
A	setStretched()	0	3	1
A	setColor()	0	3	1
A	setTextAlignment()	0	3	1
A	setValues()	0	12	4
A	createItem()	0	4	1
A	compile()	0	6	2

<?php

namespace Ajax\semantic\html\content;

use Ajax\common\html\HtmlCollection;
use Ajax\semantic\html\base\Wide;
use Ajax\semantic\html\base\Color;
use Ajax\semantic\html\base\TextAlignment;
use Ajax\JsUtils;
use Phalcon\Mvc\View;


/**
 * A row for the Semantic Grid component
 * @see http://semantic-ui.com/collections/grid.html
 * @author jc
 * @version 1.001
 */
class HtmlGridRow extends HtmlCollection{

	private $_colSize;
	private $_implicite=false;
	public function __construct( $identifier,$numCols=NULL,$colSizing=false,$implicite=false){
		parent::__construct( $identifier,"div");
		$this->_implicite=$implicite;
		$this->setClass("row");
		$width=null;
		if(isset($numCols)){
			$numCols=min(16,$numCols);
			$numCols=max(1,$numCols);
			if($colSizing)
				$width=(int)(16/$numCols);
			else
				$this->_colSize=16/$numCols;
			for ($i=0;$i<$numCols;$i++){
				$this->addItem($width);
			}
		}
	}

	/**
	 * Defines the row width
	 * @param int $width
	 * @return \Ajax\semantic\html\content\HtmlGridRow
	 */
	public function setWidth($width){

This method seems to be duplicated in your project.

		if(\is_int($width)){
			$width=Wide::getConstants()["W".$width];
		}
		$this->addToPropertyCtrl("class", $width, Wide::getConstants());
		return $this->addToPropertyCtrl("class", "column",array("column"));
	}

	/**
	 * return the col at $index
	 * @param int $index
	 * @return \Ajax\semantic\html\collections\HtmlGridCol
	 */
	public function getCol($index){
		return $this->getItem($index);
	}

	/**
	 * stretch the row contents to take up the entire column height
	 * @return \Ajax\semantic\html\content\HtmlGridRow
	 */
	public function setStretched(){
		return $this->addToProperty("class", "stretched");
	}

	/**
	 * @param string $color
	 * @return \Ajax\semantic\html\content\HtmlGridRow
	 */
	public function setColor($color){
		return $this->addToPropertyCtrl("class", $color,Color::getConstants());
	}

	public function setTextAlignment($value=TextAlignment::LEFT){
		return $this->addToPropertyCtrl("class", $value,TextAlignment::getConstants());
	}

	public function setValues($values,$force=false){
		$count=$this->count();
		if($force===true){
			for($i=$count;$i<\sizeof($values);$i++){

It seems like you are calling the size function sizeof() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

				$this->addItem(new HtmlGridCol($this->identifier."-col-".($this->count()+1),null));
			}
		}
		$count=\min(array($this->count(),\sizeof($values)));
		for($i=0;$i<$count;$i++){
			$this->content[$i]->setValue($values[$i]);
		}
	}

	/**
	 * {@inheritDoc}
	 * @see \Ajax\common\html\HtmlCollection::createItem()
	 */
	protected function createItem($value){
		$col=new HtmlGridCol($this->identifier."-col-".($this->count()+1),$value);
		return $col;
	}

	public function compile(JsUtils $js=NULL,View $view=NULL){
		if($this->_implicite===true){
			$this->_template="%wrapContentBefore%%content%%wrapContentAfter%";
		}
		return parent::compile($js,$view);
	}
}