[fenix] For https://github.com/mozilla-mobile/fenix/issues/8552 - Fixes bug where "installing/uninstalling" was using the old pref key

pull/600/head
Jeff Boek 5 years ago
parent ed57b6bdd1
commit 02f0edbfcf

@ -113,7 +113,7 @@ open class FenixSearchEngineProvider(
fun installSearchEngine(context: Context, searchEngine: SearchEngine) = runBlocking { fun installSearchEngine(context: Context, searchEngine: SearchEngine) = runBlocking {
val installedIdentifiers = installedSearchEngineIdentifiers(context).toMutableSet() val installedIdentifiers = installedSearchEngineIdentifiers(context).toMutableSet()
installedIdentifiers.add(searchEngine.identifier) installedIdentifiers.add(searchEngine.identifier)
prefs(context).edit().putStringSet(INSTALLED_ENGINES_KEY, installedIdentifiers).apply() prefs(context).edit().putStringSet(localeAwareInstalledEnginesKey(), installedIdentifiers).apply()
} }
fun uninstallSearchEngine(context: Context, searchEngine: SearchEngine) = runBlocking { fun uninstallSearchEngine(context: Context, searchEngine: SearchEngine) = runBlocking {
@ -124,7 +124,7 @@ open class FenixSearchEngineProvider(
} else { } else {
val installedIdentifiers = installedSearchEngineIdentifiers(context).toMutableSet() val installedIdentifiers = installedSearchEngineIdentifiers(context).toMutableSet()
installedIdentifiers.remove(searchEngine.identifier) installedIdentifiers.remove(searchEngine.identifier)
prefs(context).edit().putStringSet(INSTALLED_ENGINES_KEY, installedIdentifiers).apply() prefs(context).edit().putStringSet(localeAwareInstalledEnginesKey(), installedIdentifiers).apply()
} }
reload() reload()

Loading…
Cancel
Save