Adds compacting for text export

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

@@ -3,11 +3,35 @@ package net.pokeranalytics.android.model.realm.handhistory
 import io.realm.RealmObject
 import net.pokeranalytics.android.R
 import net.pokeranalytics.android.exceptions.PAIllegalStateException
+import net.pokeranalytics.android.model.handhistory.Position
 import net.pokeranalytics.android.model.handhistory.Street
+import net.pokeranalytics.android.ui.modules.handhistory.model.ActionReadRow
 import net.pokeranalytics.android.ui.view.RowRepresentable
 import net.pokeranalytics.android.ui.view.RowViewType
 import net.pokeranalytics.android.util.extensions.formatted
 
+
+/***
+ * An extension to transform a list of ComputedAction into
+ * a more compact and read-friendly list of ActionReadRow
+ */
+fun List<Action>.compact(positions: LinkedHashSet<Position>): List<ActionReadRow> {
+	val rows = mutableListOf<ActionReadRow>()
+	this.forEach {
+		if (it.type == Action.Type.FOLD && rows.lastOrNull()?.action == Action.Type.FOLD) {
+			rows.lastOrNull()?.positions?.add(positions.elementAt(it.position))
+		} else {
+			rows.add(it.toReadRow(positions))
+		}
+	}
+	return rows
+}
+
+fun Action.toReadRow(positions: LinkedHashSet<Position>): ActionReadRow {
+	val pos = positions.elementAt(this.position)
+	return ActionReadRow(mutableListOf(pos), this.type, this.amount)
+}
+
 open class Action : RealmObject() {
 
 	enum class Type(override var resId: Int) : RowRepresentable {

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

@@ -39,8 +39,8 @@ fun List<Card>.formatted(context: Context) : CharSequence? {
 open class Card : RealmObject() {
 
 	companion object {
-		fun newInstance(realm: Realm, value: Int? = null, suit: Suit? = null, index: Int = 0) : Card {
+		fun newInstance(realm: Realm?, value: Int? = null, suit: Suit? = null, index: Int = 0) : Card {
-			val card = realm.createObject(Card::class.java)
+			val card = if (realm != null) realm.createObject(Card::class.java) else Card()
 			value?.let { card.value = it }
 			suit?.let { card.suit = it}
 			card.index = index

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

@@ -17,6 +17,7 @@ import net.pokeranalytics.android.model.handhistory.Street
 import net.pokeranalytics.android.model.interfaces.Identifiable
 import net.pokeranalytics.android.model.interfaces.TimeFilterable
 import net.pokeranalytics.android.model.realm.Session
+import net.pokeranalytics.android.ui.modules.handhistory.model.ActionReadRow
 import net.pokeranalytics.android.ui.modules.handhistory.model.CardHolder
 import net.pokeranalytics.android.ui.view.RowRepresentable
 import net.pokeranalytics.android.util.extensions.addLineReturn
@@ -207,7 +208,13 @@ open class HandHistory : RealmObject(), RowRepresentable, Identifiable, Filterab
 	}
 
 	fun createPlayerSetup(positionIndex: Int): PlayerSetup {
-		val playerSetup = this.realm.createObject(PlayerSetup::class.java)
+
+		val playerSetup = if (this.realm != null) {
+			this.realm.createObject(PlayerSetup::class.java) }
+		else {
+			PlayerSetup()
+		}
+
 		playerSetup.position = positionIndex
 		this.playerSetups.add(playerSetup)
 		return playerSetup
@@ -262,7 +269,7 @@ open class HandHistory : RealmObject(), RowRepresentable, Identifiable, Filterab
 
 			string = string.addLineReturn(2)
 
-			val streetActions = sortedActions.filter { it.street == street }
+			val streetActions = sortedActions.filter { it.street == street }.compact(positions)
 			if (streetActions.isNotEmpty()) {
 
 				val streetItems = mutableListOf<CharSequence>(context.getString(street.resId))
@@ -280,7 +287,7 @@ open class HandHistory : RealmObject(), RowRepresentable, Identifiable, Filterab
 				string = string.addLineReturn()
 
 				streetActions.forEach { action ->
-					string = string.plus(localizedAction(action, positions, context))
+					string = string.plus(localizedAction(action, context))
 					string = string.addLineReturn()
 				}
 			}
@@ -299,12 +306,13 @@ open class HandHistory : RealmObject(), RowRepresentable, Identifiable, Filterab
 		return playerItems.joinToString(" ")
 	}
 
-	private fun localizedAction(action: Action, positions: LinkedHashSet<Position>, context: Context): String {
+	private fun localizedAction(actionReadRow: ActionReadRow, context: Context): String {
-		val actionItems = mutableListOf(positions.elementAt(action.position).value)
+		val formattedPositions = actionReadRow.positions.map { it.value }.joinToString(", ")
-		action.type?.let { type ->
+		val actionItems = mutableListOf(formattedPositions)
+		actionReadRow.action?.let { type ->
 			actionItems.add(context.getString(type.resId))
 		}
-		action.amount?.let { amount ->
+		actionReadRow.amount?.let { amount ->
 			actionItems.add(amount.formatted())
 		}
 		return actionItems.joinToString(" ")