Draft mail doesn't remember the From account

[humdinger]

This is hrev33529.

When opening a saved draft mail, the From field always uses the default mail account. Instead the account that's been set for the mail when it was saved should be remembered.

[jalopeura]

This patch fixes it so the drafts now remember the account, and the encoding as well.

[phoudoin]

What will happens if the account at draft save time don't exists anymore now (or was renamed)? Same goes for encoding, even if it's a lesser risk.

In such case, account or encoding should bet set to default one. And no need to put in comment previous code.

Correct these issues, and I'll be glad to commit your patch.

[jalopeura]

Hm.

It looks like BMenu::FindItem() will return NULL...which will cause a problem.

I left the commented code in because I wasn't sure which version was the preferred style. Now I need to use it anyway, so I can check for NULL.

I'm in the middle of something on Windows at the moment; expect an updated patch in a few hours.

[jalopeura]

Patch

[jalopeura]

Okay, fixed the problem with my last patch. I verified by manually changing the account and encoding attributes to "none", and it worked.

I also noticed that changing the account and/or encoding doesn't mark the mail as changed (i.e., doesn't enable the Save button), so I fixed that as well.

[phoudoin]

Thanks, looks fine now. I'll commit it soon.

[phoudoin]

Applied in hrev41266. Thanks.