Eccube\Application::initRendering()

Complexity

Conditions	20
Paths	1

Size

Total Lines	139
Code Lines	85

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	73
CRAP Score	20.2679

Importance

Changes

0

<?php
    public function initRendering()

Missing function doc comment

    {
        $this->register(new \Silex\Provider\TwigServiceProvider(), array(
            'twig.form.templates' => array('Form/form_layout.twig'),
        ));
        $this->extend('twig', function (\Twig_Environment $twig, \Silex\Application $app) {
            $twig->addExtension(new \Eccube\Twig\Extension\EccubeExtension($app));
            $twig->addExtension(new \Twig_Extension_StringLoader());

            return $twig;
        });

        $this->before(function (Request $request, \Silex\Application $app) {
            $app['admin'] = $app['front'] = false;
            $pathinfo = rawurldecode($request->getPathInfo());
            if (strpos($pathinfo, '/'.trim($app['config']['admin_route'], '/').'/') === 0) {
                $app['admin'] = true;
            } else {
                $app['front'] = true;
            }

            // フロント or 管理画面ごとにtwigの探索パスを切り替える.
            if ($app->isAdminRequest()) {

It seems like you code against a specific sub-type and not the parent class Silex\Application as the method isAdminRequest() does only exist in the following sub-classes of Silex\Application: Eccube\Application, Eccube\InstallApplication. Maybe you want to instanceof check for one of these explicitly?

                if (file_exists(__DIR__.'/../../app/template/admin')) {
                    $paths[] = __DIR__.'/../../app/template/admin';

$paths was never initialized. Although not strictly required by PHP, it is generally a good practice to add $paths = array(); before regardless.

                }
                $paths[] = $app['config']['template_admin_realdir'];

The variable $paths does not seem to be defined for all execution paths leading up to this point.

                $paths[] = __DIR__.'/../../app/Plugin';
                $cacheDir =  __DIR__.'/../../app/cache/twig/admin';
            } else {
                // モバイル端末時、smartphoneディレクトリを探索パスに追加する.
                if ($app['mobile_detect.device_type'] == \Eccube\Entity\Master\DeviceType::DEVICE_TYPE_SP) {
                    if (file_exists(__DIR__.'/../../app/template/smartphone')) {
                        $paths[] = __DIR__.'/../../app/template/smartphone';
                    }
                    $paths[] = __DIR__.'/Resource/template/smartphone';
                }

                if (file_exists($app['config']['template_realdir'])) {
                    $paths[] = $app['config']['template_realdir'];
                }
                $paths[] = $app['config']['template_default_realdir'];
                $paths[] = __DIR__.'/../../app/Plugin';
                $cacheDir =  __DIR__.'/../../app/cache/twig/'.$app['config']['template_code'];
            }
            $app['twig']->setCache($app['debug'] ? null : $cacheDir);
            $app['twig.loader']->addLoader(new \Twig_Loader_Filesystem($paths));

            // 管理画面のIP制限チェック.
            if ($app->isAdminRequest()) {

It seems like you code against a specific sub-type and not the parent class Silex\Application as the method isAdminRequest() does only exist in the following sub-classes of Silex\Application: Eccube\Application, Eccube\InstallApplication. Maybe you want to instanceof check for one of these explicitly?

                // IP制限チェック
                $allowHost = $app['config']['admin_allow_host'];
                if (count($allowHost) > 0) {
                    if (array_search($app['request_stack']->getCurrentRequest()->getClientIp(), $allowHost) === false) {
                        throw new \Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException();
                    }
                }
            }
        }, self::EARLY_EVENT);

        // twigのグローバル変数を定義.
        $this->on(\Symfony\Component\HttpKernel\KernelEvents::CONTROLLER, function (\Symfony\Component\HttpKernel\Event\FilterControllerEvent $event) {
            // 未ログイン時にマイページや管理画面以下にアクセスするとSubRequestで実行されるため,
            // $event->isMasterRequest()ではなく、グローバル変数が初期化済かどうかの判定を行う
            if (isset($this['twig_global_initialized']) && $this['twig_global_initialized'] === true) {
                return;
            }
            // ショップ基本情報
            $this['twig']->addGlobal('BaseInfo', $this[BaseInfo::class]);

            if ($this->isAdminRequest()) {
                // 管理画面
                // 管理画面メニュー
                $menus = array('', '', '');
                $this['twig']->addGlobal('menus', $menus);

                $Member = $this->user();
                if (is_object($Member)) {
                    // ログインしていれば管理者のロールを取得
                    $AuthorityRoles = $this['eccube.repository.authority_role']->findBy(array('Authority' => $Member->getAuthority()));

                    $roles = array();
                    $request = $event->getRequest();
                    foreach ($AuthorityRoles as $AuthorityRole) {
                        // 管理画面でメニュー制御するため相対パス全てをセット
                        $roles[] = $request->getBaseUrl().'/'.$this['config']['admin_route'].$AuthorityRole->getDenyUrl();
                    }

                    $this['twig']->addGlobal('AuthorityRoles', $roles);
                }

Blank line found at end of control structure

            } else {
                // フロント画面
                $request = $event->getRequest();
                $route = $request->attributes->get('_route');

                // ユーザ作成画面
                if ($route === 'user_data') {
                    $params = $request->attributes->get('_route_params');
                    $route = $params['route'];
                    // プレビュー画面
                } elseif ($request->get('preview')) {
                    $route = 'preview';
                }

                try {
                    $device_type_id = $this['mobile_detect.device_type'];

                    // TODO デバッグ用
                    if ($request->query->has('device_type_id')) {
                        $device_type_id = $request->get('device_type_id', \Eccube\Entity\Master\DeviceType::DEVICE_TYPE_PC);
                    }

                    $DeviceType = $this['eccube.repository.master.device_type']
                        ->find($device_type_id);
                    $qb = $this['eccube.repository.page_layout']->createQueryBuilder('p');
                    $PageLayout = $qb->select('p, pll,l, bp, b')
                        ->leftJoin('p.PageLayoutLayouts', 'pll')
                        ->leftJoin('pll.Layout', 'l')
                        ->leftJoin('l.BlockPositions', 'bp')
                        ->leftJoin('bp.Block', 'b')
                        ->where('p.url = :route')
                        ->andWhere('l.DeviceType = :DeviceType')
                        ->orderBy('bp.block_row', 'ASC')
                        ->setParameter('route', $route)
                        ->setParameter('DeviceType', $DeviceType)
                        ->getQuery()
                        ->getSingleResult();
                } catch (\Doctrine\ORM\NoResultException $e) {
                    $PageLayout = $this['eccube.repository.page_layout']->newPageLayout($DeviceType);

The variable $DeviceType does not seem to be defined for all execution paths leading up to this point.

                }

                $this['twig']->addGlobal('PageLayout', $PageLayout);
                $this['twig']->addGlobal('title', $PageLayout->getName());
            }

            $this['twig_global_initialized'] = true;
        });
    }