Bug 331896

Description Richard Gregory 2010-12-06 08:47:45 EST

Build Identifier: 3.2.1.v201007071800

Server.getChildModules doesn't call server delegate for non-workspace modules:

	/*
	 * @see IServer#getChildModule(IModule[])
	 */
	public IModule[] getChildModules(IModule[] module, IProgressMonitor monitor) {
		if (module == null || module.length == 0)
			throw new IllegalArgumentException("Module cannot be null or empty");
		try {
			int i = module.length - 1;
			if (module[i].getProject() == null || !module[i].getProject().isAccessible())
				return null;  <== This is the case for non-workspace modules

                        // This doesn't get a chance to run...
			ServerDelegate sd = getDelegate(monitor); 
			if (sd == null)
				return null;
			IModule[] children = sd.getChildModules(module);


Reproducible: Always

Steps to Reproduce:
1. Create a new module type that does not have an IProject.
2. Try to add child modules to it.
3. getChildModules will always return null per the comments above.