Help with 24hr code

[Simplyput]

Greetings ones who know more then I

I am writing a pice of code for my members to be able to update information and restrict them from updating it again until 24 hours have passed. I have everyting else working except this part. Below is that I have for the time code.

$max_update = 1; 

         $time_from = time() - (24*60*60);
         $time_to = time();
      $count = f(q("SELECT update_count AS count FROM members WHERE email='$email'AND time>='$time_from' AND time<='$time_to'"));
      $count = $count[count];
         if($count > $max_update){
         include "templates/members/twenty_hours.php";
            return;}

As I said everything else works up until this point. the code ignores the time factor and only looks at the count. I.E. I ran it two times for the same member email and it went straight to template twenty_hours.php which is what its suppose to do. I feel I have my time statement logic incorrect. Any assistance will be greatly appreciated.

Thanks

P.S. time = time()

[dagon]

Simplyput;10903876 wrote:

Greetings ones who know more then I

      $count = f(q("SELECT update_count AS count FROM members WHERE email='$email'AND time>='$time_from' AND time<='$time_to'"));

fatal error, unless you have a function called f(), so you need to turn error checking on

also you never actully run the query.

[djjjozsi]

In a real world (246060) is not a day 🙂
mysql has better date-time functions.

such as interval

 SELECT (`date_field` - INTERVAL 1 DAY) AS oneday_min FROM timestable;

No need to hardcode these things in php, BETWEEN is a better interval function in mySQL.
time is a timestamp? becouse time() gives you back a simple number. What is your database field: time ?
See your code, the email index in the $count array is undefined: Use quote.marks.

You should use error reporting while you troubleshoot your code:

// Report all PHP errors
error_reporting(E_ALL);

// Same as error_reporting(E_ALL);
ini_set('error_reporting', E_ALL);

Good luck 🙂

[coldwerturkey]

djjjozsi;10903909 wrote:

In a real world (246060) is not a day

What do you mean? 246060 = 86400 and according to the all knowing google, that accurate

[djjjozsi]

coldwerturkey;10903910 wrote:

What do you mean? 246060 = 86400 and according to the all knowing google, that accurate

86400 is a day with exceptions.

If we are talking about midwinter , and leap years , you must handle these events if you're making timetables.
And in mysql the date-time functions are better then a hardcoded substaction becouse of these reasons.

In my country if the time is 18:05 what is in your's , if the server is running in London 🙂

[Weedpacket]

Most days are 86400 seconds long, but some are 82800 seconds long and some are 90000 seconds long. That can make a difference. If you want to be really fancy, and there can even be a few that are 86399 seconds long (though there haven't been any yet) or 86401 seconds long (such as December 31, 2008).

Then again, the original posting said "24 hours" - that's not the same thing as "1 day" for exactly the above reasons.