cvs commit: incubator-geronimo/modules/core/src/java/org/apache/geronimo/deployment/plan CreateMBeanInstance.java

[dain]

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);
                       }