PushSubscription::subscribable() - Code Metrics - Inspection of "Implement polymorphic relation and add report hand..." - laravel-notification-channels/webpush - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( fe092f...f32e18 )

by Cretu

created 2019-07-27 15:28 UTC

PushSubscription::subscribable() A

↳ Parent: PushSubscription

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	4
ccs	0
cts	0
cp	0
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	0
crap	2

<?php

namespace NotificationChannels\WebPush;

use Illuminate\Database\Eloquent\Model;

class PushSubscription extends Model
{
    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'endpoint',
        'public_key',
        'auth_token',
        'content_encoding',
    ];

    /**
     * Create a new model instance.
     *
     * @param  array $attributes
     * @return void

     */
    public function __construct(array $attributes = [])
    {
        if (! isset($this->connection)) {
            $this->setConnection(config('webpush.database_connection'));
        }

        if (! isset($this->table)) {
            $this->setTable(config('webpush.table_name'));
        }

        parent::__construct($attributes);
    }

    /**
     * Get the model related to the subscription.
     *
     * @return \Illuminate\Database\Eloquent\Relations\MorphTo
     */
    public function subscribable()
    {
        return $this->morphTo();
    }

    /**
     * Find a subscription by the given endpint.
     *
     * @param  string $endpoint
     * @return static|null
     */
    public static function findByEndpoint($endpoint)
    {
        return static::where('endpoint', $endpoint)->first();
    }
}


1		<?php
2
3		namespace NotificationChannels\WebPush;
4
5		use Illuminate\Database\Eloquent\Model;
6
7		class PushSubscription extends Model
8		{
9		/**
10		* The attributes that are mass assignable.
11		*
12		* @var array
13		*/
14		protected $fillable = [
15		'endpoint',
16		'public_key',
17		'auth_token',
18		'content_encoding',
19		];
20
21		/**
22		* Create a new model instance.
23		*
24		* @param array $attributes
25		* @return void
		0 ignored issues – show Comprehensibility Best Practice introduced 2019-07-27 15:37 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
26		*/
27	1	public function __construct(array $attributes = [])
28		{
29	1	if (! isset($this->connection)) {
30		$this->setConnection(config('webpush.database_connection'));
31		}
32
33		if (! isset($this->table)) {
34		$this->setTable(config('webpush.table_name'));
35		}
36
37		parent::__construct($attributes);
38	9	}
39
40	9	/**
41		* Get the model related to the subscription.
42		*
43		* @return \Illuminate\Database\Eloquent\Relations\MorphTo
44		*/
45		public function subscribable()
46		{
47		return $this->morphTo();
48		}
49
50		/**
51		* Find a subscription by the given endpint.
52		*
53		* @param string $endpoint
54		* @return static\|null
55		*/
56		public static function findByEndpoint($endpoint)
57		{
58		return static::where('endpoint', $endpoint)->first();
59		}
60		}
61

laravel-notification-channels / webpush

GitHub Access Token became invalid

Push — master ( fe092f...f32e18 )

PushSubscription::subscribable() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like