#1146 closed улучшение (fixed)
При клике "Несоотв." на главной странице открывать несоответствия конкретного изделия
Reported by: | alx | Owned by: | Denis_N |
---|---|---|---|
Priority: | major | Component: | БД изделий АДС |
Keywords: | Cc: |
Description
В r213/base при клике ячейки "Несоотв." на главной странице открывается страница с интерфейсом "Несоответствия", на которой вперемежку отображаются записи разных изделий.
Мне кажется, что если пользователь кликает ячейку в строке какого-то изделия, то он хочет видеть записи, касающиеся именно этого изделия, а не какого-то другого. Сейчас для этого приходится вручную набирать серийный номер нужного изделия, что неудобно...
Предлагаю при открытии интерфейса "Несоответствия" указанным выше способом сразу отображать записи именно того изделия, которое было кликнуто. Интерфейсы "Тестирование" и "Отк" работают именно так. Если я ничего не путаю, то раньше (r180/base ?) интерфейс "Несоответствия" тоже так и работал, но потом это, видимо, было сломано...
Change History (3)
comment:1 by , 20 months ago
Summary: | Пот клике "Несоотв." на главной странице открывать несоответствия конкретного изделия → При клике "Несоотв." на главной странице открывать несоответствия конкретного изделия |
---|
comment:2 by , 19 months ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:3 by , 17 months ago
Не могу не сделать пару замечаний.
Во-первых, можно было написать проще и более эффективно:
let serialFromMain = GetURLParameter('serialFromMain') ?? '';
Здесь одна строка кода вместо трех, и GetURLParameter() выполняется один раз, а не два.
Во-вторых, насколько я понимаю, GetURLParameter() возвращает URL-кодированное значение параметра, которое потом используется как текст. Правильнее и удобнее было бы возвращать уже декодированное значение параметра, например так:
return decodeURIComponent(sParameterName[1].replace(/\+/g, '%20'));
In 279/base: