Verify::__invoke() - Code Metrics - Inspection of "Version 2.x" - kingscode/laravel-api-starter - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#7)

by Koen

created 2020-02-17 16:10 UTC

Verify::__invoke() A

↳ Parent: Verify

Complexity

Conditions	3
Paths	6

Size

Total Lines	17
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	3

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	9
c	1
b	0
f	0
nc	6
nop	2
dl	0
loc	17
ccs	9
cts	9
cp	1
crap	3
rs	9.9666

<?php

declare(strict_types=1);

namespace App\Http\Controllers\Api\Profile\Email;

use App\Auth\Dispensary\Exceptions\TokenExpiredException;
use App\Auth\EmailDispensary;
use App\Http\Requests\Api\Profile\Email\VerifyRequest;
use Illuminate\Contracts\Auth\Guard;
use Illuminate\Contracts\Routing\ResponseFactory;
use Illuminate\Http\Response;

final class Verify
{
    /**
     * @var \App\Auth\EmailDispensary
     */
    private EmailDispensary $dispensary;

    /**
     * @var \Illuminate\Contracts\Routing\ResponseFactory
     */
    private ResponseFactory $responseFactory;

    public function __construct(EmailDispensary $dispensary, ResponseFactory $responseFactory)
    {
        $this->dispensary = $dispensary;
        $this->responseFactory = $responseFactory;
    }

    public function __invoke(Guard $guard, VerifyRequest $request): Response
    {
        /** @var \App\Models\User $user */
        $user = $guard->user();

        $email = $request->input('email');

        try {
            if ($this->dispensary->verify($user, $email, $request->input('token'))) {
                $user->update(['email' => $email]);

                return $this->responseFactory->noContent(Response::HTTP_OK);
            }

            return $this->responseFactory->noContent(Response::HTTP_BAD_REQUEST);
        } catch (TokenExpiredException $e) {
            return $this->responseFactory->noContent(Response::HTTP_BAD_REQUEST);
        }
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace App\Http\Controllers\Api\Profile\Email;
6
7		use App\Auth\Dispensary\Exceptions\TokenExpiredException;
8		use App\Auth\EmailDispensary;
9		use App\Http\Requests\Api\Profile\Email\VerifyRequest;
10		use Illuminate\Contracts\Auth\Guard;
11		use Illuminate\Contracts\Routing\ResponseFactory;
12		use Illuminate\Http\Response;
13
14		final class Verify
15		{
16		/**
17		* @var \App\Auth\EmailDispensary
18		*/
19		private EmailDispensary $dispensary;
20
21		/**
22		* @var \Illuminate\Contracts\Routing\ResponseFactory
23		*/
24		private ResponseFactory $responseFactory;
25
26	4	public function __construct(EmailDispensary $dispensary, ResponseFactory $responseFactory)
27		{
28	4	$this->dispensary = $dispensary;
29	4	$this->responseFactory = $responseFactory;
30	4	}
31
32	3	public function __invoke(Guard $guard, VerifyRequest $request): Response
33		{
34		/** @var \App\Models\User $user */
35	3	$user = $guard->user();
36
37	3	$email = $request->input('email');
38
39		try {
40	3	if ($this->dispensary->verify($user, $email, $request->input('token'))) {
41	1	$user->update(['email' => $email]);
42
43	1	return $this->responseFactory->noContent(Response::HTTP_OK);
44		}
45
46	1	return $this->responseFactory->noContent(Response::HTTP_BAD_REQUEST);
47	1	} catch (TokenExpiredException $e) {
48	1	return $this->responseFactory->noContent(Response::HTTP_BAD_REQUEST);
49		}
50		}
51		}
52

kingscode / laravel-api-starter

Pull Request — master (#7)

Verify::__invoke() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like