Make WordPress Core

Opened 15 years ago

Closed 15 years ago

Last modified 15 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:


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 15 years ago.

Download all attachments as: .zip

Change History (4)

#1 @ryan
15 years ago

  • Owner changed from anonymous to ryan

#2 @ryan
15 years ago

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

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

#3 @ryan
15 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.