BEAR\Resource\Module\LoggerModule

Complexity

Total Complexity

4

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	configure()	0	10	3

<?php

declare(strict_types=1);

namespace BEAR\Resource\Module;

use Ray\Di\AbstractModule;

final class LoggerModule extends AbstractModule
{
    /**
     * Log Level for Production
     *
     * Log only unsafe method.
     */
    const LOG_LEVEL_PROD = 0;

    /**
     * Log level for development L0
     *
     * Log all request
     */
    const LOG_LEVEL_DEV = 1;

    /**
     * Log level for development L1
     *
     * Log all request and request result
     */
    const LOG_LEVEL_DEV_RESPONSE = 2;

    /**
     * @var string
     */
    private $level;

    /**
     * @var AbstractModule
     */
    private $module;

    public function __construct(string $level, AbstractModule $module)
    {
        $this->level = $level;
        $this->module = $module;
        parent::__construct($module);
    }

    protected function configure()
    {
        if ($this->level === self::LOG_LEVEL_PROD) {

The strict comparison === seems to always evaluate to false as the types of $this->level (string) and self::LOG_LEVEL_PROD (integer) can never be identical. Maybe you want to use a loose comparison == instead?

            $this->bind(LoggerInterface::class)->to(ProdLogger::class);
        }
        if ($this->level === self::LOG_LEVEL_DEV) {

The strict comparison === seems to always evaluate to false as the types of $this->level (string) and self::LOG_LEVEL_DEV (integer) can never be identical. Maybe you want to use a loose comparison == instead?

            $this->bind(LoggerInterface::class)->to(DevZeroLogger::class);
        }
        $this->bind(LoggerInterface::class)->to(DevOneLogger::class);
    }
}