Jaxon\Di\Traits\CallableTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getCallableFunctionPlugin()	0	3	1
A	registerCallables()	0	32	1
A	getClassRegistry()	0	3	1

<?php

Missing file doc comment

namespace Jaxon\Di\Traits;

use Jaxon\Config\ConfigManager;
use Jaxon\Di\Container;
use Jaxon\Request\Handler\RequestHandler;
use Jaxon\Request\Plugin\CallableClass\CallableRegistry;
use Jaxon\Request\Plugin\CallableClass\CallableRepository;
use Jaxon\Request\Plugin\CallableClass\CallableClassPlugin;
use Jaxon\Request\Plugin\CallableClass\CallableDirPlugin;
use Jaxon\Request\Plugin\CallableFunction\CallableFunctionPlugin;
use Jaxon\Request\Validator;
use Jaxon\Response\ResponseManager;
use Jaxon\Utils\Template\TemplateEngine;

The type Jaxon\Utils\Template\TemplateEngine was not found. Maybe you did not declare it correctly or list all dependencies?

use Jaxon\Utils\Translation\Translator;

trait CallableTrait

Missing doc comment for trait CallableTrait

{
    /**
     * Register the values into the container
     *
     * @return void
     */
    private function registerCallables()

Expected 2 blank lines before function; 0 found

    {
        // Validator
        $this->set(Validator::class, function($c) {

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

            return new Validator($c->g(ConfigManager::class), $c->g(Translator::class));
        });
        // Callable objects repository
        $this->set(CallableRepository::class, function($c) {
            return new CallableRepository($c->g(Container::class));
        });
        // Callable objects registry
        $this->set(CallableRegistry::class, function($c) {
            return new CallableRegistry($c->g(Container::class),
                $c->g(CallableRepository::class), $c->g(Translator::class));
        });
        // Callable class plugin
        $this->set(CallableClassPlugin::class, function($c) {
            $sPrefix = $c->g(ConfigManager::class)->getOption('core.prefix.class');
            return new CallableClassPlugin($sPrefix, $c->g(RequestHandler::class),
                $c->g(ResponseManager::class), $c->g(CallableRegistry::class), $c->g(CallableRepository::class),
                $c->g(TemplateEngine::class), $c->g(Translator::class), $c->g(Validator::class));
        });
        // Callable dir plugin
        $this->set(CallableDirPlugin::class, function($c) {
            return new CallableDirPlugin($c->g(CallableRegistry::class), $c->g(Translator::class));
        });
        // Callable function plugin
        $this->set(CallableFunctionPlugin::class, function($c) {
            $sPrefix = $c->g(ConfigManager::class)->getOption('core.prefix.function');
            return new CallableFunctionPlugin($sPrefix, $c->g(RequestHandler::class),
                $c->g(ResponseManager::class), $c->g(TemplateEngine::class),
                $c->g(Translator::class), $c->g(Validator::class));
        });
    }

Expected 2 blank lines after function; 1 found

    /**
     * Get the callable registry
     *
     * @return CallableRegistry
     */
    public function getClassRegistry(): CallableRegistry
    {
        return $this->g(CallableRegistry::class);

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

    }

Expected 2 blank lines after function; 1 found

    /**
     * Get the callable function plugin
     *
     * @return CallableFunctionPlugin
     */
    public function getCallableFunctionPlugin(): CallableFunctionPlugin
    {
        return $this->g(CallableFunctionPlugin::class);
    }

Expected 2 blank lines after function; 0 found

}