summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLeif Åstrand <leif@vaadin.com>2012-07-30 09:12:37 +0000
committerLeif Åstrand <leif@vaadin.com>2012-07-30 09:12:37 +0000
commitcc65635fe483cba01d718a152f77e4d5be91c162 (patch)
tree96c7ff702c11fd234436466fab3f99c3be397c43
parentfbddc9eff67a8392246d333d0c826d4e71edafa8 (diff)
downloadvaadin-framework-cc65635fe483cba01d718a152f77e4d5be91c162.tar.gz
vaadin-framework-cc65635fe483cba01d718a152f77e4d5be91c162.zip
Don't make changes directly to the visible itemids collection (#8109, #8787)
svn changeset:24032/svn branch:6.8
-rw-r--r--src/com/vaadin/ui/AbstractSelect.java3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/com/vaadin/ui/AbstractSelect.java b/src/com/vaadin/ui/AbstractSelect.java
index 71624804fa..1a781dd4f3 100644
--- a/src/com/vaadin/ui/AbstractSelect.java
+++ b/src/com/vaadin/ui/AbstractSelect.java
@@ -424,8 +424,9 @@ public abstract class AbstractSelect extends AbstractField implements
// Limits the deselection to the set of visible items
// (non-visible items can not be deselected)
- final Collection<?> visible = getVisibleItemIds();
+ Collection<?> visible = getVisibleItemIds();
if (visible != null) {
+ visible = new HashSet<Object>(visible);
// Don't remove those that will be added to preserve order
visible.removeAll(s);