svn commit: r356750 - /geronimo/trunk/modules/connector-builder/src/java/org/apache/geronimo/connector/deployment/ConnectorModuleBuilder.java

[dj...@apache.org]

Author: djencks
Date: Wed Dec 14 00:30:55 2005
New Revision: 356750

URL: http://svn.apache.org/viewcvs?rev=356750&view=rev
Log:
GERONIMO-1359 ParentId added in wrong deploy phase

Modified:
    geronimo/trunk/modules/connector-builder/src/java/org/apache/geronimo/connector/deployment/ConnectorModuleBuilder.java

Modified: geronimo/trunk/modules/connector-builder/src/java/org/apache/geronimo/connector/deployment/ConnectorModuleBuilder.java
URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/connector-builder/src/java/org/apache/geronimo/connector/deployment/ConnectorModuleBuilder.java?rev=356750&r1=356749&r2=356750&view=diff
==============================================================================
--- geronimo/trunk/modules/connector-builder/src/java/org/apache/geronimo/connector/deployment/ConnectorModuleBuilder.java (original)
+++ geronimo/trunk/modules/connector-builder/src/java/org/apache/geronimo/connector/deployment/ConnectorModuleBuilder.java Wed Dec 14 00:30:55 2005
@@ -243,16 +243,16 @@
         }
 
         List parentId = ServiceConfigBuilder.getParentID(gerConnector.getParentId(), gerConnector.getImportArray());
+        //suppressing the default parentid is mostly useful for deploying standalone connectors on the app client.
+        //The defaultParentId normally pulls in and tries to start all the base server gbeans.
+        if (!gerConnector.getSuppressDefaultParentId()) {
+            parentId.addAll(defaultParentId);
+        }
         return new ConnectorModule(standAlone, configId, parentId, moduleFile, targetPath, connector, gerConnector, specDD);
     }
 
     public void installModule(JarFile earFile, EARContext earContext, Module module) throws DeploymentException {
         GerConnectorType vendorConnector = (GerConnectorType) module.getVendorDD();
-        //suppressing the default parentid is mostly useful for deploying standalone connectors on the app client.
-        //The defaultParentId normally pulls in and tries to start all the base server gbeans.
-        if (!vendorConnector.getSuppressDefaultParentId()) {
-            earContext.addParentId(defaultParentId);
-        }
         try {
             JarFile moduleFile = module.getModuleFile();