NotificationChannels\WebPush\HasPushSubscriptions

Complexity

Total Complexity

10

Size/Duplication

Total Lines	83
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

42.86%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	pushSubscriptions()	0	4	1
A	updatePushSubscription()	0	25	5
A	ownsPushSubscription()	0	5	2
A	deletePushSubscription()	0	6	1
A	routeNotificationForWebPush()	0	4	1

<?php

namespace NotificationChannels\WebPush;

trait HasPushSubscriptions
{
    /**
     *  Get all of the subscriptions.
     *
     * @return \Illuminate\Database\Eloquent\Relations\MorphMany
     */
    public function pushSubscriptions()
    {
        return $this->morphMany(config('webpush.model'), 'subscribable');

It seems like morphMany() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * Update (or create) subscription.
     *
     * @param  string $endpoint
     * @param  string $endpoint_key

There is no parameter named $endpoint_key. Did you maybe mean $endpoint?

     * @param  string|null $key
     * @param  string|null $token
     * @param  string|null $contentEncoding
     * @return \NotificationChannels\WebPush\PushSubscription
     */
    public function updatePushSubscription($endpoint, $key = null, $token = null, $contentEncoding = null)
    {
        $subscription = app(config('webpush.model'))->findByEndpoint($endpoint);

        if ($subscription && $this->ownsPushSubscription($subscription)) {
            $subscription->public_key = $key;
            $subscription->auth_token = $token;
            $subscription->content_encoding = $contentEncoding;
            $subscription->save();

            return $subscription;
        }

        if ($subscription && ! $this->ownsPushSubscription($subscription)) {
            $subscription->delete();
        }

        return $this->pushSubscriptions()->create([
            'endpoint' => $endpoint,
            'endpoint_key' => md5($endpoint),
            'public_key' => $key,
            'auth_token' => $token,
            'content_encoding' => $contentEncoding,
        ]);
    }

    /**
     * Determine if the model owns the given subscription.
     *
     * @param  \NotificationChannels\WebPush\PushSubscription $subscription
     * @return bool
     */
    public function ownsPushSubscription($subscription)
    {
        return (string) $subscription->subscribable_id === (string) $this->getKey() &&

It seems like getKey() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                        $subscription->subscribable_type === $this->getMorphClass();

It seems like getMorphClass() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * Delete subscription by endpoint.
     *
     * @param  string $endpoint
     * @return void
     */
    public function deletePushSubscription($endpoint)
    {
        $this->pushSubscriptions()
            ->where('endpoint_key', md5($endpoint))
            ->delete();
    }

    /**
     * Get all of the subscriptions.
     *
     * @return \Illuminate\Database\Eloquent\Collection
     */
    public function routeNotificationForWebPush()
    {
        return $this->pushSubscriptions;

The property pushSubscriptions does not exist. Did you maybe forget to declare it?

    }
}