#963 closed улучшение (готово)
Лимит размера комментария мешает работать с базой
Reported by: | alx | Owned by: | Denis_N |
---|---|---|---|
Priority: | major | Component: | БД изделий АДС |
Keywords: | Cc: |
Description
При написании комментария по результату диагностики неисправной платы я столкнулся с ограничением на размер текста комментария. Пришлось разбивать комментарий на части, как в SMS. :(
Считаю, что такое ограничение создает неудобства, и предлагаю его убрать.
Change History (7)
comment:1 by , 2 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:2 by , 22 months ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:3 by , 22 months ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Как странно... Ограничения сделаны меньше чем было! Насколько я вижу, значения 100000 исправлены на 65535... :)
С другой стороны, я вряд ли мог написать комментарий более 100000 символов. Подозреваю, что на момент создания тикета ограничение было равно 1000 символов... Догадываюсь, что оно уже было исправлено до r104/base.
С третьей стороны, сейчас ограничение все равно сделано неправильно, так как 65535 - это максимальный размер значения типа TEXT в байтах, а атрибут maxlength
поля ввода выражается в символах. Таким образом, даже сейчас можно набрать комментарий, который будет отрезан при записи. ИМХО для значения столбца типа TEXT корректное ограничение должно быть 16383 или менее символов.
С четвертой стороны, ограничение в 16383 символа уже не выглядит достаточно большим (вдруг кто-то захочет, например, процитировать вывод платы в консоль?).
Предлагаю:
- заменить тип столбца
comment
на MEDIUMTEXT (или даже LONGTEXT); - установить соответствующее правильное значение атрибута
maxlength
(IMHO в случае LONGTEXT его можно вообще убрать, ибо немыслимо ввести гигабайт текста).
comment:6 by , 20 months ago
Resolution: | → готово |
---|---|
Status: | reopened → closed |
comment:7 by , 20 months ago
Ну хорошо. Пусть остается лимит 65535 символов. Если когда-нибудь этого окажется мало, переоткроют тикет.
In 104/base: