mobi/seedvault
1
0
Fork 0
Code Pull requests Activity

Report total number of user apps when showing final notification

Browse source 
Before, we showed the number of apps we requested the backup for which in case of non-d2d may be much lower than the number of installed apps.

In the future we may decide to also include certain system apps in that count.
This commit is contained in:
Torsten Grote 2024-03-13 09:55:46 -03:00
parent 0b021e3b48
commit f7730d3034
No known key found for this signature in database
GPG key ID: 3E5F77D92CF891FF
1 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
app/src/main/java/com/stevesoltys/seedvault/ui/notification/NotificationBackupObserver.kt
View file

@ -10,6 +10,7 @@ import android.util.Log.isLoggable
import com.stevesoltys.seedvault.MAGIC_PACKAGE_MANAGER
import com.stevesoltys.seedvault.R
import com.stevesoltys.seedvault.metadata.MetadataManager
import com.stevesoltys.seedvault.transport.backup.PackageService
import com.stevesoltys.seedvault.worker.BackupRequester
import org.koin.core.component.KoinComponent
import org.koin.core.component.inject
@ -24,6 +25,7 @@ internal class NotificationBackupObserver(
private val nm: BackupNotificationManager by inject()
private val metadataManager: MetadataManager by inject()
private val packageService: PackageService by inject()
private var currentPackage: String? = null
private var numPackages: Int = 0
private var pmCounted: Boolean = false
@ -81,7 +83,13 @@ internal class NotificationBackupObserver(
val success = status == 0
val numBackedUp = if (success) metadataManager.getPackagesNumBackedUp() else null
val size = if (success) metadataManager.getPackagesBackupSize() else 0L
nm.onBackupFinished(success, numBackedUp, requestedPackages, size)
val total = try {
packageService.allUserPackages.size
} catch (e: Exception) {
Log.e(TAG, "Error getting number of all user packages: ", e)
requestedPackages
}
nm.onBackupFinished(success, numBackedUp, total, size)
}
}