Thread

history seems to delete both, I notice it does the same thing on the codeigniter forum.

Have you noticed when you use the recaptcha, and set it to something other than custom that it shows the captcoa above the form?

http://dev3.digitalfusionstudios.ca/auth/register

Very nice. Been looking through your code. It seems very solid + flexible enough. THANKS!

Ok, I see. That’s right, after you clear history (as a part of private browsing session, for example) the autologin doesn’t work.
Otherwise browsing wouldn’t be private. 😊

Have you noticed when you use the recaptcha, and set it to something other than custom that it shows the captcoa above the form?

http://dev3.digitalfusionstudios.ca/auth/register

Yes, on your page I see it above the form. Don’t know why. Probably the problem is in view or CSS-styles.

Here is my example:
http://fs.lifetweak.ru/auth/register

if I use the custom captcha theme i get the same as you. Change your theme to “clean” and see what happens.

Hmm, I see.
Sorry, I am not well aware of the reCAPTCHA library to answer your question now. It needs more research.

When adding recaptcha support to tank_auth I followed API documentation at http://recaptcha.net/apidocs/captcha/client.html Maybe your question already answered there?

Just wanted to echo that I think it would be helpful for the library to allow non-activated users to use the forgot password functionality.  Otherwise, if a user registers but doesn’t receive or misplaces their activation email, they’re dead in the water.  They can’t delete, re-register, re-send the activation email, or use the forgot password functionality.  (The forgot password functionality tells them that the email doesn’t exist, while the register functionality tells them it already exists - confusing.)

In the meantime, this reply tells us how to code in support for the library (hooray forums! 😊 ) - but chalk this up as another vote to officially support this option.

Just wanted to echo that I think it would be helpful for the library to allow non-activated users to use the forgot password functionality.  Otherwise, if a user registers but doesn’t receive or misplaces their activation email, they’re dead in the water.  They can’t delete, re-register, re-send the activation email, or use the forgot password functionality.  (The forgot password functionality tells them that the email doesn’t exist, while the register functionality tells them it already exists - confusing.)

In the meantime, this reply tells us how to code in support for the library (hooray forums! 😊 ) - but chalk this up as another vote to officially support this option.

Ok, ok, I agree with you guys. It’s really silly when you can’t do anything with your account because of missed activation email and forgotten password.

I’ve just added such option, please check the newest version of the library - 1.0.6. In this version if non-activated user forgot his password then he may reset it using “Forgot password” feature as well as user with activated account. Clicking a link in resetting password email will activate user account as well as ordinary activation email.

Only 3 files were changed: controllers/auth.php, libraries/Tank_auth.php, models/tank_auth/users.php. Database isn’t affected.

Download library in a zip-file from official site
Download library from SVN repository

hi,,

i have problem.. with this.. im cannot load library file from :

System->Application->libraries->Tank_auth.php

“Unable to load the requested class: Tank_auth”

can anyone help me..

How do I allow different characters in the password? because imo it’s ridiculous to restrict something like @

thanks a lot

How do I allow different characters in the password? because imo it’s ridiculous to restrict something like @

thanks a lot

Get rid of ‘alpha_dash’ validation rules for ‘password’ and ‘new_password’ fields in auth controller (lines 130, 302, 350).

I’ve got my admin section that I’m trying to work out right now. If I’ve got all sorts of methods, it makes no sense to add the if/else statement to EVERY method - so instead i’ve added it to the constructor, like so:

class Welcome extends Controller
{
    function __construct()
    {
        parent::__construct();

        $this->load->library('tank_auth');
                if (!$this->tank_auth->is_logged_in()) {
            redirect('/auth/login/');
        } else {
            $data['user_id']    = $this->tank_auth->get_user_id();
            $data['username']    = $this->tank_auth->get_username();
        }
    }

    function index()
    {
        $this->load->view('welcome', $data);
    }
}

However, this doesn’t work. How come my $data variable isn’t available to index?  I want the user_id and username to be available to all of my methods, but it goes against the DRY concept if I have to put that $data info in every method.

If you use $data in more than one method, it should be declared as a class member, right?

If you use $data in more than one method, it should be declared as a class member, right?

I’m not sure :(