GloBee\PaymentApi\Models\PropertyTrait

Complexity

Total Complexity

11

Size/Duplication

Total Lines	86
Duplicated Lines	0 %

Test Coverage

Coverage

96.3%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__set()	0	3	1
A	setProperty()	0	20	4
A	__get()	0	3	1
A	getProperty()	0	16	4
A	strToStudlyCase()	0	3	1

<?php

namespace GloBee\PaymentApi\Models;

use GloBee\PaymentApi\Exceptions\LockedPropertyException;
use GloBee\PaymentApi\Exceptions\UnknownPropertyException;

trait PropertyTrait
{
    /**
     * @param $name
     *
     * @return mixed
     * @throws UnknownPropertyException
     */
    public function getProperty($name)
    {
        $methodName = 'get'.$this->strToStudlyCase($name);
        if (method_exists($this, $methodName)) {
            return $this->{$methodName}();
        }

        if (array_key_exists($name, $this->properties)) {
            return $this->properties[$name];
        }

        if (array_key_exists($name, $this->readonlyProperties)) {

The property readonlyProperties does not exist on GloBee\PaymentApi\Models\PropertyTrait. Since you implemented __get, consider adding a @property annotation.

            return $this->readonlyProperties[$name];
        }

        throw new UnknownPropertyException($name);
    }

    /**
     * @param $name
     * @param $value
     *
     * @throws LockedPropertyException
     * @throws UnknownPropertyException
     */
    protected function setProperty($name, $value)
    {
        $methodName = 'set'.$this->strToStudlyCase($name);
        if (method_exists($this, $methodName)) {
            $this->{$methodName}($value);

            return;
        }

        if (array_key_exists($name, $this->properties)) {
            $this->properties[$name] = $value;

The property properties does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            return;
        }

        if (array_key_exists($name, $this->readonlyProperties)) {

The property readonlyProperties does not exist on GloBee\PaymentApi\Models\PropertyTrait. Since you implemented __get, consider adding a @property annotation.

            throw new LockedPropertyException($name);
        }

        throw new UnknownPropertyException($name);
    }

    /**
     * @param $name
     *
     * @return mixed
     * @throws UnknownPropertyException
     */
    public function __get($name)
    {
        return $this->getProperty($name);
    }

    /**
     * @param $name
     * @param $value
     *
     * @throws LockedPropertyException
     * @throws UnknownPropertyException
     */
    public function __set($name, $value)
    {
        $this->setProperty($name, $value);
    }

    /**
     * @param $key
     *
     * @return mixed
     */
    protected function strToStudlyCase($key)
    {
        return str_replace(' ', '', ucwords(str_replace('_', ' ', $key)));
    }
}