Issue with MYSQL Select and CURDATE()

[infinity0]

EDIT: I'm new here and for some reason, it looks like I created two threads. How do I delete one?

Hi, I have a mysql select from database when the date is today that I then put into a javascript text rotator. Unfortunately, it isn't displaying anything. Why would that be?

PHP Code:

include("config.php");
$result = mysql_query("SELECT * FROM events WHERE 'when' = CURDATE()");
if($result === FALSE) {
    die(mysql_error()); // TODO: better error handling
}

if ($result) {
  // array to hold all the output
  $events = array();
  while ($row = mysql_fetch_assoc($result)) {
    // Add the event to your array
    $events[] = $row['tag'];
  }
  // After building the array, encode it as JSON
  // Later you'll echo this into your JavaScript in place of the array...
  $events = json_encode($events);
}
echo $events;
?>
<script type='text/javascript'>
function rotateEvery(sec)
{
  // The JSON from PHP output here
  // Would look something like
  // ["Event 1","Event 2","Event 3"]
  var Quotations = <?php echo $events; ?>;

  var which = Math.round(Math.random()*(Quotation.length - 1));
  document.getElementById('textrotator').innerHTML = Quotation[which];

  setTimeout('rotateEvery('+sec+')', sec*1000);
}
</script>

[guelphdad]

I deleted the other thread.
Welcome to CodingForums.

I can tell you that if your query is actually like this:
"SELECT * FROM events WHERE 'when' = CURDATE()

you have two problems:
1) WHEN is a reserved word so really you should have a different column name than that
2) by putting single quotes around your column name you have created a string, the word "when", the word "when" will never match a date so you will never get results.

You need backticks ` which is under the ~ key on the top left of your keyboard, and not single quotes as you have them. Better yet, give the column a new name and you won't need them at all.

[Fou-Lu]

This condition is invalid: 'when' = CURDATE(). 'when' is literally the string 'when'. Since when is a reserved word for case handling, you need to change the condition to `when` = CURDATE(), assuming its a datetime type of course.

Edit:
Nuts, beaten to the punch :P

[guelphdad]

Originally Posted by Fou-Lu

Nuts, beaten to the punch :P

I am in Ontario after all.

[infinity0]

Worked Great, thanks so much.