Jaxon\Di\Traits\AnnotationTrait

Complexity

Total Complexity

9

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	hp$1 ➔ registerAnnotations()	0	42	4
A	hp$1 ➔ onChanges()	0	5	2
	registerAnnotations()	0	42	?
A	hp$1 ➔ onChange()	0	5	3
A	hp$0 ➔ getAttributes()	0	3	1

<?php

Missing file doc comment

namespace Jaxon\Di\Traits;

use Jaxon\Annotations\AnnotationReader;
use Jaxon\App\Config\ConfigEventManager;
use Jaxon\App\Config\ConfigListenerInterface;
use Jaxon\Plugin\AnnotationReaderInterface;
use Jaxon\Utils\Config\Config;

use function class_exists;

trait AnnotationTrait

Missing doc comment for trait AnnotationTrait

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

Expected 2 blank lines before function; 0 found

    {
        // By default, register a fake annotation reader.
        $this->set(AnnotationReaderInterface::class, function() {

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

            return new class implements AnnotationReaderInterface
            {
                public function getAttributes(string $sClass, array $aMethods): array

Missing doc comment for function getAttributes()

Expected 2 blank lines before function; 0 found

                {
                    return [false, [], []];
                }

Expected 2 blank lines after function; 0 found

            };
        });

        if(class_exists(AnnotationReader::class))
        {
            $sEventListenerKey = AnnotationReader::class . '\\ConfigListener';
            // The annotation package is installed, register the real annotation reader,
            // but only if the feature is activated in the config.
            $this->set($sEventListenerKey, function() {
                return new class implements ConfigListenerInterface
                {
                    public function onChanges(Config $xConfig)

Missing doc comment for function onChanges()

Expected 2 blank lines before function; 0 found

                    {
                        if($xConfig->getOption('core.annotations.on'))
                        {
                            AnnotationReader::register(jaxon()->di());
                        }
                    }

Expected 2 blank lines after function; 1 found

                    public function onChange(Config $xConfig, string $sName)

Missing doc comment for function onChange()

                    {
                        if($sName === 'core.annotations.on' && $xConfig->getOption('core.annotations.on'))
                        {
                            AnnotationReader::register(jaxon()->di());
                        }
                    }

Expected 2 blank lines after function; 0 found

                };
            });

            // Register the event listener
            $xEventManager = $this->g(ConfigEventManager::class);

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

            $xEventManager->addListener($sEventListenerKey);
        }
    }

Expected 2 blank lines after function; 0 found

}