courses: smoother step progress saving (fixes #8992) (#8958)

Author: dogi

app/build.gradle

@@ -9,8 +9,8 @@ android {
         applicationId "org.ole.planet.myplanet"
         minSdk = 26
         targetSdk = 36
-        versionCode = 3708
-        versionName = "0.37.8"
+        versionCode = 3709
+        versionName = "0.37.9"
         ndkVersion = '26.3.11579264'
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
         vectorDrawables.useSupportLibrary = true

app/src/main/java/org/ole/planet/myplanet/ui/courses/CourseStepFragment.kt

@@ -11,6 +11,7 @@ import androidx.fragment.app.Fragment
 import androidx.lifecycle.lifecycleScope
 import java.util.Date
 import java.util.UUID
+import kotlinx.coroutines.Job
 import kotlinx.coroutines.launch
 import org.ole.planet.myplanet.MainApplication
 import org.ole.planet.myplanet.R
@@ -39,6 +40,7 @@ class CourseStepFragment : BaseContainerFragment(), ImageCaptureCallback {
     private lateinit var stepSurvey: List<RealmStepExam>
     var user: RealmUserModel? = null
     private var stepNumber = 0
+    private var saveInProgress: Job? = null
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
         if (arguments != null) {
@@ -55,12 +57,11 @@ class CourseStepFragment : BaseContainerFragment(), ImageCaptureCallback {
         return fragmentCourseStepBinding.root
     }
 
-    private fun saveCourseProgress() {
-        databaseService.withRealm { realm ->
-            if (!realm.isInTransaction) realm.beginTransaction()
+    private suspend fun saveCourseProgress(userId: String?, planetCode: String?, parentCode: String?) {
+        databaseService.executeTransactionAsync { realm ->
             var courseProgress = realm.where(RealmCourseProgress::class.java)
                 .equalTo("courseId", step.courseId)
-                .equalTo("userId", user?.id)
+                .equalTo("userId", userId)
                 .equalTo("stepNum", stepNumber)
                 .findFirst()
             if (courseProgress == null) {
@@ -72,12 +73,22 @@ class CourseStepFragment : BaseContainerFragment(), ImageCaptureCallback {
             if (stepExams.isEmpty()) {
                 courseProgress?.passed = true
             }
-            courseProgress?.createdOn = user?.planetCode
+            courseProgress?.createdOn = planetCode
             courseProgress?.updatedDate = Date().time
-            courseProgress?.parentCode = user?.parentCode
-            courseProgress?.userId = user?.id
-            realm.commitTransaction()
+            courseProgress?.parentCode = parentCode
+            courseProgress?.userId = userId
+        }
+    }
+
+    private fun launchSaveCourseProgress() {
+        if (saveInProgress?.isActive == true) return
+        val userId = user?.id
+        val planetCode = user?.planetCode
+        val parentCode = user?.parentCode
+        saveInProgress = lifecycleScope.launch {
+            saveCourseProgress(userId, planetCode, parentCode)
         }
+        saveInProgress?.invokeOnCompletion { saveInProgress = null }
     }
     override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
         super.onViewCreated(view, savedInstanceState)
@@ -132,7 +143,7 @@ class CourseStepFragment : BaseContainerFragment(), ImageCaptureCallback {
             }
         }
         if (isVisible && userHasCourse) {
-            saveCourseProgress()
+            launchSaveCourseProgress()
         }
     }
 
@@ -171,7 +182,7 @@ class CourseStepFragment : BaseContainerFragment(), ImageCaptureCallback {
                     isMyCourse(user?.id, step.courseId, realm)
                 }
                 if (userHasCourse) {
-                    saveCourseProgress()
+                    launchSaveCourseProgress()
                 }
             }
         } catch (e: Exception) {