diff --git a/src/main/java/com/sparrowwallet/sparrow/io/SpecterDIY.java b/src/main/java/com/sparrowwallet/sparrow/io/SpecterDIY.java index 30db6b38..c7465f89 100644 --- a/src/main/java/com/sparrowwallet/sparrow/io/SpecterDIY.java +++ b/src/main/java/com/sparrowwallet/sparrow/io/SpecterDIY.java @@ -26,16 +26,16 @@ public class SpecterDIY implements KeystoreFileImport, WalletExport { Wallet wallet = outputDescriptor.toWallet(); if(wallet.getKeystores().size() != 1) { - throw new ImportException("Could not determine keystore from import"); + throw new IllegalArgumentException("Could not determine keystore from import"); } - Keystore keystore = wallet.getKeystores().get(0); + Keystore keystore = wallet.getKeystores().getFirst(); keystore.setLabel(getName()); keystore.setWalletModel(getWalletModel()); keystore.setSource(KeystoreSource.HW_AIRGAPPED); return keystore; - } catch(IOException e) { + } catch(Exception e) { throw new ImportException("Error getting " + getName() + " keystore", e); } } diff --git a/src/main/java/com/sparrowwallet/sparrow/keystoreimport/HwAirgappedController.java b/src/main/java/com/sparrowwallet/sparrow/keystoreimport/HwAirgappedController.java index 39cdeb94..78afe002 100644 --- a/src/main/java/com/sparrowwallet/sparrow/keystoreimport/HwAirgappedController.java +++ b/src/main/java/com/sparrowwallet/sparrow/keystoreimport/HwAirgappedController.java @@ -26,12 +26,10 @@ public class HwAirgappedController extends KeystoreImportDetailController { public void initializeView() { List fileImporters = Collections.emptyList(); - if(getMasterController().getWallet().getPolicyType().equals(PolicyType.SINGLE_HD)) { + if(getMasterController().getWallet().getPolicyType().equals(PolicyType.SINGLE_HD) || getMasterController().getWallet().getPolicyType().equals(PolicyType.SINGLE_SP)) { fileImporters = List.of(new ColdcardSinglesig(), new CoboVaultSinglesig(), new Jade(), new KeystoneSinglesig(), new PassportSinglesig(), new SeedSigner(), new GordianSeedTool(), new SpecterDIY(), new Krux(), new AirGapVault(), new KeycardShellSinglesig()); } else if(getMasterController().getWallet().getPolicyType().equals(PolicyType.MULTI_HD)) { fileImporters = List.of(new Bip129(), new ColdcardMultisig(), new CoboVaultMultisig(), new JadeMultisig(), new KeystoneMultisig(), new PassportMultisig(), new SeedSigner(), new GordianSeedTool(), new SpecterDIY(), new Krux(), new KeycardShellMultisig()); - } else if(getMasterController().getWallet().getPolicyType().equals(PolicyType.SINGLE_SP)) { - fileImporters = List.of(new ColdcardSinglesig()); } for(KeystoreFileImport importer : fileImporters) { @@ -43,10 +41,7 @@ public class HwAirgappedController extends KeystoreImportDetailController { } } - List cardImporters = Collections.emptyList(); - if(getMasterController().getWallet().getPolicyType().equals(PolicyType.SINGLE_HD) || getMasterController().getWallet().getPolicyType().equals(PolicyType.MULTI_HD)) { - cardImporters = List.of(new Tapsigner(), new Satochip(), new Satschip(), new Keycard()); - } + List cardImporters = List.of(new Tapsigner(), new Satochip(), new Satschip(), new Keycard()); for(KeystoreCardImport importer : cardImporters) { if(!importer.isDeprecated() || Config.get().isShowDeprecatedImportExport()) { CardImportPane importPane = new CardImportPane(getMasterController().getWallet(), importer, getMasterController().getDefaultDerivation(), getMasterController().getRequiredDerivation());