diff options
| author | Hygna <hygna@proton.me> | 2022-10-21 19:24:55 +0100 |
|---|---|---|
| committer | Hygna <hygna@proton.me> | 2022-10-21 19:25:50 +0100 |
| commit | 51bb4e1557b892f94132131aa6d19e5cb2b148d0 (patch) | |
| tree | 5b48f4fa91293340a0c9c0562d07bd2858d4b8ac /src | |
| parent | Bump versiom 2.3.1 => 2.3.2 (diff) | |
| download | libredirect-51bb4e1557b892f94132131aa6d19e5cb2b148d0.zip | |
Fixed redirect toggles set to false being reset
Closes https://github.com/libredirect/libredirect/issues/499
Diffstat (limited to 'src')
| -rw-r--r-- | src/assets/javascripts/services.js | 40 | ||||
| -rw-r--r-- | src/config/config.json | 7 | ||||
| -rw-r--r-- | src/pages/background/background.js | 1 |
3 files changed, 16 insertions, 32 deletions
diff --git a/src/assets/javascripts/services.js b/src/assets/javascripts/services.js index 7b73d9d5..8aa044ea 100644 --- a/src/assets/javascripts/services.js +++ b/src/assets/javascripts/services.js @@ -716,7 +716,11 @@ function processUpdate() { for (const service in config.services) { if (!options[service]) options[service] = {} if (config.services[service].targets == "datajson") targets[service] = redirects[service] - for (const defaultOption in config.services[service].options) if (!options[service][defaultOption]) options[service][defaultOption] = config.services[service].options[defaultOption] + for (const defaultOption in config.services[service].options) { + if (options[service][defaultOption] === undefined) { + options[service][defaultOption] = config.services[service].options[defaultOption] + } + } for (const frontend in config.services[service].frontends) { if (config.services[service].frontends[frontend].instanceList) { if (!options[frontend]) options[frontend] = {} @@ -733,6 +737,11 @@ function processUpdate() { } } } + } else { + for (const instance of options[frontend][network].enabled) { + let i = redirects[frontend][network].indexOf(instance) + if (i < 0) options[frontend][network].enabled.splice(i, 1) + } } } } @@ -784,34 +793,6 @@ function modifyContentSecurityPolicy(details) { } } -function processEnabledInstanceList() { - return new Promise(resolve => { - fetch("/config/config.json") - .then(response => response.text()) - .then(configData => { - const config = JSON.parse(configData) - browser.storage.local.get(["redirects", "options"], r => { - let options = r.options - for (const service in config.services) { - for (const frontend in config.services[service].frontends) { - if (config.services[service].frontends[frontend].instanceList) { - for (const network in config.networks) { - for (const instance of options[frontend][network].enabled) { - let i = redirects[frontend][network].indexOf(instance) - if (i < 0) options[frontend][network].enabled.splice(i, 1) - } - } - } - } - } - browser.storage.local.set({ options }, () => { - resolve() - }) - }) - }) - }) -} - export default { redirect, computeService, @@ -823,5 +804,4 @@ export default { upgradeOptions, processUpdate, modifyContentSecurityPolicy, - processEnabledInstanceList, } diff --git a/src/config/config.json b/src/config/config.json index 5f70b85c..d9463206 100644 --- a/src/config/config.json +++ b/src/config/config.json @@ -163,7 +163,12 @@ "instanceList": true } }, - "targets": ["^https?:\\/{2}(www\\.|mobile\\.|)twitter\\.com(\\/|$)", "^https?:\\/{2}(pbs\\.|video\\.|)twimg\\.com(\\/|$)", "^https?:\\/{2}platform\\.twitter\\.com/embed(\\/|$)", "^https?:\\/{2}t\\.co(\\/|$)"], + "targets": [ + "^https?:\\/{2}(www\\.|mobile\\.|)twitter\\.com(\\/|$)", + "^https?:\\/{2}(pbs\\.|video\\.|)twimg\\.com(\\/|$)", + "^https?:\\/{2}platform\\.twitter\\.com/embed(\\/|$)", + "^https?:\\/{2}t\\.co(\\/|$)" + ], "name": "Twitter", "options": { "enabled": true, diff --git a/src/pages/background/background.js b/src/pages/background/background.js index d807fa0b..bed0055d 100644 --- a/src/pages/background/background.js +++ b/src/pages/background/background.js @@ -37,7 +37,6 @@ browser.runtime.onInstalled.addListener(details => { break default: await servicesHelper.processUpdate() - await servicesHelper.processEnabledInstanceList() } } }) |