UserRegistration::register() - Code Metrics - Inspection of "Preparing 1.18.0" - AsgardCms/User - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 2.0 ( 2d1915 )

by Nicolas

created 2016-06-16 15:16 UTC

UserRegistration::register() A

↳ Parent: UserRegistration

Complexity

Conditions	2
Paths	2

Size

Total Lines	16
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	2

Metric	Value
c	2
b	0
f	2
dl	0
loc	16
rs	9.4285
cc	2
eloc	8
nc	2
nop	1

<?php namespace Modules\User\Services;

use Modules\Core\Contracts\Authentication;
use Modules\User\Events\UserHasRegistered;
use Modules\User\Repositories\RoleRepository;

class UserRegistration
{
    /**
     * @var Authentication
     */
    private $auth;
    /**
     * @var RoleRepository
     */
    private $role;
    /**
     * @var array
     */
    private $input;

    public function __construct(Authentication $auth, RoleRepository $role)
    {
        $this->auth = $auth;
        $this->role = $role;
    }

    /**
     * @param array $input
     * @return mixed
     */
    public function register(array $input)
    {
        $this->input = $input;

        $user = $this->createUser();

        if ($this->hasProfileData()) {
            $this->createProfileForUser($user);
        }

        $this->assignUserToUsersGroup($user);

        event(new UserHasRegistered($user));

        return $user;
    }

    private function createUser()
    {
        return $this->auth->register((array) $this->input);
    }

    private function assignUserToUsersGroup($user)
    {
        $role = $this->role->findByName('User');

        $this->auth->assignRole($user, $role);
    }

    /**
     * Check if the request input has a profile key
     * @return bool
     */
    private function hasProfileData()
    {
        return isset($this->input['profile']);
    }

    /**
     * Create a profile for the given user
     * @param $user
     */
    private function createProfileForUser($user)
    {
        $profileData = array_merge($this->input['profile'], ['user_id' => $user->id]);
        app('Modules\Profile\Repositories\ProfileRepository')->create($profileData);
    }
}


1			<?php namespace Modules\User\Services;
2
3			use Modules\Core\Contracts\Authentication;
4			use Modules\User\Events\UserHasRegistered;
5			use Modules\User\Repositories\RoleRepository;
6
7			class UserRegistration
8			{
9			/**
10			* @var Authentication
11			*/
12			private $auth;
13			/**
14			* @var RoleRepository
15			*/
16			private $role;
17			/**
18			* @var array
19			*/
20			private $input;
21
22			public function __construct(Authentication $auth, RoleRepository $role)
23			{
24			$this->auth = $auth;
25			$this->role = $role;
26			}
27
28			/**
29			* @param array $input
30			* @return mixed
31			*/
32			public function register(array $input)
33			{
34			$this->input = $input;
35
36			$user = $this->createUser();
37
38			if ($this->hasProfileData()) {
39			$this->createProfileForUser($user);
40			}
41
42			$this->assignUserToUsersGroup($user);
43
44			event(new UserHasRegistered($user));
45
46			return $user;
47			}
48
49			private function createUser()
50			{
51			return $this->auth->register((array) $this->input);
52			}
53
54			private function assignUserToUsersGroup($user)
55			{
56			$role = $this->role->findByName('User');
57
58			$this->auth->assignRole($user, $role);
59			}
60
61			/**
62			* Check if the request input has a profile key
63			* @return bool
64			*/
65			private function hasProfileData()
66			{
67			return isset($this->input['profile']);
68			}
69
70			/**
71			* Create a profile for the given user
72			* @param $user
73			*/
74			private function createProfileForUser($user)
75			{
76			$profileData = array_merge($this->input['profile'], ['user_id' => $user->id]);
77			app('Modules\Profile\Repositories\ProfileRepository')->create($profileData);
78			}
79			}
80

AsgardCms / User

Push — 2.0 ( 2d1915 )

UserRegistration::register() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like