Deleting an event in the view after opening the event dialog via double-click causes a MODIFICATION_FAILED error

RESOLVED FIXED in 4.0.0.1

Status

Calendar
Calendar Views
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: mmecca, Assigned: mmecca)

Tracking

unspecified
4.0.0.1

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
This is a follow up to Bug 1120258.

Opening an event for editing by double-clicking the event in the view, then deleting the event in the view while the dialog is still open, results in a MODIFICATION_FAILED error. Opening the event dialog using the view context menu, or from the today pane or unifinder, does not trigger the same error.
(Assignee)

Comment 1

3 years ago
Created attachment 8574360 [details] [diff] [review]
Fix v1

Currently, opening the event dialog by double-clicking an item in the view is the only method that calls into the createPendingModification function, which tracks pending modification jobs in the event dialog, and attempts to complete those changes first if a subsequent modification is made before the dialog is closed. Since the fix for Bug 1120258 landed, that is no longer necessary, as the event dialog will observe those changes.
Attachment #8574360 - Flags: review?(philipp)
Attachment #8574360 - Flags: approval-calendar-aurora?(philipp)
Comment on attachment 8574360 [details] [diff] [review]
Fix v1

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

Lets do it, thanks for looking into this! r/a=philipp
Attachment #8574360 - Flags: review?(philipp)
Attachment #8574360 - Flags: review+
Attachment #8574360 - Flags: approval-calendar-aurora?(philipp)
Attachment #8574360 - Flags: approval-calendar-aurora+
(Assignee)

Updated

3 years ago
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/10ae8ea49f4e
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 4.1
Checkin for comm-aurora (4.0) is still missing!
Backported to releases/comm-aurora changeset 8b4abb999d29
Target Milestone: 4.1 → 4.0
You need to log in before you can comment on or make changes to this bug.