#42364 closed task (blessed) (fixed)
Customize: Remove deprecated nav menu classes
Reported by: | westonruter | Owned by: | desrosj |
---|---|---|---|
Milestone: | 5.4 | Priority: | low |
Severity: | normal | Version: | 4.9 |
Component: | Customize | Keywords: | early has-patch commit has-dev-note |
Focuses: | Cc: |
Description
The following classes are deprecated in 4.9 and can be removed in 5.0:
wp.customize.Menus.NewMenuControl
(in JS)WP_Customize_New_Menu_Section
(in PHP)WP_Customize_New_Menu_Control
(in PHP)
These classes are not used by plugins and they are not used in core. For more, see #42357.
Attachments (3)
Change History (15)
This ticket was mentioned in Slack in #core-customize by dlh. View the logs.
6 years ago
#5
@
6 years ago
@Matthias Reuter are you able to refresh the patch? It is not applying to trunk
currently.
#8
@
5 years ago
- Milestone changed from 5.2 to Future Release
As 5.2 is now in beta and this was something that needed early attention, I'm removing it from the milestone.
#9
@
5 years ago
- Keywords needs-refresh removed
- Milestone changed from Future Release to 5.4
I have create a new patch for the removal of the wp.customize.Menus.NewMenuControl
(in JS) and the formal deprecation of the PHP WP_Customize_New_Menu_Section
and WP_Customize_New_Menu_Control
classes.
As those PHP classes have now been in Core for much longer than originally intended, I'm hesitant about removing them outright and have opted for aggressive deprecation instead.
#10
@
5 years ago
- Keywords commit needs-dev-note added
Thanks @jrf! I tend to agree with you on not straight up removing the classes. If someone is able to do a scan of the plugin directory and there are no plugins using these classes, I think we could reconsider.
42364.2.diff just adds the @deprecated
tag to the function docblocks in addition to the _deprecated_file()
and _deprecated_function()
calls.
These changes should also get a brief dev note, just in case any plugins or themes are using them.
Since this is marked early
, I'm going to commit this so it can have some soak time.
#11
@
5 years ago
- Owner set to desrosj
- Resolution set to fixed
- Status changed from new to closed
In 46637:
In 42034: