Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#326 closed баг (fixed)

Неверное отображение имен конференций на вкладке записей

Reported by: alx Owned by: dimag
Priority: critical Milestone: 2 очередь
Component: ПО MC04-Dispatcher. Пульт диспетчера/техника Keywords: audio, interface
Cc: san

Description (last modified by alx)

  1. Запускаем программу.
  2. Входим в сеть.
  3. Нажимаем Настройки программы.
  4. Переключаемся на подвкладку "Конференции".
  5. В поле "Основная конференция" вводим "5", поле "Переименовать в" - пустое.
  6. Нажимаем "Сохранить настройки".
  7. Нажимаем "Записи переговоров".
  8. В параметрах фильтра вводим с 10.08.2016 08:15 по 10.08.2016 09:16.
  9. Нажимаем Enter. В списке появляется 6 записей конференции с именем "5".

Это неправильно, так как на самом деле (согласно содержимому столбца conference табилцы cdr базы данных) только 4 из этих записей имеют имя "5", две оставшиеся имеют имя "0".

Надо сделать так, чтобы имена конференций выводились правильно, то есть "как есть".

Проверялось в r356.

Change History (9)

comment:1 by alx, 8 years ago

Description: modified (diff)

comment:2 by san, 8 years ago

Priority: majorcritical

Дима утверждает что значение введённое в поле "Конференция" вообще ни на что не влияет

in reply to:  2 comment:3 by alx, 8 years ago

Replying to san:

Дима утверждает что значение введённое в поле "Конференция" вообще ни на что не влияет

Проверь его утверждение, воспроизведя действия из ticket:299#comment:24.

comment:4 by san, 8 years ago

Повторил действия в r357
Да, действительно повторяется.
Причём если в поле "переименовать" указать например Вася, то две из этих конференций станут Вася.
Отсюда делаю вывод, что при значении "пусто" в поле переименовать, вместо того чтобы не переименовывать ничего программа переименовывает конфенцию "0" в значение из поля "Основная конференция". Это какая-то глупость.

comment:5 by dimag, 8 years ago

Что вы предлагаете, Александр? Что нужно отображать, например, содержимое соотвествующего поля [cdr.conference], если параметр конфигурации "Переименовать в" пустой?

comment:6 by san, 8 years ago

Если поле переименовать пустое - значит не нужно ничего переименовывать и отображать "как есть"

Т.е. если имя конференции "0", значит и отображаем "0"; если имя конференции "5", значит и отображаем "5"...

comment:7 by dimag, 8 years ago

Keywords: audio interface added
Resolution: fixed
Status: newclosed

comment:8 by san, 8 years ago

Дима, в очередной раз прошу тебя писать боле полные коментарии к комитам
"ошибка 327" не даёт по сути никакой информации, что было сделано и где

В очередной раз рекомендую к прочтению: TracEffectiveWork


Делая коммит, описывайте в комментарии суть сделанных изменений

Это поможет, просматривая впоследствии журнал, находить нужное исправление. Не ограничивайтесь фразами типа "теперь работает" или "исправление 168". Когда через год вы (или другой разработчик) будете читать журнал в поисках какого-то исправления, вы не сможете понять, что этот комментарий означает. Не ограничивайтесь ссылкой на тикет - у читающего журнал может не быть доступа к trac, а даже если и есть, дублирование текста в журнале репозитория сэкономит время, затраченное на поиски тикета.
Если коммит связан с тикетом, дайте на него ссылку в комментарии

Если коммит полностью исправляет заявленную в тикете проблему, укажите в комментарии номер(а) тикета(ов) с одним из слов "close", "closed", "closes", "fix", "fixed" или "fixes": Closes #123 или Fixes #23,#45. В этом случае упомянутые тикеты будут автоматически закрыты с резолюцией fixed и со ссылкой на данный коммит в комментарии.

Если коммит связан с тикетов (тикетами), но не завершает работу над ними, дайте ссылку на тикеты с одним из слов "addresses", "re", "references", "refs" или "see": "See #27" или Refs #125,#52. В перечисленные тикеты будет добавлен комментарий со ссылкой на данный коммит, но состояние тикетов не будет изменено (они не будут закрыты).

comment:9 by san, 7 years ago

Milestone: Текущее2 очередь

Milestone renamed

Note: See TracTickets for help on using tickets.