diff options
author | Leif Åstrand <leif@vaadin.com> | 2012-07-30 09:12:37 +0000 |
---|---|---|
committer | Leif Åstrand <leif@vaadin.com> | 2012-07-30 09:12:37 +0000 |
commit | cc65635fe483cba01d718a152f77e4d5be91c162 (patch) | |
tree | 96c7ff702c11fd234436466fab3f99c3be397c43 | |
parent | fbddc9eff67a8392246d333d0c826d4e71edafa8 (diff) | |
download | vaadin-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.java | 3 |
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); |