Version 1.3: bugfix, add shutdown for SyncTarget threads.

* 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
+#Fri, 13 Sep 2013 08:25:17 -0500
 name=timestamper-lib
-version=1.2
+version=1.3
 lib.local=true
-build.number=1
+build.number=10

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

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

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

@@ -91,11 +91,19 @@ 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() {
+        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", "");