diff options
author | Allan Wang <me@allanwang.ca> | 2019-05-19 19:43:26 +0800 |
---|---|---|
committer | Allan Wang <me@allanwang.ca> | 2019-05-19 19:43:26 +0800 |
commit | 17a6a36279f20974e4db8d7355e747f4f59c1af1 (patch) | |
tree | cb0dd25bcd7dd5dd56219e1b1325b92045e2dc63 /app/src/main | |
parent | 51588778a3fc83a69ef766efb79c7458ec47a039 (diff) | |
download | frost-17a6a36279f20974e4db8d7355e747f4f59c1af1.tar.gz frost-17a6a36279f20974e4db8d7355e747f4f59c1af1.tar.bz2 frost-17a6a36279f20974e4db8d7355e747f4f59c1af1.zip |
Restrict reportable image errorss
Diffstat (limited to 'app/src/main')
-rw-r--r-- | app/src/main/kotlin/com/pitchedapps/frost/activities/ImageActivity.kt | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/app/src/main/kotlin/com/pitchedapps/frost/activities/ImageActivity.kt b/app/src/main/kotlin/com/pitchedapps/frost/activities/ImageActivity.kt index f7dcf845..b94acb63 100644 --- a/app/src/main/kotlin/com/pitchedapps/frost/activities/ImageActivity.kt +++ b/app/src/main/kotlin/com/pitchedapps/frost/activities/ImageActivity.kt @@ -53,7 +53,6 @@ import com.pitchedapps.frost.services.LocalService import com.pitchedapps.frost.utils.ARG_COOKIE import com.pitchedapps.frost.utils.ARG_IMAGE_URL import com.pitchedapps.frost.utils.ARG_TEXT -import com.pitchedapps.frost.utils.L import com.pitchedapps.frost.utils.Prefs import com.pitchedapps.frost.utils.frostSnackbar import com.pitchedapps.frost.utils.frostUriFromFile @@ -71,6 +70,7 @@ import kotlinx.coroutines.async import kotlinx.coroutines.launch import kotlinx.coroutines.withContext import java.io.File +import java.io.FileNotFoundException import java.io.IOException import java.text.SimpleDateFormat import java.util.Date @@ -294,16 +294,18 @@ internal enum class FabStates( ) { ERROR(GoogleMaterial.Icon.gmd_error, Color.WHITE, Color.RED) { override fun onClick(activity: ImageActivity) { + val err = + activity.errorRef?.takeIf { it !is FileNotFoundException && it.message != "Image failed to decode using JPEG decoder" } + ?: return activity.materialDialogThemed { title(R.string.kau_error) content(R.string.bad_image_overlay) positiveText(R.string.kau_yes) onPositive { _, _ -> - if (activity.errorRef != null) - L.e(activity.errorRef) { "ImageActivity error report" } activity.sendFrostEmail(R.string.debug_image_link_subject) { addItem("Url", activity.imageUrl) - addItem("Message", activity.errorRef?.message ?: "Null") + addItem("Type", err.javaClass.name) + addItem("Message", err.message ?: "Null") } } negativeText(R.string.kau_no) @@ -356,11 +358,6 @@ internal enum class FabStates( fab.show() } }) - -// fab.fadeScaleTransition { -// setIcon(iicon, color = iconColor) -//// backgroundTintList = ColorStateList.valueOf(tint) -// } } } |