diff options
author | Zander Brown <zbrown@gnome.org> | 2021-04-02 04:39:31 +0100 |
---|---|---|
committer | Zander Brown <zbrown@gnome.org> | 2021-04-02 13:13:34 +0100 |
commit | 21e2f76449085e099afac0f452ec32867e8c1fca (patch) | |
tree | 4e849819438d9323f85eb10c3877044630d7f695 | |
parent | b3f34d12f19c819eda127af71c399da089a4568b (diff) | |
download | gnome-shell-zbrown/search-change.tar.gz |
remoteSearch: Fetch enabled/disabled list oncezbrown/search-change
GLib should be caching the values but it seems unnecessary to fetch
and marshal what should be the same lists N times instead of just once
-rw-r--r-- | js/ui/remoteSearch.js | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/js/ui/remoteSearch.js b/js/ui/remoteSearch.js index 77ad317e1..137516bdc 100644 --- a/js/ui/remoteSearch.js +++ b/js/ui/remoteSearch.js @@ -142,16 +142,16 @@ function loadRemoteSearchProviders(searchSettings, callback) { // Special case gnome-control-center to be always active and always first sortOrder.unshift('gnome-control-center.desktop'); + const disabled = searchSettings.get_strv('disabled'); + const enabled = searchSettings.get_strv('enabled'); + loadedProviders = loadedProviders.filter(provider => { let appId = provider.appInfo.get_id(); - if (provider.defaultEnabled) { - let disabled = searchSettings.get_strv('disabled'); + if (provider.defaultEnabled) return !disabled.includes(appId); - } else { - let enabled = searchSettings.get_strv('enabled'); + else return enabled.includes(appId); - } }); loadedProviders.sort((providerA, providerB) => { |