Kohana_Auth_Jam

Complexity

Total Complexity

75

Size/Duplication

Total Lines	429
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	10

Test Coverage

Coverage

67.15%

Importance

Changes

0

21 Methods

Rating	Name	Duplication	Size	Complexity
A	clear_cache()	0	4	1
A	access()	0	11	6
A	__construct()	0	10	2
A	services()	0	4	2
B	logged_in()	0	29	8
A	session()	0	8	2
B	_login()	0	27	6
A	remember()	0	9	1
A	force_login()	0	18	3
B	login_with_token()	0	29	7
A	login_with_service()	0	4	1
A	complete_login_with_service()	0	16	3
A	get_user()	0	14	3
A	_load_user()	0	7	3
A	_load_token()	0	7	3
A	_autologin_cookie()	0	16	3
A	password()	0	4	1
A	complete_login()	0	14	1
A	check_password()	0	9	2
B	auto_login()	0	32	9
B	logout()	0	33	8

<?php defined('SYSPATH') OR die('No direct access allowed.');
/**
 * Jam Auth driver.
 *
 * @package    Kohana/Auth
 * @author     Ivan Kerin
 * @copyright  (c) 2012 Despark Ltd.
 * @license    http://creativecommons.org/licenses/by-sa/3.0/legalcode
 */
abstract class Kohana_Auth_Jam extends Auth {

	protected $_services = array();

	public static function clear_cache()
	{
		Auth_Jam::$_instance = NULL;

The property _instance cannot be accessed from this context as it is declared protected in class Kohana_Auth.

	}

	public static function access($action, $access = array())
	{
		$access = (array) $access;
		$default_access = Arr::get($access, 0, 'public');
		$opposite_access = $default_access === 'public' ? 'private' : 'public';

		$except_actions = (array) Arr::get($access, 'except', array());
		$only_actions = (array) Arr::get($access, 'only', array());

		return (($except_actions AND in_array($action, $except_actions)) OR ( $only_actions AND ! in_array($action, $only_actions))) ? $opposite_access : $default_access;
	}

	public function __construct($config = array())
	{
		parent::__construct($config);

		foreach ($config['services'] as $service => $config)
		{
			$class = 'Auth_Service_'.Jam::capitalize_class_name($service);
			$this->_services[$service] = new $class($config);
		}
	}

	/**
	 * Get all the available services, or only one service if provided a name
	 * @param  string $name the name of the service, e.g. 'facebook'
	 * @return array|Auth_Service
	 */
	public function services($name = NULL)
	{
		return $name === NULL ? $this->_services : Arr::get($this->_services, $name);
	}

	/**
	 * Checks if a session is active.
	 *
	 * @param   mixed    $role Role name string, role Jam object, or array with role names
	 * @return  boolean
	 */
	public function logged_in($role = NULL)
	{
		// Get the user from the session
		$user = $this->get_user();

		if ( ! $user)
			return FALSE;

		if ($user instanceof Model_Auth_User AND $user->loaded())
		{
			// If we don't have a roll no further checking is needed
			if ( ! $role)
				return TRUE;

			if (is_array($role))
			{
				return ! array_diff($role, $user->roles->as_array(NULL, 'name'));

The property roles does not exist on object<Model_Auth_User>. Since you implemented __get, maybe consider adding a @property annotation.

			}
			elseif (is_string($role) OR $role instanceof Model_Auth_Role)
			{
				return $user->roles->has($role);

The property roles does not exist on object<Model_Auth_User>. Since you implemented __get, maybe consider adding a @property annotation.

			}
			else
			{
				throw new Kohana_Exception('Invalid Role ":role"', array("role" => (string) $role));
			}
		}
		return FALSE;
	}

	/**
	 * Getter
	 * The session instance for the configured session_type
	 * @return Session
	 */
	public function session()
	{
		if ( ! $this->_session)
		{
			$this->_session = Session::instance($this->_config['session_type']);
		}
		return $this->_session;
	}

	/**
	 * Logs a user in.
	 *
	 * @param   string   $username  username

There is no parameter named $username. Was it maybe removed?

	 * @param   string   $password  password
	 * @param   boolean  $remember  enable autologin
	 * @return  boolean
	 */
	protected function _login($user, $password, $remember)
	{
		if (is_string($password))
		{
			// Create a hashed password
			$password = $this->hash($password);
		}

		$user = $this->_load_user($user);

		// If the passwords match, perform a login
		if ($user AND $user->roles->has('login') AND $user->password === $password)
		{
			if ($remember === TRUE)
			{
				$this->remember($user);
			}

			// Finish the login
			$this->complete_login($user);

			return TRUE;
		}

		// Login failed
		return FALSE;
	}

	/**
	 * Create autologin token
	 * @param  Model_User $user
	 * @return Model_User_Token
	 */
	public function remember($user)
	{
		$token = $user->build_user_token(array('user_agent' => sha1(Request::$user_agent)))->save();

		// Set the autologin cookie
		$this->_autologin_cookie($token->token, $this->_config['lifetime']);

		return $token;
	}

	/**
	 * Forces a user to be logged in, without specifying a password.
	 *
	 * @param   mixed    $user                    username string, or user Jam object
	 * @param   boolean  $mark_session_as_forced  mark the session as forced
	 * @param   boolean  $remember                force to remeber the user
	 * @return  boolean
	 */
	public function force_login($user, $mark_session_as_forced = FALSE, $remember = FALSE)
	{
		$user = $this->_load_user($user);

		if ($mark_session_as_forced === TRUE)
		{
			// Mark the session as forced, to prevent users from changing account information
			$this->session()->set('auth_forced', TRUE);
		}

		if ($remember)
		{
			$this->remember($user);

$user is of type null|object, but the function expects a object<Model_User>.

		}

		// Run the standard completion
		return $this->complete_login($user);

It seems like $user defined by $this->_load_user($user) on line 163 can also be of type null; however, Kohana_Auth_Jam::complete_login() does only seem to accept object, maybe add an additional type check?

	}

	/**
	 * login using a spesific token
	 * @param  string $token token hash
	 * @return Model_User|NULL
	 */
	public function login_with_token($token)
	{
		if ( ! $token)
			return NULL;

		// Load the token and user
		$token = $this->_load_token($token);

		if ($token AND $token->user AND $token->user->loaded())
		{
			if ( ! $token->user_agent OR $token->user_agent === sha1(Request::$user_agent))
			{
				// Save the token to create a new unique token
				$token->save();

				// Set the new token
				$this->_autologin_cookie($token->token, $token->expires - time());

				// Complete the login with the found data
				$this->complete_login($token->user);

				// Automatic login was successful
				return $token->user;
			}

			// Token is invalid
			$token->delete();
		}
	}

	/**
	 * Attempt to login the user using specific service
	 * @param  string  $name     the name of the service
	 * @return Model_User|NULL
	 */
	public function login_with_service($name)
	{
		return $this->services($name)->login();
	}

	/**
	 * Attempt to login the user using specific service
	 * @param  string  $name     the name of the service
	 * @param  boolean $remember create autologin token
	 * @return Model_User|NULL
	 */
	public function complete_login_with_service($name, $remember = FALSE)
	{
		if ($user = $this->services($name)->complete_login())
		{
			if ($remember === TRUE)
			{
				$this->remember($user);
			}

			$this->complete_login($user);

			return $user;
		}

		return NULL;
	}

	/**
	 * Logs a user in, based on the authautologin cookie.
	 *
	 * @return  mixed
	 */
	public function auto_login()
	{
		if ($token = $this->_autologin_cookie())
		{
			if ($user = $this->login_with_token($token))

It seems like $token defined by $this->_autologin_cookie() on line 255 can also be of type this<Kohana_Auth_Jam>; however, Kohana_Auth_Jam::login_with_token() does only seem to accept string, maybe add an additional type check?

			{
				return $user;
			}
			else
			{
				$this->_autologin_cookie(FALSE);

FALSE is of type boolean, but the function expects a string|null.

			}
		}

		if (Request::current() AND $token = Arr::get(Request::current()->query(), '_token'))
		{
			if ($user = $this->login_with_token($token))

Are you sure the assignment to $user is correct as $this->login_with_token($token) (which targets Kohana_Auth_Jam::login_with_token()) seems to always return null.

				return $user;
		}

		foreach ($this->services() as $service)

The expression $this->services() of type array|object<Auth_Service> is not guaranteed to be traversable. How about adding an additional type check?

		{
			if ($service->auto_login_enabled() AND $user = $service->get_user())
			{
				// $this->remember($user);
				$this->complete_login($user);
				return $user;
			}
		}

		return FALSE;
	}

	/**
	 * Gets the currently logged in user from the session (with auto_login check).
	 * Returns FALSE if no user is currently logged in.
	 *
	 * @return  mixed
	 */
	public function get_user($default = NULL)
	{
		// Load the session for the parent method
		$this->session();
		$user = $this->_load_user(parent::get_user($default));

		if ( ! ($user AND $user->loaded()))
		{
			// check for "remembered" login
			$user = $this->auto_login();
		}

		return $user;
	}

	protected function _load_user($user)
	{
		if ( ! $user)
			return NULL;

		return is_object($user) ? $user : Jam::find('user', $user);
	}

	protected function _load_token($token)
	{
		if ( ! $token)
			return NULL;

		return is_object($token) ? $token : Jam::all('user_token')->valid_token($token)->first();
	}

	/**
	 * Getter / Setter of the autologin cookie
	 * Extend this method in your tests so you can remove dependance on cookies there
	 *
	 * @param  string $token
	 * @param  integer $expires days lifetime
	 * @return mixed
	 */
	protected function _autologin_cookie($token = NULL, $expires = NULL)
	{
		if ($token === FALSE)
		{
			Cookie::delete('authautologin');
		}
		elseif ($token !== NULL)
		{
			Cookie::set('authautologin', $token, $expires);
		}
		else
		{
			return Cookie::get('authautologin');
		}
		return $this;
	}

	/**
	 * Log a user out and remove any autologin cookies, and goes to each service to log out the user there
	 *
	 * @param   boolean  $destroy     completely destroy the session
	 * @param	boolean  $logout_all  remove all tokens for user
	 * @return  boolean
	 */
	public function logout($destroy = FALSE, $logout_all = FALSE)
	{
		// Set by force_login()
		$this->session()->delete('auth_forced');

		if ($token = $this->_autologin_cookie())
		{
			// Delete the autologin cookie to prevent re-login
			$this->_autologin_cookie(FALSE);

FALSE is of type boolean, but the function expects a string|null.

			// Clear the autologin token from the database
			$token = $this->_load_token($token);

			if ($token AND $token->loaded() AND $token->user)
			{
				if ($logout_all)
				{
					$token->user->user_tokens->clear();
				}
				$token->delete();
			}
		}

		foreach ($this->services() as $service)

The expression $this->services() of type array|object<Auth_Service> is not guaranteed to be traversable. How about adding an additional type check?

		{
			if ($user = $service->get_user())

$user is not used, you could remove the assignment.

			{
				$service->logout();
			}
		}

		return parent::logout($destroy);
	}

	/**
	 * Get the stored password for a username.
	 *
	 * @param   mixed   $user  username string, or user Jam object
	 * @return  string
	 */
	public function password($user)
	{
		return $this->_load_user($user)->password;
	}

	/**
	 * Complete the login for a user by incrementing the logins and setting
	 * session data: user_id, username, roles.
	 *
	 * @param   object  $user Jam object
	 * @return  bool
	 */
	protected function complete_login($user)
	{
		$user->last_login_ip = Request::$client_ip;

		$user->complete_login();

		// Regenerate session_id
		$this->session()->regenerate();

		// Store username in session
		$this->session()->set($this->_config['session_key'], $user->id);

		return TRUE;
	}

	/**
	 * Compare password with original (hashed). Works for current (logged in) user
	 *
	 * @param   string  $password
	 * @return  boolean
	 */
	public function check_password($password)
	{
		$user = $this->get_user();

		if ( ! $user)
			return FALSE;

		return ($this->hash($password) === $user->password);
	}

} // End Auth Jam