Closed Bug 1363738 Opened 4 years ago Closed 4 years ago

No cancellation message to a removed attendee if another attendee was added at the same time

Categories

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

Lightning 5.4
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: c.fluegel, Assigned: MakeMyDay)

Details

(Keywords: regression)

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:53.0) Gecko/20100101 Firefox/53.0
Build ID: 20170413192749

Steps to reproduce:

1) Create an appointment with at least one attendee (on an ICAL Server)
2.1) Edit the appointment and remove the / an attendee from the attendee list ...
2.2) ... while adding another person to the appointment.
2.3) Now save the changes

Comparing the working case: 
3) Remove an attendee from the list but do not add another attendee


Actual results:

At 1) there will be a message box that asks whether you want to send a mail to the attendee. The attendee receives an invite. 
At 2.x) Only the newly added attendee will get an invite, the removed attendee does not receive the information that the appointment has been cancelled (for the attendee) 

Working Case:
At 3) The removed attendee gets an cancellation mail for the appoitment. (


Expected results:

The newly added attendee should still get an invite, of course, and the removed attendee should get cancellation information.
Component: Untriaged → E-mail based Scheduling (iTIP/iMIP)
Product: Thunderbird → Calendar
Version: 52 Branch → Lightning 5.4
Confirming, this is a regression of bug 1280898.
Assignee: nobody → makemyday
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Keywords: regression
Attachment #8869821 - Flags: review?(philipp)
Comment on attachment 8869821 [details] [diff] [review]
FixAutoResponseInItipProcessor-V1.diff

Review of attachment 8869821 [details] [diff] [review]:
-----------------------------------------------------------------

I suspect this needs some uplifts, I'd be fine with anything down to esr.
Attachment #8869821 - Flags: review?(philipp) → review+
Comment on attachment 8869821 [details] [diff] [review]
FixAutoResponseInItipProcessor-V1.diff

Yes, all of my today's review requests need to go down to esr - I just wanted await the review before requesting the approvals.

I'm adjusting adjusting the flags for this patch according to comment 3.

Btw, can we configure the flag list to simply remove the aurora approval flag from the bmo view or do we need to file a bug for that?
Attachment #8869821 - Flags: approval-calendar-esr+
Attachment #8869821 - Flags: approval-calendar-beta+
Keywords: checkin-needed
We need to file a bug to manipulate flags, I'm happy to remove the aurora flag for calendar.
https://hg.mozilla.org/comm-central/rev/fd2be9029992fc12bde1ad70f1cdcd4fe0f3c08f
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 5.7
TB 52 ESR, Calendar 5.4.x:
https://hg.mozilla.org/releases/comm-esr52/rev/4b42e257be29
Target Milestone: 5.6 → 5.4.2
You need to log in before you can comment on or make changes to this bug.