Closed
Bug 1186547
Opened 10 years ago
Closed 10 years ago
mReadOnly and mDisabled do not reflect readOnly and disabled properties
Categories
(Calendar :: Provider: CalDAV, defect)
Calendar
Provider: CalDAV
Tracking
(Not tracked)
RESOLVED
FIXED
4.0.2
People
(Reporter: Fallen, Assigned: Fallen)
References
Details
Attachments
(1 file)
|
3.77 KB,
patch
|
Fallen
:
review+
Fallen
:
approval-calendar-aurora+
Fallen
:
approval-calendar-beta+
Fallen
:
approval-calendar-esr+
|
Details | Diff | Splinter Review |
See bug 674088 for details:
"""
Then, I looked at the "minor error" code, this is obviously not a minor error. I've noticed that changing mReadOnly and mDisabled does not actually change the properties "readOnly" and "disabled". I've fixed this by removing mDisabled and mReadOnly and just using the property getters directly.
"""
|
Assignee | |
Comment 1•10 years ago
|
||
r+ from the other bug
|
|
Assignee | |
Updated•10 years ago
|
Attachment #8637361 -
Flags: approval-calendar-release+
Attachment #8637361 -
Flags: approval-calendar-beta+
Attachment #8637361 -
Flags: approval-calendar-aurora+
|
|
Assignee | |
Comment 2•10 years ago
|
||
Backported to releases/comm-aurora changeset c53da8a616fa
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 4.3
|
|
Assignee | |
Comment 3•10 years ago
|
||
Backported to releases/comm-beta changeset 18d4427d4203
Target Milestone: 4.3 → 4.2
|
|
Assignee | |
Comment 4•10 years ago
|
||
Backported to releases/comm-esr38 changeset 59bfcc074820
Target Milestone: 4.2 → 4.0.2
|
|
Assignee | |
Comment 5•10 years ago
|
||
Backported to releases/comm-aurora changeset 3638ae3dc946
Target Milestone: 4.0.2 → 4.3
|
|
Assignee | |
Comment 6•10 years ago
|
||
Backported to releases/comm-beta changeset 3a6adf92dcac
Target Milestone: 4.3 → 4.2
|
|
Assignee | |
Comment 7•10 years ago
|
||
Backported to releases/comm-esr38 changeset e08fb68b3474
Target Milestone: 4.2 → 4.0.2
|
||
Comment 8•10 years ago
|
||
This still needs to be checked in to cc - setting checkin-needed for this.
Blocks: ltn402
Keywords: checkin-needed
|
|
||
Comment 9•10 years ago
|
||
This patch is bitrotted for cc when applyied on top of the one from https://bugzilla.mozilla.org/show_bug.cgi?id=674088#c21
Whiteboard: [comm-centra]
|
|
Assignee | |
Comment 10•10 years ago
|
||
url: https://hg.mozilla.org/comm-central/rev/bdc31ac73a9af77519a613c794cf8c7a2d989a8b
changeset: bdc31ac73a9af77519a613c794cf8c7a2d989a8b
user: Philipp Kewisch <mozilla@kewis.ch>
date: Sat Aug 08 11:05:38 2015 +0100
description:
Fix bug 1186547 - mReadOnly and mDisabled do not reflect readOnly and disabled properties. r=MakeMyDay
|
|
Assignee | |
Updated•10 years ago
|
Keywords: checkin-needed
Whiteboard: [comm-centra]
|
||
Comment 11•10 years ago
|
||
Is the 2nd checkin from comment 6/7/8 required for comm-central too?
Flags: needinfo?(philipp)
|
|
Assignee | |
Comment 12•10 years ago
|
||
Looking at http://mxr.mozilla.org/comm-central/source/calendar/providers/caldav/calDavCalendar.js#1888 this has already happened in the patch for bug 674088. There was a bit of a mess with the push messages here.
Flags: needinfo?(philipp)
|
|
||
Comment 13•10 years ago
|
||
Could this be responsible for the regression reported in Bug 1195800 / Bug 1195974?
|
|
Assignee | |
Comment 14•10 years ago
|
||
Yes, quite likely. I've set bug 1195974 to be tracked for 4.0.3
|
|
||
Comment 15•10 years ago
|
||
Patch was backed-out from Lightning 4.5, 4.4, 4.3 and 4.0 with Bug 1195974. Follow up in Bug 1201862.
You need to log in
before you can comment on or make changes to this bug.
Description
•