chillerlan\PrototypeDOM\Traits\HTMLElementTrait

Complexity

Total Complexity

22

Size/Duplication

Total Lines	159
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	magic_get_id()	0	3	1
A	magic_set_id()	0	3	1
A	magic_get_class()	0	3	1
A	magic_set_class()	0	3	1
A	magic_get_innerHTML()	0	3	1
A	identify()	0	9	2
B	classNames()	0	23	5
A	hasClassName()	0	3	1
A	addClassName()	0	3	1
A	removeClassName()	0	3	1
A	toggleClassName()	0	6	2
A	getStyle()	0	9	2
A	setStyle()	0	15	3

<?php
/**
 * Trait HTMLElementTrait
 *
 * @filesource   HTMLElementTrait.php
 * @created      11.05.2017
 * @package      chillerlan\PrototypeDOM\Traits
 * @author       Smiley <smiley@chillerlan.net>
 * @copyright    2017 Smiley
 * @license      MIT
 */

namespace chillerlan\PrototypeDOM\Traits;

use chillerlan\PrototypeDOM\Node\PrototypeHTMLElement;

/**
 * @property string $id
 * @property string $class
 * @property string $innerHTML
 */
trait HTMLElementTrait{
	use Magic, ElementTrait;

	protected function magic_get_id():string {
		return trim($this->getAttribute('id'));

It seems like getAttribute() must be provided by classes using this trait. How about adding it as abstract method to this trait?

	}

	protected function magic_set_id($id) {
		return $this->setAttribute('id', $id);

It seems like setAttribute() must be provided by classes using this trait. How about adding it as abstract method to this trait?

	}

	protected function magic_get_class():string {
		return trim($this->getAttribute('class'));

It seems like getAttribute() must be provided by classes using this trait. How about adding it as abstract method to this trait?

	}

	protected function magic_set_class($class) {
		return $this->setAttribute('class', $class);

It seems like setAttribute() must be provided by classes using this trait. How about adding it as abstract method to this trait?

	}

	protected function magic_get_innerHTML():string {
		return $this->inspect();
	}

	/**
	 * http://api.prototypejs.org/dom/Element/identify/
	 *
	 * @param string|null $newID
	 *
	 * @return string
	 */
	public function identify(string $newID = null):string {
		$oldID = $this->id;

		if(!is_null($newID)){
			$this->id = $newID;
		}

		return $oldID;
	}

	/**
	 * @link http://api.prototypejs.org/dom/Element/classNames/
	 *
	 * @return array
	 */
	public function classNames():array{
		$currentClassnames = [];

		if($this->hasAttributes()){

It seems like hasAttributes() must be provided by classes using this trait. How about adding it as abstract method to this trait?

			$classnames = explode(' ', $this->class);

			if(!empty($classnames)){

				foreach($classnames as $classname){

					if(empty($classname)){
						continue;
					}

					$currentClassnames[] = $classname;
				}

			}

		}

		return $currentClassnames;
	}

	/**
	 * @link http://api.prototypejs.org/dom/Element/hasClassName/
	 *
	 * @param string $classname
	 *
	 * @return bool
	 */
	public function hasClassName(string $classname):bool{
		return in_array($classname, $this->classNames(), true);
	}

	/**
	 * @link http://api.prototypejs.org/dom/Element/addClassName/
	 *
	 * @param string $classname
	 *
	 * @return \chillerlan\PrototypeDOM\Node\PrototypeHTMLElement
	 */
	public function addClassName(string $classname):PrototypeHTMLElement{
		return $this->addClassNames([$classname]);

The method addClassNames() does not exist on chillerlan\PrototypeDOM\Traits\HTMLElementTrait. Did you maybe mean classNames()?

	}

	/**
	 * @link http://api.prototypejs.org/dom/Element/removeClassName/
	 *
	 * @param string $classname
	 *
	 * @return \chillerlan\PrototypeDOM\Node\PrototypeHTMLElement
	 */
	public function removeClassName(string $classname):PrototypeHTMLElement{
		return $this->removeClassNames([$classname]);

The method removeClassNames() does not exist on chillerlan\PrototypeDOM\Traits\HTMLElementTrait. Did you maybe mean classNames()?

	}

	/**
	 * @link http://api.prototypejs.org/dom/Element/toggleClassName/
	 *
	 * @param string $classname
	 *
	 * @return \chillerlan\PrototypeDOM\Node\PrototypeHTMLElement
	 */
	public function toggleClassName(string $classname):PrototypeHTMLElement{

		return $this->hasClassName($classname)
			? $this->removeClassName($classname)
			: $this->addClassName($classname);
	}

	/**
	 * @link http://api.prototypejs.org/dom/Element/getStyle/
	 *
	 * @param string $property
	 *
	 * @return null|string
	 */
	public function getStyle(string $property){
		$currentStyle = $this->getStyles();

The method getStyles() does not exist on chillerlan\PrototypeDOM\Traits\HTMLElementTrait. Did you maybe mean getStyle()?

		if(array_key_exists(strtolower($property), $currentStyle)){
			return $currentStyle[$property];
		}

		return null;
	}

	/**
	 * @link http://api.prototypejs.org/dom/Element/setStyle/
	 *
	 * @param array $style
	 * @param bool  $replace
	 *
	 * @return \chillerlan\PrototypeDOM\Node\PrototypeHTMLElement
	 */
	public function setStyle(array $style, bool $replace = false):PrototypeHTMLElement{
		$currentStyle = $this->getStyles();

The method getStyles() does not exist on chillerlan\PrototypeDOM\Traits\HTMLElementTrait. Did you maybe mean getStyle()?

		if(!$replace){
			$style = array_merge($currentStyle, $style);
		}

		foreach($style as $property => $value){
			$style[$property] = $property.': '.$value.';';
		}

		$this->setAttribute('style', implode(' ', $style));

It seems like setAttribute() must be provided by classes using this trait. How about adding it as abstract method to this trait?

		return $this;
	}


}