FileTranslationRepository - Code Metrics - Inspection of "Preparing 1.3.0 release" - AsgardCms/Translation - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c62883...795ab8 )

by Nicolas

created 2015-11-30 15:03 UTC

FileTranslationRepository A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	5
Bugs	0	Features	0

Metric	Value
wmc	10
c	5
b	0
f	0
lcom	1
cbo	1
dl	0
loc	67
rs	10

3 Methods

Rating	Name	Size	Complexity
A	__construct()	5	1
A	all()	19	4
B	getTranslationFilenamesFromPaths()	19	5

<?php namespace Modules\Translation\Repositories\File;

use Illuminate\Filesystem\Filesystem;
use Illuminate\Translation\LoaderInterface;
use Modules\Translation\Repositories\FileTranslationRepository as FileTranslationRepositoryInterface;

class FileTranslationRepository implements FileTranslationRepositoryInterface
{
    /**
     * @var Filesystem
     */
    private $finder;
    /**
     * @var LoaderInterface
     */
    private $loader;

    public function __construct(Filesystem $finder, LoaderInterface $loader)
    {
        $this->finder = $finder;
        $this->loader = $loader;
    }

    /**
     * Get all the translations for all modules on disk
     * @return array
     */
    public function all()
    {
        $files = $this->getTranslationFilenamesFromPaths($this->loader->paths());
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}

        $translations = [];

        foreach ($files as $locale => $files) {
            foreach ($files as $namespace => $file) {
                $trans = $this->finder->getRequire($file);
                $trans = array_dot($trans);

                foreach ($trans as $key => $value) {
                    $translations[$locale]["{$namespace}.{$key}"] = $value;
                }
            }
        }

        return $translations;
    }

    /**
     * Get all of the names of the Translations files from an array of Paths.
     * Returns [ 'translationkeyprefix' => 'filepath' ]
     * @param array $paths
     * @return array
     */
    protected function getTranslationFilenamesFromPaths(array $paths)
    {
        $files   = [];
        $locales = config('laravellocalization.supportedLocales');

        foreach ($paths as $hint => $path) {
            foreach ($locales as $locale => $language) {
                foreach ($this->finder->glob("{$path}/{$locale}/*.php") as $file) {
                    $category = str_replace(["$path/", ".php", "{$locale}/"], "", $file);
                    $category = str_replace("/", ".", $category);
                    $category = !is_int($hint) ? "{$hint}::{$category}" : $category;

                    $files[$locale][$category] = $file;
                }
            }
        }

        return $files;
    }
}


1			<?php namespace Modules\Translation\Repositories\File;
2
3			use Illuminate\Filesystem\Filesystem;
4			use Illuminate\Translation\LoaderInterface;
5			use Modules\Translation\Repositories\FileTranslationRepository as FileTranslationRepositoryInterface;
6
7			class FileTranslationRepository implements FileTranslationRepositoryInterface
8			{
9			/**
10			* @var Filesystem
11			*/
12			private $finder;
13			/**
14			* @var LoaderInterface
15			*/
16			private $loader;
17
18			public function __construct(Filesystem $finder, LoaderInterface $loader)
19			{
20			$this->finder = $finder;
21			$this->loader = $loader;
22			}
23
24			/**
25			* Get all the translations for all modules on disk
26			* @return array
27			*/
28			public function all()
29			{
30			$files = $this->getTranslationFilenamesFromPaths($this->loader->paths());
			0 ignored issues – show Bug introduced 2015-11-30 15:08 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Illuminate\Translation\LoaderInterface` as the method `paths()` does only exist in the following implementations of said interface: `Modules\Translation\Services\TranslationLoader`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
31
32			$translations = [];
33
34			foreach ($files as $locale => $files) {
35			foreach ($files as $namespace => $file) {
36			$trans = $this->finder->getRequire($file);
37			$trans = array_dot($trans);
38
39			foreach ($trans as $key => $value) {
40			$translations[$locale]["{$namespace}.{$key}"] = $value;
41			}
42			}
43			}
44
45			return $translations;
46			}
47
48			/**
49			* Get all of the names of the Translations files from an array of Paths.
50			* Returns [ 'translationkeyprefix' => 'filepath' ]
51			* @param array $paths
52			* @return array
53			*/
54			protected function getTranslationFilenamesFromPaths(array $paths)
55			{
56			$files = [];
57			$locales = config('laravellocalization.supportedLocales');
58
59			foreach ($paths as $hint => $path) {
60			foreach ($locales as $locale => $language) {
61			foreach ($this->finder->glob("{$path}/{$locale}/*.php") as $file) {
62			$category = str_replace(["$path/", ".php", "{$locale}/"], "", $file);
63			$category = str_replace("/", ".", $category);
64			$category = !is_int($hint) ? "{$hint}::{$category}" : $category;
65
66			$files[$locale][$category] = $file;
67			}
68			}
69			}
70
71			return $files;
72			}
73			}
74

AsgardCms / Translation

Push — master ( c62883...795ab8 )

FileTranslationRepository A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

3 Methods

Available Fixes

Duplication Side-by-Side

Filter issues like