itstructure / yii2-template-multilanguage

controllers/RedirectController.php

<?php

namespace app\controllers;

use Yii;
use yii\web\Controller;
use Itstructure\AdminModule\models\Language;
use app\helpers\BaseHelper;

/**
 * Class RedirectController
 *
 * @package app\controllers
 */
class RedirectController extends Controller
{
    /**
     * Redirect to home page with default language.
     */
    public function actionHomePage()
    {
        $clientShortLanguage = BaseHelper::detectClientShortLanguage();

        if (null === $clientShortLanguage) {
            $shortLanguage = Language::getDefaultLanguage()->shortName;

        } else {
            $check = Language::find()->where([
                'shortName' => $clientShortLanguage
            ])->count();

            $shortLanguage = $check == 0 ? Language::getDefaultLanguage()->shortName : $clientShortLanguage;

It seems like you are loosely comparing $check of type integer|null|string to 0; this is ambiguous as not only 0 == 0 is true, but null == 0 is true, too. Consider using a strict comparison ===.

        }

        Yii::$app->response->redirect('/'.$shortLanguage, 302)->send();
    }
}