Simplified code. Removed pixivMoe, old.reddit.com #239

2 files changed, 23 insertions, 54 deletions

diff --git a/src/pages/options/sendTargets/sendTargets.html b/src/pages/options/sendTargets/sendTargets.html
index c54c1582..b63c8233 100644
--- a/src/pages/options/sendTargets/sendTargets.html
+++ b/src/pages/options/sendTargets/sendTargets.html
@@ -23,7 +23,6 @@
       <div class="title"> <img src="../../../assets/images/tiktok-icon.png"><a href="../tiktok/tiktok.html" data-localise="__MSG_tiktok__">TikTok</a></div>
       <div class="title"> <img src="../../../assets/images/reddit-icon.png"><a href="../reddit/reddit.html" data-localise="__MSG_reddit__">Reddit</a></div>
       <div class="title"> <img src="../../../assets/images/imgur-icon.png"><a href="../imgur/imgur.html" data-localise="__MSG_imgur__">Imgur</a></div>
-      <div class="title"> <img src="../../../assets/images/pixiv-icon.svg"><a href="../pixiv/pixiv.html" data-localise="__MSG_pixiv__">Pixiv</a></div>
       <div class="title"> <img src="../../../assets/images/librespeed-icon.svg"><a href="../speedtest/speedtest.html" data-localise="__MSG_speedtest__">Speedtest</a></div>
       <div class="title"> <img src="../../../assets/images/spotify-icon.png"><a href="../spotify/spotify.html" data-localise="__MSG_spotify__">Spotify</a></div>
       <div class="title"> <img src="../../../assets/images/wikipedia-icon.svg"><a href="../wikipedia/wikipedia.html" data-localise="__MSG_wikipedia__">Wikipedia</a></div>
diff --git a/src/pages/options/sendTargets/sendTargets.js b/src/pages/options/sendTargets/sendTargets.js
index ae6f9e77..dc40f83f 100644
--- a/src/pages/options/sendTargets/sendTargets.js
+++ b/src/pages/options/sendTargets/sendTargets.js
@@ -2,18 +2,28 @@ import sendTargetsHelper from "../../../assets/javascripts/helpers/sendTargets.j
 import commonHelper from "../../../assets/javascripts/helpers/common.js";
 
 let disableSendTargetsElement = document.getElementById("disable-sendTargets");
-disableSendTargetsElement.addEventListener("change",
-    (event) => sendTargetsHelper.setDisable(!event.target.checked)
-);
-
 let protocolElement = document.getElementById("protocol")
-protocolElement.addEventListener("change",
-    (event) => {
-        let protocol = event.target.options[protocolElement.selectedIndex].value
-        sendTargetsHelper.setProtocol(protocol);
-        changeProtocolSettings(protocol);
+
+browser.storage.local.get(
+    [
+        "disableSendTarget",
+        "sendTargetsProtocol",
+    ],
+    r => {
+        disableSendTargetsElement.checked = !r.disableSendTarget;
+
+        protocolElement.value = r.sendTargetsProtocol;
+        changeProtocolSettings(r.sendTargetsProtocol);
     }
-);
+)
+
+document.addEventListener("change", async () => {
+    await browser.storage.local.set({
+        disableSendTarget: !disableSendTargetsElement.checked,
+        sendTargetsProtocol: protocolElement.value,
+    })
+    changeProtocolSettings(protocolElement.value);
+})
 
 function changeProtocolSettings(protocol) {
     let normalDiv = document.getElementsByClassName("normal")[0];
@@ -28,38 +38,8 @@ function changeProtocolSettings(protocol) {
     }
 }
 
-sendTargetsHelper.init().then(() => {
-    disableSendTargetsElement.checked = !sendTargetsHelper.getDisable();
-
-    let protocol = sendTargetsHelper.getProtocol();
-    protocolElement.value = protocol;
-    changeProtocolSettings(protocol);
-
-    browser.storage.local.get("sendLatency").then(r => {
-        commonHelper.processDefaultCustomInstances(
-            'send',
-            'normal',
-            sendTargetsHelper,
-            document,
-            sendTargetsHelper.getSendNormalRedirectsChecks,
-            sendTargetsHelper.setSendNormalRedirectsChecks,
-            sendTargetsHelper.getSendNormalCustomRedirects,
-            sendTargetsHelper.setSendNormalCustomRedirects,
-            r.sendLatency,
-        );
-    })
-
-    commonHelper.processDefaultCustomInstances(
-        'send',
-        'tor',
-        sendTargetsHelper,
-        document,
-        sendTargetsHelper.getSendTorRedirectsChecks,
-        sendTargetsHelper.setSendTorRedirectsChecks,
-        sendTargetsHelper.getSendTorCustomRedirects,
-        sendTargetsHelper.setSendTorCustomRedirects
-    )
-})
+commonHelper.processDefaultCustomInstances('send', 'normal', sendTargetsHelper, document);
+commonHelper.processDefaultCustomInstances('send', 'tor', sendTargetsHelper, document,)
 
 let latencyElement = document.getElementById("latency");
 let latencyLabel = document.getElementById("latency-label");
@@ -74,17 +54,7 @@ latencyElement.addEventListener("click",
         commonHelper.testLatency(latencyLabel, redirects.send.normal).then(r => {
             browser.storage.local.set({ sendLatency: r });
             latencyLabel.innerHTML = oldHtml;
-            commonHelper.processDefaultCustomInstances(
-                'send',
-                'normal',
-                sendTargetsHelper,
-                document,
-                sendTargetsHelper.getSendNormalRedirectsChecks,
-                sendTargetsHelper.setSendNormalRedirectsChecks,
-                sendTargetsHelper.getSendNormalCustomRedirects,
-                sendTargetsHelper.setSendNormalCustomRedirects,
-                r,
-            )
+            commonHelper.processDefaultCustomInstances('send', 'normal', sendTargetsHelper, document)
             latencyElement.removeEventListener("click", reloadWindow)
         });
     }