Changeset 10087 in josm for trunk/src/org/openstreetmap

Timestamp:

2016-03-30T20:18:14+02:00 (9 years ago)

Author:

wiktorn

Message:

Make minimum and maximum cache expires for imagery adjustable

Location:

trunk/src/org/openstreetmap/josm/data

Files:

• imagery/TMSCachedTileLoaderJob.java (modified) (3 diffs)
• preferences/LongProperty.java (added)

trunk/src/org/openstreetmap/josm/data/imagery/TMSCachedTileLoaderJob.java

@@ -30 +30 @@
 import org.openstreetmap.josm.data.cache.ICachedLoaderListener;
 import org.openstreetmap.josm.data.cache.JCSCachedTileLoaderJob;
+import org.openstreetmap.josm.data.preferences.LongProperty;
 import org.openstreetmap.josm.tools.HttpClient;
 
@@ -40 +41 @@
 public class TMSCachedTileLoaderJob extends JCSCachedTileLoaderJob<String, BufferedImageCacheEntry> implements TileJob, ICachedLoaderListener  {
     private static final Logger LOG = FeatureAdapter.getLogger(TMSCachedTileLoaderJob.class.getCanonicalName());
-    private static final long MAXIMUM_EXPIRES = 30 /*days*/ * 24 /*hours*/ * 60 /*minutes*/ * 60 /*seconds*/ *1000L /*milliseconds*/;
-    private static final long MINIMUM_EXPIRES = 1 /*hour*/ * 60 /*minutes*/ * 60 /*seconds*/ *1000L /*milliseconds*/;
+    private static final LongProperty MAXIMUM_EXPIRES = new LongProperty("imagery.generic.maximum_expires",
+            30 /*days*/ * 24 /*hours*/ * 60 /*minutes*/ * 60 /*seconds*/ *1000L /*milliseconds*/);
+    private static final LongProperty MINIMUM_EXPIRES = new LongProperty("imagery.generic.minimum_expires",
+            1 /*hour*/ * 60 /*minutes*/ * 60 /*seconds*/ *1000L /*milliseconds*/);
     private final Tile tile;
     private volatile URL url;
@@ -244 +247 @@
         // keep the expiration time between MINIMUM_EXPIRES and MAXIMUM_EXPIRES, so we will cache the tiles
         // at least for some short period of time, but not too long
-        if (ret.getExpirationTime() < now + MINIMUM_EXPIRES) {
-            ret.setExpirationTime(now + MINIMUM_EXPIRES);
-        }
-        if (ret.getExpirationTime() > now + MAXIMUM_EXPIRES) {
-            ret.setExpirationTime(now + MAXIMUM_EXPIRES);
+        if (ret.getExpirationTime() < now + MINIMUM_EXPIRES.get()) {
+            ret.setExpirationTime(now + MINIMUM_EXPIRES.get());
+        }
+        if (ret.getExpirationTime() > now + MAXIMUM_EXPIRES.get()) {
+            ret.setExpirationTime(now + MAXIMUM_EXPIRES.get());
         }
         return ret;