From b624f1cbb599dc5225fd608f3eec3bd64a016002 Mon Sep 17 00:00:00 2001
From: MohitMali <futuretechnoworld.mohit@gmail.com>
Date: Fri, 4 Aug 2023 18:59:14 +0530
Subject: [PATCH] Hide storage selection dialog from OnlineLibraryFragment if
 user already configured the storage in settings

---
 .../kiwix/kiwixmobile/core/settings/CorePrefsFragment.kt   | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/core/src/main/java/org/kiwix/kiwixmobile/core/settings/CorePrefsFragment.kt b/core/src/main/java/org/kiwix/kiwixmobile/core/settings/CorePrefsFragment.kt
index 1d835b566..f84d31536 100644
--- a/core/src/main/java/org/kiwix/kiwixmobile/core/settings/CorePrefsFragment.kt
+++ b/core/src/main/java/org/kiwix/kiwixmobile/core/settings/CorePrefsFragment.kt
@@ -314,6 +314,7 @@ abstract class CorePrefsFragment :
         findPreference<Preference>(SharedPreferenceUtil.PREF_STORAGE)?.title =
           getString(R.string.internal_storage)
         sharedPreferenceUtil.putStoragePosition(INTERNAL_SELECT_POSITION)
+        setShowStorageOption()
       } else {
         if (sharedPreferenceUtil.isPlayStoreBuild) {
           setExternalStoragePath(storageDevice)
@@ -339,6 +340,7 @@ abstract class CorePrefsFragment :
     findPreference<Preference>(SharedPreferenceUtil.PREF_STORAGE)?.title =
       getString(R.string.external_storage)
     sharedPreferenceUtil?.putStoragePosition(EXTERNAL_SELECT_POSITION)
+    setShowStorageOption()
   }
 
   private fun selectFolder() {
@@ -362,11 +364,16 @@ abstract class CorePrefsFragment :
             findPreference<Preference>(SharedPreferenceUtil.PREF_STORAGE)?.title =
               getString(R.string.external_storage)
             sharedPreferenceUtil?.putStoragePosition(EXTERNAL_SELECT_POSITION)
+            setShowStorageOption()
           }
         }
       }
     }
 
+  private fun setShowStorageOption() {
+    sharedPreferenceUtil?.showStorageOption = false
+  }
+
   companion object {
     const val PREF_VERSION = "pref_version"
     const val PREF_CLEAR_ALL_HISTORY = "pref_clear_all_history"