• Resolved Vova Druzhaev

    (@otshelnik-fm)


    Привет.

    1. При удалении плагина – “деактивировать” -> “удалить” – не происходит удаление опции из wp_options “wptrt_notice_dismissed_mihdan-mailru-pulse-feed” – я так понимаю эта опция была из-за вордпресс нотиса что я закрыл: “Поставьте 5 звёзд плагину”.

    2. Ставить эту опцию без флага “autoload” true. Она нужна не везде – а только в админке (еще лучше чтоб только на странице настроек плагина, а не по всей админке мозолила глаз). А так получается что мы ее тянем и в фронтенде.

    К таким вещам надо быть внимательней.

    3. Вот скриншот https://yadi.sk/i/geHU5KmwruEU6A – это последние опции в БД. Я так понимаю они тоже от плагина остались после его удаления?
    Смотри – сервис смок теста https://plugintests.com/plugins/mihdan-mailru-pulse-feed/latest долистай до секции “Options wp_options” (4)
    – я так понимаю и эти 4-ре опции плагин добавляет? им бы ваш префикс добавить – чтоб было понятно что это от вас и снова к autoload – пересмотрите использование с флагом true – не нужен там autoload в настройки. Стоит получать их по необходимости. А то все это на фронтенд части где это не нужно получается.

    Спасибо.

Viewing 12 replies - 1 through 12 (of 12 total)
  • Plugin Author mihdan

    (@mihdan)

    Владимир, спасибо за замечания, прям в следующей версии учтём все пожелания и напишем авторам зависимых проектов про autoload.

    Thread Starter Vova Druzhaev

    (@otshelnik-fm)

    Не используете апи настроек вордпресс? А что стороннее используете для этого?

    блин это тогда сколько их использует решений – что вот так просто загоняют в autoload. А не все настройки везде нужны. Таких плагинов 5-ть поставишь и будешь гонять половину опций просто так – а это тоже время бекенда.

    Plugin Author mihdan

    (@mihdan)

    Используем обертку над Settings API от Ахмада https://github.com/ahmadawais/WP-OOP-Settings-API.

    Это не то место, где проседает производительность.

    Thread Starter Vova Druzhaev

    (@otshelnik-fm)

    Понял кто это. Ну иногда и индусы ошибаются.

    Получать и гонять ненужные данные – так апи вордпресс не предлагает. У них не спроста есть этот флаг и это влияет на производительность. Не в твоем случае с 5-ю полями. yoast массив глянь и сколько полей.

    Plugin Author mihdan

    (@mihdan)

    Тяжелее ёста только вука и wpml

    Plugin Author mihdan

    (@mihdan)

    @otshelnik-fm, PR для ребят из WPTRT готов.

    Plugin Author mihdan

    (@mihdan)

    Ошибки исправлены в 0.2.2

    Thread Starter Vova Druzhaev

    (@otshelnik-fm)

    Спасибо. А что по автолоаду для опций не из нотиса? Они исправлены?

    Обновления пока не вижу (вернул топик в not resolved) – у меня тут по пунктам были замечания.

    Что обновление решает?
    Тут я вижу твое решение + 2 зависимости – это придется ждать пока они PR все пройдут (ага именно из-за этой волокиты с чужими решениями не люблю завязываться)

    Plugin Author mihdan

    (@mihdan)

    Все автолоад исправлены, pr отправлены кому надо.

    Обновление выйдет позже сегодня.

    Опенсорс он такой да, но писать все руками в 2020 году, когда все уже написано, даже не странно, а слова не подберу. PR там быстро принимают. Тем более, что уведомления эти – рекомендация команды ревью.

    Plugin Author mihdan

    (@mihdan)

    Вышла версия 0.2.2 с исправлениями. Нашел пару багов в зависимостях пока делал, отправил им PR, поправив руками в самом плагине.

    Запись о том, что окно закрыто с просьбой проголосовать теперь лежит не в настройках сайта, а мете отдельного юзера-админа.

    Все autoload пофиксил, сделал заготовку под миграции, чтобы переименовать в следующей версии настройки в wp_options.

    Thread Starter Vova Druzhaev

    (@otshelnik-fm)

    Спасибо Миш. Теперь все норм. Ща рецензию отправлю

    Plugin Author mihdan

    (@mihdan)

    Спасибо, Владимир, за помощь. Будут вопросы – пиши, поправив, доработаем.

Viewing 12 replies - 1 through 12 (of 12 total)
  • The topic ‘При удалении плагина – удалять за собой’ is closed to new replies.