Fix for issue #47

Author: curioustechizen

CHANGELOG.md

@@ -1,51 +1,57 @@
-v1.3.3 [Apr 2, 2017]
-========
+## v1.3.4
+
+  - [FIX]: Issue [#47](https://github.com/curioustechizen/android-ago/issues/47)
+  - [TRANSLATIONS]: Correct translations for Dutch, added Danish
+
+
+## v1.3.3 [Apr 2, 2017]
+
 
   - [CHANGE]: Bumped versions of compileSdkVersion, targetSdkVersion, Gradle, Android Gradle plugin and buildToolsVersion
   - [TRANSLATIONS]: Spanish, Ukrainian, Norwegian, Arabic, Dutch, Romanian
 
 
-v1.3.2 [Jun 12, 2016]
-========
+## v1.3.2 [Jun 12, 2016]
+
 
   - [BUGFIX]: Fixed memory leak in `RelativeTimeTextView`
   - [TRANSLATIONS]: French, Portuguese
 
 
 
-v1.3.1 [May 15, 2016]
-========
+## v1.3.1 [May 15, 2016]
+
 
   - [TRANSLATIONS]: Hebrew, Russian, Chinese, Slovak, German
 
 
-v1.3.0 [Feb 14, 2014]
-========
+## v1.3.0 [Feb 14, 2014]
+
 
   - [CHANGE]: Set minSdk to 8
   - [CHANGE]: Cleaned up unnecessary resources
   - [MISC]: Update to latest build tools
   - [NEW]: Included tasks for publishing to Maven Central
 
 
-v1.2.0 [Dec 16, 2014]
-========
+## v1.2.0 [Dec 16, 2014]
+
 
   - [BUGFIX]: NPE seen when the RelativeTimeTextView has an initial visibility="gone"
   - [BUGFIX]: RelativeTimeTextView does not show anything if the reference time was only set using the XML attribute reference_time
   - [MISC]: Updated gradle files to 1.0.0
 
 
-v1.1.1 [Sep 2, 2014]
-========
+## v1.1.1 [Sep 2, 2014]
+
 
   - [FEATURE]: Added option to add prefix and suffix to the RelativeTimeTextView (Credit @xiprox )
   - [MISC]: Updated to recent versions of gradle, gradle plugin, build tools.
 
 
 
-v1.1.0 [Aug 21, 2014]
-========
+## v1.1.0 [Aug 21, 2014]
+
 
   - [FEATURE]: Added Gradle Support. (Credit to @intrications )
   - [BUGFIX]: #2

android-ago/src/com/github/curioustechizen/ago/RelativeTimeTextView.java

@@ -128,7 +128,7 @@ public void setReferenceTime(long referenceTime) {
         /*
          * Instantiate a new runnable with the new reference time
          */
-        mUpdateTimeTask = new UpdateTimeRunnable(this, mReferenceTime);
+        initUpdateTimeTask();
 
         /*
          * Start a new schedule.
@@ -186,10 +186,15 @@ protected void onVisibilityChanged(View changedView, int visibility) {
     }
 
     private void startTaskForPeriodicallyUpdatingRelativeTime() {
+        if(mUpdateTimeTask.isDetached()) initUpdateTimeTask();
         mHandler.post(mUpdateTimeTask);
         isUpdateTaskRunning = true;
     }
 
+    private void initUpdateTimeTask() {
+        mUpdateTimeTask = new UpdateTimeRunnable(this, mReferenceTime);
+    }
+
     private void stopTaskForPeriodicallyUpdatingRelativeTime() {
         if(isUpdateTaskRunning) {
             mUpdateTimeTask.detach();
@@ -247,26 +252,30 @@ private SavedState(Parcel in) {
             referenceTime = in.readLong();
         }
     }
-    
-    private static class UpdateTimeRunnable implements Runnable{
 
-    	private long mRefTime;
-        private final WeakReference<RelativeTimeTextView>  weakRefRttv;
-    	
-    	UpdateTimeRunnable(RelativeTimeTextView rttv, long refTime){
-    		this.mRefTime = refTime;
+    private static class UpdateTimeRunnable implements Runnable {
+
+        private long mRefTime;
+        private final WeakReference<RelativeTimeTextView> weakRefRttv;
+
+        UpdateTimeRunnable(RelativeTimeTextView rttv, long refTime) {
+            this.mRefTime = refTime;
             weakRefRttv = new WeakReference<>(rttv);
         }
 
+        boolean isDetached() {
+            return weakRefRttv.get() == null;
+        }
+
         void detach() {
             weakRefRttv.clear();
         }
 
-		@Override
-		public void run() {
+        @Override
+        public void run() {
             RelativeTimeTextView rttv = weakRefRttv.get();
-            if(rttv == null)   return;
-			long difference = Math.abs(System.currentTimeMillis() - mRefTime);
+            if (rttv == null) return;
+            long difference = Math.abs(System.currentTimeMillis() - mRefTime);
             long interval = INITIAL_UPDATE_INTERVAL;
             if (difference > DateUtils.WEEK_IN_MILLIS) {
                 interval = DateUtils.WEEK_IN_MILLIS;
@@ -277,7 +286,7 @@ public void run() {
             }
             rttv.updateTextDisplay();
             rttv.mHandler.postDelayed(this, interval);
-			
-		}
+
+        }
     }
 }