Неприятный момент с ошибками кода в коментах / Вопросы, проблемы и их решения / Alto CMS

Неприятный момент с ошибками кода в коментах (comment #24286)

vshemarov@gmail.com — Thu, 22 Oct 2015 23:53:55 +0300

В результате форматирование сайта сломалось (что очевидно).

Да вот как-то не очень очевидно. Вот, попробовал воспроизвести проблему:
demo.altocms.com/new/39.html

Не получилось. Т.е. сам текст топика, конечно, сломался, но верстка страницы в целом не пострадала. Испробованные варианты:


https://www.youtube.com/watch?v=uT3SBzmDxGk
https://www.youtube.com/watch?v=uT3SBzmDxGk/video>
https://www.youtube.com/watch?v=uT3SBzmDxGk video>
https://www.youtube.com/watch?v=uT3SBzmDxGk >
https://www.youtube.com/watch?v=uT3SBzmDxGk

Возможно, там не просто тег был не закрыт, а какая-то особая комбинация, которая все сломала

Неприятный момент с ошибками кода в коментах (comment #24288)

j.j.lefebvre@yandex.ru — Fri, 23 Oct 2015 10:40:32 +0300

Так там же тэг был незакрыт не у <видео>, а у того на что поменялось… а поменялось на <ифрэйм> и куча всего в ифрэйме. Я конечно сам виноват что так произошло, но вот оказывается есть такой момент… обновлять чтото в коде лучше при выключенном сайте…

Неприятный момент с ошибками кода в коментах (comment #24289)

vshemarov@gmail.com — Fri, 23 Oct 2015 11:04:52 +0300

Так правки прямо в базе что ли выполнялись или как? Можете описать пошагово, как получить ошибку? Я не из праздного любопытства спрашиваю. Мне важно понять, можно ли как-то на уровне движка предотвратить ошибку или тут сугубо «человеческий фактор» и с этим уже ничего не поделаешь

Неприятный момент с ошибками кода в коментах (comment #24290)

j.j.lefebvre@yandex.ru — Fri, 23 Oct 2015 11:23:07 +0300

В файле engine/classes/modules/text/Text.class.php

Ниже кода
$sText = preg_replace(
'/<видео>http(?:s|):\/\/(?:www\.|)rutube\.ru\/video\/(\w+)\/?<\/видео>/Ui',
'<ифрэйм src="//rutube.ru/play/embed/$1" width="'. $iWidth. '" height="'. $iHeight. '" '. $sIframeAttr. '>',
$sText
);

Был добавлен собственный код, аналогично обрабатывающий ссылку с другого медиаресурса.
В коде который был в ифрэйме- была ошибка (тэг ифрейма не закрыт).
Поэтому форматировнаие сломалось напрочь.

А вот почему в момент вставки поста, в котором был тэг который обработался через этот этого preg_replace, текст заменился и в других коментариях к совершенно другим постам которые в этот момент набирали люди на сайте — я не совсем понимаю.

Неприятный момент с ошибками кода в коментах (comment #24292)

vshemarov@gmail.com — Fri, 23 Oct 2015 11:46:28 +0300

Ага, теперь стало понятней. Буду смотреть

Неприятный момент с ошибками кода в коментах (comment #24293)

alto@hitagi.ru — Fri, 23 Oct 2015 12:46:27 +0300

если в этих шаблонах будут допущены ошибки со стилями или тэгами — то вычищать из базы и восстанавливать нормальный вид сайта придется руками

Потому что подобные изменения надо проводить на локальном или тестовом сервере, а не в продакшене, где сидят живые люди, и база важна. Это проблема не движка, но подхода к работе.

Неприятный момент с ошибками кода в коментах (comment #24298)

j.j.lefebvre@yandex.ru — Fri, 23 Oct 2015 18:02:31 +0300

А почему вы решили что это продакшен?
Если бы это был продакшен — ситуация называлась бы не «неприятностью» а как-то иначе :)
Тестирование ресурса происходит по многим критериям. И по функционалу, и по юзабилити… Поэтому на ресурсе присутствует много тестеров, занимающихся своими участками работ. Оценивается интуитивность интерфейса, отзывчивость, эргономичность. Понятно что при этих тестах с системой работают живые люди. В том числе создающие контент и пишущие коменты. Параллельно в систему добавляется новый функционал, но при этом перезагрузки сервера не происходит а просто сбрасываются кэши (если надо).

Например было выдано замечание что в коментах не работает переход на новую строку по Энтер.
На самом деле после сохранения коментария в тех местах где жали Энтер потом будет вставлен тэг брейк, но в момент редактирования это не очевидно.