fix(LSK): include nonce in transaction fee calculation

Adamant-im · Dec 25, 2023 · 18b6bd9 · 18b6bd9
1 parent c2b2c2b
commit 18b6bd9
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/src/lib/lisk/lisk-utils.ts b/src/lib/lisk/lisk-utils.ts
@@ -219,6 +219,10 @@ type EstimateFeeParams = {
    */
   data?: string
   isNewAccount?: boolean
+  /**
+   * Current nonce
+   */
+  nonce: number | string
 }
 
 /**
@@ -232,14 +236,15 @@ export function estimateFee(params?: EstimateFeeParams) {
     keyPair = LSK_DEMO_ACCOUNT.keyPair,
     recipientAddress = LSK_DEMO_ACCOUNT.address,
     data = '',
+    nonce = 0,
     isNewAccount
   } = params || {}
 
   const unsignedTransaction = {
     module: 'token',
     command: 'transfer',
     fee: BigInt(0),
-    nonce: BigInt(0),
+    nonce: BigInt(nonce),
     senderPublicKey: Buffer.from(keyPair.publicKey, 'hex'),
     params: {
       tokenID: Buffer.from(LSK_TOKEN_ID, 'hex'),

diff --git a/src/store/modules/lsk/lsk-getters.js b/src/store/modules/lsk/lsk-getters.js
@@ -4,8 +4,13 @@ import baseGetters from '../lsk-base/lsk-base-getters'
 export default {
   ...baseGetters,
 
-  fee: () => (amount, _recipientAddress, data, isNewAccount) => {
-    return estimateFee({ amount, data, isNewAccount })
+  fee: (state) => (amount, recipientAddress, data, isNewAccount) => {
+    return estimateFee({
+      amount,
+      data,
+      isNewAccount,
+      nonce: state.nonce
+    })
   },
 
   height(state) {