although looking at this change, the other check was not modified:

````
 jmxEnabled = "true".equals(context.getEnvironment().getProperty(XDPropertyKeys.XD_JMX_ENABLED));
                if (jmxEnabled) {
````

Also, it relies on the fact that "preProcessSharedContext" is always called prior to "preProcessModule" since the former sets the jmxEnabled boolean value. So, maybe it's better to have the following for supports and then remove that other if check also:

````
public boolean supports(Module module) {
    return "true".equals(context.getEnvironment().getProperty(XDPropertyKeys.XD_JMX_ENABLED));
}
````