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

RESOLVED FIXED in 5.4.2

Status

Calendar
E-mail based Scheduling (iTIP/iMIP)
RESOLVED FIXED
8 months ago
8 months ago

People

(Reporter: c.fluegel, Assigned: MakeMyDay)

Tracking

({regression})

Lightning 5.4
5.4.2
regression

Details

Attachments

(1 attachment)

(Reporter)

Description

8 months ago
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.

Updated

8 months ago
Component: Untriaged → E-mail based Scheduling (iTIP/iMIP)
Product: Thunderbird → Calendar
Version: 52 Branch → Lightning 5.4
(Assignee)

Comment 1

8 months ago
Confirming, this is a regression of bug 1280898.
Assignee: nobody → makemyday
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Keywords: regression
(Assignee)

Comment 2

8 months ago
Created attachment 8869821 [details] [diff] [review]
FixAutoResponseInItipProcessor-V1.diff
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+
(Assignee)

Comment 4

8 months ago
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+
(Assignee)

Updated

8 months ago
Keywords: checkin-needed
We need to file a bug to manipulate flags, I'm happy to remove the aurora flag for calendar.

Comment 6

8 months ago
https://hg.mozilla.org/comm-central/rev/fd2be9029992fc12bde1ad70f1cdcd4fe0f3c08f
Status: ASSIGNED → RESOLVED
Last Resolved: 8 months ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 5.7

Comment 7

8 months ago
Beta (TB 54, Calendar 5.6):
https://hg.mozilla.org/releases/comm-beta/rev/d513b033c0d0aa2a714bcf16a2aeeb3dfe057391
Target Milestone: 5.7 → 5.6

Comment 8

8 months ago
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.