WordPress.org

Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 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 Owned by: 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 9 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 @ryan9 years ago

  • Owner changed from anonymous to ryan

comment:2 @ryan9 years ago

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

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

comment:3 @ryan9 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.