feat(wallet): hook sendTransferTokens to the wallet Send

Fixes #4650

src/app/modules/main/wallet_section/transactions/controller.nim

@@ -84,3 +84,9 @@ method transferEth*(self: Controller, from_addr: string, to_addr: string, value:
   password: string, uuid: string): bool =
   result = self.transactionService.transferEth(from_addr, to_addr, value, gas, gasPrice,
     maxPriorityFeePerGas, maxFeePerGas, password, uuid)
+
+method transferTokens*(self: Controller, from_addr: string, to_addr: string, contractAddress: string,
+    value: string, gas: string, gasPrice: string, maxPriorityFeePerGas: string,maxFeePerGas: string,
+    password: string, uuid: string): bool =
+  result = self.transactionService.transferTokens(from_addr, to_addr, contractAddress, value, gas,
+    gasPrice, maxPriorityFeePerGas, maxFeePerGas, password, uuid)

src/app/modules/main/wallet_section/transactions/controller_interface.nim

@@ -37,6 +37,12 @@ method transferEth*(self: AccessInterface, from_addr: string, to_addr: string, v
     password: string, uuid: string): bool {.base.} =
   raise newException(ValueError, "No implementation available")
 
+method transferTokens*(self: AccessInterface, from_addr: string, to_addr: string,
+    contractAddress: string, value: string, gas: string, gasPrice: string,
+    maxPriorityFeePerGas: string, maxFeePerGas: string, password: string,
+    uuid: string): bool {.base.} =
+  raise newException(ValueError, "No implementation available")
+
 type
   ## Abstract class (concept) which must be implemented by object/s used in this 
   ## module.

src/app/modules/main/wallet_section/transactions/io_interface.nim

@@ -48,6 +48,12 @@ method transferEth*(self: AccessInterface, from_addr: string, to_addr: string, v
     password: string, uuid: string): bool {.base.} =
   raise newException(ValueError, "No implementation available")
 
+method transferTokens*(self: AccessInterface, from_addr: string, to_addr: string,
+    contractAddress: string, value: string, gas: string, gasPrice: string,
+    maxPriorityFeePerGas: string, maxFeePerGas: string, password: string,
+    uuid: string): bool {.base.} =
+  raise newException(ValueError, "No implementation available")
+
 # View Delegate Interface
 # Delegate for the view must be declared here due to use of QtObject and multi 
 # inheritance, which is not well supported in Nim.

src/app/modules/main/wallet_section/transactions/module.nim

@@ -90,4 +90,10 @@ method transferEth*(self: Module, from_addr: string, to_addr: string, value: str
     gasPrice: string, maxPriorityFeePerGas: string, maxFeePerGas: string, password: string,
     uuid: string): bool =
   result = self.controller.transferEth(from_addr, to_addr, value, gas, gasPrice,
+    maxPriorityFeePerGas, maxFeePerGas, password, uuid)
+
+method transferTokens*(self: Module, from_addr: string, to_addr: string, contractAddress: string,
+    value: string, gas: string, gasPrice: string, maxPriorityFeePerGas: string,
+    maxFeePerGas: string, password: string, uuid: string): bool =
+  result = self.controller.transferTokens(from_addr, to_addr, contractAddress, value, gas, gasPrice,
     maxPriorityFeePerGas, maxFeePerGas, password, uuid)

src/app/modules/main/wallet_section/transactions/view.nim

@@ -105,9 +105,16 @@ QtObject:
 
   proc estimateGas*(self: View, from_addr: string, to: string, assetAddress: string, value: string, data: string): string {.slot.} =
     result = self.delegate.estimateGas(from_addr, to, assetAddress, value, data)
+    result = self.delegate.estimateGas(from_addr, to, assetAddress, value, data)
 
   proc transferEth*(self: View, from_addr: string, to_addr: string, value: string, gas: string,
       gasPrice: string, maxPriorityFeePerGas: string, maxFeePerGas: string, password: string,
       uuid: string): bool {.slot.} =
     result = self.delegate.transferEth(from_addr, to_addr, value, gas, gasPrice,
       maxPriorityFeePerGas, maxFeePerGas, password, uuid)
+
+  proc transferTokens*(self: View, from_addr: string, to_addr: string, contractAddress: string,
+      value: string, gas: string, gasPrice: string, maxPriorityFeePerGas: string,
+      maxFeePerGas: string, password: string, uuid: string): bool {.slot.} =
+    result = self.delegate.transferTokens(from_addr, to_addr, contractAddress, value, gas, gasPrice,
+      maxPriorityFeePerGas, maxFeePerGas, password, uuid)

src/app_service/service/eth/dto/contract.nim

@@ -74,4 +74,4 @@ proc tokenSymbol*(contract: ContractDto): string =
   getTokenString(contract, "symbol")
 
 proc getMethod*(contract: ContractDto, methodName: string): MethodDto = 
-  return contract.methods["methodName"]
+  return contract.methods[methodName]

src/app_service/service/eth/dto/method_dto.nim

@@ -93,7 +93,7 @@ proc send*(self: MethodDto, tx: var TransactionDataDto, methodDescriptor: object
   tx.data = self.encodeAbi(methodDescriptor)
   # this call should not be part of this file, we need to move it to appropriate place, or this should not be a DTO class.
   let response = status_eth.sendTransaction($(%tx), password)
-  return $response.result
+  return response.result.getStr
 
 proc call[T](self: MethodDto, tx: var TransactionDataDto, methodDescriptor: object, success: var bool): T =
   success = true

src/app_service/service/transaction/service.nim

@@ -151,7 +151,8 @@ QtObject:
       value: string,
       data: string = ""
     ): string {.slot.} =
-    var response: RpcResponse[JsonNode] 
+    var response: RpcResponse[JsonNode]
+    # TODO make this async
     try:
       if assetAddress != ZERO_ADDRESS and not assetAddress.isEmptyOrWhitespace:
         var tx = buildTokenTransaction(
@@ -160,16 +161,17 @@ QtObject:
           )
         let networkType = self.settingsService.getCurrentNetwork().toNetworkType()
         let network = self.networkService.getNetwork(networkType)
-        let contract = self.ethService.findErc20Contract(network.chainId, assetAddress)
+        let contract = self.ethService.findErc20Contract(network.chainId, parseAddress(assetAddress))
         if contract == nil:
           raise newException(ValueError, fmt"Could not find ERC-20 contract with address '{assetAddress}' for the current network")
 
         let transfer = Transfer(to: parseAddress(to), value: conversion.eth2Wei(parseFloat(value), contract.decimals))
-        let methodThing = contract.getMethod("transfer")
+        let transferMethod = contract.getMethod("transfer")
         var success: bool
-        let gas = methodThing.estimateGas(tx, transfer, success)
+        let gas = transferMethod.estimateGas(tx, transfer, success)
 
-        result = $(%* { "result": gas, "success": success })
+        let res = fromHex[int](gas)
+        result = $(%* { "result": res, "success": success })
       else:
         var tx = ens_utils.buildTransaction(
           parseAddress(from_addr),
@@ -217,4 +219,43 @@ QtObject:
     except Exception as e:
       error "Error sending eth transfer transaction", msg = e.msg
       return false
+    return true
+
+  proc transferTokens*(
+      self: Service,
+      from_addr: string,
+      to_addr: string,
+      assetAddress: string,
+      value: string,
+      gas: string,
+      gasPrice: string,
+      maxPriorityFeePerGas: string,
+      maxFeePerGas: string,
+      password: string,
+      uuid: string
+      ): bool =
+    try:
+      let eip1559Enabled = self.settingsService.isEIP1559Enabled()
+      eth_utils.validateTransactionInput(from_addr, to_addr, assetAddress, value, gas,
+        gasPrice, data = "", eip1559Enabled, maxPriorityFeePerGas, maxFeePerGas, uuid)
+
+      # TODO move this to another thread
+      let networkType = self.settingsService.getCurrentNetwork().toNetworkType()
+      let network = self.networkService.getNetwork(networkType)
+      let contract = self.eth_service.findErc20Contract(network.chainId, parseAddress(assetAddress))
+
+      var tx = ens_utils.buildTokenTransaction(parseAddress(from_addr), parseAddress(assetAddress),
+        gas, gasPrice, eip1559Enabled, maxPriorityFeePerGas, maxFeePerGas)
+
+      var success: bool
+      let transfer = Transfer(to: parseAddress(to_addr),
+        value: conversion.eth2Wei(parseFloat(value), contract.decimals))
+      let transferMethod = contract.getMethod("transfer")
+      let response = transferMethod.send(tx, transfer, password, success)
+
+      self.trackPendingTransaction(response, from_addr, to_addr,
+        $PendingTransactionTypeDto.WalletTransfer, data = "")
+    except Exception as e:
+      error "Error sending token transfer transaction", msg = e.msg
+      return false
     return true

src/app_service/service/wallet_account/service.nim

@@ -48,6 +48,10 @@ proc fetchPrice(crypto: string, fiat: string): float64 =
     client.headers = newHttpHeaders({ "Content-Type": "application/json" })
 
     let response = client.request(url)
+    let parsedResponse = parseJson(response.body)
+    if (parsedResponse{"Response"} != nil and parsedResponse{"Response"}.getStr == "Error"):
+      error "Error while getting price", message = parsedResponse["Message"].getStr
+      return 0.0
     result = parsefloat($parseJson(response.body)[fiat.toUpper])
     priceCache[key] = result
   except Exception as e:

ui/app/AppLayouts/stores/RootStore.qml

@@ -73,4 +73,9 @@ QtObject {
        return walletSectionTransactions.transferEth(from, to, amount, gasLimit, gasPrice, tipLimit,
         overallLimit, password, uuid);
     }
+
+    function transferTokens(from, to, address, amount, gasLimit, gasPrice, tipLimit, overallLimit, password, uuid) {
+       return walletSectionTransactions.transferTokens(from, to, address, amount, gasLimit,
+        gasPrice, tipLimit, overallLimit, password, uuid);
+    }
 }

ui/imports/shared/popups/SendModal.qml

@@ -51,18 +51,17 @@ ModalPopup {
                         transactionSigner.enteredPassword,
                         stack.uuid)
         } else {
-            // Not Refactored Yet
+            success = root.store.transferTokens(
-            //            success = RootStore.transferTokens(
+                        selectFromAccount.selectedAccount.address,
-            //                        selectFromAccount.selectedAccount.address,
+                        selectRecipient.selectedRecipient.address,
-            //                        selectRecipient.selectedRecipient.address,
+                        txtAmount.selectedAsset.address,
-            //                        txtAmount.selectedAsset.address,
+                        txtAmount.selectedAmount,
-            //                        txtAmount.selectedAmount,
+                        gasSelector.selectedGasLimit,
-            //                        gasSelector.selectedGasLimit,
+                        gasSelector.eip1599Enabled ? "" : gasSelector.selectedGasPrice,
-            //                        gasSelector.eip1599Enabled ? "" : gasSelector.selectedGasPrice,
+                        gasSelector.selectedTipLimit,
-            //                        gasSelector.selectedTipLimit,
+                        gasSelector.selectedOverallLimit,
-            //                        gasSelector.selectedOverallLimit,
+                        transactionSigner.enteredPassword,
-            //                        transactionSigner.enteredPassword,
+                        stack.uuid)
-            //                        stack.uuid)
         }
 
         if(!success){
@@ -144,6 +143,7 @@ ModalPopup {
                 id: txtAmount
                 selectedAccount: selectFromAccount.selectedAccount
                 currentCurrency: root.store.currentCurrency
+                // TODO make those use a debounce
                 getFiatValue: root.store.getFiatValue
 //                getCryptoValue: RootStore.cryptoValue
                 width: stack.width

ui/imports/shared/stores/RootStore.qml

@@ -62,14 +62,6 @@ QtObject {
         localAccountSensitiveSettings.isTenorWarningAccepted = value;
     }
 
-    function transferEth(from, to, amount, gasLimit, gasPrice, tipLimit, overallLimit, password, uuid) {
-//        return walletModelInst.transactionsView.transferEth(from, to, amount, gasLimit, gasPrice, tipLimit, overallLimit, password, uuid);
-    }
-
-    function transferTokens(from, to, address, amount, gasLimit, gasPrice, tipLimit, overallLimit, password, uuid) {
-//        return walletModelInst.transactionsView.transferTokens(from, to, address, amount, gasLimit, gasPrice, tipLimit, overallLimit, password, uuid);
-    }
-
     function copyToClipboard(text) {
         globalUtils.copyToClipboard(text)
     }