Author: scooter
Date: 2011-05-24 17:31:01 -0700 (Tue, 24 May 2011)
New Revision: 25523
Modified:
cytoscape/trunk/coreplugins/browser/src/main/java/browser/ui/AttributeBrowserToolBar.java
Log:
Fix bug where using "Select All" breaks the CheckBoxJList by completely
removing attributes that have been deselected from the list.
Modified:
cytoscape/trunk/coreplugins/browser/src/main/java/browser/ui/AttributeBrowserToolBar.java
===================================================================
---
cytoscape/trunk/coreplugins/browser/src/main/java/browser/ui/AttributeBrowserToolBar.java
2011-05-25 00:24:47 UTC (rev 25522)
+++
cytoscape/trunk/coreplugins/browser/src/main/java/browser/ui/AttributeBrowserToolBar.java
2011-05-25 00:31:01 UTC (rev 25523)
@@ -1090,7 +1090,10 @@
public void updateList(List<String> newSelection) {
- orderedCol = newSelection;
+ // We need to make sure we use a *copy* of the selection or
when we remove
+ // attributes from the list in the table browser, we also
remove them from the
+ // list that the popup dialog uses -- which we *definitely*
don't want to do
+ orderedCol = new ArrayList<String>(newSelection);
attributeList.setSelectedItems(orderedCol);
}
--
You received this message because you are subscribed to the Google Groups
"cytoscape-cvs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/cytoscape-cvs?hl=en.
