Opened 10 years ago
Last modified 9 years ago
#37929 new enhancement
Add `pre_delete_option` filter
| Reported by: | flixos90 | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Awaiting Review |
| Component: | Options, Meta APIs | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
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
@
10 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.
#3
@
10 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.
9 years ago
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
37929.diff is a first take on introducing the filter.