Wp-rocket: Уведомления PHP при обновлении

Созданный на 17 мар. 2020 · 5Комментарии · Источник: wp-media/wp-rocket

Опишите ошибку
В процессе обновления в журналах появилось 2 уведомления PHP:

[17-Mar-2020 13:37:45 UTC] PHP Notice:  Undefined variable: home_root_escaped in /****/wp-content/plugins/wp-rocket/inc/functions/options.php on line 252
[17-Mar-2020 13:37:45 UTC] PHP Stack trace:
[17-Mar-2020 13:37:45 UTC] PHP   1. {main}() /****/wp-admin/admin-ajax.php:0
[17-Mar-2020 13:37:45 UTC] PHP   2. do_action() /****/wp-admin/admin-ajax.php:45
[17-Mar-2020 13:37:45 UTC] PHP   3. WP_Hook->do_action() /****/wp-includes/plugin.php:478
[17-Mar-2020 13:37:45 UTC] PHP   4. WP_Hook->apply_filters() /****/wp-includes/class-wp-hook.php:312
[17-Mar-2020 13:37:45 UTC] PHP   5. rocket_upgrader() /****/wp-includes/class-wp-hook.php:288
[17-Mar-2020 13:37:45 UTC] PHP   6. flush_rocket_htaccess() /****/wp-content/plugins/wp-rocket/inc/admin/upgrader.php:27
[17-Mar-2020 13:37:45 UTC] PHP   7. get_rocket_htaccess_marker() /****/wp-content/plugins/wp-rocket/inc/functions/htaccess.php:56
[17-Mar-2020 13:37:45 UTC] PHP   8. get_rocket_htaccess_mod_rewrite() /****/wp-content/plugins/wp-rocket/inc/functions/htaccess.php:148
[17-Mar-2020 13:37:45 UTC] PHP   9. get_rocket_cache_reject_uri() /****/wp-content/plugins/wp-rocket/inc/functions/htaccess.php:271
[17-Mar-2020 13:37:45 UTC] PHP Notice:  Undefined variable: home_root_len in /****/wp-content/plugins/wp-rocket/inc/functions/options.php on line 254
[17-Mar-2020 13:37:45 UTC] PHP Stack trace:
[17-Mar-2020 13:37:45 UTC] PHP   1. {main}() /****/wp-admin/admin-ajax.php:0
[17-Mar-2020 13:37:45 UTC] PHP   2. do_action() /****/wp-admin/admin-ajax.php:45
[17-Mar-2020 13:37:45 UTC] PHP   3. WP_Hook->do_action() /****/wp-includes/plugin.php:478
[17-Mar-2020 13:37:45 UTC] PHP   4. WP_Hook->apply_filters() /****/wp-includes/class-wp-hook.php:312
[17-Mar-2020 13:37:45 UTC] PHP   5. rocket_upgrader() /****/wp-includes/class-wp-hook.php:288
[17-Mar-2020 13:37:45 UTC] PHP   6. flush_rocket_htaccess() /****/wp-content/plugins/wp-rocket/inc/admin/upgrader.php:27
[17-Mar-2020 13:37:45 UTC] PHP   7. get_rocket_htaccess_marker() /****/wp-content/plugins/wp-rocket/inc/functions/htaccess.php:56
[17-Mar-2020 13:37:45 UTC] PHP   8. get_rocket_htaccess_mod_rewrite() /****/wp-content/plugins/wp-rocket/inc/functions/htaccess.php:148
[17-Mar-2020 13:37:45 UTC] PHP   9. get_rocket_cache_reject_uri() /****/wp-content/plugins/wp-rocket/inc/functions/htaccess.php:271

Воспроизводить

Установите старую версию WP Rocket
Очистить файл журнала
Обновите плагин

Ожидаемое поведение
В файле журнала нет новых строк.

Дополнительный контекст

Я обновлял WPR с 3.4.4 до 3.5.0.3
WordPress 5.3.2

Обработка бэклога

[x] Воспроизвести проблему
[x] Определите основную причину
[x] Найдите решение
[x] Оцените усилия

[S] low bug

Источник

Screenfeed

Все 5 Комментарий

Определите основную причину ✅
inc/functions/options.php - get_rocket_cache_reject_uri() функция
Основная причина - объявление

        $home_root_escaped = preg_quote( $home_root, '/' );
        $home_root_len     = strlen( $home_root );

что выполняется в условии if:
if ( '' !== $home_root && $uris ) {
Эти переменные используются ниже и, похоже, не определены:
https://github.com/wp-media/wp-rocket/blob/8b2a567efcab3925dba6fff72c5a622ec2d16386/inc/functions/options.php#L198 -L199

Найдите решение ✅
Решением будет переместить объявление этих двух варов глобально в этой функции.

Оцените усилия ✅
Усилия]
Усилия пойдут на модульные и интеграционные тесты.

crystinutzaa 23 мар. 2020

Я только что заметил проблему с mega/Divi и могу воспроизвести ее, включив предварительную загрузку.

@GeekPress @hellofromtonya Должны ли мы повысить приоритет этого вопроса? Это тривиально, но поскольку мы знаем об этом и поскольку это легко исправить, я считаю, что мы не должны оставлять это в нашем коде.

arunbasillal 20 мая 2020

👍1

@arunbasillal Приоритет устанавливается в зависимости от критичности проблемы. Это не критично и будет «низким», потому что:

ошибки отображаются только в журналах
это не нарушает ни одной из наших функций
количество обращений по этой проблеме очень мало

GeekPress 20 мая 2020

Связанный билет: https://secure.helpscout.net/conversation/1171062698/166759?folderId=273764#thread -3340520759

juricazuanovic 27 мая 2020

👍1

Связанный билет:
https://secure.helpscout.net/conversation/1238067915/183094/

Эта ошибка регистрируется при включении нашего плагина:
[01-Aug-2020 11:54:49 UTC] PHP Notice Undefined variable: home_root_len in /var/www/vhosts/managed.hosting.com/managed.hosting.com/site/wp-content/plugins/wp-rocket/inc/functions/options.php on line 261