[jboss-dev-forums] [Design of POJO Server] - Re: loader-repository name required in JBAS5

[charles.crouch@jboss.com]

Looking at 

http://fisheye.jboss.com/browse/JBossAS/trunk/server/src/main/org/jboss/deployment/JBossWebAppParsingDeployer.java?r=66523

it appears that line 121 prevents the loader-repository name from ever being null

109   	    protected void init(VFSDeploymentUnit unit, JBossWebMetaData metaData, VirtualFile file) throws Exception
  | 110 	   {
  | 111 	      // Add the loader repository config
  | 112 	      ClassLoadingMetaData classLoading = metaData.getClassLoading();
  | 113 	      LoaderRepositoryMetaData lrmd = null;
  | 114 	      if(classLoading != null)
  | 115 	         lrmd = classLoading.getLoaderRepository();
  | 116 	      if (lrmd != null)
  | 117 	      {
  | 118 	         LoaderRepositoryConfig loaderConfig = new LoaderRepositoryConfig();
  | 119 	         
  | 120 	         loaderConfig.repositoryClassName = lrmd.getLoaderRepositoryClass();
  | 121 	         loaderConfig.repositoryName = new ObjectName(lrmd.getName());
  | 122 	         Set<LoaderRepositoryConfigMetaData> configs = lrmd.getLoaderRepositoryConfig();
  | 123 	         if (configs != null && configs.isEmpty() == false)
  | 124 	         {
  | 125 	            LoaderRepositoryConfigMetaData config = configs.iterator().next();
  | 126 	            loaderConfig.configParserClassName = config.getConfigParserClass();
  | 127 	            loaderConfig.repositoryConfig = config.getConfig();
  | 128 	         }
  | 129 	         unit.addAttachment(LoaderRepositoryConfig.class, loaderConfig);
  | 130 	      }      
  | 131 	   }

View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4117776#4117776

Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4117776