Tickets :: [#8821] Not all users are getting a reminder

6.0.0-alpha14

7/1/25

Summary	Not all users are getting a reminder
Queue	Horde Base
Queue Version	3.3.6
Type	Bug
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	80602 (at) heygel (dot) ch
Created	01/08/2010 (5653 days ago)
Due
Updated	02/10/2010 (5620 days ago)
Assigned	01/10/2010 (5651 days ago)
Resolved	02/04/2010 (5626 days ago)
Github Issue Link
Github Pull Request
Milestone	3.3.7
Patch	No

02/10/2010 07:27:24 PM	CVS Commit	Comment #6	Reply to this comment
Changes have been made in Git for this ticket: MFB: Fix sending alarms to multiple users (~~Bug #8821~~). http://git.horde.org/diff.php/framework/Alarm/Alarm/sql.php?rt=horde-git&r1=07abb73a15d7b9510cdfe5063d6d9b125d336efc&r2=e6108831f8dbb72eeecfb05b2048f9728cc27ad5 http://git.horde.org/diff.php/framework/Alarm/tests/001.phpt?rt=horde-git&r1=9cb1db2414871d3c98a0ebed0d11288c84cb384f&r2=e6108831f8dbb72eeecfb05b2048f9728cc27ad5 http://git.horde.org/diff.php/framework/Alarm/tests/setup.inc.dist?rt=horde-git&r1=9cb1db2414871d3c98a0ebed0d11288c84cb384f&r2=e6108831f8dbb72eeecfb05b2048f9728cc27ad5 http://git.horde.org/diff.php/horde/docs/CHANGES?rt=horde-git&r1=f7352a17a24e05b79e17b37d42d5ea5162f37658&r2=e6108831f8dbb72eeecfb05b2048f9728cc27ad5

02/04/2010 10:27:53 PM	CVS Commit	Comment #5	Reply to this comment
Changes have been made in Git for this ticket: MFB: Fix sending alarms to multiple users (~~Bug #8821~~). http://git.horde.org/diff.php/framework/Alarm/Alarm/sql.php?rt=horde-git&r1=07abb73a15d7b9510cdfe5063d6d9b125d336efc&r2=e6108831f8dbb72eeecfb05b2048f9728cc27ad5 http://git.horde.org/diff.php/framework/Alarm/tests/001.phpt?rt=horde-git&r1=9cb1db2414871d3c98a0ebed0d11288c84cb384f&r2=e6108831f8dbb72eeecfb05b2048f9728cc27ad5 http://git.horde.org/diff.php/framework/Alarm/tests/setup.inc.dist?rt=horde-git&r1=9cb1db2414871d3c98a0ebed0d11288c84cb384f&r2=e6108831f8dbb72eeecfb05b2048f9728cc27ad5 http://git.horde.org/diff.php/horde/docs/CHANGES?rt=horde-git&r1=f7352a17a24e05b79e17b37d42d5ea5162f37658&r2=e6108831f8dbb72eeecfb05b2048f9728cc27ad5

02/04/2010 06:03:43 PM	Jan Schneider	Comment #4 State ⇒ Resolved Milestone ⇒ 3.3.7	Reply to this comment
Fixed for the next release.

02/04/2010 06:03:28 PM	CVS Commit	Comment #3	Reply to this comment
Changes have been made in CVS for this ticket: Fix sending alarms to multiple users (~~Bug #8821~~). http://cvs.horde.org/diff.php/framework/Alarm/Alarm/sql.php?rt=horde&r1=1.11.2.11&r2=1.11.2.12&ty=u http://cvs.horde.org/diff.php/framework/Alarm/tests/001.phpt?rt=horde&r1=1.5.2.2&r2=1.5.2.3&ty=u http://cvs.horde.org/diff.php/framework/Alarm/tests/setup.inc.dist?rt=horde&r1=1.1.2.1&r2=1.1.2.2&ty=u http://cvs.horde.org/diff.php/horde/docs/CHANGES?rt=horde&r1=1.515.2.596&r2=1.515.2.597&ty=u

01/10/2010 07:31:07 PM	Chuck Hagenbuch	State ⇒ Assigned Assigned to Jan Schneider

01/08/2010 05:57:09 PM	80602 (at) heygel (dot) ch	Comment #2	Reply to this comment
Of course "all except the LAST found alarm gets lost in this array" is correct.

01/08/2010 05:54:37 PM	80602 (at) heygel (dot) ch	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ Not all users are getting a reminder Queue ⇒ Horde Base Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
On shared calendars with multiple users having reading access and having configured "On all calendars I have read access to" for setting "Choose if you want to receive reminders for events with alarms:", an alarm is always sent only to one of those users. I think, I found the cause to this problem: On line 164 of /lib/Horde/Alarm/sql.php the $alarms array is filled with $alarm['alarm_id'] as key. However it's possible, that multiple records in horde_alarms share the same "alarm_id" and all except the first found alarm gets lost in this array. So it's not a good idea to use this alarm_id as array key.