TwigInitializeListener::setFrontVaribales() - Code Metrics - Inspection of "Disable coverage" - nanasess/ec-cube - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Pull Request — experimental/sf (#31)

by Kentaro

created 2018-08-08 04:16 UTC

TwigInitializeListener::setFrontVaribales() B

↳ Parent: TwigInitializeListener

Complexity

Conditions	6
Paths	30

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	17
CRAP Score	6

Importance

Changes

Metric	Value
cc	6
nc	30
nop	1
dl	0
loc	31
rs	8.8017
c	0
b	0
f	0
ccs	17
cts	17
cp	1
crap	6

<?php

/*
 * This file is part of EC-CUBE
 *
 * Copyright(c) LOCKON CO.,LTD. All Rights Reserved.
 *
 * http://www.lockon.co.jp/
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Eccube\EventListener;

use Doctrine\ORM\NoResultException;
use Eccube\Common\EccubeConfig;
use Eccube\Entity\AuthorityRole;
use Eccube\Entity\Master\DeviceType;
use Eccube\Entity\Member;
use Eccube\Repository\AuthorityRoleRepository;
use Eccube\Repository\BaseInfoRepository;
use Eccube\Repository\Master\DeviceTypeRepository;
use Eccube\Repository\PageRepository;
use Eccube\Request\Context;
use SunCat\MobileDetectBundle\DeviceDetector\MobileDetector;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpKernel\Event\GetResponseEvent;
use Symfony\Component\HttpKernel\KernelEvents;
use Twig\Environment;

class TwigInitializeListener implements EventSubscriberInterface
{
    /**
     * @var Environment
     */
    protected $twig;

    /**
     * @var BaseInfoRepository
     */
    protected $baseInfoRepository;

    /**
     * @var DeviceTypeRepository
     */
    protected $deviceTypeRepository;

    /**
     * @var PageRepository
     */
    protected $pageRepository;

    /**
     * @var Context
     */
    protected $requestContext;

    /**
     * @var AuthorityRoleRepository
     */
    private $authorityRoleRepository;

    /**
     * @var EccubeConfig
     */
    private $eccubeConfig;

    /**
     * @var MobileDetector
     */
    private $mobileDetector;

    /**
     * TwigInitializeListener constructor.
     *
     * @param Environment $twig
     * @param BaseInfoRepository $baseInfoRepository
     * @param PageRepository $pageRepository
     * @param DeviceTypeRepository $deviceTypeRepository
     * @param AuthorityRoleRepository $authorityRoleRepository
     * @param EccubeConfig $eccubeConfig
     * @param Context $context
     * @param MobileDetector $mobileDetector
     */
    public function __construct(
        Environment $twig,
        BaseInfoRepository $baseInfoRepository,
        PageRepository $pageRepository,
        DeviceTypeRepository $deviceTypeRepository,
        AuthorityRoleRepository $authorityRoleRepository,
        EccubeConfig $eccubeConfig,
        Context $context,
        MobileDetector $mobileDetector
    ) {
        $this->twig = $twig;
        $this->baseInfoRepository = $baseInfoRepository;
        $this->pageRepository = $pageRepository;
        $this->deviceTypeRepository = $deviceTypeRepository;
        $this->authorityRoleRepository = $authorityRoleRepository;
        $this->eccubeConfig = $eccubeConfig;
        $this->requestContext = $context;
        $this->mobileDetector = $mobileDetector;
    }

    /**
     * @param GetResponseEvent $event
     *
     * @throws NoResultException
     * @throws \Doctrine\ORM\NonUniqueResultException
     */
    public function onKernelRequest(GetResponseEvent $event)
    {
        $globals = $this->twig->getGlobals();
        if (array_key_exists('BaseInfo', $globals) && $globals['BaseInfo'] === null) {
            $this->twig->addGlobal('BaseInfo', $this->baseInfoRepository->get());
        }

        if (!$event->isMasterRequest()) {
            return;
        }

        if ($this->requestContext->isAdmin()) {
            $this->setAdminGlobals($event);
        } else {
            $this->setFrontVaribales($event);
        }
    }

    /**
     * @param GetResponseEvent $event
     *
     * @throws \Doctrine\ORM\NonUniqueResultException
     */
    public function setFrontVaribales(GetResponseEvent $event)
    {
        /** @var \Symfony\Component\HttpFoundation\ParameterBag $attributes */
        $attributes = $event->getRequest()->attributes;
        $route = $attributes->get('_route');
        if ($route == 'user_data') {
            $routeParams = $attributes->get('_route_params', []);
            $route = isset($routeParams['route']) ? $routeParams['route'] : $attributes->get('route', '');
        }

        $type = DeviceType::DEVICE_TYPE_PC;
        if ($this->mobileDetector->isMobile()) {
            $type = DeviceType::DEVICE_TYPE_SP;
        }
        $DeviceType = $this->deviceTypeRepository->find($type);

        try {
            $Page = $this->pageRepository->getByUrl($DeviceType, $route);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        } catch (NoResultException $e) {
            try {
                log_info('fallback to PC layout');
                $DeviceType = $this->deviceTypeRepository->find(DeviceType::DEVICE_TYPE_PC);
                $Page = $this->pageRepository->getByUrl($DeviceType, $route);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            } catch (NoResultException $e) {
                $Page = $this->pageRepository->newPage($DeviceType);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            }
        }

        $this->twig->addGlobal('Page', $Page);
        $this->twig->addGlobal('title', $Page->getName());
    }

    /**
     * @param GetResponseEvent $event
     */
    public function setAdminGlobals(GetResponseEvent $event)
    {
        // メニュー表示用配列.
        $menus = [];
        $this->twig->addGlobal('menus', $menus);

        // メニューの権限制御.
        $Member = $this->requestContext->getCurrentUser();
        $AuthorityRoles = [];
        if ($Member instanceof Member) {
            $AuthorityRoles = $this->authorityRoleRepository->findBy(['Authority' => $this->requestContext->getCurrentUser()->getAuthority()]);
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
        }
        $roles = array_map(function (AuthorityRole $AuthorityRole) use ($event) {
            return $event->getRequest()->getBaseUrl().'/'.$this->eccubeConfig['eccube_admin_route'].$AuthorityRole->getDenyUrl();
        }, $AuthorityRoles);
        $this->twig->addGlobal('AuthorityRoles', $roles);
    }

    /**
     * {@inheritdoc}
     */
    public static function getSubscribedEvents()
    {
        return [
            KernelEvents::REQUEST => [
                // SecurityServiceProviderで、認証処理が完了した後に実行.
                ['onKernelRequest', 6],
            ],
        ];
    }
}


1		<?php
2
3		/*
4		* This file is part of EC-CUBE
5		*
6		* Copyright(c) LOCKON CO.,LTD. All Rights Reserved.
7		*
8		* http://www.lockon.co.jp/
9		*
10		* For the full copyright and license information, please view the LICENSE
11		* file that was distributed with this source code.
12		*/
13
14		namespace Eccube\EventListener;
15
16		use Doctrine\ORM\NoResultException;
17		use Eccube\Common\EccubeConfig;
18		use Eccube\Entity\AuthorityRole;
19		use Eccube\Entity\Master\DeviceType;
20		use Eccube\Entity\Member;
21		use Eccube\Repository\AuthorityRoleRepository;
22		use Eccube\Repository\BaseInfoRepository;
23		use Eccube\Repository\Master\DeviceTypeRepository;
24		use Eccube\Repository\PageRepository;
25		use Eccube\Request\Context;
26		use SunCat\MobileDetectBundle\DeviceDetector\MobileDetector;
27		use Symfony\Component\EventDispatcher\EventSubscriberInterface;
28		use Symfony\Component\HttpKernel\Event\GetResponseEvent;
29		use Symfony\Component\HttpKernel\KernelEvents;
30		use Twig\Environment;
31
32		class TwigInitializeListener implements EventSubscriberInterface
33		{
34		/**
35		* @var Environment
36		*/
37		protected $twig;
38
39		/**
40		* @var BaseInfoRepository
41		*/
42		protected $baseInfoRepository;
43
44		/**
45		* @var DeviceTypeRepository
46		*/
47		protected $deviceTypeRepository;
48
49		/**
50		* @var PageRepository
51		*/
52		protected $pageRepository;
53
54		/**
55		* @var Context
56		*/
57		protected $requestContext;
58
59		/**
60		* @var AuthorityRoleRepository
61		*/
62		private $authorityRoleRepository;
63
64		/**
65		* @var EccubeConfig
66		*/
67		private $eccubeConfig;
68	431
69		/**
70		* @var MobileDetector
71		*/
72		private $mobileDetector;
73
74		/**
75		* TwigInitializeListener constructor.
76		*
77	431	* @param Environment $twig
78	431	* @param BaseInfoRepository $baseInfoRepository
79	431	* @param PageRepository $pageRepository
80	431	* @param DeviceTypeRepository $deviceTypeRepository
81	431	* @param AuthorityRoleRepository $authorityRoleRepository
82	431	* @param EccubeConfig $eccubeConfig
83	431	* @param Context $context
84		* @param MobileDetector $mobileDetector
85		*/
86		public function __construct(
87		Environment $twig,
88		BaseInfoRepository $baseInfoRepository,
89	429	PageRepository $pageRepository,
90		DeviceTypeRepository $deviceTypeRepository,
91	429	AuthorityRoleRepository $authorityRoleRepository,
92	429	EccubeConfig $eccubeConfig,
93	429	Context $context,
94		MobileDetector $mobileDetector
95		) {
96	429	$this->twig = $twig;
97	118	$this->baseInfoRepository = $baseInfoRepository;
98		$this->pageRepository = $pageRepository;
99		$this->deviceTypeRepository = $deviceTypeRepository;
100	428	$this->authorityRoleRepository = $authorityRoleRepository;
101	270	$this->eccubeConfig = $eccubeConfig;
102		$this->requestContext = $context;
103	159	$this->mobileDetector = $mobileDetector;
104		}
105
106		/**
107		* @param GetResponseEvent $event
108		*
109		* @throws NoResultException
110	159	* @throws \Doctrine\ORM\NonUniqueResultException
111		*/
112		public function onKernelRequest(GetResponseEvent $event)
113	159	{
114	159	$globals = $this->twig->getGlobals();
115	159	if (array_key_exists('BaseInfo', $globals) && $globals['BaseInfo'] === null) {
116	2	$this->twig->addGlobal('BaseInfo', $this->baseInfoRepository->get());
117	2	}
118
119		if (!$event->isMasterRequest()) {
120		return;
121	159	}
122
123		if ($this->requestContext->isAdmin()) {
124	159	$this->setAdminGlobals($event);
125	159	} else {
126	159	$this->setFrontVaribales($event);
127	159	}
128	159	}
129	159
130	159	/**
131	159	* @param GetResponseEvent $event
132	159	*
133	159	* @throws \Doctrine\ORM\NonUniqueResultException
134	159	*/
135	159	public function setFrontVaribales(GetResponseEvent $event)
136	159	{
137	93	/** @var \Symfony\Component\HttpFoundation\ParameterBag $attributes */
138	93	$attributes = $event->getRequest()->attributes;
139		$route = $attributes->get('_route');
140		if ($route == 'user_data') {
141	159	$routeParams = $attributes->get('_route_params', []);
142	149	$route = isset($routeParams['route']) ? $routeParams['route'] : $attributes->get('route', '');
143		}
144
145		$type = DeviceType::DEVICE_TYPE_PC;
146		if ($this->mobileDetector->isMobile()) {
147		$type = DeviceType::DEVICE_TYPE_SP;
148	270	}
149		$DeviceType = $this->deviceTypeRepository->find($type);
150
151	270	try {
152	270	$Page = $this->pageRepository->getByUrl($DeviceType, $route);
		0 ignored issues – show Documentation introduced 2018-08-08 04:23 UTC by Report Bug Copy Issue Report `$DeviceType` is of type `object\|null`, but the function expects a `object<Eccube\Entity\Master\DeviceType>`. 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...
153		} catch (NoResultException $e) {
154		try {
155	270	log_info('fallback to PC layout');
156	270	$DeviceType = $this->deviceTypeRepository->find(DeviceType::DEVICE_TYPE_PC);
157	270	$Page = $this->pageRepository->getByUrl($DeviceType, $route);
		0 ignored issues – show Documentation introduced 2018-08-08 04:23 UTC by Report Bug Copy Issue Report `$DeviceType` is of type `object\|null`, but the function expects a `object<Eccube\Entity\Master\DeviceType>`. 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...
158	269	} catch (NoResultException $e) {
159		$Page = $this->pageRepository->newPage($DeviceType);
		0 ignored issues – show Documentation introduced 2018-06-05 10:54 UTC by Report Bug Copy Issue Report `$DeviceType` is of type `object\|null`, but the function expects a `object<Eccube\Entity\Master\DeviceType>`. 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...
160	270	}
161	3	}
162	270
163	270	$this->twig->addGlobal('Page', $Page);
164		$this->twig->addGlobal('title', $Page->getName());
165		}
166
167		/**
168		* @param GetResponseEvent $event
169	1	*/
170		public function setAdminGlobals(GetResponseEvent $event)
171		{
172	1	// メニュー表示用配列.
173		$menus = [];
174		$this->twig->addGlobal('menus', $menus);
175
176		// メニューの権限制御.
177		$Member = $this->requestContext->getCurrentUser();
178		$AuthorityRoles = [];
179		if ($Member instanceof Member) {
180		$AuthorityRoles = $this->authorityRoleRepository->findBy(['Authority' => $this->requestContext->getCurrentUser()->getAuthority()]);
		0 ignored issues – show Bug introduced 2018-06-26 07:27 UTC by Report Bug Copy Issue Report The method `getAuthority` does only exist in `Eccube\Entity\Member`, but not in `Eccube\Entity\Customer`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
181		}
182		$roles = array_map(function (AuthorityRole $AuthorityRole) use ($event) {
183		return $event->getRequest()->getBaseUrl().'/'.$this->eccubeConfig['eccube_admin_route'].$AuthorityRole->getDenyUrl();
184		}, $AuthorityRoles);
185		$this->twig->addGlobal('AuthorityRoles', $roles);
186		}
187
188		/**
189		* {@inheritdoc}
190		*/
191		public static function getSubscribedEvents()
192		{
193		return [
194		KernelEvents::REQUEST => [
195		// SecurityServiceProviderで、認証処理が完了した後に実行.
196		['onKernelRequest', 6],
197		],
198		];
199		}
200		}
201

nanasess / ec-cube

Pull Request — experimental/sf (#31)

TwigInitializeListener::setFrontVaribales() B

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like