lint fixes and behvaiour consistency for eventhandle

Author: Maksims

src/core/event-handle.js

@@ -5,16 +5,16 @@
  *     console.log(a + b);
  * });
  * obj.fire('test');
- * 
+ *
  * evt.off(); // easy way to remove this event
  * obj.fire('test'); // this will not trigger an event
  * @example
  * // store an array of event handles
  * let events = [ ];
- * 
+ *
  * events.push(objA.on('testA', () => { }));
  * events.push(objB.on('testB', () => { }));
- * 
+ *
  * // when needed, remove all events
  * events.forEach((evt) => {
  *     evt.off();
@@ -55,9 +55,9 @@ class EventHandle {
     /**
      * True if event has been removed.
      * @type {boolean}
-     * @readonly
+     * @private
      */
-    removed = false;
+    _removed = false;
 
     /**
      * @param {EventHandler} handler - source object of the event.
@@ -78,20 +78,26 @@ class EventHandle {
      * Remove references.
      */
     destroy() {
-        if (this.removed) return;
-        this.removed = true;
-        this.handler = null;
-        this.callback = null;
-        this.scope = null;
+        if (this._removed) return;
+        this._removed = true;
     }
 
     /**
      * Remove this event from its handler.
      */
     off() {
-        if (this.removed) return;
+        if (this._removed) return;
         this.handler.off(this.name, this.callback, this.scope);
     }
+
+    /**
+     * True if event has been removed.
+     * @type {boolean}
+     * @readonly
+     */
+    get removed() {
+        return this._removed;
+    }
 }
 
-export { EventHandle };
+export { EventHandle };

src/core/event-handler.js

@@ -33,7 +33,6 @@ class EventHandler {
     /**
      * When set to true, `on` and `once` methods will return `this`.
      * When set to false, `on` and `once` will return `EventHandle`.
-     * 
      * @type {boolean}
      */
     static chaining = true;
@@ -82,7 +81,7 @@ class EventHandler {
         if (this._callbackActive[name] && this._callbackActive[name] === this._callbacks[name])
             this._callbackActive[name] = this._callbackActive[name].slice();
 
-        let evt = new EventHandle(this, name, callback, scope, once);
+        const evt = new EventHandle(this, name, callback, scope, once);
         this._callbacks[name].push(evt);
         return evt;
     }
@@ -143,9 +142,9 @@ class EventHandler {
         }
 
         if (!name) {
-            for(let name in this._callbacks) {
+            for (const name in this._callbacks) {
                 const handles = this._callbacks[name];
-                for(let i = 0; i < handles.length; i++) {
+                for (let i = 0; i < handles.length; i++) {
                     handles[i].destroy();
                 }
             }
@@ -154,7 +153,7 @@ class EventHandler {
         } else if (!callback) {
             const handles = this._callbacks[name];
             if (handles) {
-                for(let i = 0; i < handles.length; i++) {
+                for (let i = 0; i < handles.length; i++) {
                     handles[i].destroy();
                 }
                 this._callbacks[name] = [];