Ajax\semantic\html\collections\form\HtmlFormFields

Complexity

Total Complexity

13

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	5

Importance

Changes	2
Bugs	0	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	setLabel()	0	8	2
A	compile()	0	8	2
A	setWidth()	0	4	1
A	setInline()	0	4	1
A	setGrouped()	0	4	1
A	getName()	0	3	1
A	setName()	0	4	1
A	radios()	0	11	3

<?php

namespace Ajax\semantic\html\collections\form;

use Ajax\semantic\html\base\HtmlSemCollection;
use Ajax\semantic\html\base\constants\Wide;
use Ajax\JsUtils;
use Phalcon\Mvc\View;
use Ajax\semantic\html\base\HtmlSemDoubleElement;

class HtmlFormFields extends HtmlSemCollection {

	use FieldsTrait;
	protected $_equalWidth;
	protected $_name;

	public function __construct($identifier, $fields=array(),$equalWidth=true) {
		parent::__construct($identifier, "div");
		$this->_equalWidth=$equalWidth;
		$this->addItems($fields);
	}

	/**
	 * @param string|HtmlSemDoubleElement $label
	 * @return \Ajax\semantic\html\base\HtmlSemDoubleElement
	 */
	public function setLabel($label){
		$labelO=$label;
		if(\is_string($label)){
			$labelO=new HtmlSemDoubleElement("","label","",$label);
		}
		$this->insertItem($labelO,0);
		return $labelO;

The expression return $labelO; of type string|Ajax\semantic\html\base\HtmlSemDoubleElement is incompatible with the return type documented by Ajax\semantic\html\collections\form\HtmlFormFields::setLabel of type Ajax\semantic\html\base\HtmlSemDoubleElement as it can also be of type string which is not included in this return type.

	}

	public function compile(JsUtils $js=NULL,View $view=NULL){
		if($this->_equalWidth){
			$count=$this->count();
			$this->addToProperty("class", Wide::getConstants()["W".$count]." fields");
		}else
			$this->addToProperty("class","fields");
		return parent::compile($js,$view);
	}

	public function setWidth($index,$width){
		$this->_equalWidth=false;
		return $this->getItem($index)->setWidth($width);

It seems like you code against a specific sub-type and not the parent class Ajax\common\html\HtmlDoubleElement as the method setWidth() does only exist in the following sub-classes of Ajax\common\html\HtmlDoubleElement: Ajax\bootstrap\html\content\HtmlGridCol, Ajax\semantic\html\collections\form\HtmlFormCheckbox, Ajax\semantic\html\collections\form\HtmlFormDropdown, Ajax\semantic\html\collections\form\HtmlFormField, Ajax\semantic\html\collections\form\HtmlFormFields, Ajax\semantic\html\collections\form\HtmlFormInput, Ajax\semantic\html\collections\form\HtmlFormRadio, Ajax\semantic\html\collections\form\HtmlFormTextarea, Ajax\semantic\html\content\HtmlGridCol, Ajax\semantic\html\content\HtmlGridRow. Maybe you want to instanceof check for one of these explicitly?

	}

	public function setInline(){
		$this->_equalWidth=false;
		$this->addToProperty("class", "inline");
	}

	public function setGrouped(){
		$this->_equalWidth=false;
		$this->addToProperty("class", "grouped");
	}

	public function getName() {
		return $this->_name;
	}

	public function setName($_name) {
		$this->_name=$_name;
		return $this;
	}

	public static function radios($name,$items=array(),$label=NULL){
		$fields=array();
		$i=0;
		foreach ($items as $item){
			$fields[]=new HtmlFormRadio($name."-".$i++,$name,$item,$item);
		}
		$radios=new HtmlFormFields("fields-".$name,$fields);
		if(isset($label))
			$radios->setLabel($label)->setProperty("for", $name);
		return $radios;
	}

}