6.0.0-alpha14
6/26/25

[#8638] ListAlarms: retrieve only calendars with events having alarm
Summary ListAlarms: retrieve only calendars with events having alarm
Queue Kronolith
Queue Version 2.3.2
Type Enhancement
State Resolved
Priority 1. Low
Owners jan (at) horde (dot) org
Requester patrick.abiven (at) apitech (dot) fr
Created 10/14/2009 (5734 days ago)
Due
Updated 01/13/2010 (5643 days ago)
Assigned
Resolved 10/14/2009 (5734 days ago)
Milestone 2.3.3
Patch No

History
10/14/2009 01:43:44 PM Jan Schneider Comment #4
Assigned to Jan Schneider
State ⇒ Resolved
Milestone ⇒ 2.3.3
Reply to this comment
That's really a nice catch and an impressive improvement, thanks a lot!
10/14/2009 01:12:31 PM patrick (dot) abiven (at) apitech (dot) fr Comment #1
Priority ⇒ 1. Low
Type ⇒ Enhancement
Summary ⇒ ListAlarms: retrieve only calendars with events having alarm
Queue ⇒ Kronolith
Milestone ⇒
Patch ⇒ No
State ⇒ New
Reply to this comment
Hello

For reducing the duration of the Horde_Alarm reminder script, in 
kronolith/lib/Kronolith.php, listAlarms(): instead of storing all 
calendars, can we only store calendars in which exist events with 
alarm ?

In our case, that will improve performances for about 40 seconds.



function listAlarms

- $alarms[$cal] = $kronolith_driver->listAlarms($date, $fullevent);



+ $result = $kronolith_driver->listAlarms($date, $fullevent);

+ if (! empty($result) ) {

+   $alarms[$cal] = $result;

+ }

Saved Queries