From c44b3f3d2d23bb8915d0a32adda8f5b1bf5b1ced Mon Sep 17 00:00:00 2001 From: Laurent Date: Wed, 3 Jul 2019 16:07:20 +0200 Subject: [PATCH] Fixes an issue where launching 2 reports was possible making the UI blocked --- .../ui/fragment/report/ComposableTableReportFragment.kt | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/report/ComposableTableReportFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/report/ComposableTableReportFragment.kt index aca25041..f550272f 100644 --- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/report/ComposableTableReportFragment.kt +++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/report/ComposableTableReportFragment.kt @@ -42,6 +42,8 @@ open class ComposableTableReportFragment : RealmFragment(), StaticRowRepresentab private var statsAdapter: RowRepresentableAdapter? = null var report: Report? = null + private var hasComputationInProgress: Boolean = false + companion object { /** @@ -168,6 +170,11 @@ open class ComposableTableReportFragment : RealmFragment(), StaticRowRepresentab override fun onRowSelected(position: Int, row: RowRepresentable, fromAction: Boolean) { + if (this.hasComputationInProgress) { + return + } + this.hasComputationInProgress = true + val cr = getRealm().where(ComputableResult::class.java).findAll() if (cr.size < 2) { Toast.makeText(context, R.string.less_then_2_values_for_display, Toast.LENGTH_LONG).show() @@ -223,6 +230,7 @@ open class ComposableTableReportFragment : RealmFragment(), StaticRowRepresentab throw PAIllegalStateException("Report should never be null here") } } + hasComputationInProgress = false } }