Incorrect Reminder Message Log Time

We are using Horde Webmail Group Edition 1.2.3. When testin

Thu, 13 Aug 2009 21:06:49 +0000

We are using Horde Webmail Group Edition 1.2.3. When testing the script horde/kronolith/scripts/reminders.php , the log lines associated with the sending of the reminder (from line 476 in horde/kronolith/lib/Scheduler/kronolith.php is using the timezone specified by the user, not the system. The date and time associated with the message (in the message's headers) and the syslog lines should be consistent with the system, not the individual user's locale settings.

This has nothing to do with reminders, this is how Horde log

Thu, 20 Aug 2009 16:32:24 +0000

This has nothing to do with reminders, this is how Horde logging currently works.

Changes have been made in CVS for this ticket: http://cvs.h

Thu, 20 Aug 2009 16:33:17 +0000

Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/framework/Horde/Attic/Horde.php?rt=horde&r1=1.489.2.114&r2=1.489.2.115&ty=u

Try what I committed?

Thu, 20 Aug 2009 16:33:47 +0000

Try what I committed?

Really close. I like your conservative approach. :) The

Thu, 20 Aug 2009 21:03:19 +0000

Really close. I like your conservative approach. :) The only problem I ran into was that the TZ environment variable was still set, so it continued to override the system settings. I must admit to not being an expert, but, I think, that something like the following is needed: lib/Horde.php (starting line 77) /* Make sure to log in the system's locale and timezone. */ $locale = setlocale(LC_TIME, 0); setlocale(LC_TIME, 'C'); $tz = getenv('TZ'); --> @putenv('TZ'); Thereby unsetting the TZ environment variable and letting the PHP mechanisms and system settings to be applied to logging.

Changes have been made in CVS for this ticket: http://cvs.h

Thu, 20 Aug 2009 21:43:05 +0000

Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/framework/Horde/Attic/Horde.php?rt=horde&r1=1.489.2.115&r2=1.489.2.116&ty=u

Bummer, that's what this ticket was about. :)

Thu, 20 Aug 2009 21:44:25 +0000

Bummer, that's what this ticket was about. :)

It appears to be working. Thanks!

Fri, 21 Aug 2009 20:25:59 +0000

It appears to be working. Thanks!

I was wrong. :( I failed you. The error is that the uns

Thu, 27 Aug 2009 21:05:36 +0000

I was wrong. :( I failed you. The error is that the unsetting of the TZ environment variable should only take place if it is set. If it isn't, then it will generate a segmentation violation. So, we are close. I modified the clearing of the variable to be in a conditional statement...like what you use to restore the value. if ($tz) { @putenv('TZ'); } I still need to verify whether it corrects the logging issue, but without the conditional, the behavior is quite undesirable. :)

This is a serious bug in PHP then.

Thu, 27 Aug 2009 21:41:36 +0000

This is a serious bug in PHP then.

Changes have been made in Git (master): commit 49b1c20490f2

Wed, 06 Sep 2017 08:18:02 +0000

Changes have been made in Git (master): commit 49b1c20490f26decc17f4c303498908f7398d67e Author: Moritz Venn Date: Fri Aug 18 11:37:04 2006 +0000 Fix Bug #8496 (thanks Brian) git-svn-id: https://svn.php.net/repository/pear/packages/Text_Wiki/trunk@218410 c90b9560-bf6c-de11-be94-00142212c4b1 Text/Wiki/Parse/Mediawiki/Table.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) http://github.com/horde/horde/commit/49b1c20490f26decc17f4c303498908f7398d67e