Opened 3 years ago

Closed 20 months ago

Last modified 20 months ago

#963 closed улучшение (готово)

Лимит размера комментария мешает работать с базой

Reported by: alx Owned by: Denis_N
Priority: major Component: БД изделий АДС
Keywords: Cc:

Description

При написании комментария по результату диагностики неисправной платы я столкнулся с ограничением на размер текста комментария. Пришлось разбивать комментарий на части, как в SMS. :(

Считаю, что такое ограничение создает неудобства, и предлагаю его убрать.

Change History (7)

comment:1 by san, 2 years ago

Owner: changed from san to Denis_N
Status: newassigned

comment:2 by Denis_N, 22 months ago

Resolution: fixed
Status: assignedclosed

In 104/base:

Исправлено: лимит размера комментария мешает работать с базой
Добавил лимит 65535 символов. Столбцы comment должны быть типа text
closes #963

comment:3 by alx, 22 months ago

Resolution: fixed
Status: closedreopened

Как странно... Ограничения сделаны меньше чем было! Насколько я вижу, значения 100000 исправлены на 65535... :)

С другой стороны, я вряд ли мог написать комментарий более 100000 символов. Подозреваю, что на момент создания тикета ограничение было равно 1000 символов... Догадываюсь, что оно уже было исправлено до r104/base.

С третьей стороны, сейчас ограничение все равно сделано неправильно, так как 65535 - это максимальный размер значения типа TEXT в байтах, а атрибут maxlength поля ввода выражается в символах. Таким образом, даже сейчас можно набрать набрать комментарий, который будет отрезан при записи. ИМХО для значения столбца типа TEXT корректное ограничение должно быть 16383 или менее символов.

С четвертой стороны, ограничение в 16383 символа уже не выглядит достаточно большим (вдруг кто-то захочет, например, процитировать вывод платы в консоль?).

Предлагаю:

  • заменить тип столбца comment на MEDIUMTEXT (или даже LONGTEXT);
  • установить соответствующее правильное значение атрибута maxlength (IMHO в случае LONGTEXT его можно вообще убрать, ибо немыслимо ввести гигабайт текста).
Version 0, edited 22 months ago by alx (next)

comment:4 by san, 22 months ago

In 121/base:

В таблицах продукты, история и заказы тип поля комментария изменен с text на longtext
See #963

comment:5 by san, 22 months ago

milestone: 1 очередь

Milestone deleted

comment:6 by Denis_N, 20 months ago

Resolution: готово
Status: reopenedclosed

comment:7 by alx, 20 months ago

Ну хорошо. Пусть остается лимит 65535 символов. Если когда-нибудь этого окажется мало, переоткроют тикет.

Note: See TracTickets for help on using tickets.