maint: Fix eslint errors.

Author: thet

.eslintignore

@@ -1 +1,11 @@
-module.exports = require("@patternslib/dev/eslint.config.js");
+const config_eslint = require("@patternslib/dev/eslint.config.js");
+
+module.exports = [
+    ...config_eslint,
+    {
+        ignores: [
+            // Ignore auto-generated depends_parse.js file.
+            "src/lib/depends_parse.js",
+        ],
+    },
+]

eslint.config.js

@@ -322,7 +322,7 @@ class ArgumentParser {
         if (parameter.match(this.json_param_pattern)) {
             try {
                 return JSON.parse(parameter);
-            } catch (e) {
+            } catch {
                 this.log.warn(`Invalid JSON argument found: ${parameter}.`);
             }
         }
@@ -348,7 +348,7 @@ class ArgumentParser {
                 try {
                     result[name] = this.parameters[name].value($el, name);
                     this.parameters[name].type = typeof result[name];
-                } catch (e) {
+                } catch {
                     this.log.error(`Default function for ${name} failed.`);
                 }
             } else {

src/core/parser.js

@@ -125,7 +125,7 @@ var store = {
 // reported in #326
 try {
     store.supported = typeof window.sessionStorage !== "undefined";
-} catch (e) {
+} catch {
     // just ignore.
 }
 

src/core/store.js

@@ -608,10 +608,12 @@ const debounce = (func, ms, timer = { timer: null }, postpone = true) => {
     };
 };
 
+// TODO: Remove in next major release.
 const isIE = () => {
     // See: https://stackoverflow.com/a/9851769/1337474
     // Internet Explorer 6-11
-    return /*@cc_on!@*/ false || !!document.documentMode;
+    // eslint-disable-next-line no-constant-binary-expression
+    return /*@cc_on!@*/false || !!document.documentMode;
 };
 
 const jqToNode = (el) => {

src/core/utils.js

@@ -77,10 +77,14 @@ export default Base.extend({
                     selectionClasses = JSON.parse(this.options.selectionClasses)[
                         obj.text
                     ];
-                } catch (SyntaxError) {
-                    log.error(
-                        "SyntaxError: non-JSON data given to pat-autosuggest (selection-classes)"
-                    );
+                } catch (e) {
+                    if (e instanceof SyntaxError) {
+                        log.error(
+                            "SyntaxError: non-JSON data given to pat-autosuggest (selection-classes)"
+                        );
+                    } else {
+                        throw e;
+                    }
                 }
                 if (selectionClasses) {
                     // According to Cornelis the classes need to be applied on
@@ -161,9 +165,13 @@ export default Base.extend({
         if (this.options.wordsJson?.length) {
             try {
                 words = JSON.parse(this.options.wordsJson);
-            } catch (SyntaxError) {
-                words = [];
-                log.error("SyntaxError: non-JSON data given to pat-autosuggest");
+            } catch (e) {
+                if (e instanceof SyntaxError) {
+                    words = [];
+                    log.error("SyntaxError: non-JSON data given to pat-autosuggest");
+                } else {
+                    throw e;
+                }
             }
             if (!Array.isArray(words)) {
                 words = words.map((v, k) => {
@@ -243,8 +251,12 @@ export default Base.extend({
                     }
                     callback(_data);
                 };
-            } catch (SyntaxError) {
-                log.error("SyntaxError: non-JSON data given to pat-autosuggest");
+            } catch (e) {
+                if (e instanceof SyntaxError) {
+                    log.error("SyntaxError: non-JSON data given to pat-autosuggest");
+                } else {
+                    throw e;
+                }
             }
         }
 

src/pat/auto-suggest/auto-suggest.js

@@ -30,7 +30,7 @@ var testutils = {
         var cfg = c || {};
         return $("<input/>", {
             "id": cfg.id || "select2",
-            "data-pat-autosuggest": "" || cfg.data,
+            "data-pat-autosuggest": cfg.data,
             "class": "pat-autosuggest",
             "type": "text",
         }).appendTo($("div#lab"));

src/pat/auto-suggest/auto-suggest.test.js

@@ -280,7 +280,7 @@ const inject = {
             if (cfg.delay) {
                 try {
                     cfg.delay = utils.parseTime(cfg.delay);
-                } catch (e) {
+                } catch {
                     log.warn("Invalid delay value: ", cfg.delay);
                     cfg.delay = null;
                 }

src/pat/inject/inject.js

@@ -96,7 +96,7 @@ export default Base.extend({
         var containerStyle;
         try {
             containerStyle = JSON.parse(this.options.containerStyle);
-        } catch (e) {
+        } catch {
             containerStyle = { position: "relative" };
             log.warn(
                 "Invalid value passed in as containerStyle. Needs to " +

src/pat/masonry/masonry.js

@@ -55,7 +55,7 @@ export default Base.extend({
         try {
             const response = await fetch(this.options.url);
             data = await response.text();
-        } catch (e) {
+        } catch {
             logger.error(
                 `Could not fetch from ${this.options.url} on push-id ${this.options.pushId}.`
             );
@@ -110,7 +110,7 @@ export default Base.extend({
         try {
             const response = await fetch(this.options.url);
             data = await response.json();
-        } catch (e) {
+        } catch {
             logger.error(
                 `Could not fetch from ${this.options.url} on push-id ${this.options.pushId}.`
             );