Cortex\Fort\Http\Requests\Frontend\EmailVerificationProcessRequest

Complexity

Total Complexity

10

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	authorize()	0	12	7
A	rules()	0	6	1
A	getRedirectUrl()	0	10	2

<?php

declare(strict_types=1);

namespace Cortex\Fort\Http\Requests\Frontend;

use Cortex\Fort\Models\User;
use Rinvex\Support\Http\Requests\FormRequest;
use Cortex\Foundation\Exceptions\GenericException;

class EmailVerificationProcessRequest extends FormRequest
{
    /**
     * Determine if the user is authorized to make this request.
     *
     * @throws \Cortex\Foundation\Exceptions\GenericException
     *
     * @return bool
     */
    public function authorize()
    {
        $userVerified = $this->user() && $this->user()->email_verified;
        $guestVerified = ($email = $this->get('email')) && ($user = User::where('email', $email)->first()) && $user->email_verified;

This line exceeds maximum limit of 120 characters; contains 132 characters

        if ($userVerified || $guestVerified) {
            // Redirect users if their email already verified, no need to process their request
            throw new GenericException(trans('cortex/fort::messages.verification.email.already'), $userVerified ? route('frontend.account.settings') : route('frontend.auth.login'));

This line exceeds maximum limit of 120 characters; contains 181 characters

        }

        return true;
    }

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array

Consider making the return type a bit more specific; maybe use array<string,string>.

     */
    public function rules()
    {
        // Do not validate `token` here since at this stage we can NOT generate viewable
        // notification/error, and it is been processed through EmailVerificationBroker anyway
        return ['email' => 'required|email|max:250|exists:'.config('rinvex.fort.tables.users').',email'];
    }

    /**
     * {@inheritdoc}
     */
    protected function getRedirectUrl()

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

    {
        // This is route('frontend.verification.email.send')
        if ($this->isMethod('post')) {
            return parent::getRedirectUrl();
        }

        // This is route('frontend.verification.email.verify')
        return $this->redirector->getUrlGenerator()->route('frontend.verification.email.request');
    }
}