LocalizationLoaderTrait - Code Metrics - Porto-SAP/Hello-API - Measure and Improve Code Quality continuously with Scrutinizer

LocalizationLoaderTrait A
last analyzed 2017-03-23 00:40 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	4
lcom	0
cbo	1
dl	0
loc	34
rs	10
c	1
b	0
f	0

3 Methods

Rating	Name	Size	Complexity
A	loadLocalsFromContainers()	6	1
A	loadLocalsFromShip()	4	1
A	loadLocals()	8	2

<?php

namespace App\Ship\Engine\Loaders;

use App;
use File;

/**
 * Class LocalizationLoaderTrait
 *
 * @author  Mahmoud Zalt  <[email protected]>
 */
trait LocalizationLoaderTrait
{

    /**
     * @param $containerName
     */
    public function loadLocalsFromContainers($containerName)
    {
        $containerMigrationDirectory = base_path('app/Containers/' . $containerName . '/Resources/Languages');

        $this->loadLocals($containerMigrationDirectory, $containerName);
    }

    /**
     * @void
     */
    public function loadLocalsFromShip()
    {
        // ..
    }

    /**
     * @param $directory
     * @param $containerName
     */
    private function loadLocals($directory, $containerName)
    {
        if (File::isDirectory($directory)) {

            $this->loadTranslationsFrom($directory, strtolower($containerName));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        }
    }
}


1			<?php
2
3			namespace App\Ship\Engine\Loaders;
4
5			use App;
6			use File;
7
8			/**
9			* Class LocalizationLoaderTrait
10			*
11			* @author Mahmoud Zalt <[email protected]>
12			*/
13			trait LocalizationLoaderTrait
14			{
15
16			/**
17			* @param $containerName
18			*/
19			public function loadLocalsFromContainers($containerName)
20			{
21			$containerMigrationDirectory = base_path('app/Containers/' . $containerName . '/Resources/Languages');
22
23			$this->loadLocals($containerMigrationDirectory, $containerName);
24			}
25
26			/**
27			* @void
28			*/
29			public function loadLocalsFromShip()
30			{
31			// ..
32			}
33
34			/**
35			* @param $directory
36			* @param $containerName
37			*/
38			private function loadLocals($directory, $containerName)
39			{
40			if (File::isDirectory($directory)) {
41
42			$this->loadTranslationsFrom($directory, strtolower($containerName));
			0 ignored issues – show Bug introduced 2017-02-28 16:19 UTC by Report Bug Copy Issue Report It seems like `loadTranslationsFrom()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
43
44			}
45			}
46			}
47

Porto-SAP / Hello-API

LocalizationLoaderTrait A last analyzed 2017-03-23 00:40 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

3 Methods

Duplication Side-by-Side

Filter issues like

LocalizationLoaderTrait A
last analyzed 2017-03-23 00:40 UTC