MedianetDev\LaravelAuthApi\Http\Controllers\Traits\MustVerifyEmail

Complexity

Total Complexity

4

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	markEmailAsVerified()	0	5	1
A	hasVerifiedEmail()	0	3	1
A	getEmailForVerification()	0	3	1
A	sendEmailVerificationNotification()	0	4	1

<?php

namespace MedianetDev\LaravelAuthApi\Http\Controllers\Traits;

trait MustVerifyEmail
{
    /**
     * Determine if the user has verified their email address.
     *
     * @return bool
     */
    public function hasVerifiedEmail()
    {
        return ! is_null($this->email_verified_at);
    }

    /**
     * Mark the given user's email as verified.
     *
     * @return bool
     */
    public function markEmailAsVerified()
    {
        return $this->forceFill([

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

            'email_verified_at' => $this->freshTimestamp(),

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

        ])->save();
    }

    /**
     * Send the email verification notification.
     *
     * @return void
     */
    public function sendEmailVerificationNotification()
    {
        $emailTemplate = config('laravel-auth-api.email_notification');
        $this->notify(new $emailTemplate);

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

    }

    /**
     * Get the email address that should be used for verification.
     *
     * @return string
     */
    public function getEmailForVerification()
    {
        return $this->email;
    }
}