EntryNode::onOpen() - Code Metrics - Inspection of "[!!!] DO NOT MERGE - Fluid 3.0 diff" - TYPO3/Fluid - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#470)

by Claus

created 2019-08-25 13:03 UTC

EntryNode::onOpen() A

↳ Parent: EntryNode

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	1
dl	0
loc	5
rs	10
c	0
b	0
f	0

<?php
declare(strict_types=1);
namespace TYPO3Fluid\Fluid\Core\Parser\SyntaxTree;

/*
 * This file belongs to the package "TYPO3 Fluid".
 * See LICENSE.txt that was shipped with this package.
 */

use TYPO3Fluid\Fluid\Component\AbstractComponent;
use TYPO3Fluid\Fluid\Component\ComponentInterface;
use TYPO3Fluid\Fluid\Component\EmbeddedComponentInterface;
use TYPO3Fluid\Fluid\Core\Rendering\RenderingContextInterface;

/**
 * Entry node. Used to represent either the root
 * entry point of a template tree, or a section
 * within the template tree. Common for EntryNode
 * is that:
 *
 * - It supports arguments that become variables
 * - It supports setting the name of the node which
 *   allows it to be returned from getNamedChild.
 * - It is an EmbeddedComponent which means that it
 *   is only rendered when explicitly calling the
 *   evaluate() method on the instance itself, it
 *   is not rendered when rendering the parent node.
 *
 * EntryNode instances can be resolved directly
 * from getTypedChildren to extract all sections
 * within a tree; or getNamedChild can be used
 * to extract an EntryNode (or an EntryNode child
 * of another EntryNode to any nesting depth by
 * using dotted path as argument for getNamedChild).
 */
class EntryNode extends AbstractComponent implements EmbeddedComponentInterface
{
    protected $escapeOutput = false;

    public function setName(?string $name): ComponentInterface
    {
        $this->name = $name;
        return $this;
    }

    public function evaluate(RenderingContextInterface $renderingContext)
    {
        $renderingContext = clone $renderingContext;
        $renderingContext->setVariableProvider(
            $renderingContext->getVariableProvider()->getScopeCopy(
                $this->getArguments()->setRenderingContext($renderingContext)->getArrayCopy()
            )
        );
        return parent::evaluate($renderingContext);
    }

    public function onOpen(RenderingContextInterface $renderingContext): ComponentInterface
    {
        $this->getArguments()->setRenderingContext($renderingContext)->validate();
        return parent::onOpen($renderingContext);
class Author {
    private $name;

    public function __construct($name) {
        $this->name = $name;
    }

    public function getName() {
        return $this->name;
    }
}

abstract class Post {
    public function getAuthor() {
        return 'Johannes';
    }
}

class BlogPost extends Post {
    public function getAuthor() {
        return new Author('Johannes');
    }
}

class ForumPost extends Post { /* ... */ }

function my_function(Post $post) {
    echo strtoupper($post->getAuthor());
}
    }
}


1			<?php
2			declare(strict_types=1);
3			namespace TYPO3Fluid\Fluid\Core\Parser\SyntaxTree;
4
5			/*
6			* This file belongs to the package "TYPO3 Fluid".
7			* See LICENSE.txt that was shipped with this package.
8			*/
9
10			use TYPO3Fluid\Fluid\Component\AbstractComponent;
11			use TYPO3Fluid\Fluid\Component\ComponentInterface;
12			use TYPO3Fluid\Fluid\Component\EmbeddedComponentInterface;
13			use TYPO3Fluid\Fluid\Core\Rendering\RenderingContextInterface;
14
15			/**
16			* Entry node. Used to represent either the root
17			* entry point of a template tree, or a section
18			* within the template tree. Common for EntryNode
19			* is that:
20			*
21			* - It supports arguments that become variables
22			* - It supports setting the name of the node which
23			* allows it to be returned from getNamedChild.
24			* - It is an EmbeddedComponent which means that it
25			* is only rendered when explicitly calling the
26			* evaluate() method on the instance itself, it
27			* is not rendered when rendering the parent node.
28			*
29			* EntryNode instances can be resolved directly
30			* from getTypedChildren to extract all sections
31			* within a tree; or getNamedChild can be used
32			* to extract an EntryNode (or an EntryNode child
33			* of another EntryNode to any nesting depth by
34			* using dotted path as argument for getNamedChild).
35			*/
36			class EntryNode extends AbstractComponent implements EmbeddedComponentInterface
37			{
38			protected $escapeOutput = false;
39
40			public function setName(?string $name): ComponentInterface
41			{
42			$this->name = $name;
43			return $this;
44			}
45
46			public function evaluate(RenderingContextInterface $renderingContext)
47			{
48			$renderingContext = clone $renderingContext;
49			$renderingContext->setVariableProvider(
50			$renderingContext->getVariableProvider()->getScopeCopy(
51			$this->getArguments()->setRenderingContext($renderingContext)->getArrayCopy()
52			)
53			);
54			return parent::evaluate($renderingContext);
55			}
56
57			public function onOpen(RenderingContextInterface $renderingContext): ComponentInterface
58			{
59			$this->getArguments()->setRenderingContext($renderingContext)->validate();
60			return parent::onOpen($renderingContext);
			0 ignored issues – show Bug Best Practice introduced 2019-08-07 16:29 UTC by Report Bug Copy Issue Report The return type of `return parent::onOpen($renderingContext);` (`TYPO3Fluid\Fluid\Core\Parser\SyntaxTree\EntryNode`) is incompatible with the return type declared by the interface `TYPO3Fluid\Fluid\Compone...ponentInterface::onOpen` of type `self`. If you return a value from a function or method, it should be a sub-type of the type that is given by the parent type f.e. an interface, or abstract method. This is more formally defined by the Lizkov substitution principle, and guarantees that classes that depend on the parent type can use any instance of a child type interchangably. This principle also belongs to the SOLID principles for object oriented design. Let’s take a look at an example: class Author { private $name; public function __construct($name) { $this->name = $name; } public function getName() { return $this->name; } } abstract class Post { public function getAuthor() { return 'Johannes'; } } class BlogPost extends Post { public function getAuthor() { return new Author('Johannes'); } } class ForumPost extends Post { /* ... */ } function my_function(Post $post) { echo strtoupper($post->getAuthor()); } Our function `my_function` expects a `Post` object, and outputs the author of the post. The base class `Post` returns a simple string and outputting a simple string will work just fine. However, the child class `BlogPost` which is a sub-type of `Post` instead decided to return an `object`, and is therefore violating the SOLID principles. If a `BlogPost` were passed to `my_function`, PHP would not complain, but ultimately fail when executing the `strtoupper` call in its body. Loading history...
61			}
62			}
63

TYPO3 / Fluid

Pull Request — master (#470)

EntryNode::onOpen() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like