Opened 2 years ago
Last modified 2 years ago
#15906 new defect (bug)
validate_plugin needs all_plugins filter
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Future Release |
| Component: | Plugins | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: |
Description
If a plugin adds plugins to the plugin list in the plugin admin screens using the all_plugins filter, the added plugins will not validate. If the all_plugins filter is added to the validate function, then the plugin validates and activates fine.
Attachments (1)
Change History (8)
comment:1
SergeyBiryukov
— 2 years ago
comment:2
wpmuguru
— 2 years ago
If it was added to get_plugins, then both the $plugin_folder parameter & context (ex. plugin_editor) should be provided to the filter. So, I'm not sure that moving the filter there would provide any benefit.
comment:3
follow-up:
↓ 4
nacin
— 2 years ago
We killed off the all_plugins filters in 3.0, I believe.
get_plugins() function is used in several more places (e.g. in Plugin Editor). Shouldn't we use all_plugins filter in get_plugins() itself?