Opened 10 years ago
Closed 10 years ago
#28111 closed defect (bug) (fixed)
delete_plugins() $redirect argument unused
Reported by: | jdgrimes | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 4.0 | Priority: | normal |
Severity: | normal | Version: | 2.6 |
Component: | Plugins | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
The second argument for delete_plugins()
($redirect
), has been unused since the function was introduced in [8049]. Apparently it was unnecessarily inherited from its forerunners.
Attachments (3)
Change History (9)
#2
in reply to:
↑ description
@
10 years ago
#3
follow-up:
↓ 4
@
10 years ago
- Milestone changed from Awaiting Review to 4.0
We should probably deprecate it (like in [27162]), in case we ever need to add another argument here.
#4
in reply to:
↑ 3
@
10 years ago
Replying to SergeyBiryukov:
We should probably deprecate it (like in [27162]), in case we ever need to add another argument here.
28111.patch deprecates the argument. I included a call to _deprecated_argument()
if $redirect
is passed, though this wasn't done for [27162].
#5
@
10 years ago
- Keywords commit added
_deprecated_argument()
is probably not necessary here.
28111.2.patch also contains some PHPDoc fixes and additions.
Note: See
TracTickets for help on using
tickets.
Replying to jdgrimes: