OneSignalPayloadFactory::make() - Code Metrics - Inspection of "Add payload factory (#50)" - laravel-notification-channels/onesignal - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 741135...fdec78 )

by Lukas

created 2018-02-28 20:43 UTC

OneSignalPayloadFactory::make() A

↳ Parent: OneSignalPayloadFactory

Complexity

Conditions	3
Paths	3

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	14
ccs	8
cts	8
cp	1
rs	9.4285
cc	3
eloc	9
nc	3
nop	3
crap	3

<?php

namespace NotificationChannels\OneSignal;

use Illuminate\Notifications\Notification;

class OneSignalPayloadFactory
{
    /**
     * Make a one signal notification payload.
     *
     * @param mixed $notifiable
     * @param \Illuminate\Notifications\Notification $notification
     * @param mixed $targeting
     *
     * @return array
     */
    public static function make($notifiable, Notification $notification, $targeting) : array
    {
        $payload = $notification->toOneSignal($notifiable)->toArray();


        if (static::isTargetingEmail($targeting)) {
            $payload['filters'] = collect([['field' => 'email', 'value' => $targeting['email']]]);
        } elseif (static::isTargetingTags($targeting)) {
            $payload['tags'] = collect([$targeting['tags']]);
        } else {
            $payload['include_player_ids'] = collect($targeting);
        }

        return $payload;
    }

    /**
     * @param mixed $targeting
     *
     * @return bool
     */
    protected static function isTargetingEmail($targeting)
    {
        return is_array($targeting) && array_key_exists('email', $targeting);
    }

    protected static function isTargetingTags($targeting)
    {
        return is_array($targeting) && array_key_exists('tags', $targeting);
    }
}


1		<?php
2
3		namespace NotificationChannels\OneSignal;
4
5		use Illuminate\Notifications\Notification;
6
7		class OneSignalPayloadFactory
8		{
9		/**
10		* Make a one signal notification payload.
11		*
12		* @param mixed $notifiable
13		* @param \Illuminate\Notifications\Notification $notification
14		* @param mixed $targeting
15		*
16		* @return array
17		*/
18	5	public static function make($notifiable, Notification $notification, $targeting) : array
19		{
20	5	$payload = $notification->toOneSignal($notifiable)->toArray();
		0 ignored issues – show Bug introduced 2018-02-28 19:34 UTC by Report Bug Copy Issue Report The method `toOneSignal()` does not seem to exist on `object<Illuminate\Notifications\Notification>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
21
22	5	if (static::isTargetingEmail($targeting)) {
23	1	$payload['filters'] = collect([['field' => 'email', 'value' => $targeting['email']]]);
24	4	} elseif (static::isTargetingTags($targeting)) {
25	1	$payload['tags'] = collect([$targeting['tags']]);
26		} else {
27	3	$payload['include_player_ids'] = collect($targeting);
28		}
29
30	5	return $payload;
31		}
32
33		/**
34		* @param mixed $targeting
35		*
36		* @return bool
37		*/
38	5	protected static function isTargetingEmail($targeting)
39		{
40	5	return is_array($targeting) && array_key_exists('email', $targeting);
41		}
42
43	4	protected static function isTargetingTags($targeting)
44		{
45	4	return is_array($targeting) && array_key_exists('tags', $targeting);
46		}
47		}
48

laravel-notification-channels / onesignal

GitHub Access Token became invalid

Push — master ( 741135...fdec78 )

OneSignalPayloadFactory::make() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like