Stu\Module\Logging\StuLogger::getLogger()

Complexity

Conditions	2
Paths	2

Size

Total Lines	15
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	1
Bugs	0	Features	1

<?php
    private static function getLogger(): Logger
    {
        if (self::$logger === null) {
            self::$logger = new Logger('stu');
            self::$logger->pushHandler(

The method pushHandler() does not exist on null.

                new StreamHandler(
                    Init::getContainer()
                        ->get(StuConfigInterface::class)
                        ->getDebugSettings()
                        ->getLogfilePath()
                ),
            );
        }

        return self::$logger;

The expression return self::logger could return the type null which is incompatible with the type-hinted return Monolog\Logger. Consider adding an additional type-check to rule them out.

    }