-
-
Notifications
You must be signed in to change notification settings - Fork 121
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Таблица для настроек контроллера #1522
Comments
Отказ от yaml очень давно в планах. Но слишком много "но". Как минимум, дополнения, которые собраны общим типом (дампы sql с yaml) перестанут устанавливаться. Менять на тип json - пока что вряд ли, не хватает символов, можно сделать больший текстовый тип, хотя я слабо представляю, что такого можно записать в настройки компонента, чтобы привысить 65К символов. |
Вот и я когда разбирался с проблемой потери данных при сохранении настроек не сразу додумался. Да можно поменять тип поля на более большой тип данных. А как насчет сделать отдельную таблицу для хранения настроек, мне кажется это в этом подходе больше плюсов чем минусов. |
Ради одного поля не вижу смысла. И главное, зачем. Отдельную таблицу нет в планах делать. |
Понятно. |
Но тогда хотя бы нужно добавить уведомление пользователей при сохранении настроек о превышении допустимого максимума. |
Вероятно это нужно делать разработчику компонента в настройках полей, учитывая реалии. Я подумаю над увеличением размера поля. Переход на json тоже в планах. |
Есть предложение изменить тип поля
options
ALTER TABLE
cms_controllersCHANGE
optionsJSON NULL DEFAULT NULL COMMENT 'Массив настроек';
Тогда придется повысить версию Mysql и отказаться от YAML(Что нужно было уже давно сделать) (Лично мое мнение). Есть еще пару вариантов сделать отдельную таблицу для опций контроллеров. Т.к уже не первый раз сталкиваюсь с такой проблемой что поле превышает максимально допустимое количество символов и при сохранении настроек контроллера обрезается до максимально допустимого значения. Не всегда это явно можно отследить.The text was updated successfully, but these errors were encountered: