fixed incorrect error code by unifying error reporters
This commit is contained in:
parent
44351912b4
commit
efa9ec7bc6
|
@ -30,7 +30,9 @@ class AppInstallerService: Service() {
|
|||
}
|
||||
else -> {
|
||||
sendCloseDialog(this)
|
||||
sendFailure(intent.getIntExtra(PackageInstaller.EXTRA_STATUS, -999), intent.getStringExtra(PackageInstaller.EXTRA_STATUS_MESSAGE),this)
|
||||
intent.getStringExtra(PackageInstaller.EXTRA_STATUS_MESSAGE)?.let {
|
||||
sendFailure(it,this)
|
||||
}
|
||||
}
|
||||
}
|
||||
stopSelf()
|
||||
|
|
|
@ -2,7 +2,6 @@ package com.vanced.manager.utils
|
|||
|
||||
import android.content.Context
|
||||
import android.content.Intent
|
||||
import android.content.pm.PackageInstaller
|
||||
import android.graphics.Color
|
||||
import android.text.Spannable
|
||||
import android.text.SpannableString
|
||||
|
@ -56,28 +55,12 @@ object AppUtils: CoroutineScope by CoroutineScope(Dispatchers.IO) {
|
|||
}
|
||||
}
|
||||
|
||||
fun sendFailure(status: Int, fullError: String?, context: Context): Job {
|
||||
//Delay error broadcast until activity (and fragment) get back to the screen
|
||||
return launch {
|
||||
delay(700)
|
||||
val intent = Intent(HomeFragment.INSTALL_FAILED)
|
||||
intent.putExtra("errorMsg", getErrorMessage(status, context))
|
||||
intent.putExtra("fullErrorMsg", fullError)
|
||||
LocalBroadcastManager.getInstance(context).sendBroadcast(intent)
|
||||
}
|
||||
}
|
||||
|
||||
fun sendFailure(error: MutableList<String>, context: Context): Job {
|
||||
return launch {
|
||||
delay(700)
|
||||
val intent = Intent(HomeFragment.INSTALL_FAILED)
|
||||
intent.putExtra("errorMsg", getErrorMessage(error.joinToString(), context))
|
||||
intent.putExtra("fullErrorMsg", error.joinToString(" "))
|
||||
LocalBroadcastManager.getInstance(context).sendBroadcast(intent)
|
||||
}
|
||||
fun sendFailure(error: MutableList<String>, context: Context) {
|
||||
sendFailure(error.joinToString(), context)
|
||||
}
|
||||
|
||||
fun sendFailure(error: String, context: Context): Job {
|
||||
//Delay error broadcast until activity (and fragment) get back to the screen
|
||||
return launch {
|
||||
delay(700)
|
||||
val intent = Intent(HomeFragment.INSTALL_FAILED)
|
||||
|
@ -123,6 +106,7 @@ object AppUtils: CoroutineScope by CoroutineScope(Dispatchers.IO) {
|
|||
}
|
||||
|
||||
private fun getErrorMessage(status: String, context: Context): String {
|
||||
log("VMInstall", status)
|
||||
return when {
|
||||
status.contains("INSTALL_FAILED_ABORTED") -> context.getString(R.string.installation_aborted)
|
||||
status.contains("INSTALL_FAILED_ALREADY_EXISTS") -> context.getString(R.string.installation_conflict)
|
||||
|
@ -144,20 +128,4 @@ object AppUtils: CoroutineScope by CoroutineScope(Dispatchers.IO) {
|
|||
context.getString(R.string.installation_failed)
|
||||
}
|
||||
}
|
||||
|
||||
private fun getErrorMessage(status: Int, context: Context): String {
|
||||
return when (status) {
|
||||
PackageInstaller.STATUS_FAILURE_ABORTED -> context.getString(R.string.installation_aborted)
|
||||
PackageInstaller.STATUS_FAILURE_BLOCKED -> context.getString(R.string.installation_blocked)
|
||||
PackageInstaller.STATUS_FAILURE_CONFLICT -> context.getString(R.string.installation_conflict)
|
||||
PackageInstaller.STATUS_FAILURE_INCOMPATIBLE -> context.getString(R.string.installation_incompatible)
|
||||
PackageInstaller.STATUS_FAILURE_INVALID -> context.getString(R.string.installation_invalid)
|
||||
PackageInstaller.STATUS_FAILURE_STORAGE -> context.getString(R.string.installation_storage)
|
||||
else ->
|
||||
if (isMiui())
|
||||
context.getString(R.string.installation_miui)
|
||||
else
|
||||
context.getString(R.string.installation_failed)
|
||||
}
|
||||
}
|
||||
}
|
Loading…
Reference in New Issue