Merge pull request #2914 from gradido/feat-previous-balance

feat(backend): previous balance in transaction
2025-12-13 07:45:54 +00:00 · 2023-04-05 12:31:45 +02:00 · 2023-04-05 12:31:45 +02:00 · fcb51f353f
commit fcb51f353f
parent d7c5de6780 248737c446
21 changed files with 139 additions and 62 deletions
--- a/backend/src/graphql/model/Transaction.ts
+++ b/backend/src/graphql/model/Transaction.ts
@ -47,6 +47,10 @@ export class Transaction {
    this.linkId = transaction.contribution
      ? transaction.contribution.contributionLinkId
      : transaction.transactionLinkId || null
+    this.previousBalance =
+      (transaction.previousTransaction &&
+        transaction.previousTransaction.balance.toDecimalPlaces(2, Decimal.ROUND_DOWN)) ||
+      new Decimal(0)
  }

  @Field(() => Int)
@ -70,6 +74,9 @@ export class Transaction {
  @Field(() => Date)
  balanceDate: Date

+  @Field(() => Decimal)
+  previousBalance: Decimal
+
  @Field(() => Decay)
  decay: Decay

--- a/backend/src/graphql/resolver/TransactionResolver.ts
+++ b/backend/src/graphql/resolver/TransactionResolver.ts
@ -275,6 +275,7 @@ export class TransactionResolver {
            firstDate || now,
            lastDate || now,
            self,
+            (userTransactions.length && userTransactions[0].balance) || new Decimal(0),
          ),
        )
        logger.debug(`transactions=${transactions}`)
@ -291,6 +292,15 @@ export class TransactionResolver {
    })
    logger.debug(`TransactionTypeId.CREATION: transactions=${transactions}`)

+    transactions.forEach((transaction: Transaction) => {
+      if (transaction.typeId !== TransactionTypeId.DECAY) {
+        const { balance, previousBalance, amount } = transaction
+        transaction.decay.decay = new Decimal(
+          Number(balance) - Number(amount) - Number(previousBalance),
+        ).toDecimalPlaces(2, Decimal.ROUND_HALF_UP)
+      }
+    })
+
    // Construct Result
    return new TransactionList(await balanceResolver.balance(context), transactions)
  }
--- a/backend/src/typeorm/repository/Transaction.ts
+++ b/backend/src/typeorm/repository/Transaction.ts
@ -2,7 +2,6 @@ import { EntityRepository, Repository } from '@dbTools/typeorm'
 import { Transaction } from '@entity/Transaction'

 import { Order } from '@enum/Order'
-import { TransactionTypeId } from '@enum/TransactionTypeId'

@EntityRepository(Transaction)
 export class TransactionRepository extends Repository<Transaction> {
@ -11,22 +10,15 @@ export class TransactionRepository extends Repository<Transaction> {
    limit: number,
    offset: number,
    order: Order,
-    onlyCreation?: boolean,
  ): Promise<[Transaction[], number]> {
    const query = this.createQueryBuilder('userTransaction')
      .leftJoinAndSelect(
-        'userTransaction.contribution',
-        'contribution',
-        'userTransaction.id = contribution.transactionId',
+        'userTransaction.previousTransaction',
+        'transaction',
+        'userTransaction.previous = transaction.id',
      )
      .where('userTransaction.userId = :userId', { userId })

-    if (onlyCreation) {
-      query.andWhere('userTransaction.typeId = :typeId', {
-        typeId: TransactionTypeId.CREATION,
-      })
-    }
-
    return query
      .orderBy('userTransaction.balanceDate', order)
      .limit(limit)
--- a/backend/src/util/virtualTransactions.ts
+++ b/backend/src/util/virtualTransactions.ts
@ -38,6 +38,7 @@ const virtualLinkTransaction = (
  createdAt: Date,
  validUntil: Date,
  user: User,
+  previousBalance: Decimal,
 ): Transaction => {
  const linkDbTransaction: dbTransaction = {
    id: -2,
--- a/database/entity/0036-unique_previous_in_transactions/Transaction.ts
+++ b/database/entity/0036-unique_previous_in_transactions/Transaction.ts
@ -96,4 +96,8 @@ export class Transaction extends BaseEntity {
  @OneToOne(() => Contribution, (contribution) => contribution.transaction)
  @JoinColumn({ name: 'id', referencedColumnName: 'transactionId' })
  contribution?: Contribution | null
+
+  @OneToOne(() => Transaction)
+  @JoinColumn({ name: 'previous' })
+  previousTransaction?: Transaction | null
 }
--- a/frontend/src/components/DecayInformations/DecayInformation-Decay.vue
+++ b/frontend/src/components/DecayInformations/DecayInformation-Decay.vue
@ -12,7 +12,7 @@
          </b-col>
          <b-col offset="1" offset-md="0" offset-lg="0">
            <div>
-              {{ previousBookedBalance | GDD }}
+              {{ previousBalance | GDD }}
              {{ decay === '0' ? $t('math.minus') : '' }}
              {{ decay | GDD }} {{ $t('math.equal') }}
              <b>{{ balance | GDD }}</b>
@ -35,7 +35,7 @@ export default {
      type: String,
      required: true,
    },
-    previousBookedBalance: {
+    previousBalance: {
      type: String,
      required: true,
    },
--- a/frontend/src/components/DecayInformations/DecayInformation-Long.vue
+++ b/frontend/src/components/DecayInformations/DecayInformation-Long.vue
@ -14,7 +14,7 @@
          <b-col cols="12" lg="4" md="4">
            <div>{{ $t('decay.last_transaction') }}</div>
          </b-col>
-          <b-col offset="1" offset-md="0" offset-lg="0">
+          <b-col offset="1" offset-md="0" offset-lg="0" class="text-right mr-5">
            <div>
              <span>
                {{ $d(new Date(decay.start), 'long') }}
@ -24,30 +24,44 @@
        </b-row>
        <duration-row :decayStart="decay.start" :decayEnd="decay.end" />

+        <!-- Previous Balance -->
+        <b-row class="mt-2">
+          <b-col cols="12" lg="6" md="3">
+            <div>{{ $t('decay.old_balance') }}</div>
+          </b-col>
+          <b-col offset="1" offset-md="0" offset-lg="0" class="text-right mr-5">
+            {{ previousBalance | GDD }}
+          </b-col>
+        </b-row>
+
        <!-- Decay-->
-        <b-row>
-          <b-col cols="12" lg="4" md="4">
+        <b-row class="mt-0">
+          <b-col cols="12" lg="3" md="3">
            <div>{{ $t('decay.decay') }}</div>
          </b-col>
-          <b-col offset="1" offset-md="0" offset-lg="0">{{ decay.decay | GDD }}</b-col>
+          <b-col offset="1" offset-md="0" offset-lg="0" class="text-right mr-5">
+            {{ decay.decay | GDD }}
+          </b-col>
        </b-row>
      </b-col>
    </b-row>
    <!-- Type-->
    <b-row>
      <b-col>
-        <b-row>
+        <b-row class="mb-2">
          <!-- eslint-disable-next-line @intlify/vue-i18n/no-dynamic-keys-->
-          <b-col cols="12" lg="4" md="4">{{ $t(`decay.types.${typeId.toLowerCase()}`) }}</b-col>
-          <b-col offset="1" offset-md="0" offset-lg="0">{{ amount | GDD }}</b-col>
+          <b-col cols="12" lg="3" md="3">{{ $t(`decay.types.${typeId.toLowerCase()}`) }}</b-col>
+          <b-col offset="1" offset-md="0" offset-lg="0" class="text-right mr-5">
+            {{ amount | GDD }}
+          </b-col>
        </b-row>
        <!-- Total-->
-        <b-row>
-          <b-col cols="12" lg="4" md="4">
-            <div>{{ $t('decay.total') }}</div>
+        <b-row class="border-top pt-2">
+          <b-col cols="12" lg="3" md="3">
+            <div>{{ $t('decay.new_balance') }}</div>
          </b-col>
-          <b-col offset="1" offset-md="0" offset-lg="0">
-            <b>{{ (Number(amount) + Number(decay.decay)) | GDD }}</b>
+          <b-col offset="1" offset-md="0" offset-lg="0" class="text-right mr-5">
+            <b>{{ balance | GDD }}</b>
          </b-col>
        </b-row>
      </b-col>
@ -63,6 +77,8 @@ export default {
    DurationRow,
  },
  props: {
+    balance: { type: String, default: '0' },
+    previousBalance: { type: String, default: '0' },
    amount: { type: String, default: '0' },
    typeId: { type: String, default: '' },
    memo: { type: String, default: '' },
--- a/frontend/src/components/DecayInformations/DecayInformation.vue
+++ b/frontend/src/components/DecayInformations/DecayInformation.vue
@ -7,7 +7,15 @@
      :decay="decay"
      :typeId="typeId"
    />
-    <decay-information-long v-else :amount="amount" :decay="decay" :typeId="typeId" :memo="memo" />
+    <decay-information-long
+      v-else
+      :amount="amount"
+      :decay="decay"
+      :typeId="typeId"
+      :memo="memo"
+      :balance="balance"
+      :previousBalance="previousBalance"
+    />
  </div>
 </template>
 <script>
@ -39,6 +47,14 @@ export default {
      type: String,
      required: true,
    },
+    balance: {
+      type: String,
+      required: true,
+    },
+    previousBalance: {
+      type: String,
+      required: true,
+    },
  },
  computed: {
    isStartBlock() {
--- a/frontend/src/components/GddTransactionList.spec.js
+++ b/frontend/src/components/GddTransactionList.spec.js
@ -93,8 +93,9 @@ describe('GddTransactionList', () => {
            {
              id: -1,
              typeId: 'DECAY',
-              amount: '-0.16778637075575395772595',
-              balance: '31.59320453982945549519405',
+              amount: '-0.16',
+              balance: '31.59',
+              previousBalance: '31.75',
              balanceDate: '2022-03-03T08:54:54',
              memo: '',
              linkedUser: null,
@ -110,6 +111,7 @@ describe('GddTransactionList', () => {
              typeId: 'SEND',
              amount: '1',
              balance: '31.76099091058520945292',
+              previousBalance: '30.76',
              balanceDate: '2022-02-28T13:55:47',
              memo:
                'Um den Kessel schlingt den Reihn, Werft die Eingeweid‘ hinein. Kröte du, die Nacht und Tag Unterm kalten Steine lag,',
@ -129,6 +131,7 @@ describe('GddTransactionList', () => {
              typeId: 'RECEIVE',
              amount: '10',
              balance: '10',
+              previousBalance: '31.75',
              balanceDate: '2022-02-23T10:55:30',
              memo:
                'Monatlanges Gift sog ein, In den Topf zuerst hinein… (William Shakespeare, Die Hexen aus Macbeth)',
@ -148,6 +151,7 @@ describe('GddTransactionList', () => {
              typeId: 'CREATION',
              amount: '1000',
              balance: '32.96482231613347376132',
+              previousBalance: '31.75',
              balanceDate: '2022-02-25T07:29:26',
              memo: 'Jammern hilft nichts, sondern ich kann selber meinen Teil dazu beitragen.',
              linkedUser: {
@ -414,6 +418,7 @@ describe('GddTransactionList', () => {
        return {
          amount: '3.14',
          balanceDate: '2021-04-29T17:26:40+00:00',
+          previousBalance: '31.75',
          decay: {
            decay: '-477.01',
            start: '2021-05-13T17:46:31.000Z',
--- a/frontend/src/components/GddTransactionList.vue
+++ b/frontend/src/components/GddTransactionList.vue
@ -19,10 +19,7 @@
          class="pointer bg-white appBoxShadow gradido-border-radius px-4 pt-2 test-list-group-item"
        >
          <template #DECAY>
-            <transaction-decay
-              v-bind="transactions[index]"
-              :previousBookedBalance="previousBookedBalance(index)"
-            />
+            <transaction-decay v-bind="transactions[index]" />
          </template>
        </transaction-list-item>
      </div>
@ -34,24 +31,15 @@
            class="pointer mb-3 bg-white appBoxShadow gradido-border-radius p-3 test-list-group-item"
          >
            <template #SEND>
-              <transaction-send
-                v-bind="transactions[index]"
-                :previousBookedBalance="previousBookedBalance(index)"
-              />
+              <transaction-send v-bind="transactions[index]" />
            </template>

            <template #RECEIVE>
-              <transaction-receive
-                v-bind="transactions[index]"
-                :previousBookedBalance="previousBookedBalance(index)"
-              />
+              <transaction-receive v-bind="transactions[index]" />
            </template>

            <template #CREATION>
-              <transaction-creation
-                v-bind="transactions[index]"
-                :previousBookedBalance="previousBookedBalance(index)"
-              />
+              <transaction-creation v-bind="transactions[index]" />
            </template>

            <template #LINK_SUMMARY>
@ -124,10 +112,6 @@ export default {
      })
      window.scrollTo(0, 0)
    },
-    previousBookedBalance(idx) {
-      if (this.transactions[idx + 1]) return this.transactions[idx + 1].balance
-      return '0'
-    },
  },
  computed: {
    isPaginationVisible() {
--- a/frontend/src/components/TransactionRows/DurationRow.vue
+++ b/frontend/src/components/TransactionRows/DurationRow.vue
@ -4,7 +4,7 @@
      <b-col cols="12" lg="4" md="4">
        <div>{{ $t('decay.past_time') }}</div>
      </b-col>
-      <b-col offset="1" offset-md="0" offset-lg="0">
+      <b-col offset="1" offset-md="0" offset-lg="0" class="text-right mr-5">
        <span v-if="duration">{{ durationText }}</span>
      </b-col>
    </b-row>
--- a/frontend/src/components/Transactions/TransactionCreation.spec.js
+++ b/frontend/src/components/Transactions/TransactionCreation.spec.js
@ -13,7 +13,8 @@ const mocks = {

 const propsData = {
  amount: '12.45',
-  balance: '31.76099091058521',
+  balance: '31.76',
+  previousBalance: '19.31',
  balanceDate: '2022-02-28T13:55:47.000Z',
  decay: {
    decay: '-0.2038314055482643084',
--- a/frontend/src/components/Transactions/TransactionCreation.vue
+++ b/frontend/src/components/Transactions/TransactionCreation.vue
@ -18,7 +18,14 @@
      </b-col>
    </b-row>
    <b-collapse class="pb-4 pt-lg-3" v-model="visible">
-      <decay-information :typeId="typeId" :decay="decay" :amount="amount" :memo="memo" />
+      <decay-information
+        :typeId="typeId"
+        :decay="decay"
+        :amount="amount"
+        :memo="memo"
+        :balance="balance"
+        :previousBalance="previousBalance"
+      />
    </b-collapse>
  </div>
 </template>
@ -61,7 +68,11 @@ export default {
      type: Number,
      required: false,
    },
-    previousBookedBalance: {
+    balance: {
+      type: String,
+      required: true,
+    },
+    previousBalance: {
      type: String,
      required: true,
    },
--- a/frontend/src/components/Transactions/TransactionDecay.vue
+++ b/frontend/src/components/Transactions/TransactionDecay.vue
@ -14,7 +14,7 @@
      <decay-information-decay
        :balance="balance"
        :decay="decay.decay"
-        :previousBookedBalance="previousBookedBalance"
+        :previousBalance="previousBalance"
      />
    </b-collapse>
  </div>
@ -44,9 +44,10 @@ export default {
      type: Object,
      required: true,
    },
-    previousBookedBalance: {
-      type: String,
-      required: true,
+  },
+  computed: {
+    previousBalance() {
+      return String(Number(this.balance) - Number(this.decay.decay))
    },
  },
  data() {
--- a/frontend/src/components/Transactions/TransactionReceive.spec.js
+++ b/frontend/src/components/Transactions/TransactionReceive.spec.js
@ -13,7 +13,8 @@ const mocks = {

 const propsData = {
  amount: '12.45',
-  balance: '31.76099091058521',
+  balance: '31.76',
+  previousBalance: '19.31',
  balanceDate: '2022-02-28T13:55:47.000Z',
  decay: {
    decay: '-0.2038314055482643084',
--- a/frontend/src/components/Transactions/TransactionReceive.vue
+++ b/frontend/src/components/Transactions/TransactionReceive.vue
@ -42,7 +42,14 @@
      </b-col>
    </b-row>
    <b-collapse class="pb-4 pt-lg-3" v-model="visible">
-      <decay-information :typeId="typeId" :decay="decay" :amount="amount" :memo="memo" />
+      <decay-information
+        :typeId="typeId"
+        :decay="decay"
+        :amount="amount"
+        :memo="memo"
+        :balance="balance"
+        :previousBalance="previousBalance"
+      />
    </b-collapse>
  </div>
 </template>
@ -84,7 +91,11 @@ export default {
    typeId: {
      type: String,
    },
-    previousBookedBalance: {
+    balance: {
+      type: String,
+      required: true,
+    },
+    previousBalance: {
      type: String,
      required: true,
    },
--- a/frontend/src/components/Transactions/TransactionSend.spec.js
+++ b/frontend/src/components/Transactions/TransactionSend.spec.js
@ -13,7 +13,8 @@ const mocks = {

 const propsData = {
  amount: '12.45',
-  balance: '31.76099091058521',
+  balance: '31.76',
+  previousBalance: '19.31',
  balanceDate: '2022-02-28T13:55:47.000Z',
  decay: {
    decay: '-0.2038314055482643084',
--- a/frontend/src/components/Transactions/TransactionSend.vue
+++ b/frontend/src/components/Transactions/TransactionSend.vue
@ -41,7 +41,14 @@
      </b-col>
    </b-row>
    <b-collapse class="pb-4 pt-lg-3" v-model="visible">
-      <decay-information :typeId="typeId" :decay="decay" :amount="amount" :memo="memo" />
+      <decay-information
+        :typeId="typeId"
+        :decay="decay"
+        :amount="amount"
+        :memo="memo"
+        :balance="balance"
+        :previousBalance="previousBalance"
+      />
    </b-collapse>
  </div>
 </template>
@ -84,7 +91,11 @@ export default {
      type: String,
      required: true,
    },
-    previousBookedBalance: {
+    balance: {
+      type: String,
+      required: true,
+    },
+    previousBalance: {
      type: String,
      required: true,
    },
--- a/frontend/src/graphql/queries.js
+++ b/frontend/src/graphql/queries.js
@ -33,6 +33,7 @@ export const transactionsQuery = gql`
        typeId
        amount
        balance
+        previousBalance
        balanceDate
        memo
        linkedUser {
--- a/frontend/src/locales/de.json
+++ b/frontend/src/locales/de.json
@ -89,6 +89,8 @@
    "decay_introduced": "Die Vergänglichkeit wurde eingeführt am:",
    "decay_since_last_transaction": "Vergänglichkeit seit der letzten Transaktion",
    "last_transaction": "Letzte Transaktion",
+    "new_balance": "Neuer Kontostand",
+    "old_balance": "Vorheriger Kontostand",
    "past_time": "Vergangene Zeit",
    "Starting_block_decay": "Startblock Vergänglichkeit",
    "total": "Gesamt",
--- a/frontend/src/locales/en.json
+++ b/frontend/src/locales/en.json
@ -89,6 +89,8 @@
    "decay_introduced": "Decay was introduced on:",
    "decay_since_last_transaction": "Decay since the last transaction",
    "last_transaction": "Last transaction:",
+    "new_balance": "New balance",
+    "old_balance": "Previous balance",
    "past_time": "Time passed",
    "Starting_block_decay": "Starting Block Decay",
    "total": "Total",