spesmilo / spesmilo/electrum

AttributeError: 'str' object has no attribute 'get'

Open
#6,935 6 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug 🐞 topic-walletstorage 💾
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

  File "electrum\gui\qt\__init__.py", line 245, in wrapper
  File "electrum\gui\qt\__init__.py", line 269, in start_new_window
  File "electrum\gui\qt\__init__.py", line 314, in _start_wizard_to_select_or_create_wallet
  File "electrum\wallet_db.py", line 72, in __init__
  File "electrum\wallet_db.py", line 105, in load_data
  File "electrum\util.py", line 408, in <lambda>
  File "electrum\util.py", line 404, in do_profile
  File "electrum\wallet_db.py", line 161, in upgrade
  File "electrum\wallet_db.py", line 227, in _convert_wallet_type

AttributeError: 'str' object has no attribute 'get'

Reporter

This issue was reported by 3 user(s):

Electrum Version Python Version Operating System Wallet Type Locale Stack
4.0.9 3.7.9 (tags/v3.7.9:13c94747c7, Aug 17 2020, 18:01:55) [MSC v.1900 32 bit (Intel)] Windows-10-10.0.18362-SP0 imported en_US
4.1.5 3.7.9 (v3.7.9:13c94747c7, Aug 15 2020, 01:31:08) [Clang 6.0 (clang-600.0.57)] Darwin-18.7.0-x86_64-i386-64bit imported,standard ?
4.4.6 3.8.15 (default, Aug 17 2023, 15:47:24) [Clang 11.0.5 (https://android.googlesource.com/toolchain/llvm-project 87f1315d Android 13 on samsung a14xm (TP1A.220624.014.A146USQU2BWD1) imported,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 in electrum/wallet_db.py at upgrade and _convert_wallet_type, following load_data from the traceback. Trace why a string reaches .get() for the imported-wallet cases shown in the report. Done means wallet upgrade no longer crashes for those reported wallet types, with regression coverage for this path if applicable.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
databases
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.