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 2003/09/05 06:52:04 UTC
cvs commit: incubator-geronimo/modules/core/src/java/org/apache/geronimo/deployment/plan CreateMBeanInstance.java
dain 2003/09/04 21:52:04
Modified: modules/core/src/java/org/apache/geronimo/deployment/plan
CreateMBeanInstance.java
Log:
When using constructor args we set the arguments type back into the string array after
loading the instance, because our class loading is friendler and accecpts types not
understood by JMX.
Revision Changes Path
1.14 +5 -1 incubator-geronimo/modules/core/src/java/org/apache/geronimo/deployment/plan/CreateMBeanInstance.java
Index: CreateMBeanInstance.java
===================================================================
RCS file: /home/cvs/incubator-geronimo/modules/core/src/java/org/apache/geronimo/deployment/plan/CreateMBeanInstance.java,v
retrieving revision 1.13
retrieving revision 1.14
diff -u -r1.13 -r1.14
--- CreateMBeanInstance.java 5 Sep 2003 02:35:59 -0000 1.13
+++ CreateMBeanInstance.java 5 Sep 2003 04:52:04 -0000 1.14
@@ -158,6 +158,10 @@
Class type = null;
try {
type = Classes.loadClass(typeString, newCL);
+
+ // set the type string based on the loaded class
+ // load class code is more friendly then the Sun code
+ constructorTypeStrings.set(i, type.getName());
} catch (ClassNotFoundException e) {
throw new DeploymentException(e);
}