Ошибка «Попытка получить свойство необъекта» с Custom Walker для wp_nav_menu

8

Я пытаюсь добавить некоторые связанные с ARIA вещи в функцию wp_nav_menu. Я использую пользовательский класс ходунков для этой цели:

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 ) . '"' : '';

                // Add attributes for <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        ) .'"' : '';

                // Add attributes for <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, обработанная как объект, действительно является массивом. Но действительно странная вещь в том, что эта ошибка сохраняется, даже если я передаю исходный ходок так:

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()проходит.
Rarst
Нет, если я просто использую wp_nav_menu()функцию по умолчанию без ходунка, она работает отлично. Да, но почему тогда это массив? Я даже var_dumpредактировал это, нет объекта.
Рудольф
Я не уверен, что происходит тогда, потому что проходящий new Walker_Nav_Menuдолжен вести себя так же, как не проходящий мимо. Что-нибудь еще происходит, какие-нибудь плагины, подключающиеся к / управляющие функциональностью меню?
Первое
Я только что провел дополнительное тестирование, и я уверен, что это ошибка WordPress. В моем случае навигационное меню, которое я использовал, не было назначено в бэкэнде. Поэтому был использован запасной вариант списка страниц, похоже, что тогда переменная args является массивом. Сейчас я прослежу это до источника, а затем расскажу, что нашел.
Рудольф
Если я правильно назначил меню, оно работает!
Рудольф

Ответы:

18

Я получаю эту ошибку, когда не определены меню или меню не настроены для местоположения в Appearance->Menus. Когда это происходит, wp_nav_menuиспользуется обходчик страниц.

  1. Запасной вариант ( по умолчанию) дляwp_nav_menu ISwp_walker_page
  2. который использует wp_page_menu
  3. который использует wp_list_pages
  4. который использует walk_page_tree
  5. который использует Walker_Pageнет Walker_Nav_Menu.

И, видимо, два ходока не совместимы. Я не знаю, почему это не терпит неудачу изящно. Это похоже на ошибку для меня.

При установленном меню wp-admin->Appearance->Menusваш код работает.

Вы можете избежать ошибки, проверив, что перед тем, как использовать местоположение, есть меню, назначенное местоположению.

$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,
  ) );
}
s_ha_dum
источник
Резервный режим используется только тогда, когда все остальное не отображает страницы. Если меню работает правильно, то идут пункты, walk_nav_menu_tree()которые используют Walker_Nav_Menuпо умолчанию.
Первое
Хорошее обновление! Мне не пришло в голову, что неправильная функция может быть захватом ходока (это ме). Также есть has_nav_menu()для этой проверки.
Первое
Это именно то, что я просто хотел опубликовать: D Меню навигации $argsпреобразуются в объект в wp-includes/nav-menu-template.phpстроке 145 с помощью wp_nav_menu()функции. wp_page_menu()Функция , которая используется в качестве запасного варианта не делает этого, но вместо этого просто передает массив в Walker из меню нав , который нуждается в объекте. Если вы просто добавите $args = (object) $argsв начало start_elфункции класса Walker, вы можете это исправить. Где будет правильное место, чтобы сообщить об этом команде WordPress?
Рудольф
Об основных ошибках @Rudolf следует сообщать на core.trac.wordpress.org
Rarst
WP_PostОбъект , который передается как $itemотличается, тоже (Из - за этого, страницы не имеют имен и список как раз и состоит из пустых <li>элементов). Некоторый серьезный рефакторинг был бы необходим, чтобы эти два ходунка были совместимы.
Рудольф
0

используйте следующий код для ремонта:

add_filter( 'wp_get_nav_menu_object', 'override_wp_get_nav_menu_object', 10, 2 );
function override_wp_get_nav_menu_object( $menu_obj, $menu ) {

    if ( ! is_object( $menu_obj ) ) {
        $menu_obj = (object) array( 'name' => '' );
    }

    return $menu_obj;
}
hannanstd
источник