[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