Как исправить ошибку All untrusted data should be escaped before output?

Question

[AndrewTree]

Добрый день.

Подал заявку на добавление шаблона для WordPress в каталог https://wordpress.org/themes/ , модератор проверил и нашёл ошибки
https://themes.trac.wordpress.org/ticket/23405#com...
All untrusted data should be escaped before output.
header.php, line 40
functions.php, line 94

Уже вечность ломаю голову и не могу понять в чём именно там ошибки и как их исправить? Шаблон работает, если включить дебагер, то ошибки не появляются, как он увидел что там ошибки?

header.php, line 40 - Благодаря Александру Зачиналову этот вопрос решён.

<div class="sidebar-head3 span2">
      <?php dynamic_sidebar( 'sidebar-head' ); ?>
</div>

functions.php, line 94

<a class="cart-contents" href="<?php echo $woocommerce->cart->get_cart_url(); ?>" title="<?php _e('View your shopping cart', 'WShop'); ?>"><?php echo sprintf(_n('%d item', '%d items', $woocommerce->cart->cart_contents_count, 'WShop'), $woocommerce->cart->cart_contents_count);?> <?php echo $woocommerce->cart->get_cart_total(); ?></a>

Весь код functions.php, line 89 - 98

add_filter('add_to_cart_fragments', 'WShop_fragment');
function WShop_fragment( $fragments ) 
{
    global $woocommerce;
    ob_start(); ?>
    <a class="cart-contents" href="<?php echo $woocommerce->cart->get_cart_url(); ?>" title="<?php _e('View your shopping cart', 'WShop'); ?>"><?php echo sprintf(_n('%d item', '%d items', $woocommerce->cart->cart_contents_count, 'WShop'), $woocommerce->cart->cart_contents_count);?> <?php echo $woocommerce->cart->get_cart_total(); ?></a>
    <?php
    $fragments['a.cart-contents'] = ob_get_clean();
    return $fragments;
}

Ссылка на архив с шаблоном https://wordpress.org/themes/download/wshop.1.1.zip

Answer 1

[Александр Зачиналов]

Они ругаются, потому что вы не проверяете эти данные перед использованием.
Пример:

<?php if ( is_active_sidebar( 'sidebar-head' ) ) : ?>
  <div class="sidebar-head3 span2">
      <?php dynamic_sidebar( 'sidebar-head' ); ?>
  </div>
<?php endif; ?>

Comments

[AndrewTree]

Спасибо! Сейчас исправлю.
Но ведь сайдбары работают - в них могу добавить из админки виджеты и они отобразятся на сайте или их нужно по другому проверять?

[Александр Зачиналов]

Ну это такой порядок, работать все будет, потому что это у вас есть и активно. А если нет? что тогда произойдет? Проверка доступности данных у многих считается обязательным правилом, вот они и требуют от вас их соблюдения. По второму куску пока не могу сказать без более подробного кода, возможно вы фильтры не объявили в начале функции

[Александр Зачиналов]

Вы перепроверяйте решения у популярных шаблонов WP там все ответы на поверхности

[AndrewTree]

Александр Зачиналов: ок. Буду более внимательным. Просто я уже добавил 8 шаблонов в каталог и раньше мне не говорили что там ошибки из-за этого и был сбит с толку.

"По второму куску пока не могу сказать без более подробного кода, возможно вы фильтры не объявили в начале функции"
Обновил код в вопросе.

[Александр Зачиналов]

Да блин)) правильно вроде все

[AndrewTree]

Александр Зачиналов: как мне на английском эту фразу ему процитировать)))))

[AndrewTree]

Александр Зачиналов: ?> в конце файла есть https://themes.svn.wordpress.org/wshop/1.1/functio...

[AndrewTree]

AndrewTree: Сейчас обновлю сайдбары и отправлю ему со словами что исправил все ошибки кроме этой так как не знаю в чём она заключается и он может более подробно объяснит.

[Александр Зачиналов]

Yarr, pancake! I'm real WP-developer! You must revalidate my TPL now

[Александр Зачиналов]

Ну если раньше было хорошо, а теперь стали отказывать, то не удивлюсь если они сменили правила валидации совсем недавно. Поэтому в данный момент кроме них на этот вопрос никто не ответит.... может также нужно проверять доступность модели, а может заполненность $fraimes, а может вам менеджер попался злой)))

[AndrewTree]

Александр Зачиналов: Вот и я не знаю какой вариант тут подходит. Пару недель назад вышла новая версия WordPress 4.2 из-за обновлений пришлось на всех старых шаблонах менять код вывода сайдбаров, но а других изменений я не заметил, хотя я только начинаю изучать вёрстку под WP.