From 22c372b7a5cc6540d45b9add88d447ff52eab62d Mon Sep 17 00:00:00 2001 From: Artur Signell Date: Sat, 15 Oct 2016 22:44:27 +0300 Subject: Send selection change events when changing selection mode (#20391) Change-Id: Iaeb772981f3ac039bf0fe3cde595cfdb3691cddf --- .../src/main/java/com/vaadin/v7/ui/Grid.java | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) (limited to 'compatibility-server/src/main/java') diff --git a/compatibility-server/src/main/java/com/vaadin/v7/ui/Grid.java b/compatibility-server/src/main/java/com/vaadin/v7/ui/Grid.java index ce78fa33cf..ef35808f99 100644 --- a/compatibility-server/src/main/java/com/vaadin/v7/ui/Grid.java +++ b/compatibility-server/src/main/java/com/vaadin/v7/ui/Grid.java @@ -5621,7 +5621,8 @@ public class Grid extends AbstractComponent * Takes a new {@link SelectionModel} into use. *

* The SelectionModel that is previously in use will have all its items - * deselected. + * deselected. If any items were selected, this will fire a + * {@link SelectionEvent}. *

* If the given SelectionModel is already in use, this method does nothing. * @@ -5638,13 +5639,23 @@ public class Grid extends AbstractComponent } if (this.selectionModel != selectionModel) { + Collection oldSelection; // this.selectionModel is null on init if (this.selectionModel != null) { + oldSelection = this.selectionModel.getSelectedRows(); this.selectionModel.remove(); + } else { + oldSelection = Collections.emptyList(); } this.selectionModel = selectionModel; selectionModel.setGrid(this); + Collection newSelection = this.selectionModel + .getSelectedRows(); + + if (!SharedUtil.equals(oldSelection, newSelection)) { + fireSelectionEvent(oldSelection, newSelection); + } } } -- cgit v1.2.3