MpaFirephpWrapper\View\Helper\FirephpHelper

Complexity

Total Complexity

2

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	__invoke()	0	4	1

<?php
/*
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
 * FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
 * COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
 * IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
 * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 */

namespace MpaFirephpWrapper\View\Helper;

use MpaFirephpWrapper\Service\FirephpWrapper;
use Zend\View\Helper\AbstractHelper;

class FirephpHelper extends AbstractHelper
{
    protected $firephp;

    /**
     * @param FirephpWrapper $firephp
     */
    public function __construct(FirephpWrapper $firephp)
    {
        $this->firephp = $firephp;
    }

    /**
     * @param        $object
     * @param  string $type
     * @param  string $label
     * @param  array  $options
     * @return FirephpWrapper
     */
    public function __invoke($object, $type = 'info', $label = null, $options = [])
    {
        return $this->firephp->write($object, $type, $label, $options);

It seems like $label defined by parameter $label on line 35 can also be of type string; however, MpaFirephpWrapper\Service\FirephpWrapper::write() does only seem to accept null, maybe add an additional type check?

    }
}