You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by da...@apache.org on 2004/04/23 09:17:33 UTC
cvs commit: incubator-geronimo/modules/kernel/src/java/org/apache/geronimo/gbean/jmx CollectionProxy.java
dain 2004/04/23 00:17:33
Modified: modules/kernel/src/java/org/apache/geronimo/gbean/jmx
CollectionProxy.java
Log:
Fire member removed event before disabling the proxy so the event listener
can still invoke methods on the proxy
Revision Changes Path
1.15 +4 -4 incubator-geronimo/modules/kernel/src/java/org/apache/geronimo/gbean/jmx/CollectionProxy.java
Index: CollectionProxy.java
===================================================================
RCS file: /home/cvs/incubator-geronimo/modules/kernel/src/java/org/apache/geronimo/gbean/jmx/CollectionProxy.java,v
retrieving revision 1.14
retrieving revision 1.15
diff -u -r1.14 -r1.15
--- CollectionProxy.java 10 Mar 2004 09:59:00 -0000 1.14
+++ CollectionProxy.java 23 Apr 2004 07:17:33 -0000 1.15
@@ -130,12 +130,12 @@
public synchronized void removeTarget(ObjectName target) {
Object targetProxy = proxies.remove(target);
if (targetProxy != null) {
+ if (!stopped) {
+ proxy.fireMemberRemovedEvent(targetProxy);
+ }
ProxyMethodInterceptor interceptor = (ProxyMethodInterceptor) interceptors.remove(target);
if (interceptor != null) {
interceptor.disconnect();
- }
- if (!stopped) {
- proxy.fireMemberRemovedEvent(targetProxy);
}
}
}