User Impersonation - Ko3

gistfile1.php[345]

<?php defined('SYSPATH') or die('No direct script access.');

// Extending Auth to support user impersonation functions

class Auth extends Kohana_Auth {

	public function impersonate(ORM $user)
	{
		if ( ! $user->loaded())
			throw new Exception;
			
		$this->session->set('impersonator', $this->get_user());
		
		$this->logout();
		$this->force_login($user);
	}
	
	public function abdicate()
	{
		if ( ! $this->is_impersonating())
			return FALSE;
			
		$impersonator = $this->get_impersonator();

		$this->session->delete('impersonator');

		if ( ! $impersonator->loaded())
			throw new Exception;
		
		$this->logout();
		$this->force_login($impersonator);
	}
	
	public function is_impersonating()
	{
		return (bool) $this->session->get('impersonator');
	}
	
	public function get_impersonator()
	{
		return $this->session->get('impersonator');
	}

}

I think we generally agree to never have one-line non-curly-brace-enclosed bodies of ifs and loops. This should probably be reflected here.
Edit: Because they are an invitation to break code later, of course

This actually reflects the regular Kohana coding guidelines, which I was following when I wrote this.