chore(SharedAddressesAccountSelector): Remove workaround related to issue #14244

Additionally flow is simplified slightly by setting `type` role in
ConcatModel
This commit is contained in:
Michał Cieślak 2024-04-09 14:24:53 +02:00 committed by Michał
parent 5d4380e4d9
commit 180932b799
1 changed files with 10 additions and 16 deletions

View File

@ -153,7 +153,6 @@ StatusListView {
symbol: d.collectiblesNamesToSymbols[firstEntry.name] || "",
enabledNetworkBalance: e.length,
account: firstEntry.ownership[0].accountAddress,
type: Constants.TokenType.ERC721,
imageUrl: d.collectiblesNamesToImages[firstEntry.name] || ""
}
})
@ -210,15 +209,16 @@ StatusListView {
ConcatModel {
id: concatModel
markerRoleName: "type"
sources: [
SourceModel {
// During initialization sfpm provides rowCount not taking
// filtering into account, deferring model assignment as
// a workaround
Component.onCompleted: model = walletAccountAssetsModel
model: walletAccountAssetsModel
markerRoleValue: Constants.TokenType.ERC20
},
SourceModel {
Component.onCompleted: model = accountCollectiblesModel
model: accountCollectiblesModel
markerRoleValue: Constants.TokenType.ERC721
}
]
}
@ -254,15 +254,6 @@ StatusListView {
expression: d.getTotalBalance(model.balances, model.decimals)
expectedRoles: ["balances", "decimals"]
},
FastExpressionRole {
name: "type"
readonly property int typeVal: Constants.TokenType.ERC20
// Singletons cannot be used directly in sfpm's expressions
expression: typeVal
expectedRoles: []
},
FastExpressionRole {
name: "imageUrl"
@ -295,8 +286,11 @@ StatusListView {
StatusBaseText {
anchors.verticalCenter: parent.verticalCenter
font.pixelSize: Theme.tertiaryTextFontSize
readonly property int type: model.type
text: {
if (model.type === Constants.TokenType.ERC20)
if (type === Constants.TokenType.ERC20)
return LocaleUtils.currencyAmountToLocaleString(
root.getCurrencyAmount(model.enabledNetworkBalance,
model.symbol))