Ajax\bootstrap\html\HtmlPagination

Complexity

Total Complexity

33

Size/Duplication

Total Lines	180
Duplicated Lines	1.67 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes	1
Bugs	0	Features	0

19 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	13	2
A	createElement()	0	21	4
A	createContent()	0	10	2
A	half()	0	3	1
A	getStart()	0	7	2
A	_addEvent()	0	7	3
A	fromDispatcher()	0	15	4
A	getUrl()	0	3	1
A	setSize()	3	8	4
A	getFrom()	0	3	1
A	setFrom()	0	5	1
A	getTo()	0	3	1
A	setTo()	0	5	1
A	getActive()	0	3	1
A	setActive()	0	5	1
A	getUrlMask()	0	3	1
A	setUrlMask()	0	5	1
A	getCountVisible()	0	3	1
A	setCountVisible()	0	5	1

<?php
namespace Ajax\bootstrap\html;

use Ajax\bootstrap\html\base\HtmlDoubleElement;
use Ajax\bootstrap\html\base\HtmlNavElement;
use Ajax\bootstrap\html\base\BaseHtml;
use Ajax\bootstrap\html\base\CssRef;
use Ajax\service\JString;
use Ajax\service\PhalconUtils;

/**
 * Twitter Bootstrap Pagination component
 * @see http://getbootstrap.com/components/#pagination
 * @author jc
 * @version 1.001
 */
class HtmlPagination extends HtmlNavElement {

As per PSR2, the opening brace for this class should be on a new line.

	
	/**
	 * @var int
	 */
	protected $from;
	
	/**
	 * @var int
	 */
	protected $to;
	
	
	/**
	 * @var int
	 */
	protected $countVisible;
	
	/**
	 * @var int
	 */
	protected $active;
	
	/**
	 * @var string
	 */
	protected $urlMask;
	
	/**
	 * @param string $identifier
	 */
	public function __construct($identifier,$from=1,$to=1,$active=NULL,$countVisible=NULL){
		parent::__construct($identifier,"ul");
		$this->setProperty("class", "pagination");
		$this->active=$active;
		$this->from=$from;
		$this->to=$to;
		$this->urlMask="%page%";
		if(!isset($countVisible))
			$this->countVisible=$to-$from+1;
		else
			$this->countVisible=$countVisible;
		$this->createContent();
	}
	
	private function createElement($num,$content,$disabled=false,$current=false){
		$count=sizeof($this->content)+1;

As per coding-style, the use of the aliased function sizeof() should be avoided; please use count() instead.

		$elem=new HtmlDoubleElement("li-".$this->identifier."-".$count,"li");
		if($disabled){
			$elem->setProperty("class", "disabled");
		}
		if($current){
			$content.="<span class='sr-only'>(current)</span>";
			$elem->setProperty("class", "active");
		}
		if(!$disabled){
			$url=$this->getUrl($num);
			$href=new HtmlLink("a-".$this->identifier."-".$count,$url,$content);
			$href->setProperty($this->attr, $url);
			$elem->setContent($href);
		}else{
			$elem->setContent($content);
		}
		$this->content[]=$elem;
		return $this;
	}
	
	protected function createContent(){
		$this->content=array();
		$this->createElement($this->active-1,"<span aria-hidden='true'>&laquo;</span>",$this->active===1);
		$start=$this->getStart();
		$end=min($start+$this->countVisible-1,$this->to);
		for($index=$start;$index<=$end;$index++){
			$this->createElement($index,$index,false,$index===$this->active);
		}
		$this->createElement($this->active+1,"<span aria-hidden='true'>&raquo;</span>",$this->active===$this->to);
	}
	
	protected function half(){
		return (int)($this->countVisible/2);
	}
	
	protected function getStart(){
		$result=1;
		if($this->countVisible!==$this->to-$this->from+1){
			$result=max($this->active-$this->half(),$result);
		}
		return $result;
	}

	public function _addEvent($event, $jsCode) {

Method name "_addEvent" should not be prefixed with an underscore to indicate visibility

		foreach ($this->content as $li){
			$content=$li->getContent();
			if($content instanceof BaseHtml)
				$content->_addEvent($event,$jsCode);
		}
	}
	/**
	 * set the active page corresponding to request dispatcher : controllerName, actionName, parameters and $urlMask
	 * @param Dispatcher $dispatcher the request dispatcher
	 * @return \Ajax\bootstrap\html\HtmlPagination
	 */
	public function fromDispatcher($dispatcher){
		$items=array($dispatcher->getControllerName(),$dispatcher->getActionName());
		$items=array_merge($items,$dispatcher->getParams());
		$url=implode("/", $items);
		if($this->urlMask==="%page%"){
			$this->urlMask=preg_replace("/[0-9]/", "%page%", $url);
		}
		for($index=$this->from;$index<=$this->to;$index++){
			if($this->getUrl($index)==$url){
				$this->setActive($index);
				break;
			}
		}
		return $this;
	}
	
	public function getUrl($index){
		return str_ireplace("%page%", $index, $this->urlMask);
	}
	
	/**
	 * define the buttons size
	 * available values : "lg","","sm","xs"
	 * @param string|int $size
	 * @return HtmlPagination default : ""
	 */
	public function setSize($size) {
		if (is_int($size)) {

This code seems to be duplicated across your project.

			return $this->addToPropertyUnique("class", CssRef::sizes("pagination")[$size], CssRef::sizes("pagination"));
		}
		if(!PhalconUtils::startsWith($size, "pagination-") && $size!=="")
			$size="pagination-".$size;
		return $this->addToPropertyCtrl("class", $size, CssRef::sizes("pagination"));
	}
	
	public function getFrom() {
		return $this->from;
	}
	public function setFrom($from) {
		$this->from = $from;
		$this->createContent();
		return $this;
	}
	public function getTo() {
		return $this->to;
	}
	public function setTo($to) {
		$this->to = $to;
		$this->createContent();
		return $this;
	}
	public function getActive() {
		return $this->active;
	}
	public function setActive($active) {
		$this->active = $active;
		$this->createContent();
		return $this;
	}
	public function getUrlMask() {
		return $this->urlMask;
	}
	public function setUrlMask($urlMask) {
		$this->urlMask = $urlMask;
		$this->createContent();
		return $this;
	}
	public function getCountVisible() {
		return $this->countVisible;
	}
	public function setCountVisible($countVisible) {
		$this->countVisible = $countVisible;
		$this->createContent();
		return $this;
	}
	
	
}

According to PSR2, the closing brace of classes should be placed on the next line directly after the body.