[SOLVED] Login

[simonb]

I am creating a simple login and i can't get it to work

/index.php

PHP: Select all

    <form class="search" method="post" action="/login.php">
      <p>
        <input name="search_query" type="text" class="textbox" value="Username" />
        <input class="textbox" type="text" name="search_query" value="*******" />
        <input class="button" type="submit" name="Submit" value="Login" />
      </p>
    </form> 

And the login.php

PHP: Select all

<?php include('./Connections/Links.php'); ?>
<?php
// *** Validate request to login to this site.
if (!isset($_SESSION)) {
  session_start();
}

$loginFormAction = $_SERVER['PHP_SELF'];
if (isset($_GET['accesscheck'])) {
  $_SESSION['PrevUrl'] = $_GET['accesscheck'];
}

if (isset($_POST['username'])) {
  $loginUsername=$_POST['username'];
  $password=$_POST['password'];
  $MM_fldUserAuthorization = "userrank";
  $MM_redirectLoginSuccess = "/";
  $MM_redirectLoginFailed = "/fail.php";
  $MM_redirecttoReferrer = true;
  mysql_select_db($database_Links, $Links);
      
  $LoginRS__query=sprintf("SELECT username, userpassword, userrank FROM users WHERE username='%s' AND userpassword='%s'",
  get_magic_quotes_gpc() ? $loginUsername : addslashes($loginUsername), get_magic_quotes_gpc() ? $password : addslashes($password)); 
   
  $LoginRS = mysql_query($LoginRS__query, $Links) or die(mysql_error());
  $loginFoundUser = mysql_num_rows($LoginRS);
  if ($loginFoundUser) {
    
    $loginStrGroup  = mysql_result($LoginRS,0,'userrank');
    
    //declare two session variables and assign them
    $_SESSION['MM_Username'] = $loginUsername;
    $_SESSION['MM_UserGroup'] = $loginStrGroup;          

    if (isset($_SESSION['PrevUrl']) && true) {
      $MM_redirectLoginSuccess = $_SESSION['PrevUrl'];    
    }
    header("Location: " . $MM_redirectLoginSuccess );
  }
  else {
    header("Location: ". $MM_redirectLoginFailed );
  }
}
?>

It redirects the user to login.php but does nothing

[Jack Franklin]

No where in the code do you have the connection to the MYSQL database.
Try this tutorial
http://www.phpeasystep.com/workshopview.php?id=6

[simonb]

On the first line of login.php

PHP: Select all

<?php include('./Connections/Links.php'); ?>

Connections/Links.php

PHP: Select all

<?php
# FileName="Connection_php_mysql.htm"
# Type="MYSQL"
# HTTP="true"
$hostname_Links = "localhost";
$database_Links = "links";
$username_Links = "root";
$password_Links = "";
$Links = mysql_pconnect($hostname_Links, $username_Links, $password_Links) or trigger_error(mysql_error(),E_USER_ERROR); 
?>

[Jack Franklin]

Ahh ok. Try adding the links.php to the actual file instead of including it.

[simonb]

Ok makes not difference it still goes to that page, and its blank.

[Jack Franklin]

Code: Select all

$MM_redirectLoginSuccess = "/";

Surely you need a page there?

[simonb]

No not that tryed it


any i am sure its not that because i think the code is not be actived, On the index page i set the form to go to /login.php is that right?

[Jack Franklin]

Try removing the forward slash. Make sure the php does display text when logged in.
Try doing:
1) Set up a login success page

Code: Select all

$MM_redirectLoginSuccess = "loggedin.php"

On loggedin.php put

Code: Select all

<h2>You are logged in</h2>

Check all your links are correct. Try removing the '/' at the front of a link if the file is in the same directory.

[alexgeek]

You don't have a POST variable for username

[alexgeek]

More clearly:

HTML: Select all

        <input name="search_query" type="text" class="textbox" value="Username" />
        <input class="textbox" type="text" name="search_query" value="*******" />

One should be name="username" and the other name="password".
Nothing is happening in your script as you first check for these variables, but they don't exist so nothing happens.

[simonb]

Thanks for that Alex all working

I would give you some Rep

Solved

[alexgeek]

Glad you got it working mate.
Small errors like that can be hard to find.
If you think about it logistically though it's easy.
None of your code was being executed and all of it was dependent on that if statement therefore there's obviously something wrong with it