Запрашивается ненужная информация

[alx]

В r545/base обнаружено, что при нажатии на главной странице кнопки "Показать" помимо запроса изделий, кторые требуется отобразить, одновременно выполняется запрос истории какого-то изделия. Так как пользоватль (я) историю изделия не запрашивал, и на странице веб-интерфейса запрошенные данные не отображаются, я полагаю, что второй запрос не нужен, и выполняется по ошибке.

Предлагаю устранить ненужный запрос к серверу.

[Denis_N]

Спасибо за обратную связь. Проверил описанное поведение.

При нажатии кнопки «Показать» данные обновляются для обеих вкладок "Список изделий" и "Журнал событий". Это задумано для согласованного применения фильтров для обеих вкладок. И сделано это также для быстрого перехода к журналу событий без ожидания дополнительной загрузки.

С учётом этого намерен сохранить текущее поведение

[alx]

Replying to Denis_N:

При нажатии кнопки «Показать» данные обновляются для обеих вкладок "Список изделий" и "Журнал событий".

В моем эксперименте (о котором написано в описании тикета) вкладка была только одна. Вкладки "Журнал событий" не было.

Это задумано для согласованного применения фильтров для обеих вкладок.

Хм... Не представляю, каким макаром можно применить фильтр изделий к истории одного изделия...

И сделано это также для быстрого перехода к журналу событий без ожидания дополнительной загрузки.

В таком случае, вынужден констатировать, что это не работает - в моем эксперименте переход к истории изделия требует заметного времени ожидания загрузки (несколько секунд).

С учётом этого намерен сохранить текущее поведение

Твое священное право. :) Разработчик - ты. Мое дело - предложить...

[Denis_N]

Кажется, что что-то тут не так. А резивзия точно r545?

И ты уверен, что выполняется запрос к истории именно одного изделия?

Можешь подробнее описать как ты понимаешь, что к истории одного изделия идет запрос? И опиши пожалуйста подробнее действия, чтобы я воспроизвел поведение веб-интерфейса

[alx]

Replying to Denis_N:

А резивзия точно r545?

Нет, не точно. Так написано в веб-интерфейсе внизу страницы (только там это почему-то называется "версией" - видимо, следуя традиции АДС ничего не называть своими общепринятыми именами). Допускаю, что это может быть так называемым враньем. :)

И ты уверен, что выполняется запрос к истории именно одного изделия?

Нет, не уверен. Но даже если выполняется запрос истории семи изделий, это не становится менее странным (а только более), ведь пользователь (я), нажимая кнопку "Показать", не запрашивал историю никакого количества изделий - ни одного, ни семи. Я вообще историй каких-либо изделий показывать не просил.

Можешь подробнее описать как ты понимаешь, что к истории одного изделия идет запрос?

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

И опиши пожалуйста подробнее действия, чтобы я воспроизвел поведение веб-интерфейса

• Открываю главную страницу.
• Нажимаю кнопку "Показать".

[san]

Replying to alx:

Так написано в веб-интерфейсе внизу страницы... Допускаю, что это может быть так называемым враньем. :)

Ненене, там всё честно :-)

[alx]

Replying to san:

Допускаю, что это может быть так называемым враньем. :)

Ненене, там всё честно :-)

Ну, я немного преувеличил. :) Я не имел в виду намеренный обман. Но, так как этот номер ты вписываешь вручную, ты мог ошибиться - загрузить файлы одной ревизии, а номер по ошибке вписать другой...

[Denis_N]

Replying to alx:

В моем эксперименте (о котором написано в описании тикета) вкладка была только одна. Вкладки "Журнал событий" не было.

Алексей, а мог бы ты сказать, как ты выполняешь этот экперимент? Ты нажимаешь кнопку "Журнал событий" и совершаешь какие-то действия на странице и это занимает несколько секунд?

[Denis_N]

Replying to alx:

В таком случае, вынужден констатировать, что это не работает - в моем эксперименте переход к истории изделия требует заметного времени ожидания загрузки (несколько секунд).

Алексей, а мог бы ты сказать, как ты выполняешь этот экперимент? Ты нажимаешь кнопку "Журнал событий" и совершаешь какие-то действия на странице и происходит запрос и ожидание занимает несколько секунд?

[Denis_N]

Я подумал, Алексей, и согласен с тобой, что логичнее, если будет совершен запрос соответсвующий видимому действию пользователя.

Исправлено. Теперь:

при клике по вкладке загружается только выбранная таблица;
«Показать», «Обновить» и «Сбросить фильтры» загружают только активную вкладку;
начальная загрузка страницы больше не тянет обе таблицы сразу;

https://trac.adc-line.ru/mc-04/changeset/546/base

[Denis_N]

Не ту ссылку на решение изначально дал, исправил.

[alx]

Replying to Denis_N:

в моем эксперименте переход к истории изделия требует заметного времени ожидания загрузки (несколько секунд).

Алексей, а мог бы ты сказать, как ты выполняешь этот экперимент?

Мог бы: я кликаю серийный номер нужного мне изделия в списке.

Ты нажимаешь кнопку "Журнал событий"

Нет, такую кнопку я не нажимаю. Я до сегодняшнего дня даже не знал, что такая кнопка есть (не обращал на нее внимания).

И даже сейчас, хоть и знаю о ее наличии, ничего не знаю о ее функции и назначении (в Руководстве пользователя о ней ничего на нашел)...

и совершаешь какие-то действия на странице и происходит запрос и ожидание занимает несколько секунд?

Да. Как я уже написал выше, кликаю серийный номер нужного мне изделия и жду несколько секунд, когда загрузится журнал событий. У меня пока еще довольно медленный (и временами плохой) мобильный интернет, поэтому загрузка длится так долго, поэтому в самом факте длительной загрузки ничего ненормального нет.

[alx]

Replying to Denis_N:

Не ту ссылку на решение изначально дал, исправил.

В trac вообще не надо давать абсолютные ссылки на его собственные ресурсы. Посмотри как это делается в документации. В данном случае можно было написать changeset:546/base или r546/base.