codeigniter login de usuario en el constructor

[miktrv]

Hola buenas,

Estoy desarrollando el típico panel de administración, ( mi primer trabajo con codeigniter ), y quería saber cual es la mejor manera de comprobar si la sesión de usuario admin esta activa o no.. yo ahora estoy usando la típica comprobación

Código PHP:

  if ($this->session->userdata('logged_in')) {  
   

En cada uno de los métodos de mis controladores.. pero me parece un poco engorroso... se puede hacer esta comprobación de otra manera? por ejemplo, en el constructor de cada clase?

Añado el siguiente código que acabo de encontrar:

Código PHP:

function checkSession()
{
    if (!$this->session->userdata('is_logged_in'))
    {
        //the session has expired!
        $data['title'] = "Session expired";
 
        redirect('welcome', 'refresh');
    }
}
 

Código PHP:

$this->checkSession();
 

Así sería correcto?


Un saludo!

[RatoN7]

Claro, funcionaria. Una funcion asi la podrias colocar en algun helper, si tienes alguna libreria que controle sesiones la podrias poner ahi, o, si te animas podrias extender la libreria session que viene con Codeigniter.

Acordate que en caso de que sea helper o libreria tenes que obtener una instancia para poder usar las clases cargadas en CI, por ej:

Código PHP:

//En los controladores se hace asi
echo $this->session->userdata('is_logged_in');
 
//Pero en helpers y librerias, haces asi
$CI = & get_instance();
echo $CI->session->userdata('is_logged_in');
 

De paso te dejo un tutorial en donde crea un sistema de usuarios con una funcion parecida. Lo use bastante y anda bien, tal vez lo veas viejo, pero creo que no cambie nada y a veces lo sigo usando en la version 2.0.2. Si no quieres implementar toda la clase mira el codigo asi podes aprender una manera mas de administrar la sesion.
http://www.noquieroprogramar.com/tag/centinela/

Espero que te sirva
Saludos

[miktrv]

Hey!! muchas gracias por tu ayuda, muy amable,

Lo miro aver que tal, de momento usaré el código posteado porque me parece muy cómodo, jeje


Un saludo!

[NUCKLEAR]

Iniciado por miktrv

Hola buenas,

Estoy desarrollando el típico panel de administración, ( mi primer trabajo con codeigniter ), y quería saber cual es la mejor manera de comprobar si la sesión de usuario admin esta activa o no.. yo ahora estoy usando la típica comprobación

Código PHP:

  if ($this->session->userdata('logged_in')) {  
   

En cada uno de los métodos de mis controladores.. pero me parece un poco engorroso... se puede hacer esta comprobación de otra manera? por ejemplo, en el constructor de cada clase?

Añado el siguiente código que acabo de encontrar:

Código PHP:

function checkSession()
{
    if (!$this->session->userdata('is_logged_in'))
    {
        //the session has expired!
        $data['title'] = "Session expired";
 
        redirect('welcome', 'refresh');
    }
}
 

Código PHP:

$this->checkSession();
 

Así sería correcto?


Un saludo!

Hacerlo en una libreria o un helper te va a ahorar trabajo, pero no quita de hacerlo como lo estas haciendo. Ahora para que este disponible en todos tus metodos sin tener que repetir lo mismo, usa las funcionalidades de PHP5

Código PHP:

class tuController extends Controller{
 
public function __construct() {
        parent::__construct();
        // Check if the logged user
        $this->access_library->check_logged_in(); 
       }
 
function .....{
 
}
}
 

En tu access_library haces las operaciones pertinentes en este caso: si el usuario no esta logueado y no se encuentra en la session, rediririgir al form de login.

Ejm:

Código PHP:

class Access_library
{
    // Constructor
    public function __construct()
    {
        if (!isset($this->CI))
        {
            $this->CI =& get_instance();
        }
    }
 
    public function check_logged_in()
    {
        if ($this->CI->session->userdata('logged_in') != TRUE)
        {
            redirect('user/login', 'refresh');
            exit();
        }
    }
        }
 

Puse el codigo como referencia y segun recuerdo, no esta probado, pero la idea esta presente.-

[miktrv]

De acuerdo muchas gracias por la información,

Respecto lo de usar el get instance, es solo para la versión 4 de php, no?

Muchas gracias y un saludo!

[RatoN7]

Funciona en php5 tambien. Tal vez consultaste por el =&, pero no hay problema, si usas php5 quitalo si queres.

Saludos

[miktrv]

Iniciado por RatoN7

Funciona en php5 tambien. Tal vez consultaste por el =&, pero no hay problema, si usas php5 quitalo si queres.

Saludos

De acuerdo, muchas gracias por tu ayuda!

Un saludo!

[NUCKLEAR]

Como te dijo RatonN7 agrega compatiblididad con PHP4 pero lo podes dejar asi si trabajas con PHP5.

Código PHP:

$this->CI = get_instance();
 

Saludos.