vistart\Models\traits\NotificationTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getIsExpired()	0	4	1
A	removeExpired()	0	6	2
A	onRemoveExpired()	0	5	1
A	initNotificationEvents()	0	4	1

<?php

/**
 *  _   __ __ _____ _____ ___  ____  _____
 * | | / // // ___//_  _//   ||  __||_   _|
 * | |/ // /(__  )  / / / /| || |     | |
 * |___//_//____/  /_/ /_/ |_||_|     |_|
 * @link https://vistart.name/
 * @copyright Copyright (c) 2016 vistart
 * @license https://vistart.name/license/
 */

namespace vistart\Models\traits;

/**
 * Description of NotificationTrait
 *
 * @author vistart <i@vistart.name>
 */
trait NotificationTrait
{

    public $rangeAttribute = 'range';
    public $linkAttrbute = '';
    public $expiredAt = 10080; // in minutes.

    public function getIsExpired()
    {
        return $this->offsetDatetime($this->expiredAt * 60) < $this->createdAt;

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

It seems like offsetDatetime() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    public function removeExpired()
    {
        if ($this->getIsExpired()) {
            return $this->delete();

It seems like delete() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }

    public function onRemoveExpired($event)
    {
        $sender = $event->sender;
        $sender->removeExpired();
    }

    public function initNotificationEvents()
    {
        $this->on(static::EVENT_INIT, [$this, 'onRemoveExpired']);

It seems like on() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }
}