AND-8751 Fix issues

tangem · Oct 17, 2024 · f25f009 · f25f009
1 parent e91cf88
commit f25f009
Showing 1 changed file with 17 additions and 15 deletions.
diff --git a/...java/com/tangem/blockchain/blockchains/ethereum/txbuilder/EthereumTWTransactionBuilder.kt b/...java/com/tangem/blockchain/blockchains/ethereum/txbuilder/EthereumTWTransactionBuilder.kt
@@ -222,16 +222,7 @@ internal class EthereumTWTransactionBuilder(wallet: Wallet) : EthereumTransactio
                     .setMaxInclusionFeePerGas(ByteString.copyFrom(fee.priorityFee.toByteArray()))
             }
             is Fee.Ethereum.Legacy -> {
-                val calculateGasPrise = {
-                    val feeValue = fee.amount.value
-                        ?.movePointRight(fee.amount.decimals)
-                        ?.toBigInteger()
-                        ?: error("Transaction fee must be specified")
-
-                    feeValue.divide(fee.gasLimit)
-                }
-
-                val gasPrice = fee.gasPrice.takeUnless(BigInteger::isZero) ?: calculateGasPrise()
+                val gasPrice = fee.gasPrice.takeUnless(BigInteger::isZero) ?: fee.calculateGasPrice()
 
                 this
                     .setTxMode(Ethereum.TransactionMode.Legacy)
@@ -241,6 +232,15 @@ internal class EthereumTWTransactionBuilder(wallet: Wallet) : EthereumTransactio
         }
     }
 
+    private fun Fee.Ethereum.Legacy.calculateGasPrice(): BigInteger {
+        val feeValue = amount.value
+            ?.movePointRight(amount.decimals)
+            ?.toBigInteger()
+            ?: error("Transaction fee must be specified")
+
+        return feeValue.divide(gasLimit)
+    }
+
     private fun Ethereum.SigningInput.Builder.setTransaction(
         coinAmount: BigInteger,
         extras: EthereumTransactionExtras?,
@@ -259,15 +259,17 @@ internal class EthereumTWTransactionBuilder(wallet: Wallet) : EthereumTransactio
         return setContractGeneric(
             Ethereum.Transaction.ContractGeneric.newBuilder()
                 .setAmount(ByteString.copyFrom(coinAmount.toByteArray()))
-                .apply {
-                    if (data != null) {
-                        setData(ByteString.copyFrom(data))
-                    }
-                }
+                .setInputIfNotNull(data)
                 .build(),
         )
     }
 
+    private fun Ethereum.Transaction.ContractGeneric.Builder.setInputIfNotNull(
+        data: ByteArray?,
+    ): Ethereum.Transaction.ContractGeneric.Builder {
+        return if (data != null) setData(ByteString.copyFrom(data)) else this
+    }
+
     private fun buildTxCompilerPreSigningOutput(input: Ethereum.SigningInput): PreSigningOutput {
         val txInputData = input.toByteArray()
         val preImageHashes = TransactionCompiler.preImageHashes(coinType, txInputData)