jcheron / phalcon-jquery

Ajax/bootstrap/html/HtmlProgressbar.php

<?php

namespace Ajax\bootstrap\html;

use Ajax\bootstrap\html\base\HtmlBsDoubleElement;
use Ajax\JsUtils;

use Ajax\bootstrap\html\base\CssRef;
use Ajax\service\JArray;

class HtmlProgressbar extends HtmlBsDoubleElement {
	protected $value;
	protected $max;
	protected $min;
	protected $striped="";
	protected $active;
	protected $caption;
	protected $isStacked=false;
	protected $style="";
	protected $styleLimits=null;

	public function __construct($identifier, $style="info", $value=0, $max=100, $min=0) {
		parent::__construct($identifier);
		$this->_template=include 'templates/tplProgressbar.php';
		$this->value=$value;
		$this->min=$min;
		$this->max=$max;
		$this->setStyle($style);
	}

	public function setActive($value) {

This method seems to be duplicated in your project.

		if(is_array($this->content)){
			foreach ($this->content as $pb){
				$pb->setActive($value);
			}
		}else{
			if ($value===true)
				$this->active="active";
			else
				$this->active="";
		}
		return $this;
	}

	public function setStriped($value) {
		if(is_array($this->content)){
			foreach ($this->content as $pb){
				$pb->setStriped($value);
			}
		}else{
			if ($value===true)
				$this->striped="progress-bar-striped";
			else
				$this->striped="";
		}
		return $this;
	}

	public function showCaption($value) {

This method seems to be duplicated in your project.

		if(is_array($this->content)){
			foreach ($this->content as $pb){
				$pb->showCaption($value);
			}
		}else{
			if ($value===true)
				$this->caption="%value%%";
			else
				$this->caption='<span class="sr-only">%value%% Complete (%style%)</span>';
		}
		return $this;
	}

	public function getValue() {
		return $this->value;
	}

	public function stack(HtmlProgressbar $progressBar) {
		$this->_template='%content%';
		$progressBar->setIsStacked(true);
		$progressBar->showCaption($this->caption=="%value%%");
		$progressBar->setStriped($this->striped!=="" || $progressBar->isStriped());
		$progressBar->setActive($this->active==="active" || $progressBar->isActive());
		if (is_array($this->content)===false) {
			$this->content=array ();
		}
		$this->content []=$progressBar;
	}

	public function setValue($value) {
		$this->value=$value;
		return $this;
	}

	public function getMax() {
		return $this->max;
	}

	public function setMax($max) {
		$this->max=$max;
		return $this;
	}

	public function getMin() {
		return $this->min;
	}

	public function setMin($min) {
		$this->min=$min;
		return $this;
	}

	public function getIsStacked() {
		return $this->isStacked;
	}

	public function setIsStacked($isStacked) {
		$this->isStacked=$isStacked;
		return $this;
	}

	/**
	 * define the progressbar style
	 * avaible values : "success","info","warning","danger"
	 * @param string|int $cssStyle
	 * @return \Ajax\bootstrap\html\HtmlProgressbar default : ""
	 */
	public function setStyle($cssStyle) {
		return $this->setMemberCtrl($this->style,CssRef::getStyle($cssStyle, "progress-bar"), CssRef::Styles("progress-bar"));
	}

	/*
	 * (non-PHPdoc)
	 * @see \Ajax\bootstrap\html\base\BaseHtml::compile()
	 */
	public function compile(JsUtils $js=NULL, &$view=NULL) {
		$actualStyle=$this->style;
		if(isset($this->styleLimits)&& JArray::isAssociative($this->styleLimits)){
			foreach ($this->styleLimits as $k=>$v){
				$actualStyle=$k;
				if($v>$this->value)
					break;
			}
		}
		$this->style=$actualStyle;

It seems like $actualStyle can also be of type integer. However, the property $style is declared as type string. Maybe add an additional type check?

		$this->_template=str_replace("%caption%", $this->caption, $this->_template);
		if ($this->getIsStacked()===false) {
			$this->wrap('<div class="progress">', '</div>');
		}
		return parent::compile($js, $view);
	}

	public function isStriped() {
		return $this->striped;
	}

	public function isActive() {
		return $this->active;
	}

	/* (non-PHPdoc)
	 * @see \Ajax\bootstrap\html\base\BaseHtml::fromDatabaseObject()
	 */
	public function fromDatabaseObject($object, $function) {
		$this->stack($function($object));
	}

	public function getStyleLimits() {
		return $this->styleLimits;
	}

	/**
	 * Permet de modifier le style de la progressbar à partir de sa valeur actuelle
	 * $styleLimits est de la forme ["success"=>50, "warning"=>100] pour obtenir un style success de 0 à 50 et warning de 50 à 100
	 * @param array $styleLimits tableau associatif des couples style=>valeur possibles
	 * @return \Ajax\bootstrap\html\HtmlProgressbar
	 */
	public function setStyleLimits($styleLimits) {
		$this->styleLimits=$styleLimits;
		return $this;
	}

}