Issue #770

[abmmhasan]

This is the solution discussed in #770

[slaveek]

This solution have small issue look here -> #770 (comment)

[abmmhasan]

Well, I just changed according to your updated solution! in #770

[panique]

Thanks, I've just merged this manually via patch file as github cannot merge this automatically (merge conflicts)

[panique]

Cool feature btw! Nice, thanks! :)

[slaveek]

Hi I'm afraid that you didn't put all changes ;) in LoginController.
After your changes looks:

// check login status: if true, then redirect user to user/index, if false, then to login form again
if ($login_successful) {
    if (Request::post('redirect')) {
        Redirect::toPreviousViewedPageAfterLogin(ltrim(urldecode(Request::post('redirect')), '/'));
    } else {
        Redirect::to('user/index');
    }
} else {
    Redirect::to('login/index');
}

but to work this flawless we need to add more action when login is failed:

// check login status: if true, then redirect user to user/index, if false, then to login form again
if ($login_successful) {
    if (Request::post('redirect')) {
        Redirect::toPreviousViewedPageAfterLogin(ltrim(urldecode(Request::post('redirect')), '/'));
    } else {
        Redirect::to('user/index');
    }
} else {
    if (Request::post('redirect')) {
        Redirect::to('login?redirect=' . ltrim(urlencode(Request::post('redirect')), '/'));
    } else {
        Redirect::to('login/index');
    }
}

Here is a bit more #770 (comment)

[panique]

OH damn it :) I've just fixed this and it's now in develop branch! Thanks for the good eye for the details!

[slaveek]

I can create pull request with that fix if you don't mind.

[panique]

No prob it's already in dev branch!