6.0.0-alpha10
5/17/25

[#8136] Date_Holidays bad event start hour
Summary Date_Holidays bad event start hour
Queue Kronolith
Queue Version 2.3.1
Type Bug
State Not A Bug
Priority 1. Low
Owners
Requester dom.lalot (at) gmail (dot) com
Created 04/01/2009 (5890 days ago)
Due
Updated 04/01/2009 (5890 days ago)
Assigned
Resolved 04/01/2009 (5890 days ago)
Milestone
Patch Yes

History
04/01/2009 01:31:50 PM dom (dot) lalot (at) gmail (dot) com Comment #3 Reply to this comment
Date_holidays seems to be rather painfull! Thanks for the link. I 
tested German driver, and it was also buggy. Look at May.. And I was 
obliged to get rid of accents. I wrote a french driver, but using 
german, I could see some bad display also.



My patch has may be a little side effect without waiting for the pear update.



Thks



Dom


04/01/2009 10:29:02 AM Jan Schneider Comment #2
State ⇒ Not A Bug
Reply to this comment
04/01/2009 10:01:39 AM dom (dot) lalot (at) gmail (dot) com Comment #1
Priority ⇒ 1. Low
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Kronolith
Summary ⇒ Date_Holidays bad event start hour
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
As you can see, there is something strange on some dates. I've got 
last Date_Holidays pear package as it was said to upgrade. On some 
dates, it was working correctly.. strange!



I trace and discovered this:



Apr  1 11:39:50 testDom HORDE[13801]: [kronolith] tracedom: Spring 
Bank Holiday [pid 13801 on line 204 of 
"/var/www/perso/horde-webmail-1.2.2/kronolith/lib/Driver/holidays.php"]

Apr  1 11:39:50 testDom HORDE[13801]: [kronolith] tracedom: deb Mon 
May 25 01:00:00 2009 [pid 13801 on line 209 of 
"/var/www/perso/horde-webmail-1.2.2/kronolith/lib/Driver/holidays.php"]

Apr  1 11:39:50 testDom HORDE[13801]: [kronolith] tracedom: fin Tue 
May 26 01:00:00 2009 [pid 13801 on line 210 of 
"/var/www/perso/horde-webmail-1.2.2/kronolith/lib/Driver/holidays.php"]



the events goes from 1h am to 1h am so lasting over two days.



I've made a quick and dirty patch in kronolith/lib/Driver/holidays.php 
in order to get it working properly:



--- /root/kronolith-h3-2.3.1/lib/Driver/holidays.php    2009-02-11 
23:15:37.000000000 +0100

+++ holidays.php        2009-04-01 11:54:51.000000000 +0200

@@ -201,11 +201,12 @@ class Kronolith_Event_holidays extends K

          $this->initialized = true;

          $this->setTitle(String::convertCharset($dhEvent->getTitle(), 
'UTF-8'));

          $this->setId($dhEvent->getInternalName());

-

          $this->start = new Horde_Date($dhEvent->_date->getTime());

          $this->end = new Horde_Date($this->start);

          $this->end->mday++;

          $this->end->correct();

+                 $this->start->hour=0;

+                 $this->end->hour=0;

      }



      /**



  * @version  CVS: $Id: Holidays.php,v 1.21 2009/02/09 11:38:32 kguest Exp $




Saved Queries