Xetaravel\View\Composers\NotificationsComposer

Complexity

Total Complexity

1

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	compose()	0	8	1

<?php
namespace Xetaravel\View\Composers;

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

class NotificationsComposer
{
    /**
     * Bind data to the view.
     *
     * @param  \Illuminate\View\View  $view
     * @return void
     */
    public function compose(View $view)
    {
        $notifications = UserRepository::notificationsData(Auth::id());

It seems like Illuminate\Support\Facades\Auth::id() can also be of type string; however, parameter $userId of Xetaravel\Models\Repositories\UserRepository::notificationsData() does only seem to accept integer, maybe add an additional type check?

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