Return value of App\Post::locale() must be of the type string, null returned about laravel-translatable HOT 5 CLOSED

@Gummibeer the solution is to update a line in translatable.php file inside config
'locale' => null,
to
'locale' => 'fr',
your approach of

dump($this->config->get('translatable.locale'));
dd($this->translator->getLocale());

lead to that solution all works fine now thank you

from laravel-translatable.

The session locale was empty that's what's triggering the error i managed to solve by adding the following code in AppServiceProvider.php inside method boot() :

public function boot() {
        Schema::defaultStringLength(400);
        
       //check if current locale is empty if so a default locale must be provided 
        if(!Session::has('locale')) {
            session(['local' => 'fr']);
        }
        setlocale(LC_ALL, config('app.locale') . '.utf8');
        Carbon::setLocale(config('app.locale'));
}

any other suggestions will be appreciated

from laravel-translatable.

Hey @BAKTETE ,

by default Laravel should have a locale at every time - except you set the locale to null. The config('app.locale') is used by default for the whole app - this includes the translator which is used in Locales::current().

1. the Locales::current() is missing the return type-hint string
2. in your app the translator doesn't have a locale

Could you debug/dump the two possible return values of the Locales::current() method? Just put a dump() with both values inside the method.

public function current() 
{ 
     dump($this->config->get('translatable.locale'));
     dd($this->translator->getLocale());
     return $this->config->get('translatable.locale') ?: $this->translator->getLocale(); 
} 

from laravel-translatable.

@Gummibeer
dump($this->config->get('translatable.locale')); output : "fr"
dd($this->translator->getLocale()); output : null

i have a lang switcher based on sessions when the sesssion expires it leaves the locale empty
as for the package config('translatable.locale')
so the Default locale is covering that up when change from null to desire locale (that's the solution)

from laravel-translatable.

Ok, so you found a solution for your app or do we have to change something in the package?
If I understood it right everything fine now?

from laravel-translatable.