Add optional chaining to event method calls

jonobr1 · jonobr1 · commit 2a02ac97cc7f · 2025-12-22T10:48:03.000-05:00
Updated event-related method calls in Collection and Two classes to use optional chaining. This prevents errors if the underlying event methods are undefined.
diff --git a/src/collection.js b/src/collection.js
@@ -30,34 +30,34 @@ export class Collection extends Array {
   }
 
   addEventListener() {
-    return this.#events.addEventListener.apply(this, arguments);
+    return this.#events.addEventListener?.apply(this, arguments);
   }
   on() {
-    return this.#events.on.apply(this, arguments);
+    return this.#events.on?.apply(this, arguments);
   }
   bind() {
-    return this.#events.bind.apply(this, arguments);
+    return this.#events.bind?.apply(this, arguments);
   }
   removeEventListener() {
-    return this.#events.removeEventListener.apply(this, arguments);
+    return this.#events.removeEventListener?.apply(this, arguments);
   }
   off() {
-    return this.#events.off.apply(this, arguments);
+    return this.#events.off?.apply(this, arguments);
   }
   unbind() {
-    return this.#events.unbind.apply(this, arguments);
+    return this.#events.unbind?.apply(this, arguments);
   }
   dispatchEvent() {
-    return this.#events.dispatchEvent.apply(this, arguments);
+    return this.#events.dispatchEvent?.apply(this, arguments);
   }
   trigger() {
-    return this.#events.trigger.apply(this, arguments);
+    return this.#events.trigger?.apply(this, arguments);
   }
   listen() {
-    return this.#events.listen.apply(this, arguments);
+    return this.#events.listen?.apply(this, arguments);
   }
   ignore() {
-    return this.#events.ignore.apply(this, arguments);
+    return this.#events.ignore?.apply(this, arguments);
   }
 
   constructor() {
diff --git a/src/two.js b/src/two.js
@@ -100,34 +100,34 @@ export default class Two {
   }
 
   addEventListener() {
-    return this._events.addEventListener.apply(this, arguments);
+    return this._events.addEventListener?.apply(this, arguments);
   }
   on() {
-    return this._events.addEventListener.apply(this, arguments);
+    return this._events.addEventListener?.apply(this, arguments);
   }
   bind() {
-    return this._events.addEventListener.apply(this, arguments);
+    return this._events.addEventListener?.apply(this, arguments);
   }
   removeEventListener() {
-    return this._events.removeEventListener.apply(this, arguments);
+    return this._events.removeEventListener?.apply(this, arguments);
   }
   off() {
-    return this._events.removeEventListener.apply(this, arguments);
+    return this._events.removeEventListener?.apply(this, arguments);
   }
   unbind() {
-    return this._events.removeEventListener.apply(this, arguments);
+    return this._events.removeEventListener?.apply(this, arguments);
   }
   dispatchEvent() {
-    return this._events.dispatchEvent.apply(this, arguments);
+    return this._events.dispatchEvent?.apply(this, arguments);
   }
   trigger() {
-    return this._events.dispatchEvent.apply(this, arguments);
+    return this._events.dispatchEvent?.apply(this, arguments);
   }
   listen() {
-    return this._events.listen.apply(this, arguments);
+    return this._events.listen?.apply(this, arguments);
   }
   ignore() {
-    return this._events.ignore.apply(this, arguments);
+    return this._events.ignore?.apply(this, arguments);
   }
 
   /**

Original file line number	Diff line number	Diff line change
`@@ -30,34 +30,34 @@ export class Collection extends Array {`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`addEventListener() {`
`33`		`- return this.#events.addEventListener.apply(this, arguments);`
	`33`	`+ return this.#events.addEventListener?.apply(this, arguments);`
`34`	`34`	`}`
`35`	`35`	`on() {`
`36`		`- return this.#events.on.apply(this, arguments);`
	`36`	`+ return this.#events.on?.apply(this, arguments);`
`37`	`37`	`}`
`38`	`38`	`bind() {`
`39`		`- return this.#events.bind.apply(this, arguments);`
	`39`	`+ return this.#events.bind?.apply(this, arguments);`
`40`	`40`	`}`
`41`	`41`	`removeEventListener() {`
`42`		`- return this.#events.removeEventListener.apply(this, arguments);`
	`42`	`+ return this.#events.removeEventListener?.apply(this, arguments);`
`43`	`43`	`}`
`44`	`44`	`off() {`
`45`		`- return this.#events.off.apply(this, arguments);`
	`45`	`+ return this.#events.off?.apply(this, arguments);`
`46`	`46`	`}`
`47`	`47`	`unbind() {`
`48`		`- return this.#events.unbind.apply(this, arguments);`
	`48`	`+ return this.#events.unbind?.apply(this, arguments);`
`49`	`49`	`}`
`50`	`50`	`dispatchEvent() {`
`51`		`- return this.#events.dispatchEvent.apply(this, arguments);`
	`51`	`+ return this.#events.dispatchEvent?.apply(this, arguments);`
`52`	`52`	`}`
`53`	`53`	`trigger() {`
`54`		`- return this.#events.trigger.apply(this, arguments);`
	`54`	`+ return this.#events.trigger?.apply(this, arguments);`
`55`	`55`	`}`
`56`	`56`	`listen() {`
`57`		`- return this.#events.listen.apply(this, arguments);`
	`57`	`+ return this.#events.listen?.apply(this, arguments);`
`58`	`58`	`}`
`59`	`59`	`ignore() {`
`60`		`- return this.#events.ignore.apply(this, arguments);`
	`60`	`+ return this.#events.ignore?.apply(this, arguments);`
`61`	`61`	`}`
`62`	`62`
`63`	`63`	`constructor() {`
Original file line number	Diff line number	Diff line change
`@@ -100,34 +100,34 @@ export default class Two {`
`100`	`100`	`}`
`101`	`101`
`102`	`102`	`addEventListener() {`
`103`		`- return this._events.addEventListener.apply(this, arguments);`
	`103`	`+ return this._events.addEventListener?.apply(this, arguments);`
`104`	`104`	`}`
`105`	`105`	`on() {`
`106`		`- return this._events.addEventListener.apply(this, arguments);`
	`106`	`+ return this._events.addEventListener?.apply(this, arguments);`
`107`	`107`	`}`
`108`	`108`	`bind() {`
`109`		`- return this._events.addEventListener.apply(this, arguments);`
	`109`	`+ return this._events.addEventListener?.apply(this, arguments);`
`110`	`110`	`}`
`111`	`111`	`removeEventListener() {`
`112`		`- return this._events.removeEventListener.apply(this, arguments);`
	`112`	`+ return this._events.removeEventListener?.apply(this, arguments);`
`113`	`113`	`}`
`114`	`114`	`off() {`
`115`		`- return this._events.removeEventListener.apply(this, arguments);`
	`115`	`+ return this._events.removeEventListener?.apply(this, arguments);`
`116`	`116`	`}`
`117`	`117`	`unbind() {`
`118`		`- return this._events.removeEventListener.apply(this, arguments);`
	`118`	`+ return this._events.removeEventListener?.apply(this, arguments);`
`119`	`119`	`}`
`120`	`120`	`dispatchEvent() {`
`121`		`- return this._events.dispatchEvent.apply(this, arguments);`
	`121`	`+ return this._events.dispatchEvent?.apply(this, arguments);`
`122`	`122`	`}`
`123`	`123`	`trigger() {`
`124`		`- return this._events.dispatchEvent.apply(this, arguments);`
	`124`	`+ return this._events.dispatchEvent?.apply(this, arguments);`
`125`	`125`	`}`
`126`	`126`	`listen() {`
`127`		`- return this._events.listen.apply(this, arguments);`
	`127`	`+ return this._events.listen?.apply(this, arguments);`
`128`	`128`	`}`
`129`	`129`	`ignore() {`
`130`		`- return this._events.ignore.apply(this, arguments);`
	`130`	`+ return this._events.ignore?.apply(this, arguments);`
`131`	`131`	`}`
`132`	`132`
`133`	`133`	`/**`