BlueRegister\Log::__construct()

Complexity

Conditions	5
Paths	4

Size

Total Lines	20
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	5

Importance

Changes

0

<?php
    public function __construct($logObject)
    {
        switch (true) {
            case $logObject instanceof LogInterface:
                $this->log = $logObject;
                break;

            case is_string($logObject) && $this->classExists($logObject):
                $this->log = new $logObject();

                if (!$this->log instanceof LogInterface) {
                    $message = 'Log should be instance of SimpleLog\LogInterface: ' . get_class($this->log);
                    throw new \LogicException($message);
                }

                break;

            default:
                throw new \LogicException('Cannot create Log instance: ' . get_class($logObject));

It seems like $logObject can also be of type string; however, parameter $object of get_class() does only seem to accept object, maybe add an additional type check?

                break;
        }