From: Denis Date: Fri, 27 Jan 2017 11:13:58 +0000 (+0200) Subject: Allow to disable user selection in the grid model. (#8345) X-Git-Tag: 8.0.0.beta2~29 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=5616216306a138b3437d188e4b2df8253590abf5;p=vaadin-framework.git Allow to disable user selection in the grid model. (#8345) * Allow to disable user selection in the grid model. Fixes vaadin/framework8-issues#546 --- diff --git a/server/src/main/java/com/vaadin/ui/components/grid/AbstractSelectionModel.java b/server/src/main/java/com/vaadin/ui/components/grid/AbstractSelectionModel.java index e38c350694..7a47da0ce1 100644 --- a/server/src/main/java/com/vaadin/ui/components/grid/AbstractSelectionModel.java +++ b/server/src/main/java/com/vaadin/ui/components/grid/AbstractSelectionModel.java @@ -99,4 +99,14 @@ public abstract class AbstractSelectionModel extends AbstractGridExtension super.remove(); } + @Override + public boolean isUserSelectionAllowed() { + return getState(false).selectionAllowed; + } + + @Override + public void setUserSelectionAllowed(boolean allowed) { + getState().selectionAllowed = allowed; + } + } diff --git a/server/src/main/java/com/vaadin/ui/components/grid/GridSelectionModel.java b/server/src/main/java/com/vaadin/ui/components/grid/GridSelectionModel.java index 2913318bf8..4160b5951b 100644 --- a/server/src/main/java/com/vaadin/ui/components/grid/GridSelectionModel.java +++ b/server/src/main/java/com/vaadin/ui/components/grid/GridSelectionModel.java @@ -43,4 +43,27 @@ public interface GridSelectionModel extends SelectionModel, Extension { */ @Override public void remove(); + + /** + * Sets whether the user is allowed to change the selection. + *

+ * The check is done only for the client side actions. It doesn't affect + * selection requests sent from the server side. + * + * @param allowed + * true if the user is allowed to change the + * selection, false otherwise + */ + public void setUserSelectionAllowed(boolean allowed); + + /** + * Checks if the user is allowed to change the selection. + *

+ * The check is done only for the client side actions. It doesn't affect + * selection requests sent from the server side. + * + * @return true if the user is allowed to change the selection, + * false otherwise + */ + public boolean isUserSelectionAllowed(); } diff --git a/server/src/main/java/com/vaadin/ui/components/grid/MultiSelectionModelImpl.java b/server/src/main/java/com/vaadin/ui/components/grid/MultiSelectionModelImpl.java index 97f79a4655..122abadc5a 100644 --- a/server/src/main/java/com/vaadin/ui/components/grid/MultiSelectionModelImpl.java +++ b/server/src/main/java/com/vaadin/ui/components/grid/MultiSelectionModelImpl.java @@ -324,7 +324,7 @@ public class MultiSelectionModelImpl extends AbstractSelectionModel @Override public void setReadOnly(boolean readOnly) { - getState().selectionAllowed = !readOnly; + setUserSelectionAllowed(!readOnly); } @Override @@ -458,10 +458,6 @@ public class MultiSelectionModelImpl extends AbstractSelectionModel }, userOriginated); } - private boolean isUserSelectionAllowed() { - return getState(false).selectionAllowed; - } - private void doUpdateSelection(Consumer> handler, boolean userOriginated) { if (getParent() == null) { diff --git a/server/src/main/java/com/vaadin/ui/components/grid/NoSelectionModel.java b/server/src/main/java/com/vaadin/ui/components/grid/NoSelectionModel.java index ff51b74cfc..b239469e79 100644 --- a/server/src/main/java/com/vaadin/ui/components/grid/NoSelectionModel.java +++ b/server/src/main/java/com/vaadin/ui/components/grid/NoSelectionModel.java @@ -64,4 +64,13 @@ public class NoSelectionModel extends AbstractExtension "This selection model doesn't allow selection, cannot add selection listeners to it"); } + @Override + public void setUserSelectionAllowed(boolean allowed) { + } + + @Override + public boolean isUserSelectionAllowed() { + return false; + } + } diff --git a/server/src/main/java/com/vaadin/ui/components/grid/SingleSelectionModelImpl.java b/server/src/main/java/com/vaadin/ui/components/grid/SingleSelectionModelImpl.java index 12f3ee4de4..aae0da8e14 100644 --- a/server/src/main/java/com/vaadin/ui/components/grid/SingleSelectionModelImpl.java +++ b/server/src/main/java/com/vaadin/ui/components/grid/SingleSelectionModelImpl.java @@ -27,7 +27,6 @@ import com.vaadin.event.selection.SingleSelectionListener; import com.vaadin.shared.Registration; import com.vaadin.shared.data.selection.SelectionServerRpc; import com.vaadin.shared.ui.grid.SingleSelectionModelState; -import com.vaadin.ui.Component; import com.vaadin.ui.SingleSelect; /** @@ -227,10 +226,6 @@ public class SingleSelectionModelImpl extends AbstractSelectionModel return getState().deselectAllowed; } - private boolean isUserSelectionAllowed() { - return getState(false).selectionAllowed; - } - /** * Gets a wrapper for using this grid as a single select in a binder. * @@ -275,7 +270,7 @@ public class SingleSelectionModelImpl extends AbstractSelectionModel @Override public void setReadOnly(boolean readOnly) { - getState().selectionAllowed = !readOnly; + setUserSelectionAllowed(!readOnly); } @Override diff --git a/uitest/src/main/java/com/vaadin/tests/components/grid/GridDisallowUserSelection.java b/uitest/src/main/java/com/vaadin/tests/components/grid/GridDisallowUserSelection.java new file mode 100644 index 0000000000..caac34c550 --- /dev/null +++ b/uitest/src/main/java/com/vaadin/tests/components/grid/GridDisallowUserSelection.java @@ -0,0 +1,50 @@ +/* + * Copyright 2000-2016 Vaadin Ltd. + * + * Licensed under the Apache License, Version 2.0 (the "License"); you may not + * use this file except in compliance with the License. You may obtain a copy of + * the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT + * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the + * License for the specific language governing permissions and limitations under + * the License. + */ +package com.vaadin.tests.components.grid; + +import com.vaadin.data.ValueProvider; +import com.vaadin.server.VaadinRequest; +import com.vaadin.tests.components.AbstractTestUI; +import com.vaadin.ui.Button; +import com.vaadin.ui.Grid; +import com.vaadin.ui.Grid.SelectionMode; +import com.vaadin.ui.components.grid.GridSelectionModel; + +/** + * @author Vaadin Ltd + * + */ +public class GridDisallowUserSelection extends AbstractTestUI { + + @Override + protected void setup(VaadinRequest request) { + Grid grid = new Grid<>(); + grid.addColumn(ValueProvider.identity()); + grid.setItems("a", "b"); + GridSelectionModel model = grid + .setSelectionMode(SelectionMode.SINGLE); + model.setUserSelectionAllowed(false); + + addComponent(grid); + + Button button = new Button("Multi select", event -> { + GridSelectionModel multi = grid + .setSelectionMode(SelectionMode.MULTI); + multi.setUserSelectionAllowed(false); + }); + addComponent(button); + } +} diff --git a/uitest/src/test/java/com/vaadin/tests/components/grid/GridDisallowUserSelectionTest.java b/uitest/src/test/java/com/vaadin/tests/components/grid/GridDisallowUserSelectionTest.java new file mode 100644 index 0000000000..c8e6e39f2b --- /dev/null +++ b/uitest/src/test/java/com/vaadin/tests/components/grid/GridDisallowUserSelectionTest.java @@ -0,0 +1,51 @@ +/* + * Copyright 2000-2016 Vaadin Ltd. + * + * Licensed under the Apache License, Version 2.0 (the "License"); you may not + * use this file except in compliance with the License. You may obtain a copy of + * the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT + * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the + * License for the specific language governing permissions and limitations under + * the License. + */ +package com.vaadin.tests.components.grid; + +import org.junit.Assert; +import org.junit.Test; + +import com.vaadin.testbench.elements.ButtonElement; +import com.vaadin.testbench.elements.GridElement; +import com.vaadin.tests.tb3.MultiBrowserTest; + +/** + * @author Vaadin Ltd + * + */ +public class GridDisallowUserSelectionTest extends MultiBrowserTest { + + @Test + public void checkSelection() { + openTestURL(); + + assertNoSelection(0); + assertNoSelection(1); + + // change model from single select to mutli + $(ButtonElement.class).first().click(); + + assertNoSelection(0); + assertNoSelection(1); + } + + private void assertNoSelection(int index) { + GridElement grid = $(GridElement.class).first(); + grid.getCell(index, 0).click(); + + Assert.assertFalse(grid.getRow(0).isSelected()); + } +}