Help with login cookie...

[Anon]

Hi... i keep getting the following error:

Parse error: parse error in /usr/local/psa/home/vhosts/*****.com/httpdocs/login/index.php on line 25

I am totally unsure why this is happening!

I have included my code below so any help would be great.

Jonathan

PS: Like 25 is "("user_cookie",$username,"","/");"

===========

if (!$username || !$password)
{
header("Location: login_form.php?comments=ERROR:%20No%20Username%20or%20Password%20Entered!");
}

$username=strtolower($username);
$password=strtolower($password);

$sql="SELECT FROM users WHERE username='$username' AND password='$password'";
$result=db_query($sql);
if (!$result || db_numrows($result) < 1){
header("Location: login_form.php?comments=ERROR:%20User%20not%20found%20or%20password%20incorrect');
} else {
if (!$result || db_numrows($result) > 1){
setcookie("user_cookie",$username,"","/");
header("Location: http://www.***.com/index.php");
}
?>

=========

Thanx

[[deleted]]

$result=db_query($sql);
if it's something like mysql_db_query, it will need one more argument - the database name

[Anon]

I have the database name declared above the code that i posted😉 Thanx anyway! Any more ideas?

[[deleted]]

header("Location: login_form.php?comments=ERROR:%20User%20not%20found%20or%20password%20incorrect' !!! " !!! );

should be closed
is this the problem ? 🙂

[Anon]

Nope🙁

[Anon]

This is annoying me now! lol! Ive changed it and i get this;

Warning: Supplied argument is not a valid MySQL result resource in /usr/local/psa/home/vhosts/****.com/httpdocs/login/index.php on line 19

Warning: Cannot add header information - headers already sent by (output started at /usr/local/psa/home/vhosts/*.com/httpdocs/login/index.php:19) in /usr/local/psa/home/vhosts/**.com/httpdocs/login/index.php on line 20

===========
mysql_connect($DBhost,$DBuser,$DBpass) or die("Unable to connect to database");
mysql_select_db ($DBname);

if (!$username || !$password)
{
header("Location: login_form.php?comments=ERROR:%20No%20Username%20or%20Password%20Entered!");
}

$username=strtolower($username);
$password=strtolower($password);

$sql="SELECT * FROM users WHERE username='$username' AND password='$password'";
if(sizeof(mysql_num_rows($result)) == 0) {
header("Location: login_form.php?comments=ERROR:%20User%20not%20found%20or%20password%20incorrect");
} else {
if(sizeof(mysql_num_rows($result)) == 0) {
setcookie ("user_cookie", $username,time()+3600);

header("Location: http://www.hscparts.com/index.php");
}
}
?>

=====
any ideas?

[[deleted]]

sizeof(mysql_num_rows($result))!?!

mysql_num_rows() returns int, not an array

[[deleted]]

and if you still use mysql_db_query()
change it to mysql_query()

why don't you paste the whole code

[Anon]

ok i think thats a gud idea.... im starting to get really confused now lol.

=======

<?php
$DBhost = "localhost";
$DBuser = "*";
$DBpass = "";
$DBname = "***";

mysql_connect($DBhost,$DBuser,$DBpass) or die("Unable to connect to database");
mysql_select_db ($DBname);

if (!$username || !$password)
{
header("Location: login_form.php?comments=ERROR:%20No%20Username%20or%20Password%20Entered!");
}

$username=strtolower($username);
$password=strtolower($password);

$sql="SELECT FROM users WHERE username='$username' AND password='$password'";
if(sizeof(mysql_num_rows($result)) == 0) {
header("Location: login_form.php?comments=ERROR:%20User%20not%20found%20or%20password%20incorrect");
} else {
if(sizeof(mysql_num_rows($result)) == 0) {
setcookie ("user_cookie", $username,time()+3600);

header("Location: http://www.****.com/index.php");
}
}
?>

=========

[[deleted]]

well... i asked you to post whole code, not just the part you find important

and consider this:
$r=0;
print sizeof($r);