[Tickets #8638] ListAlarms: retrieve only calendars with events having alarm

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/8638
------------------------------------------------------------------------------
  Ticket             | 8638
  Created By         | patrick.abiven at apitech.fr
  Summary            | ListAlarms: retrieve only calendars with events having
                     | alarm
  Queue              | Kronolith
  Version            | 2.3.2
  Type               | Enhancement
  State              | New
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


patrick.abiven at apitech.fr (2009-10-14 09:12) wrote:

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;
+ }