2012/12/12 Philip Luppens <philip.lupp...@gmail.com>:
> I'd say to go for it, but please do explain what the issue was for archival
> reasons, since the original ticket[1] did not receive any update yet.

Ok, you're right. The problem is related to my FileManager
refactorings and introduced FileManagerFactory with default
implementation

ConfigurationManager#conditionalReload() from version 2.3.7 looks like this:

public synchronized void conditionalReload(Container container) {
    FileManager fileManager =
container.getInstance(FileManagerFactory.class).getFileManager();
    if (fileManager.isReloadingConfigs() || providersChanged) {
    ........
}

so each time FileManager instance was looked up just to check if
reloadConfigs flag is set to true, but right now it looks like this
and the flag is checked directly

public synchronized void conditionalReload(Container container) {
    boolean reloadConfigs =
Boolean.parseBoolean(container.getInstance(String.class,
XWorkConstants.RELOAD_XML_CONFIGURATION));
    if (reloadConfigs || providersChanged) {
    ........
}


Regards
-- 
Łukasz
+ 48 606 323 122 http://www.lenart.org.pl/

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org
For additional commands, e-mail: dev-h...@struts.apache.org

Reply via email to