PHP - Spot the Error

Posted on 2011-03-25
Last Modified: 2012-05-11
Whats wrong with this query? I get an error "invalid month"

$yesterday = mktime(0, 0, 0, date('m'), date('d')-1,   date('Y')); // determin what yesterday was
$now = time();
Also how do I find out the issue?
$yesterday = mktime(0, 0, 0, date('m'), date('d')-1,   date('Y')); // determin what yesterday was
		$now = time();

	$reports_sql = "Select log.clientid,,log.message,log.LOGSTAMP
From admin_log log, client_master cm 
log.clientid=cm.clientid AND
log.logstamp >= TO_TIMESTAMP('".date('d', $yesterday).".".date('m', $yesterday).".".date('Y', $yesterday).":18:00:00','DD.MM.YYYY:HH24:MI:SS') 
log.logstamp < TO_TIMESTAMP('".date('d', $now).".".date('m', $now).".".date('Y', $now).":".date('H', $now).":".date('i', $now).":00','DD.MM.YYYY:HH24:MI:SS')
Order by ClientId";

Open in new window

Question by:codemonkey2480
LVL 27

Expert Comment

by:Lukasz Chmielewski
ID: 35218401
I'm not sure if that shouldn't be
'".date('d', $yesterday)."-".date('m', $yesterday)."-".date('Y', $yesterday)." 18:00:00'
(- instead of . like in a date 03-25-2011 18:00
LVL 83

Accepted Solution

Dave Baldwin earned 500 total points
ID: 35218679
Looks to me like it's producing the correct values.  Here I've converted your code to just display the results. I get:


Where is your error coming from? .

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"

$yesterday = mktime(0, 0, 0, date('m'), date('d')-1,   date('Y')); // determin what yesterday was
$now = time();
echo date("d.m.Y",$yesterday)."<br>";
echo date('d', $yesterday).".".date('m', $yesterday).".".date('Y', $yesterday).":18:00:00','DD.MM.YYYY:HH24:MI:SS')" ;
echo "<br>";
echo date('d', $now).".".date('m', $now).".".date('Y', $now).":".date('H', $now).":".date('i',$now).":00','DD.MM.YYYY:HH24:MI:SS')";


Open in new window


Author Comment

ID: 35219314
Let me take  look.

Expert Comment

ID: 35220036
SELECT log.clientid,,log.message,log.logstamp
FROM admin_log log, client_master cm 
    log.clientid = cm.clientid
    log.logstamp >= CURRENT_TIMESTAMP  - INTERVAL 1 DAY
ORDER BY log.clientid

Open in new window


Featured Post

Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Popularity Can Be Measured Sometimes we deal with questions of popularity, and we need a way to collect opinions from our clients.  This article shows a simple teaching example of how we might elect a favorite color by letting our clients vote for …
Deprecated and Headed for the Dustbin By now, you have probably heard that some PHP features, while convenient, can also cause PHP security problems.  This article discusses one of those, called register_globals.  It is a thing you do not want.  …
The viewer will learn how to look for a specific file type in a local or remote server directory using PHP.
The viewer will learn how to create a basic form using some HTML5 and PHP for later processing. Set up your basic HTML file. Open your form tag and set the method and action attributes.: (CODE) Set up your first few inputs one for the name and …

919 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

16 Experts available now in Live!

Get 1:1 Help Now