status-im
/
status-desktop
mirror of https://github.com/status-im/status-desktop.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

fix(@desktop/wallet): Prevent nil conversion to qvariant (#12546)

This commit is contained in:
Cuteivist 2023-10-26 10:01:52 +02:00 committed by GitHub
parent cb703a44a4
commit b1a8fd6260
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/app/modules/main/wallet_section/activity/controller.nim
View File

@ -119,6 +119,7 @@ QtObject:
return currencyAmountToItem(self.currencyService.parseCurrencyValue(symbol, amount),
self.currencyService.getCurrencyFormat(symbol))
self.activityDetails = nil
let entry = self.model.getEntry(entryIndex)
if entry == nil:
error "failed to find entry with index: ", entryIndex
@ -127,11 +128,12 @@ QtObject:
try:
self.activityDetails = newActivityDetails(entry.getMetadata(), amountToCurrencyConvertor)
except Exception as e:
let errDescription = e.msg
error "error: ", errDescription
error "error: ", e.msg
return
proc getActivityDetails(self: Controller): QVariant {.slot.} =
if self.activityDetails == nil:
return newQVariant()
return newQVariant(self.activityDetails)
QtProperty[QVariant] activityDetails:

2
ui/app/AppLayouts/Wallet/views/TransactionDetailView.qml
View File

@ -467,7 +467,7 @@ Item {
Layout.fillHeight: true
Layout.fillWidth: true
title: qsTr("Token format")
subTitle: root.isTransactionValid ? d.details.tokenType.toUpperCase() : ""
subTitle: d.isDetailsValid ? d.details.tokenType.toUpperCase() : ""
visible: !!subTitle
}
TransactionDataTile {

2
ui/imports/shared/controls/TransactionDelegate.qml
View File

@ -52,7 +52,7 @@ StatusListItem {
readonly property int transactionStatus: isModelDataValid ? modelData.status : Constants.TransactionStatus.Pending
readonly property bool isMultiTransaction: isModelDataValid && modelData.isMultiTransaction
readonly property string currentCurrency: rootStore.currentCurrency
readonly property double cryptoValue: isModelDataValid && !isMultiTransaction ? modelData.amount : 0.0
readonly property double cryptoValue: isModelDataValid ? modelData.amount : 0.0
readonly property double fiatValue: isModelDataValid && !isMultiTransaction ? rootStore.getFiatValue(cryptoValue, modelData.symbol, currentCurrency) : 0.0
readonly property double inCryptoValue: isModelDataValid ? modelData.inAmount : 0.0
readonly property double inFiatValue: isModelDataValid && isMultiTransaction ? rootStore.getFiatValue(inCryptoValue, modelData.inSymbol, currentCurrency): 0.0