NathanGeerinck / laravel-newsletter

app/Http/Middleware/LocaleMiddleware.php

<?php

namespace App\Http\Middleware;

use Closure;
use Illuminate\Support\Facades\Auth;

class LocaleMiddleware
{
    protected $languages = ['en', 'nl'];

    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \Closure  $next
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
        if (Auth::check()) {
            $lang = Auth::user()->preferences['language'];

Accessing preferences on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

            app()->setLocale($lang);

The method setLocale() does not exist on Illuminate\Container\Container. Are you sure you never get this type here, but always one of the subclasses?

        } else {
            app()->setLocale(env('APP_LOCALE'));
        }

        return $next($request);
    }
}