Opened 6 years ago
Closed 6 years ago
#45870 closed defect (bug) (fixed)
Error in wp-cron.php $gmt_time not defined
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 5.1 | Priority: | normal |
Severity: | normal | Version: | 5.1 |
Component: | Cron API | Keywords: | good-first-bug dev-feedback has-patch |
Focuses: | Cc: |
Description
r44483 removed the line $gmt_time = microtome( true );
Now the following error appears.
PHP Notice Undefined variable: gmt_time in /app/public/wp-cron.php on line 100
Attachments (1)
Change History (9)
#1
@
6 years ago
- Keywords needs-patch added
- Milestone changed from Awaiting Review to 5.1
- Owner set to peterwilsoncc
- Status changed from new to assigned
- Version set to trunk
#4
@
6 years ago
- Keywords good-first-bug added
@afragen, thanks
- Line 81: Also uses the gmt time. I had missed this one.
- Line 100: Ass @afragen noted above. This is kept as a logic check, I am in two minds as to whether it should remain but lean towards yes.
Note: See
TracTickets for help on using
tickets.
I think changing line 100 to
if ( $timestamp > microtime( true ) ) {
should fix it.This was the only place in the file that variable was used.