Tickets :: Comment on [#1855] Email Alarm Text is Ambiguous

* Your Email Address
* Spam protection	Enter the letters below: . .. ..__ . .. . \ /\ /[ __\| \|\|\/\| \/ \/ [_./\|/\\|\| \|
Comment	> 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.
Attachment
Watch this ticket