Enhanced error display in all admin templates. Fixed minor issues.

src/admin/views.php

@@ -58,6 +58,9 @@ IncludeLanguageFile( $gl_root_path . '/lang/' . $LANG . '/admin.php' );
 //if ( !isset($_SESSION['SESSION_ISADMIN']) || $_SESSION['SESSION_ISADMIN'] == 0 ) 
 //	DieWithFriendlyErrorMsg( $content['LN_ADMIN_ERROR_NOTALLOWED'] );
 
+// Init helper variable to empty string
+$content['FormUrlAddOP'] = "";
+
 if ( isset($_GET['op']) )
 {
 	if ($_GET['op'] == "add") 
@@ -72,6 +75,7 @@ if ( isset($_GET['op']) )
 		$content['userid'] = null;
 		$content['CHECKED_ISUSERONLY'] = "";
 		$content['VIEWID'] = "";
+		$content['FormUrlAddOP'] = "?op=add";
 
 		// --- Check if groups are available
 		$content['SUBGROUPS'] = GetGroupsForSelectfield();
@@ -88,7 +92,6 @@ if ( isset($_GET['op']) )
 		$content['VIEW_FORMACTION'] = "editview";
 		$content['VIEW_SENDBUTTON'] = $content['LN_VIEWS_EDIT'];
 
-
 		// Copy Views array for further modifications
 		$content['VIEWS'] = $content['Views'];
 
@@ -99,6 +102,10 @@ if ( isset($_GET['op']) )
 			$content['VIEWID'] = DB_RemoveBadChars($_GET['id']);
 			if ( isset($content['VIEWS'][ $content['VIEWID'] ]) )
 			{
+
+				//Set the FormAdd URL
+				$content['FormUrlAddOP'] = "?op=edit&id=" . $content['VIEWID'];
+
 				$myview = $content['VIEWS'][ $content['VIEWID'] ];
 
 				$content['DisplayName'] = $myview['DisplayName'] ;

src/lang/de/main.php

@@ -57,6 +57,8 @@ $content['LN_GEN_SOURCE_DB'] = "Datenbank";
 	$content['LN_GEN_DB_SQLITE'] = "SQLite 2";
 	$content['LN_GEN_SELECTVIEW'] = "Select View";
 	$content['LN_GEN_CRITERROR_UNKNOWNTYPE'] = "The source type '%1' is not supported by phpLogCon yet. This is a critical error, please fix your configuration.";
+	$content['LN_GEN_ERRORRETURNPREV'] = "Click here to return to the previous site.";
+	$content['LN_GEN_ERRORDETAILS'] = "Error Details:";
 
 // Topmenu Entries
 $content['LN_MENU_SEARCH'] = "Suchen";

src/lang/en/main.php

@@ -58,7 +58,9 @@ $content['LN_GEN_DB_INFORMIX'] = "IBM Informix Dynamic Server";
 $content['LN_GEN_DB_SQLITE'] = "SQLite 2";
 	$content['LN_GEN_SELECTVIEW'] = "Select View";
 	$content['LN_GEN_CRITERROR_UNKNOWNTYPE'] = "The source type '%1' is not supported by phpLogCon yet. This is a critical error, please fix your configuration.";
-	
+	$content['LN_GEN_ERRORRETURNPREV'] = "Click here to return to the previous site.";
+	$content['LN_GEN_ERRORDETAILS'] = "Error Details:";
+
 // Topmenu Entries
 	$content['LN_MENU_SEARCH'] = "Search";
 	$content['LN_MENU_SHOWEVENTS'] = "Show Events";

src/lang/pt_BR/main.php

@@ -61,7 +61,8 @@ $content['LN_GEN_DB_INFORMIX'] = "IBM Informix Dynamic Server";
 $content['LN_GEN_DB_SQLITE'] = "SQLite 2";
 $content['LN_GEN_SELECTVIEW'] = "Visão";
 	$content['LN_GEN_CRITERROR_UNKNOWNTYPE'] = "The source type '%1' is not supported by phpLogCon yet. This is a critical error, please fix your configuration.";
-	$content['LN_GEN_CRITERROR_UNKNOWNTYPE'] = "The source type '%1' is not supported by phpLogCon yet. This is a critical error, please fix your configuration.";
+	$content['LN_GEN_ERRORRETURNPREV'] = "Click here to return to the previous site.";
+	$content['LN_GEN_ERRORDETAILS'] = "Error Details:";
 
 // Topmenu Entries
 	$content['LN_MENU_SEARCH'] = "Search";

src/templates/admin/admin_groups.html

@@ -1,19 +1,20 @@
 <!-- INCLUDE include_header.html --> 
 
+<!-- IF ISERROR="true" -->
+<br>
+<center>
+<h3 class="ErrorMsg">{LN_GEN_ERRORDETAILS} {ERROR_MSG}</h3>
+<a href="javascript:history.back();" target="_top">{LN_GEN_ERRORRETURNPREV}</a>
+</center>
+<br>
+<!-- ENDIF ISERROR="true" -->
+
 <table width="100%" cellpadding="0" cellspacing="0" border="0" align="center" class="with_border">
 <tr>
 	<td colspan="3" class="title" nowrap><B>{LN_GROUP_CENTER}</B></td>
 </tr>
 	<tr>
 		<td align="center" class="line2">
-
-<!-- IF ISERROR="true" -->
-		<br><br>
-		<center>
-		<h3><font color="red">{ERROR_MSG}</font></h3>
-		</center>
-<!-- ENDIF ISERROR="true" -->
-
 		<br><br>
 
 <!-- IF LISTGROUPS="true" -->

src/templates/admin/admin_searches.html

@@ -1,9 +1,12 @@
 <!-- INCLUDE include_header.html --> 
 
 <!-- IF ISERROR="true" -->
+<br>
 <center>
-<h3><font color="red">{ERROR_MSG}</font></h3>
+<h3 class="ErrorMsg">{LN_GEN_ERRORDETAILS} {ERROR_MSG}</h3>
+<a href="javascript:history.back();" target="_top">{LN_GEN_ERRORRETURNPREV}</a>
 </center>
+<br>
 <!-- ENDIF ISERROR="true" -->
 
 <table width="100%" cellpadding="0" cellspacing="0" border="0" align="center" class="with_border">

src/templates/admin/admin_sources.html

@@ -30,9 +30,12 @@
 </script>
 
 <!-- IF ISERROR="true" -->
+<br>
 <center>
-<h3><font color="red">{ERROR_MSG}</font></h3>
+<h3 class="ErrorMsg">{LN_GEN_ERRORDETAILS} {ERROR_MSG}</h3>
+<a href="javascript:history.back();" target="_top">{LN_GEN_ERRORRETURNPREV}</a>
 </center>
+<br>
 <!-- ENDIF ISERROR="true" -->
 
 <table width="100%" cellpadding="0" cellspacing="0" border="0" align="center" class="with_border">

src/templates/admin/admin_users.html

@@ -1,9 +1,12 @@
 <!-- INCLUDE include_header.html --> 
 
 <!-- IF ISERROR="true" -->
+<br>
 <center>
-<h3><font color="red">{ERROR_MSG}</font></h3>
+<h3 class="ErrorMsg">{LN_GEN_ERRORDETAILS} {ERROR_MSG}</h3>
+<a href="javascript:history.back();" target="_top">{LN_GEN_ERRORRETURNPREV}</a>
 </center>
+<br>
 <!-- ENDIF ISERROR="true" -->
 
 <table width="100%" cellpadding="0" cellspacing="0" border="0" align="center" class="with_border">

src/templates/admin/admin_views.html

@@ -1,9 +1,12 @@
 <!-- INCLUDE include_header.html --> 
 
 <!-- IF ISERROR="true" -->
+<br>
 <center>
-<h3><font color="red">{ERROR_MSG}</font></h3>
+<h3 class="ErrorMsg">{LN_GEN_ERRORDETAILS} {ERROR_MSG}</h3>
+<a href="javascript:history.back();" target="_top">{LN_GEN_ERRORRETURNPREV}</a>
 </center>
+<br>
 <!-- ENDIF ISERROR="true" -->
 
 <table width="100%" cellpadding="0" cellspacing="0" border="0" align="center" class="with_border">
@@ -57,7 +60,7 @@
 <!-- ENDIF LISTVIEWS="true" -->
 
 <!-- IF ISEDITORNEWVIEW="true" -->
-		<form action="" method="post">
+		<form action="{BASEPATH}admin/views.php{FormUrlAddOP}" method="post">
 			<table border="0" cellpadding="1" cellspacing="1" bgcolor="#DDDDDD" width="600" class="with_border_alternate">
 			<tr>
 				<td align="center" class="cellmenu1" colspan="2"><b>{LN_VIEWS_ADDEDIT}</b></td>

src/templates/details.html

@@ -88,7 +88,7 @@
 		<center>
 		<font color="red">
 		<br>
-		<h3>{LN_ERROR_NORECORDS} (code {error_code} ) - Error Details: </h3>
+		<h3>{LN_ERROR_NORECORDS} (code {error_code} ) - {LN_GEN_ERRORDETAILS}</h3>
 		{detailederror}
 		<br>
 		<br>

src/templates/index.html

@@ -343,7 +343,7 @@
 	<center>
 	<font color="red">
 	<br>
-	<h3>{LN_ERROR_NORECORDS} - Error Details: </h3>
+	<h3>{LN_ERROR_NORECORDS} - {LN_GEN_ERRORDETAILS}</h3>
 	{detailederror}
 	<br>
 	<br>

src/themes/dark/main.css

@@ -333,9 +333,6 @@ A.cellmenu1_link:hover
 {
 	font: bold 12px Verdana, Arial, Helvetica, sans-serif; 
 	COLOR: #FF2222;
-	border-top: 1px solid;
-	border-bottom: 1px solid;
-	border-color: #58363E;
 }
 .PriorityEmergency
 {

src/themes/default/main.css

@@ -339,9 +339,6 @@ A.cellmenu1_link:hover
 {
 	font: bold 12px Verdana, Arial, Helvetica, sans-serif; 
 	COLOR: #FF0000;
-	border-top: 1px solid;
-	border-bottom: 1px solid;
-	border-color: #58363E;
 }
 .PriorityEmergency
 {