search

spesmilo / electrum

Electrum Bitcoin Wallet
https://electrum.org
MIT License
7.43k stars 3.09k forks source link

TypeError: conversion from NoneType to Decimal is not supported #8876

Open TARS-bot opened 8 months ago

TARS-bot commented 8 months ago

Crash Report

This crash report was reported through the automatic crash reporting system 🤖

Traceback

Traceback (most recent call last):
  File "electrum\gui\qt\send_tab.py", line 570, in do_pay_or_get_invoice
  File "electrum\gui\qt\send_tab.py", line 597, in do_pay_invoice
  File "electrum\gui\qt\send_tab.py", line 319, in pay_onchain_dialog
  File "electrum\gui\qt\confirm_tx_dialog.py", line 622, in __init__
  File "electrum\gui\qt\util.py", line 376, in __init__
  File "electrum\gui\qt\confirm_tx_dialog.py", line 125, in update
  File "electrum\gui\qt\confirm_tx_dialog.py", line 525, in _update_widgets
  File "electrum\gui\qt\confirm_tx_dialog.py", line 673, in can_pay_assuming_zero_fees
  File "electrum\gui\qt\send_tab.py", line 311, in <lambda>
  File "electrum\util.py", line 481, in do_profile
  File "electrum\wallet.py", line 1872, in make_unsigned_transaction
TypeError: conversion from NoneType to Decimal is not supported

TypeError: conversion from NoneType to Decimal is not supported

Reporter

This issue was reported by 9 user(s):

Electrum Version Python Version Operating System Wallet Type Locale Stack
4.5.2 3.10.11 (tags/v3.10.11:7d4cc5a, Apr 5 2023, 00:20:04) [MSC v.1929 32 bit (Intel)] Windows-10-10.0.19045-SP0 imported,standard es_MX ℹ
4.5.4 3.10.11 (tags/v3.10.11:7d4cc5a, Apr 5 2023, 00:20:04) [MSC v.1929 32 bit (Intel)] Windows-10-10.0.26063-SP0 imported es_MX ℹ
4.5.5 3.10.11 (tags/v3.10.11:7d4cc5a, Apr 5 2023, 00:20:04) [MSC v.1929 32 bit (Intel)] Windows-10-10.0.19045-SP0 imported en_US ℹ
4.5.5 3.10.11 (tags/v3.10.11:7d4cc5a, Apr 5 2023, 00:20:04) [MSC v.1929 32 bit (Intel)] Windows-10-10.0.22631-SP0 imported es_MX ℹ
4.5.5 3.10.11 (tags/v3.10.11:7d4cc5a, Apr 5 2023, 00:20:04) [MSC v.1929 32 bit (Intel)] Windows-10-10.0.19045-SP0 standard en_US ℹ
4.5.5 3.10.11 (tags/v3.10.11:7d4cc5a, Apr 5 2023, 00:20:04) [MSC v.1929 32 bit (Intel)] Windows-10-10.0.19045-SP0 imported,standard en_US ℹ
4.5.5 3.10.11 (tags/v3.10.11:7d4cc5a, Apr 5 2023, 00:20:04) [MSC v.1929 32 bit (Intel)] Windows-10-10.0.19045-SP0 imported,standard en_US ℹ
4.5.5 3.10.11 (tags/v3.10.11:7d4cc5a, Apr 5 2023, 00:20:04) [MSC v.1929 32 bit (Intel)] Windows-10-10.0.22631-SP0 2of2,standard,1of2,imported en_US ℹ
4.5.5 3.10.11 (tags/v3.10.11:7d4cc5a, Apr 5 2023, 00:20:04) [MSC v.1929 32 bit (Intel)] Windows-10-10.0.22631-SP0 2of2 en_US ℹ

Additional Information

L5AVwrMEg7dq2EyRxN4rVTbS7tSvzZ1y6Mp4jhyFmeCSjbKtgbib

SomberNight commented 8 months ago

Looks like user set config.WALLET_BATCH_RBF=True, and wallet.get_unconfirmed_base_tx_for_batching() selected a base_tx to merge with, but we cannot calculate base_tx.get_fee() for it... Not sure how this can happen. https://github.com/spesmilo/electrum/blob/fe03fbf2a05bcbe7ddb263d937ad9501b929d8a4/electrum/wallet.py#L1872