app\traits\AdminBeforeActionTrait::beforeAction()

Complexity

Conditions	2
Paths	2

Size

Total Lines	25
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function beforeAction($action)
    {
        $this->view->mainMenuConfig = require __DIR__ . '/../config/admin/main-menu.php';

        $this->view->homeUrl = '/' . $this->shortLanguage . '/admin';

        $this->view->signOutLink = '/' . $this->shortLanguage . '/logout';

        $this->view->profileLink = '/' . $this->shortLanguage . '/admin/users/view?id='.Yii::$app->getUser()->id;

        Yii::$app->getUser()->loginUrl = '/' . $this->shortLanguage . '/login';

        $this->urlPrefix = '/' . $this->shortLanguage . '/' . $this->module->id . '/' . $action->controller->id . '/';

The property urlPrefix does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        if (array_key_exists($action->controller->id, $this->neighborControllers)) {
            $this->urlPrefixNeighbor = '/' . $this->shortLanguage . '/' . $this->module->id . '/' .

The property urlPrefixNeighbor does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

                $this->neighborControllers[$action->controller->id] . '/';
        }

        $this->view->userBody = ArrayHelper::map($this->languages, 'name', function ($item) {
            /* @var Language $item */
            return BaseHelper::getSwitchLanguageLink($item->getShortName(), Yii::$app->request);

It seems like Yii::app->request can also be of type yii\console\Request; however, parameter $request of app\helpers\BaseHelper::getSwitchLanguageLink() does only seem to accept yii\web\Request, maybe add an additional type check?

        });

        return parent::beforeAction($action);
    }
}