Closed Bug 450563 Opened 16 years ago Closed 16 years ago

Attendee gets the organizer alarm setting if he/she accept an iTip/iMip invitation

Categories

(Calendar :: E-mail based Scheduling (iTIP/iMIP), defect)

defect
Not set
minor

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: andreas.treumann, Assigned: dbo)

Details

Attachments

(1 file)

STEPS TO REPRODUCE:
===================

- create an event and set a reminder
- invite someone via iTip/iMip
- attendee accept the invitation
- check the event in the calendar view

RESULT:
=======

- the event in the attendees calendar shows the alarm, too

EXPECTED RESULT:
================

- no reminder

REPRODUCIBLE:
=============

- always
See Bug 431149. If the invitation event specifies an reminder it's taken over just like title, location, description, etc. I think that's how other calendar applications behave too.
Yes, but I don't think that's sensible, because alarms are a "user-local" setting.
Attached patch fix - v1 — — Splinter Review
I don't think the organizer should be able to preselect the alarm settings of attendees, and think it's more important that users get their preferred default alarm set (bug 450565, bug 445488). Setting ui-review though to have a further opinion.

Even if we send out the VALARM component, we must not send out X-MOZ- properties, especially X-MOZ-LASTACK, because otherwise attendees may not even get the alarm fired.
Assignee: nobody → daniel.boelzle
Status: NEW → ASSIGNED
Attachment #334879 - Flags: ui-review?(christian.jansen)
Attachment #334879 - Flags: review?(philipp)
Comment on attachment 334879 [details] [diff] [review]
fix - v1

Looks good, r=philipp
Attachment #334879 - Flags: review?(philipp) → review+
Comment on attachment 334879 [details] [diff] [review]
fix - v1

ui+ per discussion with Christian
Attachment #334879 - Flags: ui-review?(christian.jansen) → ui-review+
Checked in on HEAD and MOZILLA_1_8_BRANCH => FIXED.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.9
Checked with lightning 2008082719 -> VERIFIED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: