Nip\Records\Traits\HasController\HasControllerRecordsTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	generateControllerNamespaced()	0	6	1
A	initController()	0	8	2
A	setController()	0	3	1
A	generateControllerGeneric()	0	5	1
A	getController()	0	7	2

<?php

namespace Nip\Records\Traits\HasController;

/**
 * Trait HasControllerRecordsTrait
 * @package Nip\Records\Traits\HasController
 */
trait HasControllerRecordsTrait
{

    /**
     * @var null|string
     */
    protected $controller = null;
    /**
     * @return string
     */
    public function getController()
    {
        if ($this->controller === null) {
            $this->initController();
        }

        return $this->controller;
    }

    /**
     * @param null|string $controller
     */
    public function setController($controller)
    {
        $this->controller = $controller;
    }

    protected function initController()
    {
        if ($this->isNamespaced()) {

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

            $controller = $this->generateControllerNamespaced();
        } else {
            $controller = $this->generateControllerGeneric();
        }
        $this->setController($controller);
    }

    /**
     * @return string
     */
    protected function generateControllerNamespaced()
    {
        $class = $this->getModelNamespacePath();

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

        $class = trim($class, '\\');

        return inflector()->unclassify($class);
    }

    /**
     * @return string
     */
    protected function generateControllerGeneric()
    {
        $class = $this->getClassName();

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

        return inflector()->unclassify($class);
    }
}