openworld-community / ows-events

Project to promote, publish and collect events
https://afisha.peredelano.com
GNU General Public License v3.0
15 stars 1 forks source link

Bug with create/edit event - remove trim() all strings (title, description) and create variables for max lengths #1203

Closed Sinamon-maker closed 3 months ago

Sinamon-maker commented 3 months ago

Критичность

В каком релизе был добавлен функционал, в котором возникла ошибка

Описание

Вместо значений при создании ивента можно забить пробелами поля title, address, description

Ожидаемое поведение

Нельзя вместо значений при создании ивента забить пробелами поля title, address, description

Условия воспроизведение

Шаги воспроизведения

Пошаговое описание условий возникновения бага

Ссылка на родительскую задачу

[!IMPORTANT] Ссылка на родительскую задачу

Скриншоты* (необязательное)

Дополнительная информация* (необязательное)

Добавьте информацию о баге которую вы считаете важной

Elizaveta-S commented 3 months ago

Не исправлено: поле Организатор можно заполнить пробелами https://take.ms/l4nEM Как выглядит карточка https://take.ms/7lWQb

Sinamon-maker commented 3 months ago

все поля можно заполнить пробелами, но: из-за того что поля title, description обязательные - сработает валидация и не создастся ивент с полями-пробелами. В случае organizer также не создасться ивент с полями-пробелами, но поскольку поле необязательное то валидация не будет ругаться, нам все равно - поле заполнено или нет и произойдет отправка не сервер в любом случае (в случае заполнения только пробелами поле считается ТЕПЕРЬ незаполненным, они автоматом уберутся при создании ивента и в карточке пробелов по краям не будет)