Cron UTC time [#2315743]

hook_cron() function is using incorrect date calculation.
Table 'weather_forecasts' field 'time_to' is using local time, not UTC, so why check for current UTC time?

Patch attached.

Comment	File	Size	Author
#1	cron_utc_time-2315743.patch	682 bytes	rudins

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

rudins CreditAttribution: rudins commented 5 August 2014 at 12:22

File	Size
cron_utc_time-2315743.patch	682 bytes

Comment #2

30 September 2014 at 20:36

toddy committed f38f60c on 7.x-2.x

#2315743: Fix an error in cron check for outdated forecasts

Comment #3

toddy CreditAttribution: toddy commented 30 September 2014 at 20:55

Assigned:	Unassigned	» toddy
Status:	Needs review	» Fixed

Thanks for the bug report, this is fixed in the latest release. Although I did not use exactly your patch, because the local time (of the computer, not the location of the forecast) would be equally wrong. I've now settled to calculate yesterday in UTC time, that should even catch the western timezones which are some hours behind UTC.

Regards,
Tobias