[Vtigercrm-developers] [patch] Login, then "browser back" causes "mixed" screen

[Asha]

Hi Kristof,

Thanks for the patch. We shall try it out and integrate if works well.

Btw, we suggest you to submit all the patches through trac.

On Mon, Sep 1, 2008 at 10:29 PM, Kristof Kovacs <
kristof.kovacs at onlineprojects.hu> wrote:

> Hello guys,
>
> On certain browsers, if the user hits a "browser back" after logging
> in to vtiger, he is presented with a screen that has a menu on the
> top, and the login screen in the middle. This is of course a minor
> bug, but there were complaints, so we fixed, and here is the patch.
>
> To reproduce the bug on any browser: Log in, and then go to index.php?
> action=Login&module=Users .
>
> Kristóf
>
> ---
>  index.php |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/index.php b/index.php
> index ae23b93..3781a08 100644
> --- a/index.php
> +++ b/index.php
> @@ -192,6 +192,11 @@ if(isset($_SESSION["authenticated_user_id"]) &&
> (isset($_SESSION["app_unique_key
>          $use_current_login = true;
>  }
>
> +// Prevent loading Login again if there is an authenticated user in
> the session.
> +if (isset($_SESSION['authenticated_user_id']) && $module == 'Users'
> && $action == 'Login') {
> +        header("Location: index.php?action=$default_action&module=
> $default_module");
> +}
> +
>  if($use_current_login)
>  {
>
>
>
> _______________________________________________
> Reach hundreds of potential candidates - http://jobs.vtiger.com
>



-- 
Regards,
Asha
vtiger Team
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.vtigercrm.com/pipermail/vtigercrm-developers/attachments/20080904/4e7c7d11/attachment-0003.html