[darkcarnival]

hi,

I have a program that i want to log submitions so i know what people are asking for but it doesnt log it :S heres the line of code im using:

PHP Code:

while ($row = mysql_fetch_array ($results, MYSQL_ASSOC)) {

 echo "<A HREF=\"{$row['Address']}\">{$row['Name']}</A> - {$row['Description']}<br>\n";



}

mysql_query("INSERT INTO SSearch_log (Catagory, Keyword, Date) VALUES ($Choice, $keyword, $date);"); 

i think i know what i did wrong but im not 100% could someone help me thx

[NeverMind]

this peice of code doesn't help much !
but anyway the log should be done in the page that displays the item you want ..
so it gets the name and INSERT all the info about this $name in the log table ..
that what I use..

[darkcarnival]

ok i think i know what your talking about ill give it a try.thx

[darkcarnival]

hi,

i fixed it but theres still 1 small problem. the date doesnt appear it shows 0000-00-0000 heres the fixed code:

PHP Code:

while ($row = mysql_fetch_array ($results, MYSQL_ASSOC)) {

 echo "<A HREF=\"{$row['Address']}\">{$row['Name']}</A> - {$row['Description']}<br>\n";



mysql_query("INSERT INTO SSearch_log (Catagory, Keyword, Date) VALUES ('$Choice', '$keyword', '$date')");

} 

and if you need to know what $date look like:

PHP Code:

$date = Date("j-d-Y"); 

i hope someone can help me with this. thx

[mdhall]

I usually make a field named "current_date" ( field-type is date ) in the table row, and have the insert query use 'now()' to insert the date in the field. Then display it as

$current_date=$date;
$date = Date("m-d-Y");
or
$date = Date("d-m-Y");

Don't know if its the best way but it works.

[darkcarnival]

well what is the problem with the code i got now. is there a way to fix the code i got now?

[mdhall]

What is the value of $date when it is inserted? Is it coming from a form or being added automatically?

$date = Date("j-d-Y");

Change the "j" to "m", see if that helps.

Wasn't inferring that I had a better way to do it, just offering another option.

[darkcarnival]

it should be added automatically. and i guess m would be correct ill give it a try