Opened 8 years ago
Last modified 7 years ago
#37929 new enhancement
Add `pre_delete_option` filter
Reported by: | flixos90 | Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | |
Component: | Options, Meta APIs | Keywords: | has-patch |
Focuses: | Cc: |
Description
get_option()
and update_option()
already have filters to short-circuit their respective processes, so I think a similar filter should be added to delete_option()
as well.
Related: #37928
Attachments (2)
Change History (7)
#2
@
8 years ago
Interesting idea.
In your patch, maybe you should use
! is_null( $skip )
instead of
null !== $skip
to keep it consistent with the null check immediately below.
Version 0, edited 8 years ago
by
(next)
#3
@
8 years ago
37929.2.diff is an updated patch that gives parity with the latest patch on #37928.
This ticket was mentioned in Slack in #core-multisite by flixos90. View the logs.
8 years ago
Note: See
TracTickets for help on using
tickets.
37929.diff is a first take on introducing the filter.