WordPress.org

Make WordPress Core

Opened 3 years ago

Last modified 12 days ago

#37586 new defect (bug)

Menu customizer: search results not properly filtered

Reported by: virgodesign Owned by:
Milestone: 5.3 Priority: normal
Severity: normal Version: 4.3
Component: Customize Keywords: has-patch dev-feedback needs-testing
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 3 years ago.

Download all attachments as: .zip

Change History (7)

#1 @virgodesign
3 years 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
3 years 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
3 years ago

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

@virgodesign
3 years ago

#4 @virgodesign
3 years ago

  • Keywords has-patch added; needs-patch removed

#5 @virgodesign
3 years ago

  • Keywords dev-feedback added

#8 @celloexpressions
2 months ago

  • Keywords needs-testing added
  • Milestone set to 5.3

#37586.1.diff needs to be reviewed by an active developer and tested (by anyone currently able to test patches). Milestoning as this is a valid bug that has waited long enough for attention and should be resolvable with a minimal effort by an active committer.

Note: See TracTickets for help on using tickets.