Revision: 2101
          http://vexi.svn.sourceforge.net/vexi/?rev=2101&view=rev
Author:   clrg
Date:     2007-08-30 15:20:18 -0700 (Thu, 30 Aug 2007)

Log Message:
-----------
Fix group merging - objects are removed from vectors when merged so must 
reassign groupref before merge

Modified Paths:
--------------
    trunk/widgets/org.vexi.widgets/src/org/vexi/lib/role/selectable.t

Modified: trunk/widgets/org.vexi.widgets/src/org/vexi/lib/role/selectable.t
===================================================================
--- trunk/widgets/org.vexi.widgets/src/org/vexi/lib/role/selectable.t   
2007-08-30 22:17:46 UTC (rev 2100)
+++ trunk/widgets/org.vexi.widgets/src/org/vexi/lib/role/selectable.t   
2007-08-30 22:20:18 UTC (rev 2101)
@@ -173,9 +173,10 @@
             if (group.selected) group.selected = false;
             var mem = group.members;
             var obj = mem.first;
+            // reassign group references - must happen before merge
+            while (obj) { obj.groupref = g; obj = mem.after(obj); }
+            // merge vectors
             g.members.merge(mem, after);
-            // reassign group references
-            while (obj) { obj.groupref = g; obj = mem.after(obj); }
         }
         
         // select the first in the group


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
Vexi-svn mailing list
Vexi-svn@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/vexi-svn

Reply via email to