[Tickets #5660] horde/scripts/alarms.php don't work

[bugs at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/?id=5660
-----------------------------------------------------------------------
 Ticket             | 5660
 Created By         | cristian at nethesis.it
 Summary            | horde/scripts/alarms.php don't work
 Queue              | Horde Framework Packages
 Version            | HEAD
 Type               | Bug
 State              | Unconfirmed
 Priority           | 1. Low
 Owners             | 
-----------------------------------------------------------------------


cristian at nethesis.it (2007-08-23 01:45) wrote:

The horde/scripts/alarms.php script (from cron every 5 minutes) don't work,
due to (maybe) a bug in the notify function (lib/Horde/Alarm.php).

in this function you say:
        if (empty($user)) {
            $user = '';
        }
so the user is '', not null.

for this, the function _list in Alarm/sql.php append in the sql query a
wrong condition
is_null($user) ? '' : ' AND (alarm_uid = ? OR alarm_uid = ?)');

so the query executed is
SELECT alarm .... rms WHERE ala...  ) AND (alarm_uid = '' OR alarm_uid =
'')

this cause the script don't work.
if I comment the 3 code line in the notify function (line 297)
/*        if (empty($user)) {
            $user = '';
        }
*/
all works fine