diff --git a/androidApp/src/main/java/bou/amine/apps/readerforselfossv2/android/HomeActivity.kt b/androidApp/src/main/java/bou/amine/apps/readerforselfossv2/android/HomeActivity.kt index 4dea1f9..50f212c 100644 --- a/androidApp/src/main/java/bou/amine/apps/readerforselfossv2/android/HomeActivity.kt +++ b/androidApp/src/main/java/bou/amine/apps/readerforselfossv2/android/HomeActivity.kt @@ -143,11 +143,11 @@ class HomeActivity : AppCompatActivity(), SearchView.OnQueryTextListener, DIAwar handleSwipeRefreshLayout() - getElementsAccordingToTab() - - CoroutineScope(Dispatchers.Main).launch { - repository.tryToCacheItemsAndGetNewOnes() + if (appSettingsService.isItemCachingEnabled()) { + CoroutineScope(Dispatchers.Main).launch { + repository.tryToCacheItemsAndGetNewOnes() + } } } diff --git a/androidApp/src/main/res/layout/activity_add_source.xml b/androidApp/src/main/res/layout/activity_add_source.xml index 7bc31e1..9998207 100644 --- a/androidApp/src/main/res/layout/activity_add_source.xml +++ b/androidApp/src/main/res/layout/activity_add_source.xml @@ -18,7 +18,7 @@ android:id="@+id/toolbar" android:layout_width="match_parent" android:layout_height="?attr/actionBarSize" - app:theme="@style/ToolBarStyle" + android:theme="@style/ToolBarStyle" app:popupTheme="?attr/toolbarPopupTheme" /> diff --git a/androidApp/src/main/res/layout/activity_home.xml b/androidApp/src/main/res/layout/activity_home.xml index 7a84128..f194740 100644 --- a/androidApp/src/main/res/layout/activity_home.xml +++ b/androidApp/src/main/res/layout/activity_home.xml @@ -32,7 +32,7 @@ android:id="@+id/toolBar" android:layout_width="match_parent" android:layout_height="?attr/actionBarSize" - app:theme="@style/ToolBarStyle" + android:theme="@style/ToolBarStyle" app:popupTheme="?attr/toolbarPopupTheme" /> diff --git a/androidApp/src/main/res/layout/activity_image.xml b/androidApp/src/main/res/layout/activity_image.xml index d2d43ef..5703d1c 100644 --- a/androidApp/src/main/res/layout/activity_image.xml +++ b/androidApp/src/main/res/layout/activity_image.xml @@ -17,7 +17,7 @@ android:layout_width="match_parent" android:layout_height="?attr/actionBarSize" app:popupTheme="?attr/toolbarPopupTheme" - app:theme="@style/ToolBarStyle" /> + android:theme="@style/ToolBarStyle" /> diff --git a/androidApp/src/main/res/layout/activity_login.xml b/androidApp/src/main/res/layout/activity_login.xml index 4c37ea3..6fe8d2f 100644 --- a/androidApp/src/main/res/layout/activity_login.xml +++ b/androidApp/src/main/res/layout/activity_login.xml @@ -14,7 +14,7 @@ android:id="@+id/toolbar" android:layout_width="match_parent" android:layout_height="?attr/actionBarSize" - app:theme="@style/ToolBarStyle" + android:theme="@style/ToolBarStyle" app:popupTheme="?attr/toolbarPopupTheme" /> diff --git a/androidApp/src/main/res/layout/activity_reader.xml b/androidApp/src/main/res/layout/activity_reader.xml index b2cfa5a..a423f5c 100644 --- a/androidApp/src/main/res/layout/activity_reader.xml +++ b/androidApp/src/main/res/layout/activity_reader.xml @@ -18,7 +18,7 @@ android:layout_width="match_parent" android:layout_height="?attr/actionBarSize" app:popupTheme="?attr/toolbarPopupTheme" - app:theme="@style/ToolBarStyle" /> + android:theme="@style/ToolBarStyle" /> diff --git a/androidApp/src/main/res/layout/activity_sources.xml b/androidApp/src/main/res/layout/activity_sources.xml index b0dc30a..274b27d 100644 --- a/androidApp/src/main/res/layout/activity_sources.xml +++ b/androidApp/src/main/res/layout/activity_sources.xml @@ -14,7 +14,7 @@ android:id="@+id/toolbar" android:layout_width="match_parent" android:layout_height="?attr/actionBarSize" - app:theme="@style/ToolBarStyle" + android:theme="@style/ToolBarStyle" app:popupTheme="?attr/toolbarPopupTheme" /> diff --git a/androidApp/src/main/res/xml/pref_general.xml b/androidApp/src/main/res/xml/pref_general.xml index 6643cb7..7ea909a 100644 --- a/androidApp/src/main/res/xml/pref_general.xml +++ b/androidApp/src/main/res/xml/pref_general.xml @@ -6,7 +6,7 @@ { // TODO: Use the updatedSince parameter var fetchedItems: SelfossModel.StatusAndData> = SelfossModel.StatusAndData.error() + var fromDB = false if (isNetworkAvailable()) { fetchedItems = api.getItems( displayedItems.type, @@ -59,6 +60,7 @@ class Repository(private val api: SelfossApi, private val appSettingsService: Ap ) } else { if (appSettingsService.isItemCachingEnabled()) { + fromDB = true fetchedItems = SelfossModel.StatusAndData.succes( getDBItems().filter { displayedItems == ItemType.ALL || @@ -71,7 +73,9 @@ class Repository(private val api: SelfossApi, private val appSettingsService: Ap if (fetchedItems.success && fetchedItems.data != null) { items = ArrayList(fetchedItems.data!!) - sortItems() + if (fromDB) { + items.sortByDescending { dateUtils.parseDate(it.datetime) } + } } return items } @@ -92,7 +96,6 @@ class Repository(private val api: SelfossApi, private val appSettingsService: Ap if (fetchedItems.success && fetchedItems.data != null) { items.addAll(fetchedItems.data!!) - sortItems() } return items } @@ -118,10 +121,6 @@ class Repository(private val api: SelfossApi, private val appSettingsService: Ap } } - private fun sortItems() { - items.sortByDescending { dateUtils.parseDate(it.datetime) } - } - suspend fun reloadBadges(): Boolean { var success = false if (isNetworkAvailable()) { diff --git a/shared/src/commonMain/kotlin/bou/amine/apps/readerforselfossv2/service/AppSettingsService.kt b/shared/src/commonMain/kotlin/bou/amine/apps/readerforselfossv2/service/AppSettingsService.kt index 7582064..d681673 100644 --- a/shared/src/commonMain/kotlin/bou/amine/apps/readerforselfossv2/service/AppSettingsService.kt +++ b/shared/src/commonMain/kotlin/bou/amine/apps/readerforselfossv2/service/AppSettingsService.kt @@ -88,7 +88,7 @@ class AppSettingsService { } fun refreshItemsNumber() { - _itemsNumber = settings.getString("prefer_api_items_number", "200").toInt() + _itemsNumber = settings.getString("prefer_api_items_number", "20").toInt() } fun getApiTimeout(): Long {