Refactoring and cleanup

feature/top10
Laurent 7 years ago
parent 0ada714e1a
commit a8d3e85021
  1. 5
      app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
  2. 8
      app/src/main/java/net/pokeranalytics/android/ui/view/SessionRowView.kt

@ -486,7 +486,7 @@ open class Session : RealmObject(), Savable, Editable, StaticRowRepresentableDat
* Return the game title * Return the game title
* Example: NL Holdem * Example: NL Holdem
*/ */
fun getGameTitle(): String { fun getFormattedGame(): String {
var gameTitle = "" var gameTitle = ""
limit?.let { limit?.let {
if (it >= 0 && it < Limit.values().size) { if (it >= 0 && it < Limit.values().size) {
@ -523,7 +523,6 @@ open class Session : RealmObject(), Savable, Editable, StaticRowRepresentableDat
SessionSetManager.removeFromTimeline(this) SessionSetManager.removeFromTimeline(this)
// cleanup unnecessary related objects // cleanup unnecessary related objects
// set.deleteFromRealm()
this.result?.deleteFromRealm() this.result?.deleteFromRealm()
this.computableResults?.deleteAllFromRealm() this.computableResults?.deleteAllFromRealm()
@ -622,7 +621,7 @@ open class Session : RealmObject(), Savable, Editable, StaticRowRepresentableDat
SessionRow.CASHED_OUT, SessionRow.PRIZE, SessionRow.NET_RESULT -> this.result?.cashout?.toCurrency(CurrencyUtils.getCurrency(bankroll)) ?: NULL_TEXT SessionRow.CASHED_OUT, SessionRow.PRIZE, SessionRow.NET_RESULT -> this.result?.cashout?.toCurrency(CurrencyUtils.getCurrency(bankroll)) ?: NULL_TEXT
SessionRow.COMMENT -> if (this.comment.isNotEmpty()) this.comment else NULL_TEXT SessionRow.COMMENT -> if (this.comment.isNotEmpty()) this.comment else NULL_TEXT
SessionRow.END_DATE -> this.endDate?.shortDateTime() ?: NULL_TEXT SessionRow.END_DATE -> this.endDate?.shortDateTime() ?: NULL_TEXT
SessionRow.GAME -> getGameTitle() SessionRow.GAME -> getFormattedGame()
SessionRow.INITIAL_BUY_IN -> tournamentEntryFee?.toCurrency(CurrencyUtils.getCurrency(bankroll)) ?: NULL_TEXT SessionRow.INITIAL_BUY_IN -> tournamentEntryFee?.toCurrency(CurrencyUtils.getCurrency(bankroll)) ?: NULL_TEXT
SessionRow.LOCATION -> location?.name ?: NULL_TEXT SessionRow.LOCATION -> location?.name ?: NULL_TEXT
SessionRow.PLAYERS -> tournamentNumberOfPlayers?.toString() ?: NULL_TEXT SessionRow.PLAYERS -> tournamentNumberOfPlayers?.toString() ?: NULL_TEXT

@ -11,6 +11,7 @@ import net.pokeranalytics.android.R
import net.pokeranalytics.android.calculus.ComputedStat import net.pokeranalytics.android.calculus.ComputedStat
import net.pokeranalytics.android.calculus.Stat import net.pokeranalytics.android.calculus.Stat
import net.pokeranalytics.android.model.TableSize import net.pokeranalytics.android.model.TableSize
import net.pokeranalytics.android.model.TournamentType
import net.pokeranalytics.android.model.extensions.SessionState import net.pokeranalytics.android.model.extensions.SessionState
import net.pokeranalytics.android.model.extensions.getState import net.pokeranalytics.android.model.extensions.getState
import net.pokeranalytics.android.model.realm.Session import net.pokeranalytics.android.model.realm.Session
@ -73,7 +74,10 @@ class SessionRowView : FrameLayout {
session.tournamentName?.let { session.tournamentName?.let {
parameters.add(it.name) parameters.add(it.name)
} ?: run { } ?: run {
parameters.add(session.getGameTitle()) parameters.add(session.getFormattedGame())
session.tournamentType?.let { type ->
parameters.add(TournamentType.values()[type].localizedTitle(context))
}
} }
if (parameters.size == 0) { if (parameters.size == 0) {
@ -84,7 +88,7 @@ class SessionRowView : FrameLayout {
parameters.add(session.getBlinds()) parameters.add(session.getBlinds())
} }
session.game?.let { session.game?.let {
parameters.add(session.getGameTitle()) parameters.add(session.getFormattedGame())
} }
if (parameters.size == 0) { if (parameters.size == 0) {

Loading…
Cancel
Save