X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=inc%2Fmodules%2Fadmin%2Fwhat-logs.php;h=df3a5872f02f9c5d6c38b599fb337d4d18d37575;hb=00797d46cc0b7e2c4e8c084eb4c6ac26cd0336e1;hp=2556113fc2539d5a0ed01a677f217141eedd6fe0;hpb=a090e351c49fe021fb3064325694da03402332e0;p=mailer.git

diff --git a/inc/modules/admin/what-logs.php b/inc/modules/admin/what-logs.php
index 2556113fc2..df3a5872f0 100644
--- a/inc/modules/admin/what-logs.php
+++ b/inc/modules/admin/what-logs.php
@@ -38,22 +38,22 @@
 
 // Some security stuff...
 if ((!defined('__SECURITY')) || (!IS_ADMIN())) {
-	$INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), "/inc") + 4) . "/security.php";
+	$INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), '/inc') + 4) . '/security.php';
 	require($INC);
 }
 
 // Add description as navigation point
 ADD_DESCR('admin', __FILE__);
 
-if (REQUEST_ISSET_GET(('access'))) {
+if (REQUEST_ISSET_GET('access')) {
 	// Secure input and construct FQFN
 	$access = SQL_ESCAPE(strip_tags(REQUEST_GET('access')));
 	$target = sprintf("%slogs/%s", constant('PATH'), $access);
 
 	// Is the file valid and readable?
-	if (FILE_READABLE($target)) {
+	if (isFileReadable($target)) {
 		// Load it directly
-		$content = READ_FILE($target);
+		$content = readFromFile($target);
 
 		// Save old content
 		$OUTPUT = ob_get_contents();
@@ -62,15 +62,15 @@ if (REQUEST_ISSET_GET(('access'))) {
 		clearOutputBuffer();
 
 		// Set header
-		if (substr($access, -3, 3) == "log") {
+		if (substr($access, -3, 3) == 'log') {
 			// Output header
-			header("Content-Type: text/plain");
-		} elseif (substr($access, -3, 3) == ".gz") {
+			sendHeader('Content-Type: text/plain');
+		} elseif (substr($access, -3, 3) == '.gz') {
 			// @TODO Fix content-type here
-			header("Content-Type: text/plain");
-		} elseif (substr($access, -3, 3) == ".bz2") {
+			sendHeader('Content-Type: text/plain');
+		} elseif (substr($access, -3, 3) == '.bz2') {
 			// @TODO Fix content-type here
-			header("Content-Type: text/plain");
+			sendHeader('Content-Type: text/plain');
 		} else {
 			// Restore old content
 			OUTPUT_HTML($OUTPUT);
@@ -89,14 +89,16 @@ if (REQUEST_ISSET_GET(('access'))) {
 	}
 } else {
 	// List access logfiles
-	$dir = constant('PATH') . getConfig('logs_base') . "/";
+	$dir = constant('PATH') . getConfig('logs_base') . '/';
+
+	// Is the directory there?
 	if (isDirectory($dir)) {
 		// logs directory does exist
 		OUTPUT_HTML("<ol>");
-		$handle = opendir($dir) or app_die(__FILE__, __LINE__, "Cannot open directory ".getConfig('logs_base')."!");
+		$handle = opendir($dir) or app_die(__FILE__, __LINE__, 'Cannot open directory '.getConfig('logs_base') . '!');
 		while ($file = readdir($handle)) {
 			// We currenly only like files with "access" as prefix, should be more flexible!
-			if (substr($file, 0, 6) == "access") {
+			if (substr($file, 0, 6) == 'access') {
 				// Okay, let us print it out
 				OUTPUT_HTML("<li><a href=\"{!URL!}/modules.php?module=admin&amp;what=".$GLOBALS['what']."&amp;access=".urlencode($file)."\">".$file."</a></li>");
 			}