Merge pull request #979 from rahulsom/cleanup-kotlin

refactor: Remove getter/setter for revertedBy
rahulsom · Mar 3, 2023 · 194cec0 · 194cec0
2 parents b78508f + 4cce1e2
commit 194cec0
Showing 1 changed file with 5 additions and 13 deletions.
diff --git a/grooves-example-pushstyle/src/main/kotlin/grooves/example/push/Transaction.kt b/grooves-example-pushstyle/src/main/kotlin/grooves/example/push/Transaction.kt
@@ -10,28 +10,21 @@ sealed class Transaction(
     override val id: String,
     override var aggregate: Account?,
     override var timestamp: Date,
-    override var position: Long
-) :
-    BaseEvent<Account, String, Transaction> {
+    override var position: Long,
+    override var revertedBy: RevertEvent<Account, String, Transaction>? = null
+) : BaseEvent<Account, String, Transaction> {
 
     override fun getAggregateObservable(): Publisher<Account> =
         Flowable.fromIterable(listOfNotNull(aggregate))
 
-    override var revertedBy: RevertEvent<Account, String, Transaction>?
-        get() = revertedBy
-        set(value) {
-            revertedBy = value
-        }
-
     data class Deposit(
         override val id: String,
         override var aggregate: Account?,
         override var timestamp: Date,
         override var position: Long,
         val atmId: String,
         val amount: Long
-    ) :
-        Transaction(id, aggregate, timestamp, position)
+    ) : Transaction(id, aggregate, timestamp, position)
 
     data class Withdraw(
         override val id: String,
@@ -40,6 +33,5 @@ sealed class Transaction(
         override var position: Long,
         val atmId: String,
         val amount: Long
-    ) :
-        Transaction(id, aggregate, timestamp, position)
+    ) : Transaction(id, aggregate, timestamp, position)
 }