Recording cds browsed

Anon

Hi can anyone help?

I am designing a music website and want to be able to store the cdids for the cds a registered user clicks on in a cdsbrowsed database table.

I have started these sessions on previous pages.

$ valid_user = $email;
session_register("valid_user");

$valid_user2 = $customerid;
session_register("valid_user2";

Here is my code for storing cdids in cdsbrowsed for registered customers:

if (session_is_registered("valid_user2")){

//Query the database to check not repeat cd browsed for that customer id

$query= "select count(*), cdid from cdsbrowsed
where customerid='$valid_user2'
and cdid= ".$HTTP_GET_VARS['cdid'];

$result = mysql_query($query);

if(!$result)
{  
	echo 'Cannot run query.';
	exit;

At the moment it prints Cannot run query.
Can anyone see any problems with the query?

ayzee01

Nicola, check your field types. I notice that you are querying the customerid field with the value enclosed in ' '. If this field is numeric then it will not return any results for this query. If the customerid field is numeric then ommit the quotes.

Also, try this code, it is cleaner and should provide better error handling...

<?
$result = mysql_query($query)
or die("Error with SQL statement");
?>

Also, try simplifying your SQL...

$query = "SELECT cdid FROM cdsbrowsed
WHERE customerid = ".$valid_user2." AND cdid = ".$HTTP_GET_VARS['cdid'];

I have written a small database class that could make things simpler for you. If you would like a copy then please drop me an email with your picture and a phone number :o)

Only kidding, nah, if you want the class then i'll email it to you.

Hope I've been of help,

Leon.