UserModel::createUser() - Code Metrics - Inspection of "Rewrite usermodel" - mafd16/comment - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 592d4f...1560d7 )

by Martin

created 2017-10-11 16:22 UTC

UserModel::createUser() B

↳ Parent: UserModel

Complexity

Conditions	2
Paths	2

Size

Total Lines	24
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	24
ccs	0
cts	12
cp	0
rs	8.9713
cc	2
eloc	13
nc	2
nop	1
crap	6

<?php

namespace Mafd16\User;

use \Anax\Configure\ConfigureInterface;
use \Anax\Configure\ConfigureTrait;
use \Anax\DI\InjectionAwareInterface;
use \Anax\Di\InjectionAwareTrait;

/**
 * User model for Comment system.
 */
class UserModel implements
    ConfigureInterface,
    InjectionAwareInterface
{
    use ConfigureTrait,
        InjectionAwareTrait;

    /**
     * Variables
     */
    //private $session;


    /**
     * Create a new User.
     * The acronym is unique!
     *
     * @param array $newUser the values of the new user.
     *
     * $newUser = [
     *      acronym => name,
     *      password => hashing in function!,
     *      email => ,
     *      created => timestamp in function,
     *      updated => null,
     *      deleted => null,
     *      active => null,
     *      admin => 0,
     *  ];
     *
     * @return boolean true if acronym unique, false otherwise.
     */
    public function createUser($newUser)
    {
        // Connect to db
        $user = new User();
        $user->setDb($this->di->get("db"));

        // Check if acronym already exists,
        if ($user->find("acronym", $newUser->acronym)) {
            return false;
        } else {
            // else create user
            $user->acronym = $newUser->acronym;
            $user->setPassword($newUser->password);
            $user->email = $newUser->email;
            $user->created = date("Y-m-d H:i:s");
            //$user->$updated;
            //$user->$deleted;
            //$user->$active;
            $user->admin = 0;
            // Save to database
            $user->save();
            return true;
        }
    }
}


1			<?php
2
3			namespace Mafd16\User;
4
5			use \Anax\Configure\ConfigureInterface;
6			use \Anax\Configure\ConfigureTrait;
7			use \Anax\DI\InjectionAwareInterface;
8			use \Anax\Di\InjectionAwareTrait;
9
10			/**
11			* User model for Comment system.
12			*/
13			class UserModel implements
14			ConfigureInterface,
15			InjectionAwareInterface
16			{
17			use ConfigureTrait,
18			InjectionAwareTrait;
19
20			/**
21			* Variables
22			*/
23			//private $session;
24
25
26			/**
27			* Create a new User.
28			* The acronym is unique!
29			*
30			* @param array $newUser the values of the new user.
31			*
32			* $newUser = [
33			* acronym => name,
34			* password => hashing in function!,
35			* email => ,
36			* created => timestamp in function,
37			* updated => null,
38			* deleted => null,
39			* active => null,
40			* admin => 0,
41			* ];
42			*
43			* @return boolean true if acronym unique, false otherwise.
44			*/
45			public function createUser($newUser)
46			{
47			// Connect to db
48			$user = new User();
49			$user->setDb($this->di->get("db"));
50
51			// Check if acronym already exists,
52			if ($user->find("acronym", $newUser->acronym)) {
53			return false;
54			} else {
55			// else create user
56			$user->acronym = $newUser->acronym;
57			$user->setPassword($newUser->password);
58			$user->email = $newUser->email;
59			$user->created = date("Y-m-d H:i:s");
60			//$user->$updated;
61			//$user->$deleted;
62			//$user->$active;
63			$user->admin = 0;
64			// Save to database
65			$user->save();
66			return true;
67			}
68			}
69			}
70

mafd16 / comment

Push — master ( 592d4f...1560d7 )

UserModel::createUser() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like