Opened 8 years ago
Closed 8 years ago
#45870 closed defect (bug) (fixed)
Error in wp-cron.php $gmt_time not defined
| Reported by: | afragen | Owned by: | peterwilsoncc |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.1 |
| Component: | Cron API | Version: | 5.1 |
| Severity: | normal | Keywords: | good-first-bug dev-feedback has-patch |
| Cc: | Focuses: |
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
@
8 years ago
- Keywords needs-patch added
- Milestone Awaiting Review → 5.1
- Owner set to
- Status new → assigned
- Version → trunk
#4
@
8 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
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.