From 39ce70f9641f51f4f779c42e63eaf8d81343d0cb Mon Sep 17 00:00:00 2001 From: Kirill Kamakin Date: Sat, 5 Nov 2022 11:08:12 +0100 Subject: [PATCH] Remove unnecessary method --- .../mealient/ui/recipes/RecipesFragment.kt | 19 ++++++++----------- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/app/src/main/java/gq/kirmanak/mealient/ui/recipes/RecipesFragment.kt b/app/src/main/java/gq/kirmanak/mealient/ui/recipes/RecipesFragment.kt index 50342d0..3260b53 100644 --- a/app/src/main/java/gq/kirmanak/mealient/ui/recipes/RecipesFragment.kt +++ b/app/src/main/java/gq/kirmanak/mealient/ui/recipes/RecipesFragment.kt @@ -79,7 +79,14 @@ class RecipesFragment : Fragment(R.layout.fragment_recipes) { binding.refresher.isRefreshing = false } - recipesAdapter.observeLoadStateChanges() + collectWhenViewResumed(recipesAdapter.appendPaginationEnd()) { + logger.v { "onPaginationEnd() called" } + showLongToast(R.string.fragment_recipes_last_page_loaded_toast) + } + + collectWhenViewResumed(recipesAdapter.refreshErrors()) { + onLoadFailure(it) + } collectWhenViewResumed(binding.refresher.refreshRequestFlow(logger)) { logger.v { "setupRecipeAdapter: received refresh request" } @@ -96,16 +103,6 @@ class RecipesFragment : Fragment(R.layout.fragment_recipes) { } } - private fun PagingDataAdapter.observeLoadStateChanges() { - collectWhenViewResumed(appendPaginationEnd()) { onPaginationEnd() } - collectWhenViewResumed(refreshErrors()) { onLoadFailure(it) } - } - - private fun onPaginationEnd() { - logger.v { "onPaginationEnd() called" } - showLongToast(R.string.fragment_recipes_last_page_loaded_toast) - } - private fun onLoadFailure(error: Throwable) { logger.w(error) { "onLoadFailure() called" } val reason = error.toLoadErrorReasonText()?.let { getString(it) }