Make WordPress Core

Opened 18 years ago

Closed 18 years ago

Last modified 18 years ago

#3169 closed defect (bug) (fixed)

Cron makes empty argument arrays into array(array()), and then array(array(array())), and so on, upon rescheduling

Reported by: masquerade's profile masquerade Owned by: ryan's profile ryan
Milestone: 2.1 Priority: normal
Severity: normal Version: 2.1
Component: Administration Keywords:
Focuses: Cc:

Description

The way func_slice_args was used caused some issues. I've done away with it in the patch. Now all the cron functions that will pass along args to their corresponding hooks pass in an array of the args to pass to the callbacks run on the hook, as opposed to adding all those args to the end. I personally think its better this way, but it is API breakage for anyone who might've written a plugin using cron so far (only me probably =D ).

Jibber jabber jibber, patch is attached.

Attachments (1)

psuedo-cron-no-nested-array.2.diff (3.0 KB) - added by masquerade 18 years ago.

Download all attachments as: .zip

Change History (4)

#1 @ryan
18 years ago

  • Owner changed from anonymous to ryan

#2 @ryan
18 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [4362]) Change cron arg passing. Props masquerade. fixes #3169.

#3 @ryan
18 years ago

Do we need to upgrade existing entries in the cron array to get rid of empty array()s? I want to make sure already scheduled future posts will be processed.

Note: See TracTickets for help on using tickets.