Новости Joomla

Вышли релизы безопасности Joomla 6.1.1 и Joomla 5.4.6

Релиз безопасности Joomla 6.1.1 и Joomla 5.4.6

Проект Joomla! рад сообщить о выходе Joomla 6.1.1 и Joomla 5.4.6. Это релизы безопасности и исправления ошибок для серий 5.x и 6.x.

0 Пользователей и 1 Гость просматривают эту тему.
  • 2 Ответов
  • 2552 Просмотров
*

novikov82

  • Новичок
  • 9
  • 3 / 0
В контроллере \administrator\components\com_jshopping\controllers\orders.php в функции edit() есть такой кусочек:
Код
        
JFilterOutput::objectHTMLSafe($order);
        foreach($order_items as $k=>$v){
            JFilterOutput::objectHTMLSafe($order_items[$k]);
        }  

В принципе, все правильно, это экранирование спецсимволов HTML. Вот только обнаружил я, что данный код не экранирует спецсимволы в фамилии, имени, отчестве и во всех полях адреса доставки (некоторые клиенты впихивают ООО "Ромашка" в поле "Имя").
Разбираюсь:
\libraries\joomla\filter\output.php
Код
	public static function objectHTMLSafe(&$mixed, $quote_style = ENT_QUOTES, $exclude_keys = '')
{
if (is_object($mixed))
{
foreach (get_object_vars($mixed) as $k => $v)
{
if (is_array($v) || is_object($v) || $v == null || substr($k, 1, 1) == '_')
{
continue;
}

if (is_string($exclude_keys) && $k == $exclude_keys)
{
continue;
}
elseif (is_array($exclude_keys) && in_array($k, $exclude_keys))
{
continue;
}

$mixed->$k = htmlspecialchars($v, $quote_style, 'UTF-8');
}
}
}
Получается, что фильтр игнорирует все поля объекта, первый символ имен которых равен подчеркиванию. Описание функции substr из Google (я не пхп-программист, поэтому приходится подглядывать), говорит, что символы для этой функции нумеруются с нуля. Следовательно, фильтр отметает "l_name", "f_name", "d_city", "d_street" и многие другие поля!
А теперь вопрос: кто недосмотрел, где ошибка? В Джумшоппинге? В Joomla? Или косяк с кодировками какой? Или еще что-то?
« Последнее редактирование: 21.08.2013, 17:43:23 от novikov82 »
*

dmitry_stas

  • Легенда
  • 13151
  • 1234 / 8
это ж надо... все так и есть. надо писать в ошибки магазина
Тут дарят бакс просто за регистрацию! Успей получить!
Все советы на форуме раздаю бесплатно, то есть даром. Индивидуально бесплатно консультирую только по вопросам стоимости индивидуальных консультаций
*

victorantoniak

  • Moderator
  • 1579
  • 185 / 2
  • JoomShopping Team
JFilterOutput::objectHTMLSafe($order);

Меняйте на

filterHTMLSafe($order);
Чтобы оставить сообщение,
Вам необходимо Войти или Зарегистрироваться