Closed
Bug 1228438
Opened 10 years ago
Closed 9 years ago
Convert event to message is broken (ReferenceError: "aToList is not defined" when calling getRecipientList)
Categories
(Calendar :: Calendar Frontend, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
4.6
People
(Reporter: ssitter, Assigned: MakeMyDay)
References
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
6.47 KB,
patch
|
MakeMyDay
:
review+
Fallen
:
approval-calendar-aurora+
Fallen
:
approval-calendar-beta+
|
Details | Diff | Splinter Review |
Lightning 4.7a1 (20151126030234) with Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.0a1
Steps:
Thunderbird is setup with only one "Blogs & News Feeds" account but no email account. When right clicking on an event and selecting Convert To > Message... from context menu nothing happens. The error console shows:
> Error: ReferenceError: aToList is not defined
> Source File: resource://calendar/modules/calUtils.jsm Line: 367
It looks like the broken code was added with Bug 1158036, therefore this problem might exist in Lightning 4.2 and newer. I only tested Lightning 4.6a2 and Lightning 4.7a1.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → makemyday
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•10 years ago
|
||
Apart from the obvious fix, I add two unit tests for the used calUtils.jsm functions. The patch should be applied on top of that for bug 1212075.
A try push for tests of both bugs is still running.
Attachment #8692989 -
Flags: review?(philipp)
Comment 2•10 years ago
|
||
Comment on attachment 8692989 [details] [diff] [review]
FixGetRecipientList-V1.diff
Review of attachment 8692989 [details] [diff] [review]:
-----------------------------------------------------------------
Check for whitespaces and maybe use an arrow function for the filter
Attachment #8692989 -
Flags: review?(philipp) → review+
Assignee | ||
Comment 3•9 years ago
|
||
Updated patch according to the above comments.
We need this also for aurora and beta.
Attachment #8692989 -
Attachment is obsolete: true
Attachment #8702893 -
Flags: review+
Attachment #8702893 -
Flags: approval-calendar-beta?(philipp)
Attachment #8702893 -
Flags: approval-calendar-aurora?(philipp)
Updated•9 years ago
|
Attachment #8702893 -
Flags: approval-calendar-beta?(philipp)
Attachment #8702893 -
Flags: approval-calendar-beta+
Attachment #8702893 -
Flags: approval-calendar-aurora?(philipp)
Attachment #8702893 -
Flags: approval-calendar-aurora+
Updated•9 years ago
|
Keywords: checkin-needed
Comment 4•9 years ago
|
||
https://hg.mozilla.org/comm-central/rev/c73aed47dd731f55989d63db065b45c5cf51ce5e
Bug 1228438 - Convert event to message is broken;r=philipp
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 4.8
Reporter | ||
Updated•9 years ago
|
Whiteboard: [checkin-needed comm-aurora, comm-beta]
Comment 6•9 years ago
|
||
Just to be clear, I do not routinely do comm-beta checkins for calendar bugs, but I assume that the calendar team has their own process to get these checked in.
Comment 7•9 years ago
|
||
The calendar process has usually been, whoever does the push to c-c also does the backport to approved branches.
Assignee | ||
Comment 8•9 years ago
|
||
We should either skip the beta uplift for this or also uplift bug 121075.
Whiteboard: [checkin-needed comm-aurora, comm-beta] → [checkin-needed comm-aurora]
Comment 9•9 years ago
|
||
Backported to releases/comm-aurora changeset 54de083be103
Keywords: checkin-needed
Target Milestone: 4.8 → 4.7
Comment 10•9 years ago
|
||
Backported to releases/comm-beta changeset 3866c4feab99
Target Milestone: 4.7 → 4.6
Updated•9 years ago
|
Whiteboard: [checkin-needed comm-aurora]
Reporter | ||
Comment 11•9 years ago
|
||
This broke unit tests on beta: https://treeherder.mozilla.org/logviewer.html#?job_id=5710&repo=comm-beta
TEST-UNEXPECTED-FAIL | xpcshell-libical.ini:calendar/test/unit/test_calutils.js | getAttendeeEmail_test - [getAttendeeEmail_test : 59] (test #1) - "first.last@example.net" == "\\"Last, First\\" <first.last@example.net>"
Assignee | ||
Comment 12•9 years ago
|
||
yes, this is expected without the uplift at least of bug 1212075, see comment #8.
Comment 13•9 years ago
|
||
Please do not add additional beta test breakages, can we get the issues of comment 11 and comment 8 resolved please?
Assignee | ||
Comment 14•9 years ago
|
||
The additional backport is already available in the other bug.
You need to log in
before you can comment on or make changes to this bug.
Description
•