[jboss-cvs] jbosside/as/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/model ...

[Robert Stryker]

  User: rawb    
  Date: 07/01/04 17:41:19

  Modified:    as/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/model 
                        ModuleModel.java
  Log:
  Initialization of module factories needed to be refactored because, well, it didnt work. ;)   Initialization was being done too many times, and in the case of path factories, not at all. 
  
  Revision  Changes    Path
  1.9       +8 -5      jbosside/as/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/model/ModuleModel.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: ModuleModel.java
  ===================================================================
  RCS file: /cvsroot/jboss/jbosside/as/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/model/ModuleModel.java,v
  retrieving revision 1.8
  retrieving revision 1.9
  diff -u -b -r1.8 -r1.9
  --- ModuleModel.java	28 Nov 2006 17:30:01 -0000	1.8
  +++ ModuleModel.java	4 Jan 2007 22:41:19 -0000	1.9
  @@ -116,11 +116,14 @@
   		while(i.hasNext()) {
   			f = (ModuleFactory)i.next();
   			delegate = f.getDelegate(null);
  -			if( delegate instanceof JBossModuleFactory ) {
  -				((JBossModuleFactory)delegate).initialize();
  -			} else {
  -				factories.remove(f);
  -			}
  +			// initialization is now done by the factory itself in wtp
  +			if( (delegate instanceof JBossModuleFactory ))
  +				i.remove();
  +//			if( delegate instanceof JBossModuleFactory ) {
  +//				((JBossModuleFactory)delegate).initialize();
  +//			} else {
  +//				i.remove();
  +//			}
   		}
   
   	}