Как запустить миддлвар один раз?

Question

[Wiemdier_brenul]

У меня есть пакет, который определяет страну юзера по его апи и исходя из этой страны я хочу переключать на нужный язык. Так что я создал миддлвар LocalizationMiddleware и вставил его в список веб миддлваров

protected $middlewareGroups = [
        'web' => [
            ...,
            \App\Http\Middleware\LocalizationMiddleware::class
        ],

Сам миддлвар выглядит так, что я вызваю метод Location::get($ip) и исходя из свойства countryName достаю страну, и через множество if устанавливаю нужный язык:

$currentUserCountry = Location::get($ip);
        $country = $currentUserCountry->countryName;
        if($country == /*страна*/){
            app()->setLocale('/*язык*/');
            session()->put('locale', '/*язык*/');
        }

Проблема в том, что теперь этот миддлвар будет запускаться каждый раз, даже если юзер переключит язык на другой, ему установится тот, который подходит его стране. же хочу, чтобы этот миддлвар (или какая либо функция) вызывался только один раз, когда юзер зайдет на сайт, как это лучше всего реализовать?

Comments

[Антон Антон]

полуофф: Мне кажется, это дело не для мидлвари, а для сервис-провайдера. Когда нужно спросишь у сервиса и установишь язык. Сервиспровайдер на основе чего-то будет этот язык устанавливать. Например смотреть в сессию или настройки пользователя для неанонимного пользователя и, если там пусто, устанавливать на основании geo-ip. Если же есть в сессии или пользователь залогинен с настройками - то брать локаль оттуда.
А с миддварью надо либо руками отслеживать все маршруты, где это требуется, либо вешать абсолютно на все, что тоже не самый лучший выход.

Answer 1

[Сергей delphinpro]

Запускаться он будет каждый раз.
Но ему не нужно каждый раз выполнять определение страны.

if (страна_не_определена) {
  Определяем страну
}

return $next($request);

Comments

[Wiemdier_brenul]

У меня несколько языков, исходя из этих множественных if хочу поставить нужный язык. Понимаю, что миддлвар в данном случае не очень подходит, но как можно иначе поступить?

[Wiemdier_brenul]

Если вы имели в ввиду ставить дефолтную страну и язык, то в конце всех условий есть else.

}else{
        app()->setLocale('en');
        session()->put('locale', 'en');
}

То есть язык по дефолту - английский

[Сергей delphinpro]

Wiemdier_brenul, зачем?
Все просто

Добаиьте в каждую страну код локали (countryCode)

$locale = session()->get('locale');
if (!$locale) // в сессии локаль не сохранена
  $currentUserCountry = Location::get($ip);
  $locale = $currentUserCountry->countryCode; // ru или en или fr и т.д.
  session()->put('locale', $locale);
}

app()->setLocale($locale);

return $next($request);

Вот и весь миддлвар

updated

[Сергей delphinpro]

ну еще одна проверка на случай, если определение не прошло

$locale = session()->get('locale');

if (!$locale) // в сессии локаль не сохранена
  if ($currentUserCountry = Location::get($ip)) {
    $locale = $currentUserCountry->countryCode; // ru или en или fr и т.д.
  } else {
    $locale = config('app.fallback_locale');
  }
  session()->put('locale', $locale);
}

app()->setLocale($locale);

return $next($request);

Как-то так.

[Дмитрий]

А чего в сессии а не в куки?

[Сергей delphinpro]

Если php8, то попроще

$locale = session()->get('locale');

if (!$locale) // в сессии локаль не сохранена
  $locale = Location::get($ip)?->countryCode ?? config('app.fallback_locale');
  session()->put('locale', $locale);
}

app()->setLocale($locale);

return $next($request);

При условии, что метод ::get() возвращает null если не смог определить страну.

Или даже так =))

$locale = session()->get('locale')
          ?? Location::get($ip)?->countryCode
          ?? config('app.fallback_locale');

session()->put('locale', $locale);
app()->setLocale($locale);

return $next($request);

[Сергей delphinpro]

Дмитрий, хз, автор в сессии хранит

[Wiemdier_brenul]

Сергей delphinpro, Но ведь $locale будет всегда установлена, я даже не попадаю в это условие, как мне кажется это из за дефолтного языка, в конфиге 'locale' => 'en',

[Wiemdier_brenul]

То есть по идее нужно вашу функцию проверки вставить прям в 'locale' =>, но ведь так нельзя

[tukreb]

Wiemdier_brenul, нет, по идеи вам нужно подтянуть базу, если вы не понимаете, что там идёт просто присвоение переменной через тернарные операции

[Сергей delphinpro]

Wiemdier_brenul,

Но ведь $locale будет всегда установлена

Вам нужно проверять не локаль в приложении, а сохранена ли локаль в сессии/куках
Именно это и проверяется
$locale = session()->get('locale')
Здесь будет null если в сессии локали нет

[Сергей delphinpro]

Wiemdier_brenul, Вот мой работающий миддлвэр для установки языка

namespace App\Http\Middleware;

use App\Services\GeoIp;
use Closure;
use Illuminate\Http\Request;
use Illuminate\Support\Str;
use Mcamara\LaravelLocalization\Facades\LaravelLocalization;

class DetectCountry
{
  /** @var \App\Services\GeoIp */
  private GeoIp $geoIp;

  public function __construct(GeoIp $geoIp)
  {
    $this->geoIp = $geoIp;
  }

  /**
   * Handle an incoming request.
   *
   * @param  \Illuminate\Http\Request  $request
   * @param  \Closure  $next
   * @return mixed
   */
  public function handle(Request $request, Closure $next)
  {
    // Do not use for the admin panel
    if (Str::startsWith(\Route::currentRouteName(), 'platform.')) return $next($request);

    if (!session()->has(SK_USER_COUNTRY)) {
      $country = \Auth::check() && \Auth::user()->location_country
        ? \Auth::user()->location_country
        : $this->geoIp->getDetectedCountry();
      session()->put(SK_USER_COUNTRY, $country);
      return redirect(LaravelLocalization::localizeURL($request->getRequestUri(), $country->locale));
    }

    return $next($request);
  }
}

[tukreb]

Сергей delphinpro, вы не того тэгнули )

[Wiemdier_brenul]

Сергей delphinpro, Спасибо большое вы правы, а что у вас за способ определения страны? ->location_country это какой то пакет?

[Сергей delphinpro]

Wiemdier_brenul, нет. Это поле в таблице юзеров куда я сохраняю страну, которую пользователь выберет вручную. На сайте есть выбор страны расположения юзера.

[Сергей delphinpro]

Wiemdier_brenul, Определение по IP здесь $this->geoIp->getDetectedCountry()