RedirectFrontendNodeRoutePartHandler::startsWith() - Code Metrics - Inspection of "[TASK] Code cleanup" - mocdk/MOC.Redirects - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 05a7a6...c5adec )

by Aske

created 2016-03-09 05:33 UTC

RedirectFrontendNodeRoutePartHandler::startsWith() A

↳ Parent: RedirectFrontendNodeRoutePartHandler::matchValue()

Complexity

Conditions	2
Paths	2

Size

Total Lines	3
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	3
rs	10
cc	2
eloc	2
nc	2
nop	2

<?php
namespace MOC\Redirects\Routing;

use Doctrine\ORM\QueryBuilder;
use TYPO3\Flow\Annotations as Flow;
use TYPO3\Flow\Core\Bootstrap;
use TYPO3\Flow\Http\Uri;
use TYPO3\Flow\Mvc\Routing\DynamicRoutePart;
use TYPO3\TYPO3CR\Domain\Model\NodeData;

/**
 * A route part handler for finding nodes specifically in the website's frontend.
 */
class RedirectFrontendNodeRoutePartHandler extends DynamicRoutePart
{

    /**
     * @Flow\Inject
     * @var \Doctrine\Common\Persistence\ObjectManager
     */
    protected $entityManager;

    /**
     * @Flow\Inject
     * @var Bootstrap
     */
    protected $bootstrap;

    /**
     * @param string $requestPath The request path to be matched
     * @return string value to match
     */
    protected function findValueToMatch($requestPath)
    {
        return $requestPath;
    }

    /**
     * @param string $requestPath
     * @return boolean TRUE if the $requestPath could be matched, otherwise FALSE
     */
    protected function matchValue($requestPath)
    {
        /** @var Uri $uri */
        $uri = $this->bootstrap->getActiveRequestHandler()->getHttpRequest()->getUri();
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $relativeUrl = rtrim($uri->getPath(), '/');
        $relativeUrlWithQueryString = $relativeUrl . ($uri->getQuery() ? '?' . $uri->getQuery() : '');
        $absoluteUrl = $uri->getHost() . $relativeUrl;
        $absoluteUrlWithQueryString = $uri->getHost() . $relativeUrlWithQueryString;

        if (empty($relativeUrl)) {
            return false;
        }

        /** @var QueryBuilder $queryBuilder */
        $queryBuilder = $this->entityManager->createQueryBuilder();
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $queryBuilder->select('n')
            ->distinct()
            ->from('TYPO3\TYPO3CR\Domain\Model\NodeData', 'n')
            ->where('n.workspace = :workspace')
            ->setParameter('workspace', 'live')
            ->andWhere('n.properties LIKE :relativeUrl')
            ->setParameter('relativeUrl', '%"redirectUrl"%' . str_replace('/', '\\\\\\/', ltrim($relativeUrl, '/')) . '%');

        $query = $queryBuilder->getQuery();
        $nodes = $query->getResult();
        if (empty($nodes)) {
            return false;
        }

        foreach ($nodes as $node) {
            /** @var NodeData $node */
            // Prevent partial matches
            $redirectUrl = trim(preg_replace('#^https?://#', '', $node->getProperty('redirectUrl')), '/');
            if (in_array($redirectUrl, array($relativeUrl, $relativeUrlWithQueryString, $absoluteUrl, $absoluteUrlWithQueryString), true)) {
                $matchingNode = $node;
                break;
            }
        }

        if (!isset($matchingNode)) {
            return false;
        }

        $this->setName('node');
        $this->value = $matchingNode->getPath();

        return true;
    }

    /**
     * @param string $value value to resolve
     * @return boolean TRUE if value could be resolved successfully, otherwise FALSE.
     */
    protected function resolveValue($value)
    {
        return false;
    }

}


1			<?php
2			namespace MOC\Redirects\Routing;
3
4			use Doctrine\ORM\QueryBuilder;
5			use TYPO3\Flow\Annotations as Flow;
6			use TYPO3\Flow\Core\Bootstrap;
7			use TYPO3\Flow\Http\Uri;
8			use TYPO3\Flow\Mvc\Routing\DynamicRoutePart;
9			use TYPO3\TYPO3CR\Domain\Model\NodeData;
10
11			/**
12			* A route part handler for finding nodes specifically in the website's frontend.
13			*/
14			class RedirectFrontendNodeRoutePartHandler extends DynamicRoutePart
15			{
16
17			/**
18			* @Flow\Inject
19			* @var \Doctrine\Common\Persistence\ObjectManager
20			*/
21			protected $entityManager;
22
23			/**
24			* @Flow\Inject
25			* @var Bootstrap
26			*/
27			protected $bootstrap;
28
29			/**
30			* @param string $requestPath The request path to be matched
31			* @return string value to match
32			*/
33			protected function findValueToMatch($requestPath)
34			{
35			return $requestPath;
36			}
37
38			/**
39			* @param string $requestPath
40			* @return boolean TRUE if the $requestPath could be matched, otherwise FALSE
41			*/
42			protected function matchValue($requestPath)
43			{
44			/** @var Uri $uri */
45			$uri = $this->bootstrap->getActiveRequestHandler()->getHttpRequest()->getUri();
			0 ignored issues – show Bug introduced 2015-12-02 10:00 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `TYPO3\Flow\Core\RequestHandlerInterface` as the method `getHttpRequest()` does only exist in the following implementations of said interface: `TYPO3\Flow\Http\RequestHandler`, `TYPO3\Setup\Core\RequestHandler`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
46			$relativeUrl = rtrim($uri->getPath(), '/');
47			$relativeUrlWithQueryString = $relativeUrl . ($uri->getQuery() ? '?' . $uri->getQuery() : '');
48			$absoluteUrl = $uri->getHost() . $relativeUrl;
49			$absoluteUrlWithQueryString = $uri->getHost() . $relativeUrlWithQueryString;
50
51			if (empty($relativeUrl)) {
52			return false;
53			}
54
55			/** @var QueryBuilder $queryBuilder */
56			$queryBuilder = $this->entityManager->createQueryBuilder();
			0 ignored issues – show Bug introduced 2015-12-02 10:00 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Doctrine\Common\Persistence\ObjectManager` as the method `createQueryBuilder()` does only exist in the following implementations of said interface: `Doctrine\ORM\Decorator\EntityManagerDecorator`, `Doctrine\ORM\EntityManager`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
57			$queryBuilder->select('n')
58			->distinct()
59			->from('TYPO3\TYPO3CR\Domain\Model\NodeData', 'n')
60			->where('n.workspace = :workspace')
61			->setParameter('workspace', 'live')
62			->andWhere('n.properties LIKE :relativeUrl')
63			->setParameter('relativeUrl', '%"redirectUrl"%' . str_replace('/', '\\\\\\/', ltrim($relativeUrl, '/')) . '%');
64
65			$query = $queryBuilder->getQuery();
66			$nodes = $query->getResult();
67			if (empty($nodes)) {
68			return false;
69			}
70
71			foreach ($nodes as $node) {
72			/** @var NodeData $node */
73			// Prevent partial matches
74			$redirectUrl = trim(preg_replace('#^https?://#', '', $node->getProperty('redirectUrl')), '/');
75			if (in_array($redirectUrl, array($relativeUrl, $relativeUrlWithQueryString, $absoluteUrl, $absoluteUrlWithQueryString), true)) {
76			$matchingNode = $node;
77			break;
78			}
79			}
80
81			if (!isset($matchingNode)) {
82			return false;
83			}
84
85			$this->setName('node');
86			$this->value = $matchingNode->getPath();
87
88			return true;
89			}
90
91			/**
92			* @param string $value value to resolve
93			* @return boolean TRUE if value could be resolved successfully, otherwise FALSE.
94			*/
95			protected function resolveValue($value)
96			{
97			return false;
98			}
99
100			}
101

mocdk / MOC.Redirects

Push — master ( 05a7a6...c5adec )

RedirectFrontendNodeRoutePartHandler::startsWith() A

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like