Fix for logged in users

[mpdaddy]

This is my fix for the issue I brought up at Bug? Should it work this way?.

This code fixes it so that when a user is logged into IndexU,
if they access either the login.php or register.php page, they will be redirected to the index.php page.

Fixing this issue involves some minor modifications to the login.php and register.php files.

Open login.php and replace the following function:

  function ShowFormLogin() {

    // vars global configuration
    global $theme_path;

    // vars url & form
    global $f, $b;

    // vars template
    global $error_msg;

    if ($f == 1)
      DisplayTemplate($theme_path . "permission_error.html", "\$error_msg,\$username,\$password,\$f,\$b");
    else
      DisplayTemplate($theme_path . "login_form.html", "\$error_msg,\$username,\$password,\$f,\$b");
  }

With this:

  function ShowFormLogin() {

    // vars global configuration
    global $theme_path;

    // vars url & form
    global $f, $b;

    // vars template
    global $error_msg;

  $users_obj = new clsUsers;

  if ($users_obj->GetUserAuthentication() == 0) {

    if ($f == 1)
      DisplayTemplate($theme_path . "permission_error.html", "\$error_msg,\$username,\$password,\$f,\$b");
    else
      DisplayTemplate($theme_path . "login_form.html", "\$error_msg,\$username,\$password,\$f,\$b");
}
  else {
	    Redirect("index.php");
    }
}

Now open register.php and replace the following function:

  function ShowFormRegister() {

    // vars global configuration
    global $theme_path;

    // vars template
    global $error_msg;

    DisplayTemplate($theme_path . "register_form.html", "\$error_msg,\$username,\$password,\$password2,\$email");
  }

With this:

  function ShowFormRegister() {

    // vars global configuration
    global $theme_path;

    // vars template
    global $error_msg;

  $users_obj = new clsUsers;

  if ($users_obj->GetUserAuthentication() == 0) {

    DisplayTemplate($theme_path . "register_form.html", "\$error_msg,\$username,\$password,\$password2,\$email,\$logged_in");
  }
  else {
	    Redirect("index.php");
    }
}

Thats all there is to it.

You can download the login.php and register.php files for free along with the instructions in zip format at http://store.scriptdragon.com/viewitem.php?ItemID=8

Wayne

[esm]

is Redirect a standard php function?


.

[esm]

another way to do it is to add the following just after the first line <?php in the register.php and login.php files

 $url="index.php";
if ($_COOKIE['COOKIE_USER_AUTHENTICATED'])
{
  if(!headers_sent()) 
  { 
    header("Location: ". $url); 
    exit; 
  } else {
 
  // otherwise, try to redirect with javascript 
  echo 
  "<script language='javascript'>\n". 
  "document.location.href='".$url."';\n". 
  "</script>\n"; 
    exit;
  }
} 
  

[mpdaddy]

is Redirect a standard php function?


.

No the php way is with the header() way you did it. I just used the redirect function found in misc.lib.php. Either way it works.

As Bruce said

Mods have never really been added to the base code just because they were submitted as far as I know. It's probably easier for Dody to write the code and add it himself than it is to try and incorporate someone else's code.

My intention was to get it included in the core code. I tried to write it as Dody would using his built in functions. The redirect code is actually used farther down in the login.php file. I truly think this is the way the script should work right out of the box. We should not have to fix it.

What site have you ever been to that allows you to re-register when you are already logged in?

What site lets you re-login when you are already logged in?

[esm]

well, dody may end up doing it that way.

But I would remove the register button by wrapping an IF statement around it in the header file and modify the DisplayTemplate function. But that would mean the DisplayTemplate function would need to be modified in several files.

Is the login box still there after you login? I thought it just showed you status as logged in with a link to logout...?



.

[Bruceper]

I did that with my ICE theme, so that when a user logs in the register link is hidden. Of course the page is still there, but most users wouldn't go hunting for it.

[echo@]

good idea, shame it doesnt work.

modified the files acordingly and now im just redirected regardless if im logged in or not.

I must of missed something , running indexu 5.4,

Whats happed to your mods site ??

[mpdaddy]

good idea, shame it doesnt work.

modified the files acordingly and now im just redirected regardless if im logged in or not.

I must of missed something , running indexu 5.4,

Whats happed to your mods site ??

After the whole upgrade issue was botched, and I found out some information about John Milton and his company that I am not comfortable with (unconfirmed or I would post it) , I decided that IndexU was not the right script for me. I took the site down and have moved on to another venture, which I am sure you will hear about in the next couple months.

I am currently updating my mods, but finding it difficult working with the plugin system. I will release the free ones when I get them finished and I am not sure what will happen to the pay ones yet. I might turn them over to somebody else to support. I will let you know when I figure it out.

The code worked on my server, but I may have left something out. I put it together pretty fast and may have missed some code. This is a slight modification of what esm posted and it workes great. The only thing I changed was $_COOKIE['COOKIE_USER_AUTHENTICATED'] should have been $_COOKIE['COOKIE_USERNAME']. Add it to the top of your login.php and register.php just after the <?php tag at the top. Here it is...

 $url="index.php";
if ($_COOKIE['COOKIE_USERNAME']) {
  if(!headers_sent()) {
    header("Location: ". $url);
    exit;
  } else {
 
  // otherwise, try to redirect with javascript
  echo
  "<script language='javascript'>\n".
  "document.location.href='".$url."';\n".
  "</script>\n";
    exit;
  }
} 
  

Regards
Wayne

[echo@]

...............

[esm]

After the whole upgrade issue was botched, and I found out some information about John Milton and his company that I am not comfortable with (unconfirmed or I would post it) , I decided that IndexU was not the right script for me.

Regards
Wayne

I agree.


.