DeferredNodeVisitor::leaveNode() - Code Metrics - Inspection of "Add Twig 3 support and rename package" - rybakit/twig-deferred-extension - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#9)

by Eugene

created 2020-06-02 22:45 UTC

DeferredNodeVisitor::leaveNode() A

↳ Parent: DeferredNodeVisitor

Complexity

Conditions	3
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	9
rs	9.9666
c	0
b	0
f	0
cc	3
nc	2
nop	2

<?php

/**
 * This file is part of the rybakit/twig-deferred-extension package.
 *
 * (c) Eugene Leonovich <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace Twig\DeferredExtension;

use Twig\Environment;
use Twig\Node\ModuleNode;
use Twig\Node\Node;
use Twig\NodeVisitor\NodeVisitorInterface;

final class DeferredNodeVisitor implements NodeVisitorInterface
{
    private $hasDeferred = false;

    public function enterNode(Node $node, Environment $env) : Node
    {
        if (!$this->hasDeferred && $node instanceof DeferredBlockNode) {
            $this->hasDeferred = true;
        }

        return $node;
    }

    public function leaveNode(Node $node, Environment $env) : ?Node
    {
        if ($this->hasDeferred && $node instanceof ModuleNode) {
            $node->setNode('display_end', new Node([new DeferredNode(), $node->getNode('display_end')]));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $this->hasDeferred = false;
        }

        return $node;
    }

    public function getPriority() : int
    {
        return 0;
    }
}


1			<?php
2
3			/**
4			* This file is part of the rybakit/twig-deferred-extension package.
5			*
6			* (c) Eugene Leonovich <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			declare(strict_types=1);
13
14			namespace Twig\DeferredExtension;
15
16			use Twig\Environment;
17			use Twig\Node\ModuleNode;
18			use Twig\Node\Node;
19			use Twig\NodeVisitor\NodeVisitorInterface;
20
21			final class DeferredNodeVisitor implements NodeVisitorInterface
22			{
23			private $hasDeferred = false;
24
25			public function enterNode(Node $node, Environment $env) : Node
26			{
27			if (!$this->hasDeferred && $node instanceof DeferredBlockNode) {
28			$this->hasDeferred = true;
29			}
30
31			return $node;
32			}
33
34			public function leaveNode(Node $node, Environment $env) : ?Node
35			{
36			if ($this->hasDeferred && $node instanceof ModuleNode) {
37			$node->setNode('display_end', new Node([new DeferredNode(), $node->getNode('display_end')]));
			0 ignored issues – show Documentation introduced 2020-06-02 17:36 UTC by Report Bug Copy Issue Report `new \Twig\Node\Node(arra...etNode('display_end')))` is of type `object<Twig\Node\Node>`, but the function expects a `object<self>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
38			$this->hasDeferred = false;
39			}
40
41			return $node;
42			}
43
44			public function getPriority() : int
45			{
46			return 0;
47			}
48			}
49

rybakit / twig-deferred-extension

Pull Request — master (#9)

DeferredNodeVisitor::leaveNode() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like