Respect\Rest\Routines\AbstractRoutine::__construct()

Complexity

Conditions	5
Paths	3

Size

Total Lines

12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	5

Importance

Changes

0

<?php
    public function __construct($callback)
    {
        if (is_string($callback) && class_exists($callback) && method_exists($callback, '__invoke')) {
            return $this->callback = $callback;

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

        }

        if (!is_callable($callback)) {
            throw new InvalidArgumentException('Routine callback must be... guess what... callable!');
        }

        $this->callback = $callback;
    }