ActivationController::activate() - Code Metrics - Inspection of "Merge pull request #22 from AudithSoftworks/larave..." - AudithSoftworks/Basis - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 94434d...92e1c0 )

by Şəhriyar

created 2016-01-26 14:37 UTC

ActivationController::activate() B

↳ Parent: ActivationController

Complexity

Conditions	8
Paths	8

Size

Total Lines	19
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	8.2327

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	19
ccs	11
cts	13
cp	0.8462
rs	7.7777
cc	8
eloc	12
nc	8
nop	3
crap	8.2327

<?php namespace App\Http\Controllers\Users;

use App\Contracts\Registrar;
use App\Events\Users\RequestedActivationLink;
use App\Exceptions\Common\NotImplementedException;
use App\Exceptions\Users\UserAlreadyActivatedException;
use App\Exceptions\Users\UserNotFoundException;
use App\Http\Controllers\Controller;
use Illuminate\Http\Request;

class ActivationController extends Controller
{
    /**
     * Create a new authentication controller instance.
     */
    public function __construct()
    {
        $this->middleware('auth', ['only' => ['requestActivationCode']]);
    }

    /**
     * Request account activation link via email.
     *
     * @param \Illuminate\Http\Request $request
     *
     * @return \Illuminate\Http\RedirectResponse|\Illuminate\Http\JsonResponse
     * @throws \App\Exceptions\Users\UserAlreadyActivatedException
     */
    public function requestActivationCode(Request $request)
    {
        if (!($user = app('sentinel')->getUser())) {
            throw new UserNotFoundException;
        }

        if (false !== app('sentinel.activations')->completed($user)) {
            throw new UserAlreadyActivatedException;
        }

        app('events')->fire(new RequestedActivationLink($user));

        if ($request->ajax() || $request->wantsJson()) {
            return response()->json(['message' => 'Activation link sent']);
        }

        return redirect()->back()->with('message', 'Activation link sent');
    }

    /**
     * Activate an account [Web only].
     *
     * @param \Illuminate\Http\Request $request
     * @param \App\Contracts\Registrar $registrar
     * @param string|null              $token
     *
     * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector|\Illuminate\Http\JsonResponse
     */
    public function activate(Request $request, Registrar $registrar, $token = null)
    {
        switch ($requestMethod = $request->getMethod()) {
            case 'GET':
                if ($request->ajax() || $request->wantsJson()) {
                    throw new NotImplementedException;
                }
                break;
            case 'POST':
                if (!$request->ajax() && !$request->wantsJson()) {
                    throw new NotImplementedException;
                }
                break;
        }

        $registrar->activate($token);

        return $requestMethod == 'GET' ? redirect($this->redirectPath())->with('message', 'Activation successful') : response()->json(['message' => 'Activated']);
    }

    /**
     * Get the post register / login redirect path.
     *
     * @return string
     */
    private function redirectPath()
    {
        if (isset($this->redirectPath)) {
            return $this->redirectPath;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        }

        return isset($this->redirectTo) ? $this->redirectTo : '/';
class MyClass { }

$x = new MyClass();
$x->foo = true;
    }
}


1		<?php namespace App\Http\Controllers\Users;
2
3		use App\Contracts\Registrar;
4		use App\Events\Users\RequestedActivationLink;
5		use App\Exceptions\Common\NotImplementedException;
6		use App\Exceptions\Users\UserAlreadyActivatedException;
7		use App\Exceptions\Users\UserNotFoundException;
8		use App\Http\Controllers\Controller;
9		use Illuminate\Http\Request;
10
11		class ActivationController extends Controller
12		{
13		/**
14		* Create a new authentication controller instance.
15		*/
16	5	public function __construct()
17		{
18	5	$this->middleware('auth', ['only' => ['requestActivationCode']]);
19	5	}
20
21		/**
22		* Request account activation link via email.
23		*
24		* @param \Illuminate\Http\Request $request
25		*
26		* @return \Illuminate\Http\RedirectResponse\|\Illuminate\Http\JsonResponse
27		* @throws \App\Exceptions\Users\UserAlreadyActivatedException
28		*/
29	1	public function requestActivationCode(Request $request)
30		{
31	1	if (!($user = app('sentinel')->getUser())) {
32		throw new UserNotFoundException;
33		}
34
35	1	if (false !== app('sentinel.activations')->completed($user)) {
36		throw new UserAlreadyActivatedException;
37		}
38
39	1	app('events')->fire(new RequestedActivationLink($user));
40
41	1	if ($request->ajax() \|\| $request->wantsJson()) {
42	1	return response()->json(['message' => 'Activation link sent']);
43		}
44
45		return redirect()->back()->with('message', 'Activation link sent');
46		}
47
48		/**
49		* Activate an account [Web only].
50		*
51		* @param \Illuminate\Http\Request $request
52		* @param \App\Contracts\Registrar $registrar
53		* @param string\|null $token
54		*
55		* @return \Illuminate\Http\RedirectResponse\|\Illuminate\Routing\Redirector\|\Illuminate\Http\JsonResponse
56		*/
57	3	public function activate(Request $request, Registrar $registrar, $token = null)
58		{
59	3	switch ($requestMethod = $request->getMethod()) {
60	3	case 'GET':
61	1	if ($request->ajax() \|\| $request->wantsJson()) {
62	1	throw new NotImplementedException;
63		}
64		break;
65	2	case 'POST':
66	2	if (!$request->ajax() && !$request->wantsJson()) {
67		throw new NotImplementedException;
68		}
69	2	break;
70	2	}
71
72	2	$registrar->activate($token);
73
74	1	return $requestMethod == 'GET' ? redirect($this->redirectPath())->with('message', 'Activation successful') : response()->json(['message' => 'Activated']);
75		}
76
77		/**
78		* Get the post register / login redirect path.
79		*
80		* @return string
81		*/
82		private function redirectPath()
83		{
84		if (isset($this->redirectPath)) {
85		return $this->redirectPath;
		0 ignored issues – show Bug introduced 2015-11-20 21:25 UTC by Report Bug Copy Issue Report The property `redirectPath` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
86		}
87
88		return isset($this->redirectTo) ? $this->redirectTo : '/';
		0 ignored issues – show Bug introduced 2015-11-20 21:25 UTC by Report Bug Copy Issue Report The property `redirectTo` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
89		}
90		}
91

AudithSoftworks / Basis

Push — master ( 94434d...92e1c0 )

ActivationController::activate() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like