6.0.0-git
2019-03-19

[#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 2009-10-14 (3443 days ago)
Due
Updated 2010-01-13 (3352 days ago)
Assigned
Resolved 2009-10-14 (3443 days ago)
Milestone 2.3.3
Patch No

History
2009-10-14 13:43:44 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!
2009-10-14 13:12:31 patrick (dot) abiven (at) apitech (dot) fr Comment #1
Type ⇒ Enhancement
State ⇒ New
Priority ⇒ 1. Low
Summary ⇒ ListAlarms: retrieve only calendars with events having alarm
Queue ⇒ Kronolith
Milestone ⇒
Patch ⇒ No
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