![[MC-04 logo]](/mc-04/chrome/site/logo.png){kind=logo}
#1412 closed дефект (готово)
Неверная работа фильтра (сезон 3)
| Reported by: | alx | Owned by: | Denis_N |
|---|---|---|---|
| Priority: | critical | Component: | БД изделий АДС |
| Keywords: | Cc: |
Description (last modified by )
В r420/base выполняю следующие действия:
- Открываю главную страницу.
- В поле "Название изделия" ввожу "VE-02-01".
- Кликаю в панели "Фильтры по истории изделия".
- В открывшейся панели отмечаю радио-кнопку "Было протестированно".
- В появившейся строке "Кем" ввожу "Alex Mogilnikov". В результате получаю в таблице список из 30 изделий, что неверно.
Ожидалось, что я получу список всех изделий, тестировавшихся Алексом Могильниковым, которых на данный момент 39. Например в выведенном списке отсутствует изделие S/N H01568, которое имеет в истории запись о том, что его тестировал Alex Mogilnikov.
Change History (5)
comment:1 by , 9 months ago
| Description: | modified (diff) |
|---|
follow-up: 4 comment:3 by , 3 weeks ago
| Resolution: | → готово |
|---|---|
| Status: | new → closed |
Исправлено в r498/base.
В r420 фильтр “Было протестировано” на Главной странице проверял записи истории через точное сравнение h.type_write = 'testing'. Из-за этого записи со составным типом события, например mismatch,testing, не учитывались фильтром, хотя фактически это тоже записи тестирования. Поэтому часть изделий, протестированных указанным сотрудником, могла не попадать в результат.
В r498/base логика фильтров истории на Главной была переработана: тип события теперь проверяется через FIND_IN_SET('testing', h.type_write) > 0, поэтому составные записи тестирования также учитываются.
Спасибо, Алексей
follow-up: 5 comment:4 by , 3 weeks ago
Replying to Denis_N:
Исправлено в r498/base.
Хм... В сообщении коммита говорится об исправлении бага "новая главная сделана". :) Не обычный баг... :) Интересно, если через год ты прочитаешь это сообщение коммита, ты сам поймешь, чтоб было исправлено? :)
В r498/base логика фильтров истории на Главной была переработана: тип события теперь проверяется через
FIND_IN_SET('testing', h.type_write) > 0, поэтому составные записи тестирования также учитываются.
Вот в комментарии ты смог вполне ясно описать суть сделанных изменений...
Спасибо, Алексей
Да не за что. Рад был чем-то помочь.
comment:5 by , 3 weeks ago
Replying to alx:
Хм... В сообщении коммита говорится об исправлении бага "новая главная сделана". :) Не обычный баг... :) Интересно, если через год ты прочитаешь это сообщение коммита, ты сам поймешь, чтоб было исправлено? :)
Нет, сам не пойму. Буду стараться конкретнее писать
