core.Observable: addListener() => remove default values from the eventConfig #6657

Author: tobiu

src/core/Observable.mjs

@@ -101,14 +101,12 @@ class Observable extends Base {
         }
 
         if (!nameObject) {
-            eventConfig = {
-                data,
-                delay,
-                fn: listener,
-                id: eventId || Neo.getId('event'),
-                once,
-                scope
-            };
+            eventConfig = {fn: listener, id: eventId || Neo.getId('event')};
+
+            if (data)      {eventConfig.data   = data}
+            if (delay > 0) {eventConfig.delay  = delay}
+            if (once)      {eventConfig.once   = once}
+            if (scope)     {eventConfig.scope  = scope}
 
             if (existing = me.listeners?.[name]) {
                 existing.forEach(cfg => {
@@ -194,10 +192,6 @@ class Observable extends Base {
                 // look in the ownership hierarchy from me.
                 const cb = resolveCallback(handler.fn, handler.scope || me);
 
-                if (Neo.isString(cb.fn)) {
-                    console.log(name, handler, cb);
-                }
-
                 // remove the listener if the scope no longer exists
                 if (cb.scope && !cb.scope.id) {
                     NeoArray.remove(listeners[name], handler)