You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by gr...@apache.org on 2013/07/22 09:41:08 UTC

git commit: LOG4PHP-210: fixed filesize stats failed issue

Updated Branches:
  refs/heads/develop f15e0cc61 -> 1b558631d


LOG4PHP-210: fixed filesize stats failed issue


Project: http://git-wip-us.apache.org/repos/asf/logging-log4php/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4php/commit/1b558631
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4php/tree/1b558631
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4php/diff/1b558631

Branch: refs/heads/develop
Commit: 1b558631d58f3b46040ac2550f729a69337d540f
Parents: f15e0cc
Author: Christian Grobmeier <gr...@apache.org>
Authored: Mon Jul 22 09:30:24 2013 +0200
Committer: Christian Grobmeier <gr...@apache.org>
Committed: Mon Jul 22 09:30:24 2013 +0200

----------------------------------------------------------------------
 src/changes/changes.xml                              | 1 +
 src/main/php/appenders/LoggerAppenderRollingFile.php | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4php/blob/1b558631/src/changes/changes.xml
----------------------------------------------------------------------
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index cb8b3d9..850a555 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -21,6 +21,7 @@
 	</properties>
 	<body>
 		<release version="develop">
+			<action date="2013-07-22" type="fix" issue="LOG4PHP-210" dev="Christian grobmeier">Appenders cause filesize(): stat failed</action>
 			<action date="2012-12-31" type="update" issue="LOG4PHP-197" dev="Ivan Habunek" due-to="Sven Rautenberg" due-to-email="sven at rtbg dot de">Made Logger::isInitialized() public.</action>
 			<action date="2012-12-31" type="update" issue="LOG4PHP-144" dev="Ivan Habunek">Improved LoggerAppenderMail to set the Content-type header as defined in layout.</action>
 		</release>

http://git-wip-us.apache.org/repos/asf/logging-log4php/blob/1b558631/src/main/php/appenders/LoggerAppenderRollingFile.php
----------------------------------------------------------------------
diff --git a/src/main/php/appenders/LoggerAppenderRollingFile.php b/src/main/php/appenders/LoggerAppenderRollingFile.php
index 322cab8..b8b5557 100644
--- a/src/main/php/appenders/LoggerAppenderRollingFile.php
+++ b/src/main/php/appenders/LoggerAppenderRollingFile.php
@@ -219,7 +219,7 @@ class LoggerAppenderRollingFile extends LoggerAppenderFile {
 			}
 			
 			// Rollover if needed
-			if (filesize($this->file) > $this->maxFileSize) {
+			if (filesize(realpath($this->file)) > $this->maxFileSize) {
 				try {
 					$this->rollOver();
 				} catch (LoggerException $ex) {