From 808006e0744a83132592d7d1a707f7269e0ddf9b Mon Sep 17 00:00:00 2001
From: Laurent <laurent@staxriver.com>
Date: Thu, 30 Jan 2020 18:20:14 +0100
Subject: [PATCH] Disable components when necessary

---
 .../modules/handhistory/HandHistoryAdapter.kt | 37 +++++++------------
 1 file changed, 14 insertions(+), 23 deletions(-)

diff --git a/app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/HandHistoryAdapter.kt b/app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/HandHistoryAdapter.kt
index af276750..3bbd16a6 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/HandHistoryAdapter.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/HandHistoryAdapter.kt
@@ -110,19 +110,11 @@ class HandHistoryAdapter(
 	inner class RowHandAction(itemView: View) : RowHandHolder(itemView),
 		BindableHolder {
 
-//		private var listener = TextListener()
-		private var actionCanBeEdited = true
-		private var amountCanBeEdited = true
-
 		init {
 			// Action
 			itemView.findViewById<Button>(R.id.actionButton)?.let { actionButton ->
-
 				actionButton.setOnClickListener {
-
-					if (this.actionCanBeEdited) {
-						buttonEdited(actionButton, true, HHKeyboard.ACTION.ordinal)
-					}
+					buttonEdited(actionButton, true, HHKeyboard.ACTION.ordinal)
 				}
 			}
 
@@ -133,18 +125,15 @@ class HandHistoryAdapter(
 				amountEditText.isFocusableInTouchMode = true
 
 				amountEditText.setOnTouchListener { _, event ->
-//					Timber.d("touch action = ${event.action}")
-					if (this.amountCanBeEdited) {
 
-						if (event.action == MotionEvent.ACTION_UP) {
-							// Both are required, otherwise requestFocus() fails
-							amountEditText.isFocusable = true
-							amountEditText.isFocusableInTouchMode = true
+					if (event.action == MotionEvent.ACTION_UP) {
+						// Both are required, otherwise requestFocus() fails
+						amountEditText.isFocusable = true
+						amountEditText.isFocusableInTouchMode = true
 
-							amountEditText.requestFocus()
+						amountEditText.requestFocus()
 
-							editTextSelected(amountEditText, true, HHKeyboard.AMOUNT.ordinal)
-						}
+						editTextSelected(amountEditText, true, HHKeyboard.AMOUNT.ordinal)
 					}
 					return@setOnTouchListener true
 				}
@@ -171,8 +160,6 @@ class HandHistoryAdapter(
 			this.currentPosition = position
 
 			val computedAction = row as ComputedAction
-			this.actionCanBeEdited = computedAction.actionTypeCanBeEdited
-			this.amountCanBeEdited = computedAction.amountCanBeEdited
 
 			// Position
 			itemView.findViewById<Button>(R.id.positionButton)?.let { button ->
@@ -181,14 +168,16 @@ class HandHistoryAdapter(
 
 			// Action
 			itemView.findViewById<Button>(R.id.actionButton)?.let { actionButton ->
+				actionButton.isEnabled = computedAction.actionTypeCanBeEdited
+
 				val tag = HHKeyboard.ACTION.ordinal
 
 				val selected = adapter.dataSource.isSelected(position, row, tag)
 //				Timber.d("Action at $position is selected: $selected")
 				actionButton.backgroundTintList = ColorStateList.valueOf(color(selected))
-				if (selected) {
-					(itemView.context as Activity).hideKeyboard()
-				}
+//				if (selected) {
+//					(itemView.context as Activity).hideKeyboard()
+//				}
 
 				computedAction.action.type?.resId?.let {
 					actionButton.setText(it)
@@ -200,6 +189,8 @@ class HandHistoryAdapter(
 
 			// Amount
 			itemView.findViewById<EditText>(R.id.amountEditText)?.let { amountEditText ->
+				amountEditText.isEnabled = computedAction.amountCanBeEdited
+
 				val tag = HHKeyboard.AMOUNT.ordinal
 
 				val selected = adapter.dataSource.isSelected(position, row, tag)