@wavemaker/app-ng-runtime 11.14.1-3.6306 → 11.14.1-6.6324

package/components/base/fesm2022/index.mjs

@@ -1,5 +1,5 @@
 import * as i1 from '@wm/core';
-import { FormWidgetType, checkIsCustomPipeExpression, $parseExpr, isValidWebURL, _WM_APP_PROJECT, isValidImageUrl, encodeUrl, stringStartsWith, prettifyLabel, deHyphenate, initCaps, getClonedObject, DataSource, debounce, processFilterExpBindNode, isDefined, MatchMode, $parseEvent, isEqualWithFields, getFormattedDate, isAndroid, isNumberType, DataType, setCSS, IDGenerator, toBoolean, isChangeFromWatch, $unwatch, resetChangeFromWatch, isObject as isObject$1, toDimension, $appDigest, removeClass, addClass, setCSSFromObj, App, findParent, setAttr, switchClass, removeAttr, $invokeWatchers, $watch, $unwatchAll, findValueOf, validateAccessRoles, ComponentType, noop as noop$1, VALIDATOR, isIos, isMobile, hasOffsetStr, AbstractI18nService, CURRENCY_INFO, replace as replace$1, transformFileURI, isLargeTabletLandscape, isLargeTabletPortrait } from '@wm/core';
+import { FormWidgetType, checkIsCustomPipeExpression, $parseExpr, isValidWebURL, _WM_APP_PROJECT, isValidImageUrl, encodeUrl, stringStartsWith, prettifyLabel, deHyphenate, initCaps, getClonedObject, DataSource, debounce, processFilterExpBindNode, isDefined, MatchMode, $parseEvent, isEqualWithFields, getFormattedDate, isAndroid, isNumberType, DataType, setCSS, IDGenerator, toBoolean, isChangeFromWatch, $unwatch, resetChangeFromWatch, isObject as isObject$1, toDimension, $appDigest, removeClass, addClass, setCSSFromObj, App, findParent, setAttr, switchClass, removeAttr, $invokeWatchers, $watch, findValueOf, validateAccessRoles, ComponentType, noop as noop$1, VALIDATOR, isIos, isMobile, hasOffsetStr, AbstractI18nService, CURRENCY_INFO, replace as replace$1, transformFileURI, isLargeTabletLandscape, isLargeTabletPortrait } from '@wm/core';
 import * as i0 from '@angular/core';
 import { InjectionToken, forwardRef, ElementRef, inject, ViewContainerRef, Injector, Injectable, Inject, Optional, Directive, Self, ContentChildren, Attribute, Input, Component, Pipe, SecurityContext } from '@angular/core';
 import { createFocusTrap } from '@wavemaker/focus-trap';
@@ -1994,12 +1994,7 @@ const parseValue = (key, value, type) => {
         return toBoolean(value, isBooleanAttr(key) && key);
     }
     if (type === PROP_TYPE.NUMBER) {
-        const numValue = +value;
-        // Fix NaN for tabindex - default to 0 if value is invalid
-        if (key === 'tabindex' && isNaN(numValue)) {
-            return value === null || value === undefined || value === '' ? 0 : numValue;
-        }
-        return numValue;
+        return +value;
     }
     return value;
 };
@@ -2323,26 +2318,10 @@ class BaseComponent {
         this.propertyChange.subscribe(({ key, nv, ov }) => fn(key, nv, ov));
     }
     registerDestroyListener(fn, ctx) {
-        if (!fn) {
-            return;
-        }
         if (ctx) {
             fn = fn.bind(ctx);
         }
-        this.destroy.subscribe(() => { }, () => { }, () => {
-            try {
-                if (fn && typeof fn === 'function') {
-                    fn();
-                }
-            }
-            catch (e) {
-                // Silently handle cleanup errors to prevent test failures
-                // These are typically non-critical cleanup issues like:
-                // - Subscriptions already unsubscribed
-                // - Null references during cleanup
-                // - Components already destroyed
-            }
-        });
+        this.destroy.subscribe(() => { }, () => { }, () => fn());
     }
     getDisplayType() {
         return this.displayType;
@@ -2508,24 +2487,14 @@ class BaseComponent {
                 // the promise returned by the variable call.
                 boundFnVal = boundFn();
                 if (boundFnVal instanceof Promise) {
-                    return boundFnVal.then(response => response, err => {
-                        const widgetName = widget.name || widget.widgetId || 'unknown';
-                        console.error(`Error in promise returned by event callback "${eventName}" on widget "${widgetName}":`, err);
-                        return err;
-                    });
+                    return boundFnVal.then(response => response, err => err);
                 }
                 else {
                     return boundFnVal;
                 }
             }
             catch (e) {
-                // Enhanced error logging with context
-                const widgetName = widget.name || widget.widgetId || 'unknown';
-                const errorMessage = e.message || String(e);
-                console.error(`Error executing event callback "${eventName}" on widget "${widgetName}":`, errorMessage);
-                console.error('Event expression context:', { eventName, widgetName, locals });
-                // Re-throw to maintain original behavior but with better context
-                throw e;
+                console.error(e);
             }
         };
         if (child && !child.widget.eventHandlers)
@@ -2549,17 +2518,11 @@ class BaseComponent {
         if (!child)
             this.initState.delete(propName);
         let viewParent = child ? child.widget.viewParent.viewParent : this.viewParent, context = child ? child.widget.context : this.context, widget = child ? child.widget : this.widget, isMuted = child ? child.widget.isMuted : this.isMuted, widgetProps = child ? child.widget.widgetProps : this.widgetProps, widgetId = child ? child.widget.widgetId : this.widgetId;
-        // OPTIMIZATION: Apply throttling for non-critical properties
-        const watchConfig = widgetProps.get(propName) || {};
-        const throttledProps = ['class', 'style', 'ngclass', 'ngstyle'];
-        if (throttledProps.includes(propName.toLowerCase())) {
-            watchConfig.throttle = 100; // Throttle to 100ms for style/class properties
-        }
         this.registerDestroyListener($watch(expr, viewParent, context, nv => {
             if (propName.startsWith('base-') && widget.updateData)
                 widget.updateData(propName, nv);
             widget[propName] = nv;
-        }, getWatchIdentifier(widgetId, propName), propName === 'datasource', watchConfig, () => isMuted));
+        }, getWatchIdentifier(widgetId, propName), propName === 'datasource', widgetProps.get(propName), () => isMuted));
     }
     /**
      * Remove watch on the bound property
@@ -2578,18 +2541,7 @@ class BaseComponent {
             const fn = callbackInfo.callback;
             const locals = callbackInfo.locals || {};
             if (fn) {
-                try {
-                    return fn(Object.assign(locals, extraLocals));
-                }
-                catch (error) {
-                    // Log error with context information for debugging
-                    const widgetName = this.name || this.widgetId || 'unknown';
-                    const errorMessage = error?.message || String(error);
-                    console.error(`Error executing event callback "${eventName}" on widget "${widgetName}":`, errorMessage);
-                    console.error('Full error:', error);
-                    // Re-throw to maintain original behavior but with better error context
-                    throw error;
-                }
+                return fn(Object.assign(locals, extraLocals));
             }
         }
     }
@@ -2809,24 +2761,6 @@ class BaseComponent {
         this.propertyChange.complete();
         this.destroy.complete();
         this.isAttached = false;
-        // MEMORY LEAK FIX: Unwatch all watchers for this widget
-        // This ensures all watchers registered for this component are cleaned up
-        if (this.widgetId) {
-            const removedCount = $unwatchAll(this.widgetId);
-            if (removedCount > 0) {
-                console.log(`🧹 Cleaned up ${removedCount} watcher(s) for widget ${this.widgetId}`);
-            }
-        }
-        // MEMORY LEAK FIX: Clear context to release references to parent scope data
-        // The context object holds references to repeater items, wizard steps, etc.
-        // which can retain large amounts of data even after component destruction
-        if (this.context) {
-            // Clear all properties from context
-            Object.keys(this.context).forEach(key => {
-                delete this.context[key];
-            });
-            this.context = null;
-        }
     }
     static { this.ɵfac = function BaseComponent_Factory(__ngFactoryType__) { return new (__ngFactoryType__ || BaseComponent)(i0.ɵɵinject(i0.Injector), i0.ɵɵinject(WidgetConfig), i0.ɵɵinject('EXPLICIT_CONTEXT', 8), i0.ɵɵinject(Promise)); }; }
     static { this.ɵprov = /*@__PURE__*/ i0.ɵɵdefineInjectable({ token: BaseComponent, factory: BaseComponent.ɵfac }); }
@@ -5085,9 +5019,7 @@ class LazyLoadDirective {
         this.unSubscribeFn = $watch(expr, this.viewParent, this.context, (val) => {
             if (!this.embeddedView && val) {
                 this.embeddedView = this.viewContainer.createEmbeddedView(this.templateRef, this.context);
-                if (this.unSubscribeFn && typeof this.unSubscribeFn === 'function') {
-                    this.unSubscribeFn();
-                }
+                this.unSubscribeFn();
             }
         });
     }