Fix domain lookup toggles for Yat and Zcash.me providers (#3439)

Both providers checked the wrong SettingsStore flag in isEnabled(), so
their toggles in Settings > Connection and sync > Domain lookups had no
effect:

- YatAddressProvider read lookupsWellKnown instead of lookupsYatService,
  so the 'Yat' toggle did nothing and '.wellknown' unexpectedly gated it.
- ZcashMeAddressProvider read lookupsZcashNames instead of
  lookupsZcashAddress, so the 'Zcash.me' toggle did nothing and it was
  gated by the 'Zcash Names' toggle.

Each provider now reads the flag its own toggle writes, matching the
mapping in ConnectionSyncViewModel.lookupValue/setLookupValue.
This commit is contained in:
Seth For Privacy
2026-07-25 02:43:06 +03:00
committed by GitHub
parent 1a0c3e0d6b
commit 7f090c3a3a
2 changed files with 2 additions and 2 deletions
@@ -20,7 +20,7 @@ class YatAddressProvider extends AddressLookupProvider {
bool canHandle(String q) => q.hasOnlyEmojis; // Yat handle example: 🐶🐾
@override
bool isEnabled(SettingsStore settingsStore) => settingsStore.lookupsWellKnown;
bool isEnabled(SettingsStore settingsStore) => settingsStore.lookupsYatService;
@override
Future<List<ParsedAddress>> resolve({
@@ -15,7 +15,7 @@ class ZcashMeAddressProvider extends AddressLookupProvider {
List<CryptoCurrency> get supportedCurrencies => [CryptoCurrency.zec];
@override
bool isEnabled(SettingsStore settingsStore) => settingsStore.lookupsZcashNames;
bool isEnabled(SettingsStore settingsStore) => settingsStore.lookupsZcashAddress;
@override
bool canHandle(String query) {