WordPress.org

Make WordPress Core

Opened 23 months ago

Last modified 18 months ago

#37586 new defect (bug)

Menu customizer: search results not properly filtered

Reported by: virgodesign Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 4.3
Component: Customize Keywords: has-patch dev-feedback
Focuses: Cc:

Description

customize_nav_menu_available_item_types is a filter located in class-wp-customize-nav-menus.php.

The purpouse of this filter is to restrict item types available in the Menu Customizer.

This filter should be applied even if I perform a research using the search field in Menu Customizer. But this does not happens.

While in Menu Customizer, doing a research, in search results shows up even items of specific types excluded with customize_nav_menu_available_item_types filter.

We can resolve this issue using another filter always located in the same class: the customize_nav_menu_searched_items filter. With this filter we can restrict the selections of items received from the search result just before sending them to the frontend (ajax response).

But this could be considered only a workaround and not a solution, because items should be filtered/excluded by type BEFORE wordpress performs the query to the database.

We should suppose that, if we uses the customize_nav_menu_available_item_types to filter item types available on Menu Customizer, most likely we do not want search for elements of theese item types.

Attachments (1)

#37586.1.diff (1.6 KB) - added by virgodesign 20 months ago.

Download all attachments as: .zip

Change History (6)

#1 @virgodesign
23 months ago

  • Summary changed from customize_nav_menu_available_item_types should filter even search results to Menu customizer: search results not properly filtered

#2 @celloexpressions
23 months ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to Future Release
  • Version changed from 4.5.3 to 4.3

Good catch. Can you create a patch @virgodesign?

#3 @virgodesign
23 months ago

I've created the snippet code, but currently I'm not able to create a patch.

#4 @virgodesign
20 months ago

  • Keywords has-patch added; needs-patch removed

#5 @virgodesign
18 months ago

  • Keywords dev-feedback added
Note: See TracTickets for help on using tickets.