commit a6b0797fcfb322244c941e5ca0f5ea004e9a6528
author David Morgan Spenser Savage <dsavage@apache.org> Thu Jul 29 08:43:11 2010 +0000
committer David Morgan Spenser Savage <dsavage@apache.org> Thu Jul 29 08:43:11 2010 +0000
tree 06e1cd2dd5bf2ab2fc4b7247692e64cf5721befc
parent c562b65b489f3c2f261b7882c4169b1ed5c82e84

fix hardcoded update period - now checks config props and falls back to default if that fails (FELIX-1610)


git-svn-id: https://svn.apache.org/repos/asf/felix/trunk@980360 13f79535-47bb-0310-9956-ffa450edef68

sigil/common/obr/src/org/apache/felix/sigil/obr/OBRRepositoryProvider.java

diff --git a/sigil/common/obr/src/org/apache/felix/sigil/obr/OBRRepositoryProvider.java b/sigil/common/obr/src/org/apache/felix/sigil/obr/OBRRepositoryProvider.java
index 32e8c15..171e634 100644
--- a/sigil/common/obr/src/org/apache/felix/sigil/obr/OBRRepositoryProvider.java
+++ b/sigil/common/obr/src/org/apache/felix/sigil/obr/OBRRepositoryProvider.java
@@ -33,6 +33,12 @@
 
 public class OBRRepositoryProvider implements IRepositoryProvider
 {
+    private static final String IN_MEMORY = "inmemory";
+    private static final String UPDATE_PERIOD = "updatePeriod";
+    private static final String AUTH_FILE = "auth";
+    private static final String CACHE_DIRECTORY = "cache";
+    private static final String INDEX_CACHE_FILE = "index";
+
     public IBundleRepository createRepository(String id, Properties preferences)
         throws RepositoryException
     {
@@ -45,10 +51,11 @@
             File urlFile = new File(urlStr);
             URL repositoryURL = urlFile.exists() ? urlFile.toURI().toURL() : new URL(urlStr);
             URL testURL = urlFile.exists() ? urlFile.toURI().toURL() : new URL(urlStr);
-            File indexCache = new File(preferences.getProperty("index"));
-            File localCache = new File(preferences.getProperty("cache"));
-            String auth = preferences.getProperty("auth");
+            File indexCache = new File(preferences.getProperty(INDEX_CACHE_FILE));
+            File localCache = new File(preferences.getProperty(CACHE_DIRECTORY));
+            String auth = preferences.getProperty(AUTH_FILE);
             File authFile = auth == null ? null : new File(auth);
+            Long up = preferences.containsKey(UPDATE_PERIOD) ? Long.parseLong( preferences.getProperty(UPDATE_PERIOD) ) : 60 * 60 * 24 * 7; 
 
             if (testURL.openConnection().getLastModified() == 0)
             {
@@ -58,10 +65,8 @@
                 System.err.println("WARNING: " + msg + "using cache: " + urlStr);
             }
 
-            // TODO create user configurable updatePeriod
-            long updatePeriod = TimeUnit.MILLISECONDS.convert(60 * 60 * 24 * 7,
-                TimeUnit.SECONDS);
-            if (preferences.getProperty("inmemory") == null)
+            long updatePeriod = TimeUnit.MILLISECONDS.convert(up,TimeUnit.SECONDS);
+            if (preferences.getProperty(IN_MEMORY) == null)
             {
                 return new NonCachingOBRBundleRepository(id, repositoryURL, indexCache,
                     localCache, updatePeriod, authFile);