@tuempelkoenig, @sahilseta, @lenny1975 – Got it. Ok so those with the Zephyr/Imprezza, the theme itself is not the issue, but the UpSolution Core (us-core) plugin that must be installed with it.
Specifically they are doing the one scenario I didn’t outline before, they are overloading the WP core nav menu renderer (“Walker”) and not outputting the required call to do_action( 'wp_nav_menu_item_custom_fields', $item_id, $item, $depth, $args );
that core and the walker we use does. This is where we render our fields, and so do other plugins like Popup Maker, If Menus, Nav Menu Roles etc. So currently none of these plugins would render fields either.
Commenting out the following line that registers their custom admin nav menu walker fixes it: wp-content/plugins/us-core/admin/functions/nav-menu-edit.php:21
This is what they need to do long term: see Solution #2.
For examples of how we and many other plugins that are all compatible together do it see below.
Potential temporary workaround would be to unhook their walker with the following in your functions.php
when you need to edit User Menu settings, then comment it out when not needing to edit it:
remove_filter( 'wp_edit_nav_menu_walker', 'us_edit_nav_menu_walker' );
Not ideal but it works
Examples of compatible methods of adding fields
User Menus
– https://github.com/JunglePlugins/User-Menus/blob/develop/includes/classes/admin/menu-editor.php#L22-L24
– https://github.com/JunglePlugins/User-Menus/blob/develop/includes/classes/admin/menu-editor.php#L33-L53
– https://github.com/JunglePlugins/User-Menus/blob/develop/includes/classes/walker/nav-menu-edit-custom-fields.php
Popup Maker (500k+ sites)
– https://github.com/PopupMaker/Popup-Maker/blob/master/includes/modules/menus.php
– https://github.com/PopupMaker/Popup-Maker/blob/master/includes/modules/menus/class-nav-menu-edit-custom-fields.php.
—
For everyone else, so far it has 100% been theme/theme related plugins causing this for similar reasons to those above. Seems many devs never realized that they were doing things in an incompatible way or missed the WP core release notes.
-
This reply was modified 4 years, 6 months ago by Daniel Iser. Reason: Added temporary workaround
-
This reply was modified 4 years, 6 months ago by Daniel Iser. Reason: Added temporary workaround