add_plugin_settings_for_sql_validator#184
add_plugin_settings_for_sql_validator#184Masssana wants to merge 5 commits intoprocesstech:masterfrom
Conversation
plugins/ru.runa.gpd/plugin.xml
Outdated
| </validator> | ||
|
|
||
| <validator | ||
| applicable="java.lang.Object" |
There was a problem hiding this comment.
просьба посмотреть - можно ли тут через использовать множественный фильтр (нас интересуют только строки и числа думаю), чтобы валидатор не светился там где неприменим (хотя бы частично)
P.S. выше вижу applicable="java.util.List,java.util.Map", наверное можно списком задать
| validator.time.param.min = Minimum time value | ||
| validator.url.desc = Value is valid URL | ||
| validator.url.name = URL | ||
| validator.uniqueCheckInStorage = Checking uniqueness in external memory |
There was a problem hiding this comment.
"external memory" так себе :) "storage" рекомендуемый термин в develop ( ранее был external/internal storage, но это тоже так себе было
| validator.url.desc = Value is valid URL | ||
| validator.url.name = URL | ||
| validator.uniqueCheckInStorage = Checking uniqueness in external memory | ||
| validator.Unique.name = DW Uniqueness |
There was a problem hiding this comment.
DW? Storage table column uniqueness - не влезет?
plugins/ru.runa.gpd/plugin.xml
Outdated
| applicable="java.lang.Object" | ||
| description="Проверка уникальности ID во внутреннем хранилище" | ||
| label="Уникальность ХД" | ||
| applicable="java.util.List,java.util.Map" |
There was a problem hiding this comment.
не, ну не дословно же скопировать надо было, подставь свои типы (строка и число), а не список и карта
| validator.url.desc = Value is valid URL | ||
| validator.url.name = URL | ||
| validator.uniqueCheckInStorage = Checking uniqueness in data storage | ||
| validator.Unique.name = Data warehouse uniqueness |
There was a problem hiding this comment.
"Data warehouse uniqueness" - новый термин, лучше не вводить, предлагаю по названию класса сделать, причём ключи - тоже, может description не нужен, раз непонятно что написать?
ну и не был использован плагин, который делает настройки эти - не по алфавиту
ну и на русском - тоже названия непонятные
2 participants
No description provided.