ggeorg / gwt-mosaic

Automatically exported from code.google.com/p/gwt-mosaic
1 stars 0 forks source link

PagingScrollTable selected rows bug #94

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

1. Create a PagingScrollTable with a multi-select selection policy:

pagingScrollTable.getDataTable().setSelectionPolicy(SelectionPolicy.MULTI_ROW);

2. In the browser select and then deselect rows.

3. Attempt to retrieve the selected row Set from the PagingScrollTable using:

PagingScrollTable.getSelectedRowValues();

What is the expected output? What do you see instead?

I would expect only selected rows to be returned, but i get ALL rows that have 
selected and deselected.

I think there is a bug in PagingScrollTable at line 527 :

This:
Set<Row> deselected = event.getSelectedRows();

Should be this (?):
Set<Row> deselected = event.getDeselectedRows();

What version of the product are you using? On what operating system?

latest build:
gwt-mosaic-v0.4.0-rc4 

Please provide any additional information below.

Original issue reported on code.google.com by j.steven...@har.mrc.ac.uk on 25 Jan 2012 at 2:33

GoogleCodeExporter commented 9 years ago
Yes, I have found the same problem.  The error in the code is on line 527 of 
PagingScrollTable.java.  It should check for DEselected rows, not selected rows 
there.

Set<Row> deselected = event.getSelectedRows();
SHOULD BE:
Set<Row> deselected = event.getDeselectedRows();

Original comment by wendy.bu...@gmail.com on 9 Feb 2012 at 5:33