diff options
author | Lyubomir Marinov <lyubomir.marinov@jitsi.org> | 2014-02-18 16:18:35 +0200 |
---|---|---|
committer | Lyubomir Marinov <lyubomir.marinov@jitsi.org> | 2014-02-18 16:18:35 +0200 |
commit | 0c7cdf4e64ce378dd7fdb8b16bd02692843a6bdc (patch) | |
tree | 319f08aec1b975e2e5e01878896ff57f3d5cf3ec /src/net/java/sip/communicator/impl/configuration/ConfigurationActivator.java | |
parent | 8ef362f0811123acbc8f8404a1eee5fb6862c534 (diff) | |
download | jitsi-0c7cdf4e64ce378dd7fdb8b16bd02692843a6bdc.zip jitsi-0c7cdf4e64ce378dd7fdb8b16bd02692843a6bdc.tar.gz jitsi-0c7cdf4e64ce378dd7fdb8b16bd02692843a6bdc.tar.bz2 |
Fixes an IllegalStateException in FileAccessServiceImpl when executing inside Jitsi Videobridge.
Diffstat (limited to 'src/net/java/sip/communicator/impl/configuration/ConfigurationActivator.java')
-rw-r--r-- | src/net/java/sip/communicator/impl/configuration/ConfigurationActivator.java | 27 |
1 files changed, 23 insertions, 4 deletions
diff --git a/src/net/java/sip/communicator/impl/configuration/ConfigurationActivator.java b/src/net/java/sip/communicator/impl/configuration/ConfigurationActivator.java index e6cee27..fac1434 100644 --- a/src/net/java/sip/communicator/impl/configuration/ConfigurationActivator.java +++ b/src/net/java/sip/communicator/impl/configuration/ConfigurationActivator.java @@ -49,14 +49,33 @@ public class ConfigurationActivator { FileAccessService fas = ServiceUtils.getService(bundleContext, FileAccessService.class); + if (fas != null) { - File useDatabaseConfig = fas.getPrivatePersistentFile( - ".usedatabaseconfig", - FileCategory.PROFILE); + File useDatabaseConfig; + + try + { + useDatabaseConfig + = fas.getPrivatePersistentFile( + ".usedatabaseconfig", + FileCategory.PROFILE); + } + catch (IllegalStateException ise) + { + /* + * There is somewhat of a chicken-and-egg dependency between + * FileConfigurationServiceImpl and ConfigurationServiceImpl: + * FileConfigurationServiceImpl throws IllegalStateException if + * certain System properties are not set, + * ConfigurationServiceImpl will make sure that these properties + * are set but it will do that later. + */ + useDatabaseConfig = null; + } // BETA: if the marker file exists, use the database configuration - if (useDatabaseConfig.exists()) + if ((useDatabaseConfig != null) && useDatabaseConfig.exists()) { logger.info("Using database configuration store."); this.cs = new JdbcConfigService(fas); |