Eccube\Controller\Admin\Store\PluginController::enable()

Complexity

Conditions	10
Paths	88

Size

Total Lines

84

Duplication

Lines	14
Ratio	16.67 %

Code Coverage

Tests	0
CRAP Score	110

Importance

Changes

0

<?php
    public function enable(Plugin $Plugin, CacheUtil $cacheUtil, Request $request, EventDispatcherInterface $dispatcher)
    {
        $this->isTokenValid();

        // QueryString maintenance_modeがない場合
        if (!$request->query->has('maintenance_mode')) {
            // プラグイン管理の有効ボタンを押したとき
            $this->systemService->switchMaintenance(true); // auto_maintenanceと設定されたファイルを生成
            // TERMINATE時のイベントを設定
            $dispatcher->addListener(KernelEvents::TERMINATE, function () {
            $this->systemService->switchMaintenance(); // auto_maintenanceと設定されたファイルを削除
            });
        } else {
            // プラグイン管理のアップデートを実行したとき
            // TERMINATE時のイベントを設定
            $dispatcher->addListener(KernelEvents::TERMINATE, function () {
                // auto_maintenance_updateと設定されたファイルを削除
                $this->systemService->switchMaintenance(false,SystemService::AUTO_MAINTENANCE_UPDATE);
            });
        }

        $cacheUtil->clearCache();

        $log = null;

$log is not used, you could remove the assignment.

        if ($Plugin->isEnabled()) {
            if ($request->isXmlHttpRequest()) {
                return $this->json(['success' => true]);
            } else {
                $this->addError(trans('admin.store.plugin.already.enabled', ['%plugin_name%' => $Plugin->getName()]), 'admin');
                return $this->redirectToRoute('admin_store_plugin');
            }
        } else {
            // ストアからインストールしたプラグインは依存プラグインが有効化されているかを確認
            if ($Plugin->getSource()) {
                $requires = $this->pluginService->getPluginRequired($Plugin);
                $requires = array_filter($requires, function ($req) {
                    $code = preg_replace('/^ec-cube\//', '', $req['name']);
                    /** @var Plugin $DependPlugin */
                    $DependPlugin = $this->pluginRepository->findOneBy(['code' => $code]);

                    return $DependPlugin->isEnabled() == false;

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

                });
                if (!empty($requires)) {
                    $names = array_map(function ($req) {
                        return "「${req['description']}」";
                    }, $requires);
                    $message = trans('%depend_name%を先に有効化してください。', ['%name%' => $Plugin->getName(), '%depend_name%' => implode(', ', $names)]);

                    if ($request->isXmlHttpRequest()) {
                        return $this->json(['success' => false, 'message' => $message], 400);
                    } else {
                        $this->addError($message, 'admin');

                        return $this->redirectToRoute('admin_store_plugin');
                    }
                }
            }


            try {
                ob_start();

                if (!$Plugin->isInitialized()) {
                    $this->pluginService->installWithCode($Plugin->getCode());
                }

                $this->pluginService->enable($Plugin);
            } finally {
                $log = ob_get_clean();
                while (ob_get_level() > 0) {
                    ob_end_flush();
                }
            }
        }

        if ($request->isXmlHttpRequest()) {
            return $this->json(['success' => true, 'log' => $log]);
        } else {
            $this->addSuccess(trans('admin.store.plugin.enable.complete', ['%plugin_name%' => $Plugin->getName()]), 'admin');

            return $this->redirectToRoute('admin_store_plugin');
        }
    }