jeremykenedy / laravel-material-design

app/Http/ViewComposers/ThemeComposer.php

<?php

namespace App\Http\ViewComposers;

use App\Models\Theme;
use Illuminate\Support\Facades\Auth;
use Illuminate\View\View;

class ThemeComposer
{
    protected $user;
    protected $theme;

    public function __construct()
    {
        $this->user = Auth::user();
    }

    /**
     * Bind data to the view.
     *
     * @param View $view
     *
     * @return void
     */
    public function compose(View $view)
    {
        $theme = null;

        if (Auth::check()) {
            $user = $this->user;

            if ($user->profile) {

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

                $theme = Theme::find($user->profile->theme_id);

                if ($theme->status == 0) {
                    $theme = Theme::find(1);
                }
            }
        }

        $view->with('theme', $theme);
    }
}