Coco\SourceWatcher\Utils\i18n

Complexity

Total Complexity

4

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getText()	0	17	2
A	getInstance()	0	7	2

<?php

namespace Coco\SourceWatcher\Utils;

use Symfony\Component\Yaml\Yaml;

class i18n
{
    /**
     * @var i18n|null
     */
    private static ?i18n $instance = null;

    /**
     * @return i18n
     */
    public static function getInstance () : i18n
    {
        if ( is_null( static::$instance ) ) {

Since $instance is declared private, accessing it with static will lead to errors in possible sub-classes; you can either use self, or increase the visibility of $instance to at least protected.

            static::$instance = new static;
        }

        return static::$instance;

The expression return static::instance could return the type null which is incompatible with the type-hinted return Coco\SourceWatcher\Utils\i18n. Consider adding an additional type-check to rule them out.

    }

    public function getText ( string $language = "en_US", string $className, string $entry ) : string
    {
        $result = "";

The assignment to $result is dead and can be removed.

        $data = Yaml::parseFile( __DIR__ . sprintf( "/../../resources/Locales/translations_%s.yml", $language ) );

        $classPathParts = explode( "\\", $className );

        $dataCopy = $data;

        foreach ( $classPathParts as $currentPart ) {
            $dataCopy = $dataCopy[$currentPart];
        }

        $result = $dataCopy[$entry];

        return $result;
    }
}