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.
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 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+
Updated patch according to the above comments. We need this also for aurora and beta.
https://hg.mozilla.org/comm-central/rev/c73aed47dd731f55989d63db065b45c5cf51ce5e Bug 1228438 - Convert event to message is broken;r=philipp
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 4.8
Keyword "checkin-needed" set for aurora/beta
Whiteboard: [checkin-needed comm-aurora, comm-beta]
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.
The calendar process has usually been, whoever does the push to c-c also does the backport to approved branches.
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]
Backported to releases/comm-aurora changeset 54de083be103
Target Milestone: 4.8 → 4.7
Backported to releases/comm-beta changeset 3866c4feab99
Target Milestone: 4.7 → 4.6
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) - "firstname.lastname@example.org" == "\\"Last, First\\" <email@example.com>"
Please do not add additional beta test breakages, can we get the issues of comment 11 and comment 8 resolved please?
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.