spesmilo / spesmilo/electrum

KeyError: 'payment_hash'

Open
#8,952 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
8.6k
Forks
3.5k
Avg merge
2d 8h
Merged PRs (30d)
61

Description

Crash Report

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

Traceback

Traceback (most recent call last):
  File "/home/user/wspace/electrum/.buildozer/android/app/electrum/gui/qml/qetxdetails.py", line 478, in save
  File "/home/user/wspace/electrum/.buildozer/android/app/electrum/gui/qml/qewallet.py", line 612, in save_tx
  File "/home/user/wspace/electrum/.buildozer/android/app/electrum/gui/qml/qetransactionlistmodel.py", line 219, in initModel
  File "/home/user/wspace/electrum/.buildozer/android/app/electrum/gui/qml/qetransactionlistmodel.py", line 130, in tx_to_model
KeyError: 'payment_hash'

KeyError: 'payment_hash'

Reporter

This issue was reported by 4 user(s):

Electrum Version Python Version Operating System Wallet Type Locale Stack
4.5.3 3.8.18 (default, Feb 22 2024, 12:26:25) [Clang 12.0.8 (https://android.googlesource.com/toolchain/llvm-project c935d99d Android 10 on xiaomi lavender (QKQ1.190910.002 V12.5.3.0.QFGMIXM) standard en_US
4.5.4 3.8.18 (default, Mar 13 2024, 14:09:52) [Clang 12.0.8 (https://android.googlesource.com/toolchain/llvm-project c935d99d Android 12 on google bonito (SP2A.220305.012.2022033013) standard en_US
4.5.5 3.8.18 (default, May 29 2024, 15:14:46) [Clang 12.0.8 (https://android.googlesource.com/toolchain/llvm-project c935d99d Android 14 on samsung e1s (UP1A.231005.007.S921BXXS2AXD5) standard en_US
4.5.8 3.10.14 (main, Jun 18 2024, 10:31:10) [Clang 12.0.8 (https://android.googlesource.com/toolchain/llvm-project c935d99d7 Android 15 on google bluejay (2025041100) standard en_US

Additional Information

The reporting user(s) did not provide additional information.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with qetransactionlistmodel.py around tx_to_model at line 130, then trace the save path through qewallet.py line 612 and qetxdetails.py line 478. Reproduce the Android transaction-save flow and verify that it no longer raises KeyError for payment_hash.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
frontend, mobile
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.