From 497145b1b59704aefd64045e7f9146186ee072a3 Mon Sep 17 00:00:00 2001 From: "E:V:A" Date: Mon, 2 Apr 2018 14:07:25 +0300 Subject: [PATCH] null check for notification removal Make sure there is something to remove, before we attempt to remove the notifications. - This fixes #1240 --- modules/default/alert/alert.js | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/modules/default/alert/alert.js b/modules/default/alert/alert.js index c5d3e650..7845b5d0 100644 --- a/modules/default/alert/alert.js +++ b/modules/default/alert/alert.js @@ -109,12 +109,14 @@ Module.register("alert",{ }, hide_alert: function(sender) { - //Dismiss alert and remove from this.alerts - this.alerts[sender.name].dismiss(); - this.alerts[sender.name] = null; - //Remove overlay - var overlay = document.getElementById("overlay"); - overlay.parentNode.removeChild(overlay); + //Dismiss alert and remove from this.alerts + if (this.alerts[sender.name]) { + this.alerts[sender.name].dismiss(); + this.alerts[sender.name] = null; + //Remove overlay + var overlay = document.getElementById("overlay"); + overlay.parentNode.removeChild(overlay); + } }, setPosition: function(pos) { //Add css to body depending on the set position for notifications