diff options
author | Artur <artur@vaadin.com> | 2017-03-07 10:50:13 +0200 |
---|---|---|
committer | Henri Sara <henri.sara@gmail.com> | 2017-03-07 15:43:00 +0200 |
commit | 39218c989b5c9a4c0b6e568d3f32a7f26b41574f (patch) | |
tree | 9e51fc709cb61005641dc47ffd950f7ef38f5f62 | |
parent | b02b623d438480529e6d7f33168ff85a95617787 (diff) | |
download | vaadin-framework-39218c989b5c9a4c0b6e568d3f32a7f26b41574f.tar.gz vaadin-framework-39218c989b5c9a4c0b6e568d3f32a7f26b41574f.zip |
Add missing Grid.removeAllColumns (#8750)
Fixes #8704
-rw-r--r-- | server/src/main/java/com/vaadin/ui/Grid.java | 9 | ||||
-rw-r--r-- | server/src/test/java/com/vaadin/tests/server/component/grid/GridTest.java | 14 |
2 files changed, 23 insertions, 0 deletions
diff --git a/server/src/main/java/com/vaadin/ui/Grid.java b/server/src/main/java/com/vaadin/ui/Grid.java index 60abec4e45..51d6b1db61 100644 --- a/server/src/main/java/com/vaadin/ui/Grid.java +++ b/server/src/main/java/com/vaadin/ui/Grid.java @@ -2313,6 +2313,15 @@ public class Grid<T> extends AbstractListing<T> implements HasComponents, } /** + * Removes all columns from this Grid. + */ + public void removeAllColumns() { + for (Column<T, ?> column : getColumns()) { + removeColumn(column); + } + } + + /** * Sets the details component generator. * * @param generator diff --git a/server/src/test/java/com/vaadin/tests/server/component/grid/GridTest.java b/server/src/test/java/com/vaadin/tests/server/component/grid/GridTest.java index 40c753943e..c85cf5ee73 100644 --- a/server/src/test/java/com/vaadin/tests/server/component/grid/GridTest.java +++ b/server/src/test/java/com/vaadin/tests/server/component/grid/GridTest.java @@ -404,6 +404,20 @@ public class GridTest { } @Test + public void removeAllColumns() { + grid.removeAllColumns(); + + Assert.assertEquals(Collections.emptyList(), grid.getColumns()); + } + + @Test + public void removeAllColumnsInGridWithoutColumns() { + grid.removeAllColumns(); + grid.removeAllColumns(); + Assert.assertEquals(Collections.emptyList(), grid.getColumns()); + } + + @Test public void setColumns_reorder() { // Will remove other columns grid.setColumns("length", "foo"); |