From 0f1b7f2602b947d39d9fc1f81be1cd6b8a1bf3a7 Mon Sep 17 00:00:00 2001 From: Aayush Gupta Date: Mon, 21 Oct 2024 10:02:43 +0530 Subject: [PATCH] UpdateWorker: Sort the updates before caching in database Signed-off-by: Aayush Gupta --- .../java/com/aurora/store/data/work/UpdateWorker.kt | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/app/src/main/java/com/aurora/store/data/work/UpdateWorker.kt b/app/src/main/java/com/aurora/store/data/work/UpdateWorker.kt index 0faee1461..8061a6d50 100644 --- a/app/src/main/java/com/aurora/store/data/work/UpdateWorker.kt +++ b/app/src/main/java/com/aurora/store/data/work/UpdateWorker.kt @@ -35,6 +35,7 @@ import dagger.assisted.Assisted import dagger.assisted.AssistedInject import kotlinx.coroutines.Dispatchers import kotlinx.coroutines.withContext +import java.util.Locale /** * A worker to check for updates for installed apps based on saved authentication data, @@ -91,7 +92,10 @@ class UpdateWorker @AssistedInject constructor( } try { - val updates = checkUpdates() + val updates = checkUpdates().also { + // Cache the updates into the database + updateDao.insertUpdates(it) + } if (updates.isEmpty()) { Log.i(TAG, "No updates found!") @@ -158,10 +162,8 @@ class UpdateWorker @AssistedInject constructor( if (canSelfUpdate) getSelfUpdate()?.let { updates.add(it) } - return@withContext updates.map { Update.fromApp(appContext, it) }.also { - // Cache the updates into the database - updateDao.insertUpdates(it) - } + return@withContext updates.map { Update.fromApp(appContext, it) } + .sortedBy { it.displayName.lowercase(Locale.getDefault()) } } }