Fix for duplicate active menu items. #986
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There's a duplication of active menu items when using custom post types. Namely that custom posts nested beneath a page cause the blog to become
.active
while simultaneously setting the parent page.active
and the current menu item.active
. This patch fixes all that nonsense. If youprint_r($classes); print_r($new_classes)
on line 60 you'll see that there are always at least 2 active replaces prior to thearray_unique($new_classes)
. Thearray_search('current-menu-item',...
conditional check is to ensure that.active
is still set for current menu item regardless of the number of$new_class_counts
.