Opened 8 years ago

Closed 6 years ago

#14180 closed defect (patchwelcome)

EnhancedGrid - Pagination: grid.setQuery() works not properly with pagination

Reported by: elvis Owned by: Evan
Priority: high Milestone: tbd
Component: DojoX Grid Version: 1.6.1
Keywords: grid pagination Cc:
Blocked By: Blocking:

Description

The pagination bar can not be updated correctly if filter grid data by call grid.setQuery() when the current page is not first page, e.g., there is a grid with 100 items, and rows per page are 10; Steps:

  1. jump the page to 3rd,
  2. Call grid.setQuery() to filter the grid data, assume there would be only 5 items leave after filtering;

Expect: pagination bar should be updated and only show one page Observed: the pagination bar still show there are 10 pages and current page is 3rd page

Change History (3)

comment:1 Changed 8 years ago by juk

I cannot repeat this with Dojo 1.7 EnhancedGrid? - it correctly updates page stepper size on setQuery().
However, it's still not working properly, I think, as current page is not reset to first page.

Scenario:

  1. Have grid with 10 pages.
  2. Jump to 3rd page.
  3. Call grid.setQuery(), assume there would be 4 pages of filtered data. (i.e. more than 3)

Expected: page 1 is selected.
Observed: page 3 is selected.

comment:2 Changed 6 years ago by bill

DojoX Grid and EnhancedGrid are deprecated in favor of dgrid and gridx.

You should upgrade your code to use one of those two grids.

We will consider patches to the old DojoX Grid code though.

comment:3 Changed 6 years ago by bill

Resolution: patchwelcome
Status: newclosed
Note: See TracTickets for help on using tickets.