Cog\Flag\Traits\Inverse\HasExpiredFlagHelpers

Complexity

Total Complexity

5

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	initializeHasExpiredFlagHelpers()	0	3	1
A	isNotExpired()	0	3	1
A	isExpired()	0	3	1
A	expire()	0	6	1
A	undoExpire()	0	6	1

<?php

/*
 * This file is part of Laravel Eloquent Flag.
 *
 * (c) Anton Komarev <anton@komarev.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace Cog\Flag\Traits\Inverse;

trait HasExpiredFlagHelpers
{
    public function initializeHasExpiredFlagHelpers(): void
    {
        $this->casts['is_expired'] = 'boolean';

The property casts does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    public function isExpired(): bool
    {
        return $this->getAttributeValue('is_expired');

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

    }

    public function isNotExpired(): bool
    {
        return !$this->isExpired();
    }

    public function expire(): void
    {
        $this->setAttribute('is_expired', true);

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

        $this->save();

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

        $this->fireModelEvent('expired', false);

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

    }

    public function undoExpire(): void
    {
        $this->setAttribute('is_expired', false);
        $this->save();

        $this->fireModelEvent('expiredUndone', false);
    }
}