Revision: 3806
Author: [email protected]
Date: Thu Jul 29 10:59:25 2010
Log: Fixed a bug where an OLAPSession could not be removed from an
OLAPRootObject. This issue was caused by setting the child's parent
reference to null before firing the child removed event. The order of these
operations have been swapped.
http://code.google.com/p/power-architect/source/detail?r=3806
Modified:
/trunk/src/main/java/ca/sqlpower/architect/olap/OLAPRootObject.java
=======================================
--- /trunk/src/main/java/ca/sqlpower/architect/olap/OLAPRootObject.java Fri
Jul 16 07:18:35 2010
+++ /trunk/src/main/java/ca/sqlpower/architect/olap/OLAPRootObject.java Thu
Jul 29 10:59:25 2010
@@ -90,8 +90,8 @@
public OLAPSession removeOLAPSession(int pos) {
OLAPSession removedItem = olapSessions.remove(pos);
if (removedItem != null) {
- removedItem.setParent(null);
fireChildRemoved(OLAPSession.class, removedItem, pos);
+ removedItem.setParent(null);
}
return removedItem;
}
