[Tickets #1855] NEW: Email Alarm Text is Ambiguous

[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=1855
-----------------------------------------------------------------------
 Ticket             | 1855
 Created By         | derek at battams.ca
 Summary            | Email Alarm Text is Ambiguous
 Queue              | Kronolith
 Version            | 2.0.3-RC1
 State              | Unconfirmed
 Priority           | 2. Medium
 Type               | Bug
 Owners             | 
-----------------------------------------------------------------------


derek at battams.ca (2005-04-23 20:09) wrote:

The email text used in reminders (alarms) for calendar events is ambiguous. 
My patch for bug 1356 included a fix for this, but obviously wasn't included
in the commit.  The current text reads:

sprintf(_("You requested to be reminded about %s, which is at %s."),
$event->title, date('H:i', $event->start->timestamp()))

Since bug 1356 addressed fixing reminders such that they are sent at the
appropriate time, which may be days before the event in question, the text
can be quite misleading.  For example, if an alarm is set for 48 hours
before an event you will get the above text in an email reminder, which
could read:

"You requested to be reminded about Appointment X, which is at 09:30."

However, with the patch for 1356, this event may actually be scheduled for
09:30 two days from now, but this email suggests that the event is today at
09:30.  The quickest fix, as provided in my patch for 1356, is to simply add
the date of the event along with the time in the date() call.  Of course,
there is probably a better, more complete text that could be written.

Marked as medium because the text of the emails as sent can be misleading as
to the actual date/time of the event for which the alarm is being sent
about.