App\View\Composers\GlobalDataComposer

Complexity

Total Complexity

3

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	compose()	0	38	3

<?php

namespace App\View\Composers;

use App\Events\UserLoggedIn;
use App\Models\Category;
use App\Models\Settings;
use App\Models\User;
use Illuminate\Support\Facades\Auth;
use Illuminate\View\View;

class GlobalDataComposer
{
    /**
     * Bind data to the view.
     */
    public function compose(View $view): void
    {
        $settings = new Settings;

        $viewData = [
            'serverroot' => url('/'),
            'site' => $settings,
            'theme' => 'Gentele',
        ];

        if (Auth::check()) {
            $userdata = User::find(Auth::id());
            $userdata->categoryexclusions = User::getCategoryExclusionById(Auth::id());

It seems like Illuminate\Support\Facades\Auth::id() can also be of type null and string; however, parameter $userID of App\Models\User::getCategoryExclusionById() does only seem to accept integer, maybe add an additional type check?

            // Update last login every 15 mins.
            if (now()->subHours(3) > $userdata->lastlogin) {
                event(new UserLoggedIn($userdata));
            }

            $parentcatlist = Category::getForMenu($userdata->categoryexclusions);

            $viewData = array_merge($viewData, [
                'userdata' => $userdata,
                'loggedin' => true,
                'isadmin' => $userdata->hasRole('Admin'),
                'ismod' => $userdata->hasRole('Moderator'),
                'parentcatlist' => $parentcatlist,
                'header_menu_cat' => request()->input('t', ''),
            ]);
        } else {
            $viewData = array_merge($viewData, [
                'isadmin' => false,
                'ismod' => false,
                'loggedin' => false,
            ]);
        }

        $view->with($viewData);
    }
}