MeNeedz Auth

[eluser]Référencement Google[/eluser]
From your user guide example:

Code:

header("location: ".$this->config->site_url().'/welcome/login');

Better use CI redirect:

Code:

redirect('welcome/login');

Also, where in your code is PHP5?

Code:

class Auth
{
    var $CI = null;
    var $user_table = "user";
    var $user_identifier_column = "user_name";
    var $user_password_column = "user_password";
    var $user_salt_column = "user_salt";
    var $user_group_column = "user_group";
    var $use_database = TRUE;
    var $non_database_user_identifier = "";
    var $non_database_user_password = "";
    var $use_encryption = TRUE;
    var $encryption_method = "md5";
    var $encryption_order = "%salt%.%password%";
    var $user_identifier_input_name = "user_name";
    var $user_password_input_name = "user_password";
    var $user = FALSE;
    var $user_groups = array('user' => '1', 'moderator' => '10', 'admin' => '100');
    var $use_remember_me = FALSE;
    var $remember_me_input_name = "remember_me";

    var $_encryption_methods = array('md5', 'sha', 'sha1');
    var $_encryption_elements = array('%salt%', '%password%');

    function Auth($config = array())
    {
           ... etc...

Your code is also a bit of spaghetti coding without any comments. That makes it real difficult to understand, and I am sure there would be a lot lot lot of things to improve in it, first of separate the DB code in a model, flag some long successions of if / else tests instead of returning each time something.

Also I don't understand why this mixture of post code without any validation of the form. Why not just use some validation then? That said, I personally think an auth lib shouldn't mix things like you did. Keep it simple, just do auth process in this lib and add some extended class to do other things, keep it structured.

Sorry about criticism, it's to be constructive...