Cog\Flag\Traits\Classic\HasVerifiedAtHelpers

Complexity

Total Complexity

7

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	unverify()	0	5	1
A	setVerifiedFlag()	0	5	1
A	unsetVerifiedFlag()	0	5	1
A	verify()	0	5	1
A	isUnverified()	0	3	1
A	initializeHasVerifiedAtHelpers()	0	3	1
A	isVerified()	0	3	1

<?php

/*
 * This file is part of Laravel Eloquent Flag.
 *
 * (c) Anton Komarev <a.komarev@cybercog.su>
 *
 * 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\Classic;

use Illuminate\Support\Carbon;

trait HasVerifiedAtHelpers
{
    public function initializeHasVerifiedAtHelpers(): void
    {
        $this->dates[] = 'verified_at';

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

    }

    /**
     * Set verified flag.
     *
     * @return static
     */
    public function setVerifiedFlag()
    {
        $this->setAttribute('verified_at', Carbon::now());

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

        return $this;
    }

    /**
     * Unset verified flag.
     *
     * @return static
     */
    public function unsetVerifiedFlag()
    {
        $this->setAttribute('verified_at', null);

        return $this;
    }

    /**
     * If entity is verified.
     *
     * @return bool
     */
    public function isVerified(): bool
    {
        return !is_null($this->getAttributeValue('verified_at'));

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

    }

    /**
     * If entity is unverified.
     *
     * @return bool
     */
    public function isUnverified(): bool
    {
        return !$this->isVerified();
    }

    /**
     * Mark entity as verified.
     *
     * @return void
     */
    public function verify(): void
    {
        $this->setVerifiedFlag()->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('verified', false);

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

    }

    /**
     * Mark entity as unverified.
     *
     * @return void
     */
    public function unverify(): void
    {
        $this->unsetVerifiedFlag()->save();

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