Ajax\common\Widget

Complexity

Total Complexity

14

Size/Duplication

Total Lines	95
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	2
A	show()	0	3	1
A	getModel()	0	3	1
A	setModel()	0	4	1
A	getInstanceViewer()	0	3	1
A	setInstanceViewer()	0	4	1
	getHtmlComponent()	0	1	?
A	setColor()	0	3	1
A	getToolbar()	0	7	2
A	addInToolbar()	0	4	1
A	addItemInToolbar()	0	5	1
A	addButtonInToolbar()	0	4	1
A	addLabelledIconButtonInToolbar()	0	5	1

<?php

namespace Ajax\common;

use Ajax\common\html\HtmlDoubleElement;
use Ajax\semantic\html\elements\HtmlButton;
use Ajax\semantic\widgets\datatable\PositionInTable;
use Ajax\semantic\html\collections\menus\HtmlMenu;

abstract class Widget extends HtmlDoubleElement {

	/**
	 * @var string classname
	 */
	protected $_model;
	protected $_modelInstance;
	/**
	 * @var InstanceViewer
	 */
	protected $_instanceViewer;
	/**
	 * @var boolean
	 */
	protected $_toolbar;
	/**
	 * @var PositionInTable
	 */
	protected $_toolbarPosition;


	public function __construct($identifier,$model,$modelInstance=NULL) {
		parent::__construct($identifier);
		$this->_template="%wrapContentBefore%%content%%wrapContentAfter%";
		$this->setModel($model);
		if(isset($modelInstance));
			$this->show($modelInstance);
	}

	public function show($modelInstance){
		$this->_modelInstance=$modelInstance;
	}

	public function getModel() {
		return $this->_model;
	}

	public function setModel($_model) {
		$this->_model=$_model;
		return $this;
	}

	public function getInstanceViewer() {
		return $this->_instanceViewer;
	}

	public function setInstanceViewer($_instanceViewer) {
		$this->_instanceViewer=$_instanceViewer;
		return $this;
	}

	public abstract function getHtmlComponent();

The abstract declaration must precede the visibility declaration

	public function setColor($color){
		return $this->getHtmlComponent()->setColor($color);
	}

	/**
	 * @return \Ajax\semantic\html\collections\menus\HtmlMenu
	 */
	public function getToolbar(){
		if(isset($this->_toolbar)===false){
			$this->_toolbar=new HtmlMenu("toolbar-".$this->identifier);

It seems like new \Ajax\semantic\html\collections\menus\HtmlMenu('toolbar-' . $this->identifier) of type object<Ajax\semantic\html\collections\menus\HtmlMenu> is incompatible with the declared type boolean of property $_toolbar.

			$this->_toolbar->setSecondary();
		}
		return $this->_toolbar;

The expression $this->_toolbar; of type Ajax\semantic\html\collections\menus\HtmlMenu|boolean adds the type boolean to the return on line 75 which is incompatible with the return type documented by Ajax\common\Widget::getToolbar of type Ajax\semantic\html\collections\menus\HtmlMenu.

	}

	/**
	 * Adds a new element in toolbar
	 * @param mixed $element
	 * @return \Ajax\common\html\HtmlDoubleElement
	 */
	public function addInToolbar($element){
		$tb=$this->getToolbar();
		return $tb->addItem($element);
	}

	public function addItemInToolbar($caption,$icon=NULL){
		$result=$this->addInToolbar($caption);
		$result->addIcon($icon);
		return $result;
	}

	public function addButtonInToolbar($caption){
		$bt=new HtmlButton("",$caption);
		return $this->addInToolbar($bt);
	}

	public function addLabelledIconButtonInToolbar($caption,$icon,$before=true,$labeled=false){
		$bt=new HtmlButton("",$caption);
		$bt->addIcon($icon,$before,$labeled);
		return $this->addInToolbar($bt);
	}
}