Usamamuneerchaudhary\Notifier\Services\NotificationRepository

Complexity

Total Complexity

3

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

1 Method

Rating	Name	Duplication	Size	Complexity
A	findByToken()	0	17	3

<?php

namespace Usamamuneerchaudhary\Notifier\Services;

use Illuminate\Support\Facades\Cache;
use Usamamuneerchaudhary\Notifier\Models\Notification;

class NotificationRepository
{
    /**
     * Get notification by tracking token from cache/database
     */
    public function findByToken(string $token): ?Notification
    {
        $notificationId = Cache::get("notifier:tracking_token:{$token}");

        if ($notificationId) {
            return Notification::find($notificationId);

The expression return Usamamuneerchaudhary\Notifier\Models\Notification::find($notificationId) could return the type Illuminate\Database\Eloquent\PendingHasThroughRelationship which is incompatible with the type-hinted return Usamamuneerchaudhary\Notifier\Models\Notification|null. Consider adding an additional type-check to rule them out.

        }

        $notifications = Notification::whereJsonContains('data->tracking_token', $token)->get();

        if ($notifications->count() > 0) {
            $notification = $notifications->first();
            Cache::put("notifier:tracking_token:{$token}", $notification->id, now()->addDays(30));
            return $notification;
        }

        return null;
    }
}