Bernard\Router\SimpleReceiverResolver::resolve()

Complexity

Conditions	5
Paths	6

Size

Total Lines

21

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	6.3183

Importance

Changes

0

<?php
    public function resolve($receiver, Envelope $envelope)
    {
        if (null === $receiver) {
            return null;
        }

        if ($receiver instanceof Receiver) {
            return $receiver;
        }

        if (is_callable($receiver) == false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            $receiver = [$receiver, lcfirst($envelope->getName())];
        }

        // Receiver is still not a callable which means it's not a valid receiver.
        if (is_callable($receiver) == false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            return null;
        }

        return new Receiver\CallableReceiver($receiver);
    }
}