fix: Address PR comments-2

openedx · May 22, 2024 · f6ecde0 · f6ecde0
1 parent d9c54eb
commit f6ecde0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/app/src/main/java/org/openedx/app/AppActivity.kt b/app/src/main/java/org/openedx/app/AppActivity.kt
@@ -136,10 +136,6 @@ class AppActivity : AppCompatActivity(), InsetHolder, WindowSizeHolder {
             }
         }
 
-        if (viewModel.canResetAppDirectory){
-            viewModel.resetAppDirectory(this)
-        }
-
         viewModel.logoutUser.observe(this) {
             profileRouter.restartApp(supportFragmentManager, viewModel.isLogistrationEnabled)
         }

diff --git a/app/src/main/java/org/openedx/app/AppViewModel.kt b/app/src/main/java/org/openedx/app/AppViewModel.kt
@@ -34,11 +34,14 @@ class AppViewModel(
     private var logoutHandledAt: Long = 0
 
     val isBranchEnabled get() = config.getBranchConfig().enabled
-    val canResetAppDirectory = preferencesManager.canResetAppDirectory
+    private val canResetAppDirectory get() = preferencesManager.canResetAppDirectory
 
     override fun onCreate(owner: LifecycleOwner) {
         super.onCreate(owner)
         setUserId()
+        if(canResetAppDirectory) {
+            resetAppDirectory(owner as Context)
+        }
         viewModelScope.launch {
             notifier.notifier.collect { event ->
                 if (event is LogoutEvent && System.currentTimeMillis() - logoutHandledAt > 5000) {
@@ -63,7 +66,7 @@ class AppViewModel(
         )
     }
 
-    fun resetAppDirectory(context: Context) {
+    private fun resetAppDirectory(context: Context) {
         FileUtil.deleteOldAppDirectory(context)
         preferencesManager.canResetAppDirectory = false
     }