Ticket #3169 (closed defect (bug): fixed)

Opened 5 years ago

Last modified 5 years ago

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

Reported by: masquerade Owned by: ryan
Priority: normal Milestone: 2.1
Component: Administration Version: 2.1
Severity: normal Keywords:
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

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

Change History

comment:1   ryan5 years ago

  • Owner changed from anonymous to ryan

comment:2   ryan5 years ago

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

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

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