Cubiche\Core\Visitor\LinkedVisitor

Complexity

Total Complexity

12

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	3
A	canHandlerVisitee()	0	4	2
A	visit()	0	4	1
A	doVisit()	0	12	4
A	notSupportedVisiteeException()	0	8	2

<?php

/**
 * This file is part of the Cubiche package.
 *
 * Copyright (c) Cubiche
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace Cubiche\Core\Visitor;

/**
 * Abstract Linked Visitor class.
 *
 * @author Karel Osorio Ramírez <osorioramirez@gmail.com>
 */
abstract class LinkedVisitor extends DynamicDispatchVisitor
{
    /**
     * @var LinkedVisitor
     */
    private $next = null;

    /**
     * @var LinkedVisitor
     */
    private $parent = null;

    /**
     * @param LinkedVisitor                  $next
     * @param ResolverVisitorMethodInterface $resolver
     */
    public function __construct(
        LinkedVisitor $next = null,
        ResolverVisitorMethodInterface $resolver = null
    ) {
        $this->next = $next === null ? new NullVisitor() : $next;
        $this->next->parent = $this;
        $this->parent = null;
        $this->resolver = $resolver === null ? new ResolverVisitorMethod() : $resolver;
    }

    /**
     * {@inheritdoc}
     */
    public function canHandlerVisitee(VisiteeInterface $visitee)
    {
        return parent::canHandlerVisitee($visitee) || $this->next->canHandlerVisitee($visitee);
    }

    /**
     * {@inheritdoc}
     */
    public function visit(VisiteeInterface $visitee)
    {
        return $this->doVisit($visitee, \func_get_args());
    }

    /**
     * @param VisiteeInterface $visitee
     * @param array            $args
     * @param bool             $fromParent
     */
    protected function doVisit(VisiteeInterface $visitee, array $args, $fromParent = false)
    {
        if (!$fromParent && $this->parent !== null) {
            return \call_user_func_array(array($this->parent, 'visit'), $args);
        }

        if (parent::canHandlerVisitee($visitee)) {

It seems like you call parent on a different method (canHandlerVisitee() instead of doVisit()). Are you sure this is correct? If so, you might want to change this to $this->canHandlerVisitee().

            return $this->visitWith($visitee, $args);
        }

        return $this->next->doVisit($visitee, $args, true);
    }

    /**
     * {@inheritdoc}
     */
    protected function notSupportedVisiteeException(VisiteeInterface $visitee)
    {
        if ($this->parent !== null) {
            return $this->parent->notSupportedVisiteeException($visitee);
        }

        return parent::notSupportedVisiteeException($visitee);
    }
}