RestlerEnhancer::getRoutePathRedecorationPattern() - Code Metrics - Inspection of "[WIP ]Feature/typo3 v9 master wip" - AOEpeople/TYPO3_Restler - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#50)

unknown

created 2020-02-14 06:46 UTC

RestlerEnhancer::getRoutePathRedecorationPattern() A

↳ Parent: RestlerEnhancer

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace Aoe\Restler\System\TYPO3;

use Aoe\Restler\System\RestlerBuilderAware;
use TYPO3\CMS\Core\Routing\Aspect\AspectInterface;
use TYPO3\CMS\Core\Routing\Enhancer\DecoratingEnhancerInterface;
use TYPO3\CMS\Core\Routing\RouteCollection;

/**
 * Puts the URLs which should be handled by restler into the current route collection
 * and maps them to the current default route.
 *
 * Please be aware that this seems to have some side effects on other DecoratingEnhancers like PageTypeDecorator
 * routeEnhancers:
 *   RestlerEnhancer:
 *     type: Restler
 *     default: '.json'
 */
class RestlerEnhancer extends RestlerBuilderAware implements DecoratingEnhancerInterface
{
    private $default;

    public function __construct($configuration)
    {
        parent::__construct(null);
        $default = $configuration['default'] ?? '';

        if (!is_string($default)) {
            throw new \InvalidArgumentException('default must be string', 1538327508);
        }

        $this->default = $default;
    }

    /**
     * Gets pattern that can be used to redecorate (undecorate)
     * a potential previously decorated route path.
     *
     * Example:
     * + route path: 'first/second.html'
     * + redecoration pattern: '(?:\.html|\.json)$'
     * -> 'first/second' might be the redecorated route path after
     *    applying the redecoration pattern to preg_match/preg_replace
     *
     * @return string regular expression pattern
     */
    public function getRoutePathRedecorationPattern(): string
    {
        return preg_quote($this->default, '#') . '$';
    }

    /**
     * Decorates route collection to be processed during URL resolving.
     * Executed before invoking routing enhancers.
     *
     * @param RouteCollection $collection
     * @param string $routePath URL path
     */
    public function decorateForMatching(RouteCollection $collection, string $routePath): void
    {
        // set path according to typo3/sysext/core/Classes/Routing/PageRouter.php:132
        $prefixedUrlPath = '/' . trim($routePath, '/');

        if ($this->isRestlerPrefix($prefixedUrlPath)) {
            $this->getRestlerBuilder()->build(null);

            if ($this->isRestlerUrl($prefixedUrlPath)) {
                $defaultRoute = $collection->get('default');
                $defaultRoute->setPath($prefixedUrlPath);
                $collection->add('restler', $defaultRoute);
/** @return stdClass|null */
function mayReturnNull() { }

function doesNotAcceptNull(stdClass $x) { }

// With potential error.
function withoutCheck() {
    $x = mayReturnNull();
    doesNotAcceptNull($x); // Potential error here.
}

// Safe - Alternative 1
function withCheck1() {
    $x = mayReturnNull();
    if ( ! $x instanceof stdClass) {
        throw new \LogicException('$x must be defined.');
    }
    doesNotAcceptNull($x);
}

// Safe - Alternative 2
function withCheck2() {
    $x = mayReturnNull();
    if ($x instanceof stdClass) {
        doesNotAcceptNull($x);
    }
}
            }
        }
    }

    /**
     * Decorates route collection during URL URL generation.
     * Executed before invoking routing enhancers.
     *
     * @param RouteCollection $collection
     * @param array $parameters query parameters
     */
    public function decorateForGeneration(RouteCollection $collection, array $parameters): void
    {
    }

    /**
     * @param AspectInterface[] $aspects
     */
    public function setAspects(array $aspects): void
    {
    }

    /**
     * @return AspectInterface[]
     */
    public function getAspects(): array
    {
        return [];
    }

    private function isRestlerUrl($uri): bool
    {
        return \Aoe\Restler\System\Restler\Routes::containsUrl($uri);
    }
}


1			<?php
2
3			namespace Aoe\Restler\System\TYPO3;
4
5			use Aoe\Restler\System\RestlerBuilderAware;
6			use TYPO3\CMS\Core\Routing\Aspect\AspectInterface;
7			use TYPO3\CMS\Core\Routing\Enhancer\DecoratingEnhancerInterface;
8			use TYPO3\CMS\Core\Routing\RouteCollection;
9
10			/**
11			* Puts the URLs which should be handled by restler into the current route collection
12			* and maps them to the current default route.
13			*
14			* Please be aware that this seems to have some side effects on other DecoratingEnhancers like PageTypeDecorator
15			* routeEnhancers:
16			* RestlerEnhancer:
17			* type: Restler
18			* default: '.json'
19			*/
20			class RestlerEnhancer extends RestlerBuilderAware implements DecoratingEnhancerInterface
21			{
22			private $default;
23
24			public function __construct($configuration)
25			{
26			parent::__construct(null);
27			$default = $configuration['default'] ?? '';
28
29			if (!is_string($default)) {
30			throw new \InvalidArgumentException('default must be string', 1538327508);
31			}
32
33			$this->default = $default;
34			}
35
36			/**
37			* Gets pattern that can be used to redecorate (undecorate)
38			* a potential previously decorated route path.
39			*
40			* Example:
41			* + route path: 'first/second.html'
42			* + redecoration pattern: '(?:\.html\|\.json)$'
43			* -> 'first/second' might be the redecorated route path after
44			* applying the redecoration pattern to preg_match/preg_replace
45			*
46			* @return string regular expression pattern
47			*/
48			public function getRoutePathRedecorationPattern(): string
49			{
50			return preg_quote($this->default, '#') . '$';
51			}
52
53			/**
54			* Decorates route collection to be processed during URL resolving.
55			* Executed before invoking routing enhancers.
56			*
57			* @param RouteCollection $collection
58			* @param string $routePath URL path
59			*/
60			public function decorateForMatching(RouteCollection $collection, string $routePath): void
61			{
62			// set path according to typo3/sysext/core/Classes/Routing/PageRouter.php:132
63			$prefixedUrlPath = '/' . trim($routePath, '/');
64
65			if ($this->isRestlerPrefix($prefixedUrlPath)) {
66			$this->getRestlerBuilder()->build(null);
67
68			if ($this->isRestlerUrl($prefixedUrlPath)) {
69			$defaultRoute = $collection->get('default');
70			$defaultRoute->setPath($prefixedUrlPath);
71			$collection->add('restler', $defaultRoute);
			0 ignored issues – show Bug introduced 2019-10-07 06:32 UTC by Report Bug Copy Issue Report It seems like `$defaultRoute` defined by `$collection->get('default')` on line `69` can be `null`; however, `Symfony\Component\Routing\RouteCollection::add()` does not accept `null`, maybe add an additional type check? Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: /** @return stdClass\|null */ function mayReturnNull() { } function doesNotAcceptNull(stdClass $x) { } // With potential error. function withoutCheck() { $x = mayReturnNull(); doesNotAcceptNull($x); // Potential error here. } // Safe - Alternative 1 function withCheck1() { $x = mayReturnNull(); if ( ! $x instanceof stdClass) { throw new \LogicException('$x must be defined.'); } doesNotAcceptNull($x); } // Safe - Alternative 2 function withCheck2() { $x = mayReturnNull(); if ($x instanceof stdClass) { doesNotAcceptNull($x); } } Loading history...
72			}
73			}
74			}
75
76			/**
77			* Decorates route collection during URL URL generation.
78			* Executed before invoking routing enhancers.
79			*
80			* @param RouteCollection $collection
81			* @param array $parameters query parameters
82			*/
83			public function decorateForGeneration(RouteCollection $collection, array $parameters): void
84			{
85			}
86
87			/**
88			* @param AspectInterface[] $aspects
89			*/
90			public function setAspects(array $aspects): void
91			{
92			}
93
94			/**
95			* @return AspectInterface[]
96			*/
97			public function getAspects(): array
98			{
99			return [];
100			}
101
102			private function isRestlerUrl($uri): bool
103			{
104			return \Aoe\Restler\System\Restler\Routes::containsUrl($uri);
105			}
106			}
107

AOEpeople / TYPO3_Restler

Pull Request — master (#50)

RestlerEnhancer::getRoutePathRedecorationPattern() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like