Wats wrong?...

[FourthChapter]

Here is a simple login scipt:

<?php

$mysql_access = mysql_connect("localhost", "username" , "password");
mysql_select_db("databasename");


$check = mysql_query("select * from `tablename` where users='".$username."' 
and password='".$password."'");

$exists = mysql_num_rows($check);

if($exists == 1)
{
include("secure/index.php");
}
else
{
include("denied.php");
}

?>

I get the following error:

Warning: Supplied argument is not a valid MySQL result resource in /homepages/verify.php on line 10

Wats wrong? why doesnt it work 🙁

Thanks for your help (PS im quite new to PHP)

Matt

[trooper]

You should always check that the calls to mysql_connect and mysql_select_db are successful. Youd do this by changing you code to be the following

$mysql_access = mysql_connect("localhost", "username" , "password") or die mysql_error();
mysql_select_db("databasename") or die mysql_error();

Now assuming that they are successful you need to put the resource identifier (in this case $mysql_access) in the query, so it'd be

$check = mysql_query("select * from `tablename` where users='$username' and password='$password',$mysql_access);

Note that there is no need to escape the $username and $password vars in your query.

HTH

GM

Originally posted by FourthChapter
Here is a simple login scipt:

<?php

$mysql_access = mysql_connect("localhost", "username" , "password");
mysql_select_db("databasename");


$check = mysql_query("select * from `tablename` where users='".$username."' 
and password='".$password."'");

$exists = mysql_num_rows($check);

if($exists == 1)
{
include("secure/index.php");
}
else
{
include("denied.php");
}

?>

I get the following error:

Warning: Supplied argument is not a valid MySQL result resource in /homepages/verify.php on line 10

Wats wrong? why doesnt it work 🙁

Thanks for your help (PS im quite new to PHP)

Matt [/B]

[FourthChapter]

Ive done that, but i get the following error with the following code:

Parse error: parse error in /homepages/verify.php on line 3

<?php

$mysql_access = mysql_connect("localhost", "username" , "password") or die mysql_error();
mysql_select_db("databasename") or die mysql_error();

$check = mysql_query("select * from `tablename` where users='$username' and password='$password',$mysql_access);

$exists = mysql_num_rows($check);

if($exists == 1)
{
include("secure/index.php");
}
else
{
include("denied.php");
}

?>

Pardon my ignorance, im new lol. But im learning!

Thanks very much

Matt

[Sore]

You have mixed up the " and the '
Here:

$check=mysql_query("select*fromtablenamewhere_users='$username'_and_password='$password' ",$mysql_access);

[FourthChapter]

Should it look like this....

<?php

$mysql_access = mysql_connect("db114.oneandone.co.uk", "dbo71833037" , "CfQymWk2") or die mysql_error();
mysql_select_db("db71833037") or die mysql_error();

$check_=_mysql_query(" select_*_from_`4c_bdaccess`_where_users='$username'_and_password='$password' ",$mysql_access);

$exists = mysql_num_rows($check);

if($exists == 1)
{
include("secure/index.php");
}
else
{
include("denied.php");
}

?>

because it still comes up with the parse error on line 3

Thanks

Matt

[trooper]

No you have no need for the underscores

so it should ge like this

$check_=mysql_query(" select * from 4c_bdaccess where users='$username' and password='$password' ",$mysql_access) or die mysql_error();

Always always put some form of error checking on your function calls.

Note that you also do not need the ' ' around the table name - my best peice of advice for you is to get hold of a mysql book - you'll learn loads that way :-)

HTH

GM

Originally posted by FourthChapter
Should it look like this....

<?php

$mysql_access = mysql_connect("db114.oneandone.co.uk", "dbo71833037" , "CfQymWk2") or die mysql_error();
mysql_select_db("db71833037") or die mysql_error();

$check_=_mysql_query(" select_*_from_`4c_bdaccess`_where_users='$username'_and_password='$password' ",$mysql_access);

$exists = mysql_num_rows($check);

if($exists == 1)
{
include("secure/index.php");
}
else
{
include("denied.php");
}

?>

because it still comes up with the parse error on line 3

Thanks

Matt [/B]

😃 😃 😃 😃 😃

[FourthChapter]

Any recommendations for a good MySQL/PHP book?

Just lastly:

$mysql_access = mysql_connect("host", "username" , "password") or die mysql_error();

Still a parse error on this line, i cant find it ne where 🙁 😕

Thanks v much

Matt

[trooper]

post the code you have now and i will take a look at it

GM

[FourthChapter]

There ya go:

<?php

$mysql_access = mysql_connect("host", "username" , "password") or die mysql_error();
mysql_select_db("database") or die mysql_error();

$check_=mysql_query(" select * from table where users='$username' and password='$password' ",$mysql_access) or die mysql_error();

$exists = mysql_num_rows($check);

if($exists == 1)
{
include("secure/index.php");
}
else
{
include("denied.php");
}

?>

[jacob_kochi]

<?php
$mysql_access = mysql_connect("localhost", "username" , "password") or die(mysql_error());
mysql_select_db("databasename") or die (mysql_error());
$check = mysql_query("select * from tablename where users='$username' and password='$password'");

$exists = mysql_num_rows($check);

if($exists == 1)
{
include("secure/index.php");
}
else
{
include("denied.php");
}

?>

[trooper]

Appologies i mistakenly put die mysql_error()

when it should have been die(mysql_error())

[FourthChapter]

Thanks, but... lol

Warning: Supplied argument is not a valid MySQL result resource in /homepages/verify.php on line 6

with code:

<?php 
$mysql_access = mysql_connect("host", "username" , "password") or die(mysql_error()); 
mysql_select_db("database") or die(mysql_error()); 
$check = mysql_query("select * from tabel where users='$username' and password='$password'"); 

$exists = mysql_num_rows($check);  //this line has the error

if($exists == 1) 
{ 
include("secure/index.php"); 
} 
else 
{ 
include("denied.php"); 
} 

?>

lol, im sorry about this ppl, but im learning! lol

Thanks so much every 1 for ur help

Matt

[jacob_kochi]

<?php
$mysql_access = mysql_connect("localhost", "root" , "") or die(mysql_error());
mysql_select_db("database") or die(mysql_error());
$check = mysql_query("select * from table where users='$username' and password='$password'");

$exists = mysql_num_rows($check); //this line has the error

if($exists == 1)
{
include("secure/index.php");
}
else
{
include("denied.php");
}

?>

you must provide Database informations

You must create a database with name "database" in mysql
you must create a table named "table" with fields "users" and "password"

[FourthChapter]

I did that lol, i just put them there so u could see which field it is. On the one i upload i got all the real host informatio, password etc on it.

Itsjust that SQL error on line 6 thats buggin me

Thx very much

Matt

[jacob_kochi]

$check = mysql_query("select * from table where users='$username' and password='$password'") or die(mysql_error());

Please put mysql_error() in the above line so that we can find the exact error.

[Poolie]

Try putting the link in with your DB...

$mysql_access = mysql_connect("localhost", "root" , "") or die(mysql_error());
mysql_select_db("database", $mysql_access) or die(mysql_error());
$check = mysql_query("select * from table where users='$username' and password='$password'");

Poolie

[FourthChapter]

EXCELLENT!

Works PERFECTLY! thanks so much every one!

can i just make it harder now lol, this should be simple. How do i add data to the MySQL data, ie. Users last IP address, Time and Date of last logon?

Is this possible. If it is can it go on the same page as the logon script?

Thanks SO MUCH!

Matt

[Poolie]

Glad that worked for your... as for your other query that is simple..

<?php
$ip = $REMOTE_ADDR;
$query = mysql_query ("INSERT INTO table_name VALUES ($ip,NOW())");
?>

NOW() is a MySQL function that will add the current time etc into a cell.. so have it in the "last visit" field of your table (i think thats what your after) not sure if my example makes much sense though hehe

Poolie

[FourthChapter]

How does it know which colum to put the IP address in, and how do i choose which coloum to put the Date/TIme in. (soz i just need some more xplination lol)

Thanks

Matt

[Poolie]

Np's - Your probably better using an UPDATE rather than INSERT in your SQL query as well but

UPDATE table_name SET users_ip = '$ip' WHERE user_id = '$username';

Same with the last visit but use the NOW() function to log current time in MySQL

Does that make sense ? :p