Xetaravel\Http\ViewComposers\NotificationsServiceProvider

Complexity

Total Complexity

1

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	boot()	0	12	1

<?php
namespace Xetaravel\Http\ViewComposers;

use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\View;
use Illuminate\Support\ServiceProvider;
use Xetaravel\Models\Repositories\UserRepository;

class NotificationsServiceProvider extends ServiceProvider
{
    /**
     * Register bindings in the container.
     *
     * @return void
     */
    public function boot()
    {
        View::composer('elements._notifications', function ($view) {
            $notifications = UserRepository::notificationsData(Auth::user()->id);

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

            $view->with([
                'notifications' => $notifications['notifications'],
                'hasUnreadNotifications' => $notifications['hasUnreadNotifications'],
                'unredNotificationsCount' => $notifications['unredNotificationsCount']
            ]);
        });
    }
}