From bef2267641d1b43ee2089edd6b77cb92c18542bb Mon Sep 17 00:00:00 2001 From: BamaHodl Date: Thu, 18 Jul 2024 07:23:16 -0500 Subject: [PATCH] Fix issue caused by having is_sign_message method and property of same name--always returned true --- src/seedsigner/models/decode_qr.py | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/seedsigner/models/decode_qr.py b/src/seedsigner/models/decode_qr.py index 55991668a..4ac166423 100644 --- a/src/seedsigner/models/decode_qr.py +++ b/src/seedsigner/models/decode_qr.py @@ -382,7 +382,7 @@ def detect_segment_type(s, wordlist_language_code=None): return QRType.BITCOIN_ADDRESS # message signing - elif DecodeQR.is_sign_message(s): + elif s.startswith("signmessage"): return QRType.SIGN_MESSAGE # config data @@ -506,11 +506,6 @@ def is_bitcoin_address(s): return False - @staticmethod - def is_sign_message(s): - return type(s) == str and s.startswith("signmessage") - - @staticmethod def multisig_setup_file_to_descriptor(text) -> str: # sample text file, parse the contents and create descriptor