From 509b627e880a635e3c5fa22020138c31bfbec541 Mon Sep 17 00:00:00 2001
From: Ajay Ramachandran <dev@ajay.app>
Date: Wed, 1 Jan 2020 12:55:35 -0500
Subject: [PATCH] Made it reenable the permission when an invidious instance is
 added.

Right now it throws an error
---
 options/options.html |  2 +-
 options/options.js   | 41 ++++++++++++++++++++++++++---------------
 2 files changed, 27 insertions(+), 16 deletions(-)

diff --git a/options/options.html b/options/options.html
index 49b3ece7..b48aa358 100644
--- a/options/options.html
+++ b/options/options.html
@@ -21,7 +21,7 @@
 	
 		<div id="options" class="hidden">
 
-			<div option-type="toggle" sync-option="supportInvidious">
+			<div id="support-invidious" option-type="toggle" sync-option="supportInvidious">
 				<label class="switch-container" label-name="__MSG_supportInvidious__">
 					<label class="switch">
 						<input type="checkbox">
diff --git a/options/options.js b/options/options.js
index 6802448c..7e4d7010 100644
--- a/options/options.js
+++ b/options/options.js
@@ -270,24 +270,35 @@ function activateTextChange(element) {
         case "invidiousInstances":
 
             let setButton = element.querySelector(".text-change-set");
-            setButton.addEventListener("click", function(e) {
+            setButton.addEventListener("click", async function(e) {
                 if (textBox.value.includes("/") || textBox.value.includes("http") || textBox.value.includes(":")) {
                     alert(chrome.i18n.getMessage("addInvidiousInstanceError"));
                 } else {
                     // Add this
-                    chrome.storage.sync.get([option], function(result) {
-                        if (!result[option]) result[option] = [];
-
-                        result[option].push(textBox.value);
-
-                        setOptionValue(option, result[option]);
-
-                        textBox.value = "";
-
-                        // Hide this section again
-                        element.querySelector(".option-hidden-section").classList.add("hidden");
-                        button.classList.remove("disabled");
+                    //TODO Make the call to invidiousOnClick support passing the straight extra values, plus make the get not needed
+                    //OR merge the config PR and use that
+                    let result = await new Promise((resolve, reject) => {
+                        chrome.storage.sync.get([option], resolve);
                     });
+
+                    if (!result[option]) result[option] = [];
+
+                    result[option].push(textBox.value);
+
+                    await new Promise((resolve, reject) => {
+                        setOptionValue(option, result[option], resolve);
+                    });
+
+                    let checkbox = document.querySelector("#support-invidious input");
+                    checkbox.checked = true;
+
+                    invidiousOnClick(checkbox, "supportInvidious");
+
+                    textBox.value = "";
+
+                    // Hide this section again
+                    element.querySelector(".option-hidden-section").classList.add("hidden");
+                    button.classList.remove("disabled");
                 }
             });
 
@@ -319,6 +330,6 @@ function activateTextChange(element) {
  * @param {string} option 
  * @param {*} value 
  */
-function setOptionValue(option, value) {
-    chrome.storage.sync.set({[option]: value});
+function setOptionValue(option, value, callback) {
+    chrome.storage.sync.set({[option]: value}, callback);
 }
\ No newline at end of file