CheckUserRole::handle() - Code Metrics - Inspection of "Add custom exception to display message and links..." - GCTC-NTGC/TalentCloud - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — bugfix/manager-not-created-aft... ( fcd1af )

by Chris

created 2019-04-16 12:57 UTC

CheckUserRole::handle() A

↳ Parent: CheckUserRole

Complexity

Conditions	4
Paths	3

Size

Total Lines	11
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	4

Importance

Changes

Metric	Value
eloc	8
dl	0
loc	11
ccs	9
cts	9
cp	1
rs	10
c	0
b	0
f	0
cc	4
nc	3
nop	1
crap	4

<?php

namespace App\Listeners;

use App\Events\UserUpdated;
use App\Models\Manager;
use Illuminate\Support\Facades\Log;

class CheckUserRole
{
    /**
     * Create the event listener.
     *
     * @return void
     */
    public function __construct()
    {
        //
    }

    /**
     * Create a manager object for a User if one does not exist.
     *
     * @param UserUpdated $event Fires after successful database update.
     * @return void
     */
    public function handle(UserUpdated $event) : void
    {
        if ($event->user->hasRole('manager') ||
            $event->user->hasRole('admin')
        ) {
            $managerProfile = $event->user->manager;
            Log::notice("User " . $event->user->id . " manager account: " . $event->user->manager);
            if ($managerProfile === null) {
                $managerProfile = new Manager();
                $managerProfile->user_id = $event->user->id;
                $managerProfile->save();
            }
        }
    }
}


1		<?php
2
3		namespace App\Listeners;
4
5		use App\Events\UserUpdated;
6		use App\Models\Manager;
7		use Illuminate\Support\Facades\Log;
8
9		class CheckUserRole
10		{
11		/**
12		* Create the event listener.
13		*
14		* @return void
15		*/
16	3	public function __construct()
17		{
18		//
19	3	}
20
21		/**
22		* Create a manager object for a User if one does not exist.
23		*
24		* @param UserUpdated $event Fires after successful database update.
25		* @return void
26		*/
27	3	public function handle(UserUpdated $event) : void
28		{
29	3	if ($event->user->hasRole('manager') \|\|
30	3	$event->user->hasRole('admin')
31		) {
32	1	$managerProfile = $event->user->manager;
33	1	Log::notice("User " . $event->user->id . " manager account: " . $event->user->manager);
34	1	if ($managerProfile === null) {
35	1	$managerProfile = new Manager();
36	1	$managerProfile->user_id = $event->user->id;
37	1	$managerProfile->save();
38		}
39		}
40	2	}
41		}
42

GCTC-NTGC / TalentCloud

Push — bugfix/manager-not-created-aft... ( fcd1af )

CheckUserRole::handle() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like