fixes #503

Author: rdkmaster

src/app/demo/notification/full/demo.component.ts

@@ -47,7 +47,8 @@ export class NotificationFullDemoComponent {
     showWithOptions() {
         JigsawNotification.show(this.message, {
             caption: this.caption, position: this.position, icon: this.icon,
-            timeout: this.timeout * 1000, width: this.width, height: this.height
+            timeout: this.timeout * 1000, width: this.width, height: this.height,
+            innerHtmlContext: this
         });
     }
 

src/jigsaw/directive/trusted-html/trusted-html.ts

@@ -10,29 +10,36 @@ type CallbackValues = { [callbackName: string]: HtmlCallback };
 })
 export class JigsawTrustedHtml implements OnInit, OnDestroy {
     private static _callbacks = new Map<any, CallbackValues>();
-    private static _contexts = [];
+    private static _contexts: {context: any, counter: number}[] = [];
 
     private static _getContextMagicNumber(context: any): number {
-        return JigsawTrustedHtml._contexts.indexOf(context);
+        return JigsawTrustedHtml._contexts.findIndex(i => i && i.context === context);
     }
 
     private static _getContext(magicNumber: number): any {
         return JigsawTrustedHtml._contexts[magicNumber];
     }
 
     private static _registerContext(context: any): void {
-        if (CommonUtils.isDefined(context) && JigsawTrustedHtml._getContextMagicNumber(context) == -1) {
-            JigsawTrustedHtml._contexts.push(context);
+        if (CommonUtils.isUndefined(context)) {
+            return;
+        }
+        let info = JigsawTrustedHtml._contexts.find(i => i && i.context === context);
+        if (CommonUtils.isUndefined(info)) {
+            info = {context: context, counter: 1};
+            JigsawTrustedHtml._contexts.push(info);
+        } else {
+            info.counter++;
         }
     }
 
     private static _jigsawInternalCallbackWrapper(callbackName: string, contextMagicNumber: number, ...args) {
-        const context = JigsawTrustedHtml._getContext(contextMagicNumber);
-        if (CommonUtils.isUndefined(context)) {
+        const contextInfo = JigsawTrustedHtml._getContext(contextMagicNumber);
+        if (CommonUtils.isUndefined(contextInfo)) {
             console.error('no context found by magic number, callbackName = ' + callbackName);
             return;
         }
-        const callbacks = JigsawTrustedHtml._callbacks.get(context);
+        const callbacks = JigsawTrustedHtml._callbacks.get(contextInfo.context);
         if (CommonUtils.isUndefined(callbacks)) {
             console.error('no callback cache info found by magic number, callbackName = ' + callbackName);
             return;
@@ -43,7 +50,7 @@ export class JigsawTrustedHtml implements OnInit, OnDestroy {
             console.log(`Hint: add a member method named ${callbackName} to the context object.`);
             return;
         }
-        CommonUtils.safeInvokeCallback(context, callback, args);
+        CommonUtils.safeInvokeCallback(contextInfo.context, callback, args);
     }
 
     private static _declareCallback(context: any, name: string, callback: HtmlCallback) {
@@ -65,11 +72,22 @@ export class JigsawTrustedHtml implements OnInit, OnDestroy {
             callbacks = {};
             JigsawTrustedHtml._callbacks.set(context, callbacks);
         }
-        callbacks[name] = CommonUtils.isDefined(callback) ? callback : context;
+        callbacks[name] = callback;
     }
 
     private static _clearCallbacks(context: any) {
-        JigsawTrustedHtml._callbacks.delete(context);
+        const idx = JigsawTrustedHtml._contexts.findIndex(i => i && i.context === context);
+        if (idx == -1) {
+            return;
+        }
+        const info = JigsawTrustedHtml._contexts[idx];
+        info.counter--;
+        if (info.counter == 0) {
+            JigsawTrustedHtml._callbacks.delete(context);
+            info.context = null;
+            info.counter = -1;
+            JigsawTrustedHtml._contexts[idx] = null;
+        }
     }
 
     //====================================================
@@ -92,9 +110,6 @@ export class JigsawTrustedHtml implements OnInit, OnDestroy {
         if (CommonUtils.isUndefined(value)) {
             return;
         }
-        if (CommonUtils.isDefined(this._trustedHtmlContext)) {
-            JigsawTrustedHtml._clearCallbacks(this._trustedHtmlContext);
-        }
         this._trustedHtmlContext = value;
         JigsawTrustedHtml._registerContext(value);
         this._contextMagicNumber = JigsawTrustedHtml._getContextMagicNumber(value);