6.0.0-beta1
7/5/25

[#5660] horde/scripts/alarms.php don't work
Summary horde/scripts/alarms.php don't work
Queue Horde Framework Packages
Queue Version HEAD
Type Bug
State Resolved
Priority 1. Low
Owners jan (at) horde (dot) org
Requester cristian (at) nethesis (dot) it
Created 08/23/2007 (6526 days ago)
Due
Updated 09/06/2017 (2859 days ago)
Assigned 08/23/2007 (6526 days ago)
Resolved 09/20/2007 (6498 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
09/06/2017 08:17:38 AM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (master):

commit 5760ec194e34960ff1ee97710a19acc966d8f7eb
Author: bertrand Gugger <toggg@php.net>
Date:   Wed Oct 19 23:43:53 2005 +0000

     Obviously, the # is to be kept in the anchor option, #5660, merci 
jeremy Lecour


     git-svn-id: 
https://svn.php.net/repository/pear/packages/Text_Wiki/trunk@198815 
c90b9560-bf6c-de11-be94-00142212c4b1

  Text/Wiki/Parse/Default/Freelink.php | 6 ++----
  1 file changed, 2 insertions(+), 4 deletions(-)

http://github.com/horde/horde/commit/5760ec194e34960ff1ee97710a19acc966d8f7eb
09/20/2007 07:01:34 PM Jan Schneider Comment #2
State ⇒ Resolved
Reply to this comment
Fixed in CVS.
08/23/2007 01:34:17 PM Chuck Hagenbuch State ⇒ Assigned
Assigned to Jan Schneider
 
08/23/2007 08:45:51 AM cristian (at) nethesis (dot) it Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ horde/scripts/alarms.php don't work
Queue ⇒ Horde Framework Packages
State ⇒ Unconfirmed
Reply to this comment
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


Saved Queries