Version 1.5: Fixed an issue with JDBLabsWebTimeline persist.

* SyncTarget now persists the remote timeline after pushing to it.
* Added SyncTarget.shutdown() to allow callers to signal the sync timer thread
  to stop.
* Fixed a bug where a SyncTarget was added instanciated times.

project.properties

@@ -1,5 +1,5 @@
-#Fri, 09 Aug 2013 11:36:19 -0500
+#Sun, 22 Sep 2013 14:58:43 -0500
 name=timestamper-lib
-version=1.2
+version=1.5
 lib.local=true
 build.number=1

src/main/com/jdblabs/timestamper/core/JDBLabsWebTimelineSource.groovy

@@ -153,7 +153,8 @@ public class JDBLabsWebTimelineSource extends TimelineSource {
                 requestContentType = JSON
                 body = entryBody
 
-                response.success = { entryHashes.put(fullHash(entry), entry.id) }
+                response.success = { resp, json ->
+                    entryHashes.put(fullHash(entry), json?.id ?: 0) }
             }
         }
     }

src/main/com/jdblabs/timestamper/core/SyncTarget.java

@@ -91,11 +91,22 @@ public class SyncTarget {
                 remoteTimeline.addAll(diffFromLocal);
                 syncPerformed = true;
             }
+
+            // try to persist the updated remote timeline
+            SyncTarget.this.source.persist(remoteTimeline);
         }
 
         return syncPerformed;
     }
 
+    public void shutdown() throws IOException {
+        // TODO: move this onto the timer thread?
+        if (syncOnExit) sync();
+
+        syncTimer.cancel();
+        syncTimer.purge();
+    }
+
     public String getName() { return name; }
 
     public TimelineSource getSource() { return source; }

src/main/com/jdblabs/timestamper/core/TimelineProperties.java

@@ -123,9 +123,6 @@ public class TimelineProperties {
         timeline = timelineSource.read();
 
         // search keys for remote timeline entries
-        // TODO: this code will add a new sync object for every remote target
-        // property, regardless of whether the SyncTarget for that remote URI
-        // already exists
         for (Object keyObj : config.keySet()) {
             if (!(keyObj instanceof String)) continue;
 
@@ -138,6 +135,14 @@ public class TimelineProperties {
             if (!m.matches()) continue;
 
             stName = m.group(1);
+
+            // skip if we have already setup this remote sync
+            boolean stExists = false;
+            for (SyncTarget target : syncTargets)
+                if (target.getName().equals(stName))
+                    stExists = true;
+            if (stExists) continue;
+
             remoteBase = REMOTE_TIMELINE_BASE + stName;
 
             strURI = (String) config.getProperty(remoteBase + ".uri", "");