Load modules by configuration

Hi all,

XMPPServer is a graceful class where the openfire’s modules have been loaded by static statements as:

loadModule(IQBindHandler.class.getName());

I need to disable some of them for instance IQPEPHandler but it isn’t possibile because the load module call is hard coded.

I think it would be an improvement to mode module setup to xml configuration in this way:

List modules = JiveGlobals.getXMLProperties("modules"); for(String module: modules)    loadModule(module);

What do you think about?

Thanks,

Giancarlo Frison

I remember that JabberD listed it’s modules in the config file, allowing admins to select. However, JabberD warned about disabling the pre-config list because it might cause unexpected behavior as well as breaking the XMPP spec. Nevertheless, it’s not a bad idea to have some control over optional modules.