Ну если один из ведущих разработчиков модулей пишет как самый худший студент - совершенно нечитаемый код, то DEFAULT для поле TEXT - это наименьшая проблема
Они оптимизирут код перед его выкладкой - удаляют пробелы и табы, иногда переводы строк: ведущие пробелы - могут давать 30-40% к объему файла, который при работе скрипта загружается в память, так что это даже полезное дело. С одной стороны плохо, что цскартовцы этого не делают, так как объем ядра уменьшился бы процентов на 20, а с другой - хорошо, так как часто приходится лазить в код и что-то править…
Вот еще…
в файле шаблона
/design/backend/templates/addons/rus_order_logs/views/orders/components/order_logs.tpl
в блоке div -
один открыывающий тег < table >
и два (!) закрывающих < / table >
к чему бы это? проверил в обновленном тесте - всё неизменно с самого выхода модуля.
PS - еще атавизм в модуле - в таблице есть поле
`label` char(1) DEFAULT ''
которое вообще нигде и никогда не используется - видимо были планы, но потом о них забыли. кстати, не убирайте это поле, я придумал ему применение - по клику на строку комментария администратора в модуле
комментарию присваивается метка важности в этом поле, в итоге сам коммент подсвечивается, а на списке заказов на этом заказе появляется иконка с такой меткой, которая говорит, что на заказ надо обратить внимание. Повторным кликом метка снимается
Здравствуйте @alex_vp
Спасибо за ваше обращение и предоставленную информацию.
Синтаксис действительно не совсем верный, однако это не является багом.
Возможно, синтаксис будет изменен в одной из будущих версий.
Действительно, в шаблоне есть дублирующий закрывающий тег </table>. Но т. к. браузеры умные, они сами вырезали этот тег. Поэтому баг не был замечен в течение 8 лет
Мы исправили проблему с дублирующим тегом </table>. Исправление войдет в CS-Cart 4.14.3.
Спасибо за быстрый ответ и ссылку на стандарты html, но всё равно обывателю сложно разобраться в этих хитросплетениях.
Просто кажите, по вашему мнению, в конкретном случае по логике браузера, какой тег будет вырезан, первый или второй?
По человеческой логике браузер встречает первое вхождение и остальные игнорирует, т.к. тег уже закрыт. Он не будет парсить раз за разом страницу в поисках дублей закрывающих тегов для каждого из них.
Я тоже сделал такой вывод. Но, возникает вопрос ещё и в том, какой из тегов верный, как было задумано разработчиками. Расположение закрывающего тега может влиять на разметку блока или страницы, и если убрать не тот что нужно, то в каких-то браузерах может поехать вёрстка.