ExpoDatabaseDriver::forget() - Code Metrics - Inspection of "Database driver, multi-token per user and Package..." - Alymosul/laravel-exponent-push-notifications - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — master (#12)

unknown

created 2018-09-05 13:43 UTC

ExpoDatabaseDriver::forget() A

↳ Parent: ExpoDatabaseDriver

Complexity

Conditions	3
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

Metric	Value
dl	0
loc	23
ccs	0
cts	9
cp	0
rs	9.552
c	0
b	0
f	0
cc	3
nc	4
nop	2
crap	12

<?php

namespace NotificationChannels\ExpoPushNotifications\Repositories;

use ExponentPhpSDK\ExpoRepository;
use NotificationChannels\ExpoPushNotifications\Models\Interest;

class ExpoDatabaseDriver implements ExpoRepository
{
    /**
     * Stores an Expo token with a given identifier.
     *
     * @param $key
     * @param $value
     *
     * @return bool
     */
    public function store($key, $value): bool
    {
        $interest = Interest::firstOrCreate([
            'key' => $key,
            'value' => $value,
        ]);

        return $interest instanceof Interest;
    }

    /**
     * Retrieves an Expo token with a given identifier.
     *
     * @param string $key
     *
     * @return array
     */
    public function retrieve(string $key)
    {
        return Interest::where('key', $key)->pluck('value')->toArray();
    }

    /**
     * Removes an Expo token with a given identifier.
     *
     * @param string $key
     * @param string $value
     *
     * @return bool
     */
    public function forget(string $key, string $value = null): bool
    {
        // Delete interest
        $delete = Interest::where('key', $key);

        if(isset($value))
        {
            // Only delete this token
            $delete->where('value', $value);
        }

        $delete->delete();

        // Check if our interest exist
        $count = Interest::where('key', $key);

        if(isset($value))
        {
            $count->where('value', $value);
        }

        return $count->count() === 0;
    }
}


1		<?php
2
3		namespace NotificationChannels\ExpoPushNotifications\Repositories;
4
5		use ExponentPhpSDK\ExpoRepository;
6		use NotificationChannels\ExpoPushNotifications\Models\Interest;
7
8		class ExpoDatabaseDriver implements ExpoRepository
9		{
10		/**
11		* Stores an Expo token with a given identifier.
12		*
13		* @param $key
14		* @param $value
15		*
16		* @return bool
17		*/
18	1	public function store($key, $value): bool
19		{
20	1	$interest = Interest::firstOrCreate([
21	1	'key' => $key,
22	1	'value' => $value,
23		]);
24
25	1	return $interest instanceof Interest;
26		}
27
28		/**
29		* Retrieves an Expo token with a given identifier.
30		*
31		* @param string $key
32		*
33		* @return array
34		*/
35		public function retrieve(string $key)
36		{
37		return Interest::where('key', $key)->pluck('value')->toArray();
38		}
39
40		/**
41		* Removes an Expo token with a given identifier.
42		*
43		* @param string $key
44		* @param string $value
45		*
46		* @return bool
47		*/
48		public function forget(string $key, string $value = null): bool
49		{
50		// Delete interest
51		$delete = Interest::where('key', $key);
52
53		if(isset($value))
54		{
55		// Only delete this token
56		$delete->where('value', $value);
57		}
58
59		$delete->delete();
60
61		// Check if our interest exist
62		$count = Interest::where('key', $key);
63
64		if(isset($value))
65		{
66		$count->where('value', $value);
67		}
68
69		return $count->count() === 0;
70		}
71		}
72

Alymosul / laravel-exponent-push-notifications

Pull Request — master (#12)

ExpoDatabaseDriver::forget() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like