Users::status() - Code Metrics - csphere-cms/cSphere - Measure and Improve Code Quality continuously with Scrutinizer

Users::status() B
last analyzed 2017-04-21 16:53 UTC

↳ Parent: Users

Complexity

Conditions	3
Paths	3

Size

Total Lines	24
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	10
nc	3
nop	0
dl	0
loc	24
rs	8.9713
c	0
b	0
f	0

<?php

/**
 * Login support for users plugin
 *
 * PHP Version 5
 *
 * @category  Authentication
 * @package   Users
 * @author    Hans-Joachim Piepereit <[email protected]>
 * @copyright 2013 cSphere Team
 * @license   http://opensource.org/licenses/bsd-license Simplified BSD License
 * @link      http://www.csphere.eu
 **/

namespace csphere\core\authentication;

/**
 * Login support for users plugin
 *
 * @category  Authentication
 * @package   Users
 * @author    Hans-Joachim Piepereit <[email protected]>
 * @copyright 2013 cSphere Team
 * @license   http://opensource.org/licenses/bsd-license Simplified BSD License
 * @link      http://www.csphere.eu
 **/

class Users
{
    /**
     * Session object
     **/
     private $_session = null;


    /**
     * Datamapper object for users table
     **/
     private $_mapper = null;


    /**
     * Prepare values that are needed for later usage
     *
     * @return \csphere\core\authentication\Users
     **/


    public function __construct()
    {
        // Create session and user table datamapper object
        $this->_mapper  = new \csphere\core\datamapper\Model('users');
        $this->_session = new \csphere\core\session\Session();
    }

    /**
     * Whether someone is logged in at the moment
     *
     * @return boolean
     **/


    public function status()
    {
        // Check if user is still logged in
        $result  = false;
        $user_id = $this->_session->get('user_id');

        if (!empty($user_id)) {

            // Get user active info and check for existence
            $user = $this->_mapper->read($user_id, 'user_id');

            if (empty($user['user_active'])) {

                // Kick user auth
                $this->logout();

            } else {

                $result = true;
            }
        }

        return $result;
    }

    /**
     * Attempt to authenticate a user
     *
     * @param string $name     User name
     * @param string $password User password
     *
     * @return boolean
     **/


    public function login($name, $password)
    {
        // Get user from database
        $user   = $this->_mapper->read($name, 'user_name');
        $verify = false;

        // Verify password if both vars are not empty
        if (isset($user['user_password']) && !empty($password)) {

            $verify = \csphere\core\authentication\Password::compare(
                $password, $user['user_password']
            );
        }

        // Only grant access if password check is true
        if ($verify === true) {

            $this->_newLogin($user);
        }

        return $verify;
    }

    /**
     * Update content on fresh logins
     *
     * @param array $user Array with user data
     *
     * @return void
     **/


    private function _newLogin(array $user)
    {
        // Set session data
        $this->_session->set('user_id', $user['user_id']);
        $this->_session->set('user_name', $user['user_name']);
        $this->_session->set('user_lang', $user['user_lang']);

        // Notify users_logins about new visit
        $web  = \csphere\core\http\Input::get('server', 'HTTP_USER_AGENT');
        $time = time();

        $dm_logins = new \csphere\core\datamapper\Model('users', 'logins');
        $login     = $dm_logins->create();

        $login['user_id']       = $user['user_id'];
        $login['login_since']   = $time;
        $login['login_browser'] = $web;

        $dm_logins->insert($login);

        // Update date of last login
        $update = ['user_id' => $user['user_id'], 'user_laston' => $time];

        $this->_mapper->update($update);
    }

    /**
     * Clear session data
     *
     * @return boolean
     **/


    public function logout()
    {
        // Destroy current session
        $this->_session->destroy();

        return true;
    }
}


1			<?php
2
3			/**
4			* Login support for users plugin
5			*
6			* PHP Version 5
7			*
8			* @category Authentication
9			* @package Users
10			* @author Hans-Joachim Piepereit <[email protected]>
11			* @copyright 2013 cSphere Team
12			* @license http://opensource.org/licenses/bsd-license Simplified BSD License
13			* @link http://www.csphere.eu
14			**/
15
16			namespace csphere\core\authentication;
17
18			/**
19			* Login support for users plugin
20			*
21			* @category Authentication
22			* @package Users
23			* @author Hans-Joachim Piepereit <[email protected]>
24			* @copyright 2013 cSphere Team
25			* @license http://opensource.org/licenses/bsd-license Simplified BSD License
26			* @link http://www.csphere.eu
27			**/
28
29			class Users
30			{
31			/**
32			* Session object
33			**/
34			private $_session = null;
			0 ignored issues – show Coding Style introduced 2017-04-21 16:53 UTC by Report Bug Copy Issue Report Line indented incorrectly; expected 4 spaces, found 5 Loading history...
35
36			/**
37			* Datamapper object for users table
38			**/
39			private $_mapper = null;
			0 ignored issues – show Coding Style introduced 2017-04-21 16:53 UTC by Report Bug Copy Issue Report Line indented incorrectly; expected 4 spaces, found 5 Loading history...
40
41			/**
42			* Prepare values that are needed for later usage
43			*
44			* @return \csphere\core\authentication\Users
45			**/
			0 ignored issues – show Comprehensibility Best Practice introduced 2017-04-21 16:53 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history... Coding Style introduced 2017-04-21 16:53 UTC by Report Bug Copy Issue Report There must be no blank lines after the function comment Loading history...
46
47			public function __construct()
48			{
49			// Create session and user table datamapper object
50			$this->_mapper = new \csphere\core\datamapper\Model('users');
51			$this->_session = new \csphere\core\session\Session();
52			}
53
54			/**
55			* Whether someone is logged in at the moment
56			*
57			* @return boolean
58			**/
			0 ignored issues – show Coding Style introduced 2017-04-21 16:53 UTC by Report Bug Copy Issue Report There must be no blank lines after the function comment Loading history...
59
60			public function status()
61			{
62			// Check if user is still logged in
63			$result = false;
64			$user_id = $this->_session->get('user_id');
65
66			if (!empty($user_id)) {
67
68			// Get user active info and check for existence
69			$user = $this->_mapper->read($user_id, 'user_id');
70
71			if (empty($user['user_active'])) {
72
73			// Kick user auth
74			$this->logout();
75
76			} else {
77
78			$result = true;
79			}
80			}
81
82			return $result;
83			}
84
85			/**
86			* Attempt to authenticate a user
87			*
88			* @param string $name User name
89			* @param string $password User password
90			*
91			* @return boolean
92			**/
			0 ignored issues – show Coding Style introduced 2017-04-21 16:53 UTC by Report Bug Copy Issue Report There must be no blank lines after the function comment Loading history...
93
94			public function login($name, $password)
95			{
96			// Get user from database
97			$user = $this->_mapper->read($name, 'user_name');
98			$verify = false;
99
100			// Verify password if both vars are not empty
101			if (isset($user['user_password']) && !empty($password)) {
102
103			$verify = \csphere\core\authentication\Password::compare(
104			$password, $user['user_password']
105			);
106			}
107
108			// Only grant access if password check is true
109			if ($verify === true) {
110
111			$this->_newLogin($user);
112			}
113
114			return $verify;
115			}
116
117			/**
118			* Update content on fresh logins
119			*
120			* @param array $user Array with user data
121			*
122			* @return void
123			**/
			0 ignored issues – show Coding Style introduced 2017-04-21 16:53 UTC by Report Bug Copy Issue Report There must be no blank lines after the function comment Loading history...
124
125			private function _newLogin(array $user)
126			{
127			// Set session data
128			$this->_session->set('user_id', $user['user_id']);
129			$this->_session->set('user_name', $user['user_name']);
130			$this->_session->set('user_lang', $user['user_lang']);
131
132			// Notify users_logins about new visit
133			$web = \csphere\core\http\Input::get('server', 'HTTP_USER_AGENT');
134			$time = time();
135
136			$dm_logins = new \csphere\core\datamapper\Model('users', 'logins');
137			$login = $dm_logins->create();
138
139			$login['user_id'] = $user['user_id'];
140			$login['login_since'] = $time;
141			$login['login_browser'] = $web;
142
143			$dm_logins->insert($login);
144
145			// Update date of last login
146			$update = ['user_id' => $user['user_id'], 'user_laston' => $time];
147
148			$this->_mapper->update($update);
149			}
150
151			/**
152			* Clear session data
153			*
154			* @return boolean
155			**/
			0 ignored issues – show Coding Style introduced 2017-04-21 16:53 UTC by Report Bug Copy Issue Report There must be no blank lines after the function comment Loading history...
156
157			public function logout()
158			{
159			// Destroy current session
160			$this->_session->destroy();
161
162			return true;
163			}
164			}
165

csphere-cms / cSphere

Users::status() B last analyzed 2017-04-21 16:53 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

Users::status() B
last analyzed 2017-04-21 16:53 UTC