eliminate var

Author: yfdyh000

background.js

@@ -19,7 +19,7 @@ function createContextMenu(options) {
     contexts: ["image"]
   });
   for (let searchProvider of options.searchProviders) {
-    var contextMenuOptions = {
+    let contextMenuOptions = {
       parentId: "Image-Reverse-Search",
       id: searchProvider,
       icons: {
@@ -38,7 +38,7 @@ function createContextMenu(options) {
 }
 
 /* Default settings. If there is nothing in storage, use these values. */
-var defaultSettings = {
+const defaultSettings = {
   openInBackground: false,
   openTabAt: "right",
   searchProviders: ["google"]

options/options.js

@@ -47,7 +47,7 @@ function storeSettings() {
       }
   }
 
-  var searchProviders = getSearchProviders();
+  const searchProviders = getSearchProviders();
 
   // if no checkbox is selected
   const searchProvidersError = document.getElementById("searchProvidersError");
@@ -71,6 +71,7 @@ function storeSettings() {
   }
 
   /* Check if custom search provider is valid */
+  let cseProvider = "";
   if (searchProviders.includes("other")) {
       const cseForm = document.getElementById("customSearchProvider");
       if (cseForm.checkValidity() == false || cseForm.value == null || cseForm.value == "") {
@@ -80,13 +81,13 @@ function storeSettings() {
           if (cseForm.value.indexOf("%s") == "-1") { // no %s in URL!
               toggleCSEform(false);
           } else {
-            var cseProvider = cseForm.value;
+              cseProvider = cseForm.value;
           }
       }
   }
 
   /* If custom search provider is not valid, but checked */
-  if (typeof cseProvider == "undefined" && searchProviders.includes("other")) {
+  if (cseProvider === "" && searchProviders.includes("other")) {
       const status = document.getElementById("status");
       status.classList.add("alert-danger");
       status.textContent = chrome.i18n.getMessage("error");
@@ -97,8 +98,6 @@ function storeSettings() {
         status.classList.remove("alert-danger");
       }, 1800);
       return
-  } else if (typeof cseProvider == "undefined") {
-      var cseProvider = "";
   }
 
   const openInBackground = document.getElementById("openInBackground").checked;