Jaxon\Di\Traits\UtilTrait

Complexity

Total Complexity

2

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	registerUtils()	0	40	1
A	getTemplateEngine()	0	3	1

<?php

Missing file doc comment

namespace Jaxon\Di\Traits;

use Jaxon\Utils\Config\ConfigReader;

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

use Jaxon\Utils\File\FileMinifier;

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

use Jaxon\Utils\Http\UriDetector;
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;

use function rtrim;
use function trim;

trait UtilTrait

Missing doc comment for trait UtilTrait

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

Expected 2 blank lines before function; 0 found

    {
        // Translator
        $this->set(Translator::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?

            $xTranslator = new Translator();

The call to Jaxon\Utils\Translation\Translator::__construct() has too few arguments starting with sDefaultLocale.

            $sResourceDir = rtrim(trim($c->g('jaxon.core.dir.translation')), '/\\');
            // Load the Jaxon package translations
            $xTranslator->loadTranslations($sResourceDir . '/en/errors.php', 'en');
            $xTranslator->loadTranslations($sResourceDir . '/fr/errors.php', 'fr');
            $xTranslator->loadTranslations($sResourceDir . '/es/errors.php', 'es');
            // Load the config translations
            $xTranslator->loadTranslations($sResourceDir . '/en/config.php', 'en');
            $xTranslator->loadTranslations($sResourceDir . '/fr/config.php', 'fr');
            $xTranslator->loadTranslations($sResourceDir . '/es/config.php', 'es');
            // Load the upload translations
            $xTranslator->loadTranslations($sResourceDir . '/en/upload.php', 'en');
            $xTranslator->loadTranslations($sResourceDir . '/fr/upload.php', 'fr');
            $xTranslator->loadTranslations($sResourceDir . '/es/upload.php', 'es');
            return $xTranslator;
        });
        // Config reader
        $this->set(ConfigReader::class, function() {
            return new ConfigReader();
        });
        // Template engine
        $this->set(TemplateEngine::class, function($c) {
            $xTemplateEngine = new TemplateEngine();
            $sTemplateDir = rtrim(trim($c->g('jaxon.core.dir.template')), '/\\');

Equals sign not aligned with surrounding assignments; expected 4 spaces but found 1 space

            $sPaginationDir = $sTemplateDir . DIRECTORY_SEPARATOR . 'pagination';
            $xTemplateEngine->addNamespace('jaxon', $sTemplateDir, '.php');
            $xTemplateEngine->addNamespace('pagination', $sPaginationDir, '.php');
            return $xTemplateEngine;
        });
        // File minifier
        $this->set(FileMinifier::class, function() {
            return new FileMinifier();
        });
        // URI detector
        $this->set(UriDetector::class, function() {
            return new UriDetector();
        });
    }

Expected 2 blank lines after function; 1 found

    /**
     * Get the template engine
     *
     * @return TemplateEngine
     */
    public function getTemplateEngine(): TemplateEngine
    {
        return $this->g(TemplateEngine::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; 0 found

}