"Unable to save your message as a draft" and "Assembling mail information" when replying to forwarded attached .eml message

RESOLVED FIXED in Thunderbird 12.0

Status

MailNews Core
Backend
RESOLVED FIXED
10 years ago
5 years ago

People

(Reporter: Rod Whiteley, Assigned: Magnus Melin)

Tracking

(Blocks: 1 bug)

unspecified
Thunderbird 12.0
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 2 obsolete attachments)

(Reporter)

Description

10 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.5) Gecko/20070713 Firefox/2.0.0.5
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.5) Gecko/20070716 Thunderbird/2.0.0.5

Thunderbird cannot save a message as a draft when it is a reply to a message that was forwarded as an attachment.  This is worse when auto-save is set, because you get periodic error messages.

Reproducible: Always

Steps to Reproduce:
0. Write a message and verify that you can save it as a draft.
1. In Tools – Options... – Composition – General, set Thunderbird to forward messages as attachments.
2. Forward a test message to yourself.
3. Receive the new message, open it, open the attached copy of the test message, and write a reply to it.
4. Try to save your reply as a draft.
Actual Results:  
Error message: "Unable to save your message as draft.
Please verify that your Mail & Newsgroups account settings are correct and try again."


Also in: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a7pre) Gecko/2007072504 Thunderbird/3.0a1pre
(Assignee)

Comment 1

10 years ago
Confirmed on linux/trunk.
Blocks: 269826
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows XP → All
Hardware: PC → All
Reproduced with Seamonkey 1.1.3(MS Win-XP). Core bug.
(Assignee)

Updated

10 years ago
Component: Message Compose Window → MailNews: Backend
Product: Thunderbird → Core
QA Contact: message-compose → backend
(Assignee)

Updated

10 years ago
Summary: Unable to save draft when replying to forwarded attached message → Unable to save draft when replying to forwarded attached .eml message
(Assignee)

Updated

10 years ago
Duplicate of this bug: 413346

Comment 4

10 years ago
This is very annoying, since with auto-save, the error message blocks you from continuing to write (i.e., you *have to* click ok to continue), which gets very annoying.

Also, it attracts the focus to the compose window, even when it is in the background: the error message pops up in front of everything, clicking OK lifts the compose window to focus, so one has to switch to the window one was working with again.

(linux with 2.0.0.12)

Comment 5

9 years ago
My impression is that, if you have autosave and you don't service the dialogue box, Thunderbird eventually locks up (Linux).
(Assignee)

Updated

9 years ago
Duplicate of this bug: 439159

Comment 7

9 years ago
This exact problem also occurs if I open a saved .eml file and reply to that message.

version 2.0.0.16 (20080708), and I believe at least one prior version.
Product: Core → MailNews Core
(Assignee)

Updated

9 years ago
Duplicate of this bug: 391503

Comment 9

9 years ago
I confirm this bug for Thunderbird/3.0b2
It is really annoying.

Comment 10

8 years ago
Confirmed for Thunderbird 3.0.0 on Windows 7 and Thunderbird version 2.0.0.23 (20090812)on Snow Leopard.

Also confirmed as Mark Sapiro stated above

"This exact problem also occurs if I open a saved .eml file and reply to that
message."

We save a lot of email messages for auditing purposes and so this bug is pretty lame. I'd just persuaded everyone to move from Outlook as well.

Comment 11

8 years ago
Bug still exists in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.7) Gecko/20100111 Thunderbird/3.0.1
Duplicate of this bug: 629683
Duplicate of this bug: 525034
"Assembling mail information..." is also seen in status bar
Summary: Unable to save draft when replying to forwarded attached .eml message → "Unable to save your message as a draft" and "Assembling mail information" when replying to forwarded attached .eml message

Comment 15

6 years ago
Don't know if it's only me, but I have one more problem in this situation : I can not paste an image in the reply (instead it show a "broken image" icon)

Comment 16

6 years ago
TB 5 problem. This bug:
- When saving e-mails in *.eml format to the local file system
- re-opening them, 
- clicking on reply, adding text
- click on close
- click on "yes safe as draft"
- getting this message:

[Unable to save your message as draft.
Please verify that your Mail & Newsgroups account settings are correct and try again]

existed already in TB3. Work around was to not click on reply but on forward. E-mail could then be saved as a draft. 
Now in TB5 even this is not possible any longer. E-Mails saved to the local system (Win 7, 64 Bit) can´t be saved as draft any longer. This is a big problem as many companies safe important e-mails to a digital filing system and can't save them later as draft for further action. Hope there will be a solution soon. Our Company downgraded from TB5 to TB3 again to have a workable workflow.

Comment 17

6 years ago
(In reply to comment #16)

OrchidOne, this is what I reported in Bug 548070 (glad I am not alone ;-)
probably you want to have a look there.
These bugs seem to be somehow related.

However, I get the same behaviour with TB 5 (I can press "save as draft", and I get the same error as in TB 3), tested on WinXP 32, TB 5 release version.

Working with .eml files does not seem to be a recommended option (and might look a bit "oldfashioned"), but also my client here uses .eml files heavily...

Comment 18

6 years ago
I've found that this is a timeout type issues.

When composing an email (reply, forward, edit as new) and I have attachments or images in the email if I take too long (5 minutes) to edit and save or send it will start giving this error.

If I then delete and read the images and attachments it will work just fine again. I think it is loosing the connections to the files that are in the original email message.

This happens more often when I'm editing a template that has already be created (they have attachments and embedded images).

Comment 19

6 years ago
That's interesting - I think you're describing a different way to get the same repeatable error message, and I suppose they could be related. I don't actually know what these objects look like in memory, but maybe the compose/reply window is just incapable of (re-)initiating a connection or using an existing one when it doesn't have one, such as after a timeout or when the reply was launched from a parent who has no reference to the existing connection pool.

I'm sure a timeout is not a required component of this bug, though. This problem has been reproducible for me every time I try to reply to attached mails and .eml files, without waiting for a time-out, whether they are plain text, HTML, have attachments, or no attachments.
(Assignee)

Comment 20

6 years ago
Created attachment 584459 [details] [diff] [review]
proposed fix

Basically there were two problems here: currentAccountKey was undefined (set up in another if clause), and in nsMsgSend.cpp we returned a handled error for draft/template saving.

The other changes are whitespaceish, as i got rid of two unncessary if blocks - gCompose or gMsgCompose.compFields really shouldn't be null. If they are you'd want to know.

I'll attach an whitespace patch too for easier review.
Assignee: nobody → mkmelin+mozilla
Status: NEW → ASSIGNED
Attachment #584459 - Flags: review?(dbienvenu)
(Assignee)

Comment 21

6 years ago
Created attachment 584460 [details] [diff] [review]
proposed fix (whitespace changes ignored)

Comment 22

6 years ago
Comment on attachment 584459 [details] [diff] [review]
proposed fix

thx, Magnus - seems to work fine here. And the mozmill test seems OK when run by itself, but when all the compose tests are run, I get several failures in test-forward-headers.js -I'm not sure if your new test is not cleaning up after itself, or what...but I'm going to clear the review request for now, and if you could look into the composition test failure, that would be great.
Attachment #584459 - Flags: review?(dbienvenu)
(Assignee)

Comment 23

6 years ago
Created attachment 584602 [details] [diff] [review]
proposed fix v2

This deletes the drafts too, so there's no problem with other tests.
Attachment #584459 - Attachment is obsolete: true
Attachment #584460 - Attachment is obsolete: true
Attachment #584602 - Flags: review?(dbienvenu)
(Assignee)

Comment 24

6 years ago
Created attachment 584603 [details] [diff] [review]
proposed fix v2 (whitespace changes ignored)

Comment 25

6 years ago
Comment on attachment 584602 [details] [diff] [review]
proposed fix v2

thx, that makes the tests pass when run in batch.
Attachment #584602 - Flags: review?(dbienvenu) → review+
(Assignee)

Comment 26

6 years ago
http://hg.mozilla.org/comm-central/rev/6846848bd7d0
->FIXED
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 12.0
(Assignee)

Updated

6 years ago
Flags: in-testsuite+
Duplicate of this bug: 548070

Updated

6 years ago
Depends on: 738907
(Assignee)

Updated

5 years ago
Depends on: 745395
You need to log in before you can comment on or make changes to this bug.