Ошибка "Trying to get property of non-object" при использовании кастомного Walker для wp_nav_menu
Я пытаюсь добавить ARIA-атрибуты к функции wp_nav_menu. Для этого использую кастомный класс Walker:
class Walker_Nav_Menu_With_Aria extends Walker_Nav_Menu {
function start_lvl( &$output, $depth = 0, $args = array() ) {
$indent = str_repeat("\t", $depth);
$output .= "\n$indent<ul class=\"sub-menu\" role=\"group\">\n";
}
function end_lvl( &$output, $depth = 0, $args = array() ) {
$indent = str_repeat("\t", $depth);
$output .= "$indent</ul>\n";
}
function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0 ) {
$indent = ( $depth ) ? str_repeat( "\t", $depth ) : '';
$class_names = $value = '';
$classes = empty( $item->classes ) ? array() : (array) $item->classes;
$classes[] = 'menu-item-' . $item->ID;
$class_names = join( ' ', apply_filters( 'nav_menu_css_class', array_filter( $classes ), $item, $args ) );
$class_names = $class_names ? ' class="' . esc_attr( $class_names ) . '"' : '';
$id = apply_filters( 'nav_menu_item_id', 'menu-item-'. $item->ID, $item, $args );
$id = $id ? ' id="' . esc_attr( $id ) . '"' : '';
// Добавляем атрибуты для <li>
$li_attributes = ' role="treeitem"';
$li_attributes .= ' aria-expanded="false"';
$output .= $indent . '<li' . $id . $value . $class_names . $li_attributes .'>';
$attributes = ! empty( $item->attr_title ) ? ' title="' . esc_attr( $item->attr_title ) .'"' : '';
$attributes .= ! empty( $item->target ) ? ' target="' . esc_attr( $item->target ) .'"' : '';
$attributes .= ! empty( $item->xfn ) ? ' rel="' . esc_attr( $item->xfn ) .'"' : '';
$attributes .= ! empty( $item->url ) ? ' href="' . esc_attr( $item->url ) .'"' : '';
// Добавляем атрибуты для <a>
$attributes .= $depth == 0 ? ' tabindex="0"' : ' tabindex="-1"';
$item_output = $args->before;
$item_output .= '<a'. $attributes .'>';
$item_output .= $args->link_before . apply_filters( 'the_title', $item->title, $item->ID ) . $args->link_after;
$item_output .= '</a>';
$item_output .= $args->after;
$output .= apply_filters( 'walker_nav_menu_start_el', $item_output, $item, $depth, $args );
}
function end_el( &$output, $item, $depth = 0, $args = array() ) {
$output .= "</li>\n";
}
}
Я просто скопировал оригинальный класс Walker_Nav_Menu и добавил свои изменения, но получаю ошибки, потому что переменная $args, которая должна быть объектом, на самом деле является массивом. Самое странное, что эта ошибка возникает даже если использовать оригинальный Walker вот так:
wp_nav_menu( array(
'theme_location' => 'main-nav',
'walker' => new Walker_Nav_Menu,
) );
Вот какие сообщения об ошибках я получаю:
NOTICE: TRYING TO GET PROPERTY OF NON-OBJECT IN /USERS/RUDOLF/SITES/LOCALHOST/WP/WP-INCLUDES/NAV-MENU-TEMPLATE.PHP ON LINE 88
NOTICE: TRYING TO GET PROPERTY OF NON-OBJECT IN /USERS/RUDOLF/SITES/LOCALHOST/WP/WP-INCLUDES/NAV-MENU-TEMPLATE.PHP ON LINE 90
NOTICE: TRYING TO GET PROPERTY OF NON-OBJECT IN /USERS/RUDOLF/SITES/LOCALHOST/WP/WP-INCLUDES/NAV-MENU-TEMPLATE.PHP ON LINE 90
NOTICE: TRYING TO GET PROPERTY OF NON-OBJECT IN /USERS/RUDOLF/SITES/LOCALHOST/WP/WP-INCLUDES/NAV-MENU-TEMPLATE.PHP ON LINE 92
Эти строки выглядят так (в оригинальном файле!):
(88) $item_output = $args->before;
(89) $item_output .= '<a'. $attributes .'>';
(90) $item_output .= $args->link_before . apply_filters( 'the_title', $item->title, $item->ID ) . $args->link_after;
(91) $item_output .= '</a>';
(92) $item_output .= $args->after;
Я получаю эту ошибку, когда не определены меню или не заданы меню для расположения в разделе Внешний вид->Меню
. В этом случае wp_nav_menu
использует резервный вариант с постраничным навигатором (page walker).
- Резервный вариант (по умолчанию) для
wp_nav_menu
— этоwp_walker_page
- который использует
wp_page_menu
- который использует
wp_list_pages
- который использует
walk_page_tree
- который использует
Walker_Page
, а неWalker_Nav_Menu
.
И, по-видимому, эти два навигатора несовместимы. Непонятно, почему это не обрабатывается корректно. Мне кажется, это баг.
Если меню задано в wp-admin->Внешний вид->Меню
, ваш код работает.
Вы можете избежать ошибки, проверив, назначено ли меню для данного расположения, перед попыткой его использования.
$locations = get_nav_menu_locations();
if (0 !== $locations['main-nav']) {
wp_nav_menu( array(
'theme_location' => 'main-nav',
'walker' => new Walker_Nav_Menu_With_Aria,
) );
}
Или, если предпочитаете менее загруженный код (спасибо @Rarst):
if (has_nav_menu('primary')) {
wp_nav_menu( array(
'theme_location' => 'primary',
'walker' => new Walker_Nav_Menu_With_Aria,
) );
}

Fallback используется только в случае, когда все остальные способы отображения страниц не сработали. Если меню работает корректно, элементы обрабатываются с помощью walk_nav_menu_tree()
, которая по умолчанию использует Walker_Nav_Menu
.

Хорошее обновление! Я не подумал, что неправильная функция может захватывать Walker (это не очень хорошо). Также для этой проверки есть has_nav_menu()
.

Это именно то, что я хотел написать :D Аргументы навигационного меню $args
преобразуются в объект в файле wp-includes/nav-menu-template.php
на строке 145 функцией wp_nav_menu()
. Функция wp_page_menu()
, которая используется как fallback, этого не делает, а просто передает массив в Walker из навигационного меню, которому нужен объект.
Если просто добавить $args = (object) $args
в начало функции start_el
класса Walker, это можно исправить. Где лучше всего сообщить об этом команде WordPress?

@Rudolf ошибки ядра следует сообщать на http://core.trac.wordpress.org/

используйте приведённый ниже код для исправления:
add_filter( 'wp_get_nav_menu_object', 'override_wp_get_nav_menu_object', 10, 2 );
/**
* Переопределяет объект меню, если он не является объектом
*
* @param object $menu_obj Объект меню
* @param mixed $menu Идентификатор или слаг меню
* @return object Возвращает объект меню
*/
function override_wp_get_nav_menu_object( $menu_obj, $menu ) {
if ( ! is_object( $menu_obj ) {
$menu_obj = (object) array( 'name' => '' );
}
return $menu_obj;
}
