Ticket #2814 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

Another quick cron problem

Reported by: robmiller Assigned to: robmiller
Priority: high Milestone: 2.1
Component: General Version: 2.1
Severity: major Keywords: bg|has-patch bg|tested
Cc:

Description

Fixes array_keys error if cron option is not an array.

Attachments

2814.diff (0.9 kB) - added by robmiller on 06/11/06 04:00:56.
2814.2.diff (400 bytes) - added by robmiller on 06/27/06 04:06:56.

Change History

06/11/06 04:00:56 changed by robmiller

  • attachment 2814.diff added.

06/11/06 04:02:00 changed by robmiller

  • keywords set to bg|has-patch bg|tested.
  • owner changed from anonymous to robmiller.
  • status changed from new to assigned.

06/27/06 00:12:36 changed by robmiller

Any input? This would be nice for 2.0.4.

06/27/06 00:22:31 changed by ryan

  • version changed from 2.0 to 2.1.
  • milestone changed from 2.0.4 to 2.1.

Have you tested the latest cron.php in trunk? It's a bit different.

Cron doesn't exist for 2.0.x. It's 2.1 only.

06/27/06 04:06:45 changed by robmiller

Ooh, I thought it was a 2.0 thing. My bad!

The error is still there in the latest trunk, though. Here's a patch that takes into account the other fix from the other ticket.

06/27/06 04:06:56 changed by robmiller

  • attachment 2814.2.diff added.

06/27/06 06:04:28 changed by ryan

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [3929]) Fix array_keys error when cron option is not an array. Props robmiller. fixes #2814

06/27/06 07:08:50 changed by robmiller

Eek! You caught the typos, but sorry. That'll teach me to submit patches at 4am :)