Closed Bug 673049 Opened 11 years ago Closed 11 years ago
Forwarding an .eml message causes message compose error
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:5.0.1) Gecko/20100101 Firefox/5.0.1 Build ID: 20110707182747 Steps to reproduce: Save a message to .eml format Open .eml message in TB (opens in new window) Click on Forward Actual results: First attempt, compose window opens with blank subject line On second attempt, error: An error occurred while creating a message compose window. Please try again. Expected results: Compose window should open as normal without error so email can be forwarded
Does this happens in -safe-mode (see http://support.mozillamessaging.com/en-US/kb/Safe-Mode) ?
Yes same issue in safe mode
Confirmed on nightly. I'm in the process of narrowing down where this is happening...
Status: UNCONFIRMED → NEW
Ever confirmed: true
Fallout from bug 583587, which adds references to forwarded messages, but this breaks for .emls because we can't get an nsIMsgDBHdr for them.
Here's a quick one-line patch to fix this. No tests, since it's just one line (though I keep thinking when my patch queue isn't full to bursting that I should write some more tests for .emls).
Sorry, *this* is a one-line fix. (The previous patch had an extra blank line that snuck in.)
Comment on attachment 547339 [details] [diff] [review] Fix this (again) Looks like the right to do. Thanks for fixing this so quickly.
Attachment #547339 - Flags: review?(jonathan.protzenko) → review+
Assignee: nobody → squibblyflabbetydoo
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment on attachment 547339 [details] [diff] [review] Fix this (again) Asking for approval on branches. This is extremely low-risk, and high-gain for anyone who uses .eml files regularly.
Odd, this bug looks entirely mailnews core, yet my test on SeaMonkey 2.2 (roughly == to miramar) appears to be fine.
You need to log in before you can comment on or make changes to this bug.