Change the type of debounce, use arrow functions (#9328)

src/common/entity/domain_icon.ts

@@ -89,8 +89,6 @@ export const domainIcon = (
   }
 
   // eslint-disable-next-line
-  console.warn(
+  console.warn(`Unable to find icon for domain ${domain}`);
-    "Unable to find icon for domain " + domain + " (" + stateObj + ")"
-  );
   return DEFAULT_DOMAIN_ICON;
 };

src/common/util/debounce.ts

@@ -4,29 +4,25 @@
 // be triggered. The function will be called after it stops being called for
 // N milliseconds. If `immediate` is passed, trigger the function on the
 // leading edge, instead of the trailing.
-// eslint-disable-next-line: ban-types
+
-export const debounce = <T extends (...args) => unknown>(
+export const debounce = <T extends any[]>(
-  func: T,
+  func: (...args: T) => void,
-  wait,
+  wait: number,
   immediate = false
-): T => {
+) => {
-  let timeout;
+  let timeout: number | undefined;
-  // @ts-ignore
+  return (...args: T): void => {
-  return function (...args) {
-    // @ts-ignore
-    // eslint-disable-next-line @typescript-eslint/no-this-alias
-    const context = this;
     const later = () => {
-      timeout = null;
+      timeout = undefined;
       if (!immediate) {
-        func.apply(context, args);
+        func(...args);
       }
     };
     const callNow = immediate && !timeout;
     clearTimeout(timeout);
-    timeout = setTimeout(later, wait);
+    timeout = window.setTimeout(later, wait);
     if (callNow) {
-      func.apply(context, args);
+      func(...args);
     }
   };
 };