Summary | Incorrect Reminder Message Log Time |
Queue | Horde Framework Packages |
Queue Version | FRAMEWORK_3 |
Type | Bug |
State | Resolved |
Priority | 1. Low |
Owners | |
Requester | bhalsema (at) purdue (dot) edu |
Created | 08/13/2009 (5760 days ago) |
Due | |
Updated | 09/06/2017 (2814 days ago) |
Assigned | 08/20/2009 (5753 days ago) |
Resolved | 08/20/2009 (5753 days ago) |
Milestone | |
Patch | No |
commit 49b1c20490f26decc17f4c303498908f7398d67e
Author: Moritz Venn <ritzmo@php.net>
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
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. :)
State ⇒ Resolved
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
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.
State ⇒ Feedback
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
Version ⇒ FRAMEWORK_3
Queue ⇒ Horde Framework Packages
currently works.
Patch ⇒ No
State ⇒ Unconfirmed
Milestone ⇒
Queue ⇒ Kronolith
Summary ⇒ Incorrect Reminder Message Log Time
Type ⇒ Bug
Priority ⇒ 1. Low
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.