Aura\Marshal\Entity\MagicArrayAccessTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__isset()	0	3	1
A	__unset()	0	3	1
A	__set()	0	3	1
A	__get()	0	14	2

<?php
/**
 *
 * This file is part of the Aura project for PHP.
 *
 * @package Aura.Marshal
 *
 * @license https://opensource.org/licenses/mit-license.php MIT
 *
 */
namespace Aura\Marshal\Entity;

use Aura\Marshal\Lazy\LazyInterface;

/**
 *
 * Use this trait for magic __get(), __set(), __isset(), and __unset() on any
 * entity class that implements ArrayAccess for its fields.
 *
 * @package Aura.Marshal
 *
 */
trait MagicArrayAccessTrait
{
    /**
     *
     * Calls ArrayAccess::offsetGet() to get a field value; converts Lazy
     * objects for related entities and collections in place.
     *
     * @param string $field The field name to get.
     *
     * @return mixed The field value.
     *
     */
    public function __get($field)
    {
        // get the field value
        $value = $this->offsetGet($field);

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

        // is it a Lazy placeholder?
        if ($value instanceof LazyInterface) {
            // replace the Lazy placeholder with the real object
            $value = $value->get($this);
            $this->offsetSet($field, $value);

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

        }

        // done!
        return $value;
    }

    /**
     *
     * Calls ArrayAccess::offsetSet() to set a field value.
     *
     * @param string $field The field name to get.
     *
     * @param mixed $value Set the field to this value.
     *
     * @return void
     *
     */
    public function __set($field, $value)
    {
        $this->offsetSet($field, $value);
    }

    /**
     *
     * Calls ArrayAccess::offsetExists() to see if a field is set.
     *
     * @param string $field The field name to check.
     *
     * @return bool
     *
     */
    public function __isset($field)
    {
        return $this->offsetExists($field);

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

    }

    /**
     *
     * Calls ArrayAccess::offsetUnset() to unset a field.
     *
     * @param string $field The field name to unset.
     *
     * @return void
     *
     */
    public function __unset($field)
    {
        $this->offsetUnset($field);

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

    }
}