tuyakhov\notifications\NotifiableTrait

Complexity

Total Complexity

12

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	shouldReceiveNotification()	0	14	4
A	viaChannels()	0	4	1
A	routeNotificationFor()	0	14	5
A	getNotifications()	0	6	1
A	getUnreadNotifications()	0	4	1

<?php
/**
 * @copyright Anton Tuyakhov <atuyakhov@gmail.com>
 */

namespace tuyakhov\notifications;


use tuyakhov\notifications\models\Notification;
use yii\db\BaseActiveRecord;
use yii\helpers\Inflector;

trait NotifiableTrait
{
    /**
     * Determines if notifiable entity should receive the notification by checking in notification settings.
     * @param NotificationInterface $notification
     * @return bool
     */
    public function shouldReceiveNotification(NotificationInterface $notification)
    {
        $alias = Inflector::camel2id(get_class($notification));
        if (isset($this->notificationSettings)) {
            $settings = $this->notificationSettings;

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

            if (array_key_exists($alias, $settings)) {
                if ($settings[$alias] instanceof \Closure) {
                    return call_user_func($settings[$alias], $notification);
                }
                return (bool) $settings[$alias];
            }
        }
        return true;
    }

    /**
     * Send notifications via email by default
     * @return array
     */
    public function viaChannels()
    {
        return ['mail'];
    }

    /**
     * Return the notification routing information for the given channel.
     * ```php
     * public function routeNotificationForMail() {
     *      return $this->email;
     * }
     * ```
     * @param $channel string
     * @return mixed
     */
    public function routeNotificationFor($channel)
    {
        if (method_exists($this, $method = 'routeNotificationFor'.Inflector::id2camel($channel))) {
            return $this->{$method}();
        }
        switch ($channel) {
            case 'mail':
                return $this->email;

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

            case 'twilio':
                return $this->phone_number;

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

Variable "phone_number" is not in valid camel caps format

            case 'database':
                return [get_class($this), $this->id];

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

        }
    }

    public function getNotifications()

The return type could not be reliably inferred; please add a @return annotation.

    {
        /** @var $this BaseActiveRecord */
        return $this->hasMany(Notification::className(), ['notifiable_id' => 'id'])

The method addOnCondition() does not seem to exist on object<yii\db\ActiveQueryInterface>.

The method yii\base\BaseObject::className() has been deprecated with message: since 2.0.14. On PHP >=5.5, use `::class` instead.

            ->addOnCondition(['notifiable_type' => get_class($this)]);
    }

    public function getUnreadNotifications()

The return type could not be reliably inferred; please add a @return annotation.

    {
        return $this->getNotifications()->addOnCondition(['read_at' => null]);
    }
}