Do not force-sync when switching to another activity

AndBible · Jun 11, 2023 · 275ec02 · 275ec02
1 parent 0dc6396
commit 275ec02
Showing 1 changed file with 13 additions and 7 deletions.
diff --git a/app/src/main/java/net/bible/android/view/activity/page/MainBibleActivity.kt b/app/src/main/java/net/bible/android/view/activity/page/MainBibleActivity.kt
@@ -551,11 +551,6 @@ class MainBibleActivity : CustomTitlebarActivityBase() {
 
     override fun onPause() {
         CommonUtils.windowControl.windowRepository.saveIntoDb(false)
-        if(CommonUtils.isGoogleDriveSyncEnabled) {
-            syncJob?.cancel(StopSync())
-            syncJob = null
-            lifecycleScope.launch { DeviceSynchronize.start() }
-        }
         fullScreen = false;
         if(CommonUtils.showCalculator) {
             (window.decorView as ViewGroup).removeView(binding.root)
@@ -1268,7 +1263,9 @@ class MainBibleActivity : CustomTitlebarActivityBase() {
      */
     override fun onRestart() {
         super.onRestart()
-        startSync()
+        if(syncJob != null) {
+            startSync()
+        }
         if (mWholeAppWasInBackground) {
             mWholeAppWasInBackground = false
             refreshIfNightModeChange()
@@ -1286,7 +1283,11 @@ class MainBibleActivity : CustomTitlebarActivityBase() {
                 if(now - lastSynchronized > syncInterval) {
                     synchronize(true)
                 }
-                syncJob = lifecycleScope.launch { periodicSync() }
+                if(syncJob != null) {
+                    Log.e(TAG, "syncJob already exists")
+                } else {
+                    syncJob = lifecycleScope.launch { periodicSync() }
+                }
             }
         }
     }
@@ -1338,6 +1339,11 @@ class MainBibleActivity : CustomTitlebarActivityBase() {
     fun onEvent(event: AppToBackgroundEvent) {
         if (event.isMovedToBackground) {
             mWholeAppWasInBackground = true
+            syncJob?.cancel(StopSync())
+            syncJob = null
+            if(CommonUtils.isGoogleDriveSyncEnabled) {
+                lifecycleScope.launch { DeviceSynchronize.start() }
+            }
         }
         else {
             updateActions()