Closed
Bug 715890
Opened 13 years ago
Closed 12 years ago
useless refresh of the ACL entry during initialization
Categories
(Calendar :: Provider: CalDAV, defect)
Calendar
Provider: CalDAV
Tracking
(Not tracked)
RESOLVED
FIXED
1.2
People
(Reporter: WSourdeau, Assigned: WSourdeau)
Details
Attachments
(1 file)
|
509 bytes,
patch
|
Fallen
:
review+
|
Details | Diff | Splinter Review |
In calDavCalendar.js, there is code invoking "refresh" on the ACL entry. Since it's supposed to happen from the views, that code can be removed. Currently, it will force the reloading of ACL as part as the calendar intiialization but this is useless since the ACL will already have been initialized.
|
Assignee | |
Updated•13 years ago
|
Attachment #586417 -
Flags: review?(philipp)
|
|
Assignee | |
Updated•13 years ago
|
Severity: normal → minor
OS: Mac OS X → All
Hardware: x86 → All
|
|
Assignee | |
Updated•13 years ago
|
Attachment #586417 -
Attachment is patch: true
|
||
Updated•13 years ago
|
Assignee: nobody → WSourdeau
Status: NEW → ASSIGNED
|
||
Comment 1•12 years ago
|
||
Pushed to comm-central changeset a6ca255de8ff
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.4
|
|
||
Comment 2•12 years ago
|
||
Backported to releases/comm-aurora changeset c27a041a2ce4
Target Milestone: 1.4 → 1.3
|
|
||
Comment 3•12 years ago
|
||
Backported to releases/comm-beta changeset cb9de3bd0866
Target Milestone: 1.3 → 1.2
|
|
||
Updated•12 years ago
|
Attachment #586417 -
Flags: review?(philipp) → review+
You need to log in
before you can comment on or make changes to this bug.
Description
•