Ошибка "Trying to get property of non-object" при использовании кастомного Walker для wp_nav_menu

16 июн. 2013 г., 16:07:29
Просмотры: 18.5K
Голосов: 10

Я пытаюсь добавить 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;
8
Комментарии

Вы получаете ошибку, если вообще не передавать walker? Аргументы должны быть объектом, я думаю, что именно это указано во встроенной документации и что передаёт wp_nav_menu().

Rarst Rarst
16 июн. 2013 г. 18:02:38

Нет, если я просто использую стандартную функцию wp_nav_menu() без walker, всё работает идеально. Да, но почему тогда это массив? Я даже сделал var_dump, никакого объекта.

Rudolf Rudolf
16 июн. 2013 г. 18:26:16

Я не уверен, что происходит, потому что передача new Walker_Nav_Menu должна вести себя идентично отсутствию walker. Есть ли что-то ещё, какие-то плагины, которые вмешиваются в функциональность меню или модифицируют её?

Rarst Rarst
16 июн. 2013 г. 18:27:39

Я только что провел дополнительные тесты и уверен, что это баг WordPress. В моем случае навигационное меню не было назначено в админке. Поэтому использовался fallback в виде списка страниц, и похоже, что в этом случае переменная args является массивом. Сейчас я попробую найти корень проблемы и расскажу вам, что обнаружу.

Rudolf Rudolf
16 июн. 2013 г. 18:40:24

Если правильно назначить меню, то все работает!

Rudolf Rudolf
16 июн. 2013 г. 18:40:42

Сначала я не увидел ответ ниже, смотрите там решение.

Rudolf Rudolf
16 июн. 2013 г. 18:59:11

Я попробовал if(wp_nav_menu()), и код работает.

User User
1 сент. 2015 г. 09:59:09

Ты понимаешь, что этому вопросу уже два года? :D Но я создал тикет для него здесь: https://core.trac.wordpress.org/ticket/24587 Проблема всё ещё не исправлена, так что не стесняйся обсудить её там!

Rudolf Rudolf
1 сент. 2015 г. 17:15:05
Показать остальные 3 комментариев
Все ответы на вопрос 2
5
21

Я получаю эту ошибку, когда не определены меню или не заданы меню для расположения в разделе Внешний вид->Меню. В этом случае wp_nav_menu использует резервный вариант с постраничным навигатором (page walker).

  1. Резервный вариант (по умолчанию) для wp_nav_menu — это wp_walker_page
  2. который использует wp_page_menu
  3. который использует wp_list_pages
  4. который использует walk_page_tree
  5. который использует 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,
  ) );
}
16 июн. 2013 г. 17:05:02
Комментарии

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

Rarst Rarst
16 июн. 2013 г. 17:57:40

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

Rarst Rarst
16 июн. 2013 г. 18:34:35

Это именно то, что я хотел написать :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 Rudolf
16 июн. 2013 г. 18:57:34

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

Rarst Rarst
16 июн. 2013 г. 19:00:40

Объект WP_Post, передаваемый как $item, также отличается (Из-за этого страницы не имеют названий и список состоит просто из пустых элементов <li>). Для совместимости этих двух обработчиков потребуется серьезный рефакторинг.

Rudolf Rudolf
16 июн. 2013 г. 19:04:43
0

используйте приведённый ниже код для исправления:

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;
}
21 авг. 2017 г. 03:37:06