PropertyHiderInArray

Complexity

Total Complexity

5

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes	3
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	isPropertyActionAvailable()	0	4	1
A	setProperty()	0	6	1
A	getProperty()	0	3	1
A	__isset()	0	3	1
A	isContainerEmpty()	0	3	1

<?php

/**
 * Class PropertyHiderInArray
 *
 * Holding properties in internal container
 */
class PropertyHiderInArray extends PropertyHider {

  protected $_data = array();

  /**
   * Method checks if action is available for named property
   *
   * @param string $name
   * @param string $action
   *
   * @return bool
   */
  protected function isPropertyActionAvailable($name, $action = '') {
    // By default all action available for existing properties
    return true;
  }

  /**
   * Internal method that make real changes to property value
   * May be override in child class
   *
   * @param string $name
   * @param mixed  $value
   *
   * @return mixed
   */
  protected function setProperty($name, $value) {
    // TODO: Change property only if value differs ????

48% of this comment could be valid code. Did you maybe forget this after debugging?

//      if($this->{$this->getPhysicalPropertyName($name)} !== $value) {
//      }
    return $this->_data[$name] = $value;
  }

  /**
   * Internal method that make really reads property value
   * May be override in child class
   *
   * @param string $name
   * @param mixed  $value - ignored. Used for compatibility
   *
   * @return mixed
   */
  protected function getProperty($name, $value = null) {
    return $this->_data[$name];
  }

  public function __isset($name) {
    return isset($this->_data[$name]);
  }

  /**
   * Is container empty
   *
   * @return boolean
   */
  public function isContainerEmpty() {
    return empty($this->_data);
  }

}