App\Http\Middleware\GetAppLocale::handle()

Complexity

Conditions	1
Paths	1

Size

Total Lines	13
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function handle(Request $request, Closure $next)
    {
        // Try to get the current locale from the session.
        $locale = session('jaxonCurrentLocale', LaravelLocalization::getCurrentLocale());
        LaravelLocalization::setLocale($locale);

It seems like $locale can also be of type Illuminate\Session\SessionManager and Illuminate\Session\Store; however, parameter $locale of Mcamara\LaravelLocalization\Facades\LaravelLocalization::setLocale() does only seem to accept string, maybe add an additional type check?

        // Same as for SetAppLocale, excepted that the locale is read in the session.
        Carbon::setLocale($locale);

It seems like $locale can also be of type Illuminate\Session\SessionManager and Illuminate\Session\Store; however, parameter $locale of Carbon\Carbon::setLocale() does only seem to accept string, maybe add an additional type check?

        setlocale(LC_TIME, $locale);

It seems like $locale can also be of type Illuminate\Session\SessionManager and Illuminate\Session\Store; however, parameter $locales of setlocale() does only seem to accept array|integer|string, maybe add an additional type check?

        // For comparison of strings with accented characters in french
        setlocale(LC_COLLATE, LaravelLocalization::getCurrentLocaleRegional() . '.utf8');

        return $next($request);
    }