Notifable::readAllNotifications() - Code Metrics - Inspection of "Merge remote-tracking branch 'origin/version-4' in..." - fenos/Notifynder - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — version-4 ( 455d27...4e0b22 )

unknown

created 2016-07-22 12:31 UTC

Notifable::readAllNotifications() A

↳ Parent: Notifable

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	1	Features	1

Metric	Value
cc	1
eloc	2
c	1
b	1
f	1
nc	1
nop	0
dl	0
loc	4
rs	10

<?php

namespace Fenos\Notifynder\Traits;

use Fenos\Notifynder\Models\Notification;

trait Notifable
{
    public function notifications()
    {
        $model = notifynder_config()->getNotificationModel();
        if (notifynder_config()->isPolymorphic()) {
            return $this->morphMany($model, 'to');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        }

        return $this->hasMany($model, 'to_id');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    public function notifynder($category)
    {
        return app('notifynder')->category($category);
    }

    public function sendNotificationFrom($category)
    {
        return $this->notifynder($category)->from($this);
    }

    public function sendNotificationTo($category)
    {
        return $this->notifynder($category)->to($this);
    }

    public function readNotification($notification)
    {
        if (! ($notification instanceof Notification)) {
            $notification = Notification::firstOrFail($notification);
        }

        return $notification->read();
    }

    public function readAllNotifications()
    {
        return $this->notifications()->update(['read' => 1]);
    }

    public function unreadNotification($notification)
    {
        if (! ($notification instanceof Notification)) {
            $notification = Notification::firstOrFail($notification);
        }

        return $notification->unread();
    }

    public function countUnreadNotifications()
    {
        return $this->notifications()->byRead(0)->count();
    }

    public function getNotifications($limit = null, $order = 'desc')
    {
        $query = $this->notifications()->orderBy('created_at', $order);
        if (! is_null($limit)) {
            $query->limit($limit);
        }

        return $query->get();
    }
}


1			<?php
2
3			namespace Fenos\Notifynder\Traits;
4
5			use Fenos\Notifynder\Models\Notification;
6
7			trait Notifable
8			{
9			public function notifications()
10			{
11			$model = notifynder_config()->getNotificationModel();
12			if (notifynder_config()->isPolymorphic()) {
13			return $this->morphMany($model, 'to');
			0 ignored issues – show Bug introduced 2016-06-17 14:28 UTC by Report Bug Copy Issue Report It seems like `morphMany()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
14			}
15
16			return $this->hasMany($model, 'to_id');
			0 ignored issues – show Bug introduced 2016-06-17 14:28 UTC by Report Bug Copy Issue Report It seems like `hasMany()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
17			}
18
19			public function notifynder($category)
20			{
21			return app('notifynder')->category($category);
22			}
23
24			public function sendNotificationFrom($category)
25			{
26			return $this->notifynder($category)->from($this);
27			}
28
29			public function sendNotificationTo($category)
30			{
31			return $this->notifynder($category)->to($this);
32			}
33
34			public function readNotification($notification)
35			{
36			if (! ($notification instanceof Notification)) {
37			$notification = Notification::firstOrFail($notification);
38			}
39
40			return $notification->read();
41			}
42
43			public function readAllNotifications()
44			{
45			return $this->notifications()->update(['read' => 1]);
46			}
47
48			public function unreadNotification($notification)
49			{
50			if (! ($notification instanceof Notification)) {
51			$notification = Notification::firstOrFail($notification);
52			}
53
54			return $notification->unread();
55			}
56
57			public function countUnreadNotifications()
58			{
59			return $this->notifications()->byRead(0)->count();
60			}
61
62			public function getNotifications($limit = null, $order = 'desc')
63			{
64			$query = $this->notifications()->orderBy('created_at', $order);
65			if (! is_null($limit)) {
66			$query->limit($limit);
67			}
68
69			return $query->get();
70			}
71			}
72

fenos / Notifynder

Push — version-4 ( 455d27...4e0b22 )

Notifable::readAllNotifications() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like