Changeset 4596 for trunk/core/logger/file.logger.php

Timestamp:

08/22/10 10:28:57 (21 months ago)

Author:

vain

Message:

• fixed bug in returnEntriesFromLogfile() when logfile is not existing

File:

• trunk/core/logger/file.logger.php (modified) (1 diff)

trunk/core/logger/file.logger.php

@@ -181 +181 @@
         }
 
-        # get logfile as array
-        $logfile_array = file($logfilename);
-        $logfile_cnt = count($logfile_array);
-
-        if($logfile_cnt > 0)
-        {
-            # count array elements = total number of logfile entries
-            $i = $logfile_cnt - 1;
-
-            # subtract from total number of logfile entries the number to fetch
-            $max_entries = max(0, $i - $entriesToFetch);
-
-            # reverse for loop over the logfile_array
-            $logEntries = '';
-            for($i; $i > $max_entries; $i--)
+         $logEntries = '';
+
+        if(true === is_file($logfilename))
+        {
+            # get logfile as array
+            $logfile_array = file($logfilename);
+            $logfile_cnt = count($logfile_array);
+
+            if($logfile_cnt > 0)
             {
-                # remove linebreaks
-                $entry = str_replace(array('\r', '\n'), '', $logfile_array[$i]);
-
-                $logEntries .= '<b>Entry ' . $i . '</b>';
-                $logEntries .= '<br />' . htmlentities($entry) . '<br />';
+                # count array elements = total number of logfile entries
+                $i = $logfile_cnt - 1;
+
+                # subtract from total number of logfile entries the number to fetch
+                $max_entries = max(0, $i - $entriesToFetch);
+
+                # reverse for loop over the logfile_array
+                for($i; $i > $max_entries; $i--)
+                {
+                    # remove linebreaks
+                    $entry = str_replace(array('\r', '\n'), '', $logfile_array[$i]);
+
+                    $logEntries .= '<b>Entry ' . $i . '</b>';
+                    $logEntries .= '<br />' . htmlentities($entry) . '<br />';
+                }
+
+                # cleanup
+                unset($logfilename, $logfile_array, $i, $max_entries, $entry);
             }
-
-            # cleanup
-            unset($logfilename, $logfile_array, $i, $max_entries, $entry);
+            else
+            {
+                $logEntries .= '<b>No Entries</b>';
+            }
         }
         else
         {
-            $logEntries .= '<b>No Entries</b>';
+            $logEntries .= '<b>No Logfile found. No entries yet.</b>';
         }