Project

General

Profile

Actions

Bug #4090

closed

Incorrect URL for paging in Event search

Added by Radko KrkoŇ° over 4 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
High
Assignee:
Category:
Development - GUI
Target version:
Start date:
05/15/2018
Due date:
% Done:

100%

Estimated time:
To be discussed:

Description

For event searches using multiple choice dropdowns with multiple options selected, the URL for Next and Previous pages only contains the first selected option.

How to reproduce:
  1. Set up query: SELECT * FROM events WHERE "category" && ARRAY['Abusive.Spam', 'Information.UnauthorizedAccess'] ORDER BY "detecttime" DESC LIMIT 100;
  2. The URL for result page is: https://mentat-alt.cesnet.cz/mentat/events/search?dt_from=&dt_to=&source_addrs=&source_ports=&categories=Abusive.Spam&categories=Information.UnauthorizedAccess&description=&submit=Search
  3. The URL for Next page is: https://mentat-alt.cesnet.cz/mentat/events/search?page=2&submit=Search&categories=Abusive.Spam
    • This is not correct, multiple categories params should be present, just like in 2.
    • This was confirmed for all multichoice dropdowns, for each only the first selected option is included (it does not matter whether they are combined or isolated for the query).

As a consequence, results diplayed for Next/Previous pages do not match the original criteria.

Actions #1

Updated by Jan Mach about 4 years ago

  • Parent task deleted (#3734)
Actions #2

Updated by Jan Mach about 4 years ago

  • Target version set to Backlog
Actions #3

Updated by Radko KrkoŇ° about 4 years ago

  • Priority changed from Normal to High

Still present in 2.0.0, increasing priority as this makes paging unusable for multichoice selectors.

Actions #4

Updated by Jan Mach about 4 years ago

  • Target version changed from Backlog to 2.1

Moving to next release since this could pose problems to all users.

Actions #5

Updated by Jan Mach about 4 years ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

Attached commits fully resolve this issue on all existing views and forms.

Actions

Also available in: Atom PDF