Adds row refresh of automatically set actions

app/src/main/java/net/pokeranalytics/android/model/handhistory/HHBuilder.kt

@@ -5,8 +5,8 @@ import net.pokeranalytics.android.exceptions.PAIllegalStateException
 import net.pokeranalytics.android.model.realm.handhistory.Action
 import net.pokeranalytics.android.model.realm.handhistory.HandHistory
 import net.pokeranalytics.android.ui.modules.handhistory.HandRowType
-import net.pokeranalytics.android.ui.view.RowRepresentable
 import net.pokeranalytics.android.ui.modules.handhistory.views.StreetCardHeader
+import net.pokeranalytics.android.ui.view.RowRepresentable
 import net.pokeranalytics.android.ui.view.rowrepresentable.CustomizableRowRepresentable
 import timber.log.Timber
 import java.util.*
@@ -146,8 +146,9 @@ class HHBuilder {
 	 * If the user changes the current action,
 	 * for convenience we remove all the following actions to avoid managing complex cases
 	 * Also calculates the player effective amounts in proper cases
+	 * Returns the list of modified action indexes that are not the action at [index]
 	 */
-	fun selectAction(index: Int, actionType: Action.Type) : Boolean {
+	fun selectAction(index: Int, actionType: Action.Type) : List<Int> {
 
 		Timber.d(">>> Sets $actionType at index: $index")
 
@@ -160,8 +161,9 @@ class HHBuilder {
 		dropNextActionsIfNecessary(index)
 
 		// Automatically sets action for the previous empty actions
+		val modifiedActions = mutableListOf<ComputedAction>()
 		getPreviousEmptyActions(index).forEach {
-
+			modifiedActions.add(it)
 			val lastSignificant = getLastSignificantAction(index)
 			if (lastSignificant != null) {
 				it.action.type = Action.Type.FOLD
@@ -188,7 +190,7 @@ class HHBuilder {
 			else -> {}
 		}
 
-		return computedAction.requiresAmount
+		return modifiedActions.map { this.currentRowRepresentables.indexOf(it) }
 	}
 
 	private fun getPreviousEmptyActions(index: Int) : List<ComputedAction> {

app/src/main/java/net/pokeranalytics/android/model/realm/handhistory/Action.kt

@@ -28,7 +28,7 @@ open class Action : RealmObject() {
 		val isSignificant: Boolean
 			get() {
 				return when (this) {
-					POST_BB, STRADDLE, BET, RAISE, BET_ALLIN, RAISE_ALLIN -> true
+					POST_SB, POST_BB, STRADDLE, BET, RAISE, BET_ALLIN, RAISE_ALLIN -> true
 					UNDEFINED_ALLIN -> throw PAIllegalStateException("Can't ask for UNDEFINED_ALLIN")
 					else -> false
 				}

app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/HandHistoryFragment.kt

@@ -246,8 +246,10 @@ class HandHistoryFragment : RealmFragment(), RowRepresentableDataSource, RowRepr
 
 	override fun actionSelected(action: Action.Type) {
 		Timber.d(">>> action $action selected")
-		this.model.actionSelected(action)
+		val indexesToRefresh = this.model.actionSelected(action)
-//		this.handHistoryAdapter.notifyDataSetChanged()
+
+		indexesToRefresh.forEach { this.handHistoryAdapter.notifyItemChanged(it) }
+
 		this.findNextActionToEdit()
 	}
 

app/src/main/java/net/pokeranalytics/android/ui/modules/handhistory/HandHistoryViewModel.kt

@@ -49,8 +49,10 @@ class HandHistoryViewModel : ViewModel() {
 			} ?: throw PAIllegalStateException("No selection")
 		}
 
-	fun actionSelected(action: Action.Type) {
+	fun actionSelected(action: Action.Type) : List<Int> {
-		builder.value?.selectAction(this.actionIndexForSelection, action)
+		builder.value?.let {
+			return it.selectAction(this.actionIndexForSelection, action)
+		} ?: throw PAIllegalStateException("Builder not found")
 	}
 
 	fun amountValidated() {