Opened 20 months ago

Closed 19 months ago

Last modified 17 months ago

#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 alx, 20 months ago

Summary: Пот клике "Несоотв." на главной странице открывать несоответствия конкретного изделияПри клике "Несоотв." на главной странице открывать несоответствия конкретного изделия

comment:2 by Denis_N, 19 months ago

Resolution: fixed
Status: newclosed

In 279/base:

Улучшено: При клике "Несоотв." на главной странице открывать несоответствия конкретного изделия

При клике на главной на ячейку в столбце несоответствия открывается и-с "Несоответствия" с серийным номером с главной страницы

close #1146

comment:3 by alx, 17 months ago

Не могу не сделать пару замечаний.

Во-первых, можно было написать проще и более эффективно:

    let serialFromMain = GetURLParameter('serialFromMain') ?? '';

Здесь одна строка кода вместо трех, и GetURLParameter() выполняется один раз, а не два.

Во-вторых, насколько я понимаю, GetURLParameter() возвращает URL-кодированное значение параметра, которое потом используется как текст. Правильнее и удобнее было бы возвращать уже декодированное значение параметра, например так:

    return decodeURIComponent(sParameterName[1].replace(/\+/g, '%20'));
Note: See TracTickets for help on using tickets.