Поддержка WooCommerce В логах нотис

  • В логах вот такой нотис

    Функция name вызвана неправильно. Свойства товара не должны быть доступны непосредственно. Backtrace: do_action_ref_array, WP_Hook->do_action, WP_Hook->apply_filters, YmlforYandexMarket->yfym_do_this_seventy_sec, YmlforYandexMarket::yfym_construct_yml, yfym_onlygluing, yfym_gluing, yfym_unit, apply_filters(‘yfym_variable_change_name’), WP_Hook->apply_filters, YmlforYandexMarketPro->yfymp_variable_change_name_func, WC_Abstract_Legacy_Product->__get, wc_doing_it_wrong

    Ругается на строку (если комментирую, ошибка уходит)
    $result_yml_name = $product->name;
    при этом в $result_yml_name попадает как раз то, что нужно, т.е. работает как надо, но выводит нотис.

    Не пойму, что блин ему не нравится. И что мне сделать. На что поменять $product->name?

Просмотр 5 ответов — с 1 по 5 (всего 5)
  • Обратитесь в техподдержку плагина Yml for Yandex Market Pro (если я не ошибаюсь с виновником)

    Модератор Yui

    (@fierevere)

    永子

    kw56, это и есть автор плагина 🙂

    Модератор Yuri

    (@yube)

    Не пойму, что блин ему не нравится.

    То, что написано:

    Свойства товара не должны быть доступны непосредственно.

    На что поменять $product->name?

    $product->get_name() наверно

    WC нынче требует, чтобы свойства объекта вынимались исключительно через его методы.

    Да) Это мой плагин. Потому и парюсь.
    Я сначала думал заменить на
    $result_yml_name = get_the_title($postID);
    и проблема как бы решена т.к в 90% случаев значения будут совпадать. НО! Если у юзера какие-то фильтры для названий товаров от других плагинов установлены, нужные ему. Тогда это не поможет.
    Т.е нужно использовать именно $product;

    Можно, конечно, и так оставить ибо работает же. Но все же речь о PRO версии. Да и вообще. Не гоже нотисы в плагине оставлять. Должно все как часы работать.

    Мне не совсем понятна сама природа нотиса. Линк, который в логах вукомерц дает — ведет на пустой раздел справки.

    И главное. Работает же.

    Юрий! Спасибо! Теперь нотис ушел!

Просмотр 5 ответов — с 1 по 5 (всего 5)
  • Тема «В логах нотис» закрыта для новых ответов.