Modification for datalog notification header (uniform naming / stability when losing connection)

Author: Pascal Dreissen

interface/web/themes/default/assets/javascripts/ispconfig.js

@@ -8,6 +8,7 @@ var ISPConfig = {
 	indicatorCompleted: false,
 	registeredHooks: new Array(),
 	new_tpl_add_id: 0,
+	dataLogTimer: 0,
 
 	options: {
 		useLoadIndicator: false,
@@ -175,6 +176,8 @@ var ISPConfig = {
 						ISPConfig.onAfterContentLoad(target, $('#'+formname).serialize());
 						ISPConfig.pageFormChanged = false;
 					}
+					clearTimeout(dataLogTimer);
+					ISPConfig.dataLogNotification();
 					ISPConfig.hideLoadIndicator();
 				},
 				error: function(jqXHR, textStatus, errorThrown) {
@@ -282,6 +285,8 @@ var ISPConfig = {
 					ISPConfig.onAfterContentLoad(pagename, (params ? params : null));
 					ISPConfig.pageFormChanged = false;
 				}
+				clearTimeout(dataLogTimer); // clear running dataLogTimer
+				ISPConfig.dataLogNotification();
 				ISPConfig.hideLoadIndicator();
 			},
 			error: function() {
@@ -346,7 +351,7 @@ var ISPConfig = {
 
 		ISPConfig.loadMenus();
 		ISPConfig.keepalive();
-		ISPConfig.datalognotification();
+		ISPConfig.dataLogNotification();
 		setTimeout(function() {
 			try {
 				$('form#pageForm').find('input[name="username"]').focus();
@@ -487,32 +492,35 @@ var ISPConfig = {
 			url: "keepalive.php",
 			dataType: "html",
 			success: function(data, textStatus, jqXHR) {
+				//alert(textStatus);
 				setTimeout( function() { ISPConfig.keepalive(); }, 1000000 );
 			},
 			error: function() {
 				ISPConfig.reportError('Session expired. Please login again.');
 			}
 		});
 	},
-	datalognotification: function() {
+	dataLogNotification: function() {
+		console.log(ISPConfig.options);
 	    var notificationContent = $.ajax({
 			type: "GET",
 			url: "datalogstatus.php",
 			dataType: "json",
 			success: function(data, textStatus, jqXHR) {
 				var dataLogItems = [];
 				$.each( data['entries'], function( key, val ) {
-						dataLogItems.push('<li><strong>' + val['text'] + ':</strong> ' + val['count'] + '</li>');
+					dataLogItems.push('<li><strong>' + val['text'] + ':</strong> ' + val['count'] + '</li>');
 				});
 				if(data['count'] > 0) {
 					$('.modal-body').html(dataLogItems.join(""));
 					$('.notification_text').text(data['count']);
 					$('.notification').css('display','');
-					setTimeout( function() { ISPConfig.datalognotification(); }, 2000 );
+					dataLogTimer = setTimeout( function() { ISPConfig.dataLogNotification(); }, 2000 );
 				} else {
 					$('.notification').css('display','none');
 					$('.modal-body').html('');
-					setTimeout( function() { ISPConfig.datalognotification(); }, 5000 );
+					$('#datalogModal').modal('hide');
+					dataLogTimer = setTimeout( function() { ISPConfig.dataLogNotification(); }, 5000 );
 				}
 			},
 			error: function() {
@@ -530,7 +538,7 @@ var ISPConfig = {
 		if(addTplId > 0) {
 			var newVal = tpl_add.split('/');
 			ISPConfig.new_tpl_add_id += 1;
-			var delbtn = $('<a href="#"><span class="glyphicon glyphicon-remove-circle" aria-hidden="true"></span></a>').attr('class', 'btn btn-danger btn-xs').click(function(e) {
+			var delbtn = $('&nbsp;<a href="#"><span class="glyphicon glyphicon-remove-circle" aria-hidden="true"></span></a>').attr('class', 'btn btn-danger btn-xs').click(function(e) {
 				e.preventDefault();
 				ISPConfig.delAdditionalTemplate($(this).parent().attr('rel'));
 			});