[wp-trac] [WordPress Trac] #53937: delete the un-used function _upgrade_cron_array()

WordPress Trac noreply at wordpress.org
Wed Aug 18 03:23:28 UTC 2021


#53937: delete the un-used function _upgrade_cron_array()
--------------------------+----------------------
 Reporter:  pbearne       |       Owner:  (none)
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:
Component:  Cron API      |     Version:  trunk
 Severity:  normal        |  Resolution:  wontfix
 Keywords:  has-patch     |     Focuses:
--------------------------+----------------------
Changes (by peterwilsoncc):

 * status:  new => closed
 * resolution:   => wontfix
 * milestone:  Awaiting Review =>


Comment:

 Replying to [comment:3 SergeyBiryukov]:
 > I think we should not remove the `version` field from
 `_set_cron_array()` though, as that would block us from changing the
 option structure in the future.

 I agree with Sergey's view here, both in terms of allow for future format
 changes and allowing for very old installs to update. It's also a pretty
 harmless function to keep around.

 Ticket #34913 contains a patch that proposes upgrading the cron array
 format to version 3 within `_upgrade_cron_array()` so it may happen at
 some point.

 I'm going to close this ticket as ''wontfix'' but thank you for taking an
 interest in the cron component. It's great to see it getting some
 attention.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/53937#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list