Closed
Bug 103732
Opened 23 years ago
Closed 21 years ago
mail status "replied" not set if mail replied "offline"
Categories
(SeaMonkey :: MailNews: Backend, defect, P3)
SeaMonkey
MailNews: Backend
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla2007, Assigned: Bienvenu)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
The mail-status "replied" is not set correct, if you reply the mail offline.
Steps to reproduce:
1.) Get new mail
2.) select "Work offline"
3.) reply mail
4.) go online, send mail
The status of the just replied mail stays "read" instead of "replied".
Better: The status should be converted to replied, after the mail is sent
successfully.
Always reproducable with Builds 2001100306, 2001091311
Confirmed on commercial trunk
2001-10-08-09-trunk NT 4.0
2001-10-08-08-trunk linux 2.2
2001-10-08-08-trunk mac 9.1
Replying to a downloaded mesg while offline results in
the status still being listed as 'read' rather than
'replied' when you go back online.
Replying to a downloaded mesg while online, results
in status being changed to 'replied'.
changing status to new
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 2•23 years ago
|
||
the problem is that the mark replied stuff is handled from
nsMsgComposeSendListener, and there isn't one when we're doing a send later.
I'll try to look at extending the send later code to handle this.
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•23 years ago
|
||
When we sent the unsent message, we no longer know that it's a reply to a
particular message. To fix this, we could store the replied flag when the send
later operation happens, and play it back later when going back online, though
we don't really know if the actual send has happened (this seems to be what 4.x
did). Or we could add extra info to the unsent message in the unsent messages
folder to know the uri of the msg being replied to, strip that out of the unsent
message before sending it, and use the info to mark the original message as
replied. The latter approach is more reliable, but a lot more work.
Updated•23 years ago
|
Updated•23 years ago
|
Priority: -- → P3
Updated•23 years ago
|
Assignee | ||
Comment 5•23 years ago
|
||
*** Bug 126544 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 6•23 years ago
|
||
the same problem happens, for the same reason, when you save a reply as a draft.
Assignee | ||
Comment 7•23 years ago
|
||
*** Bug 126544 has been marked as a duplicate of this bug. ***
Updated•22 years ago
|
OS: Linux → All
Hardware: PC → All
*** Bug 184647 has been marked as a duplicate of this bug. ***
*** Bug 149395 has been marked as a duplicate of this bug. ***
Comment 10•22 years ago
|
||
*** Bug 120791 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 11•22 years ago
|
||
*** Bug 202126 has been marked as a duplicate of this bug. ***
Comment 12•21 years ago
|
||
The solution (and behaviour) proposed in comment #3 seems logical. But some
people are used to the behaviour in old Communicator 4.x. They want a message to
be marked replied even when the reply was never actually sent. They want it as
an indication that they there was some message derived from it (reply, forward,
etc.). My proposal is: as a temporary (or even permanent) workaround, we could
add some new states of a message, which would indicate this situation - reply to
a message created, but not yet really sent. They can be called 'Reply prepared',
'To be forwarded', or something like that. Would new states broke something or
be too much work? Thanks
Assignee | ||
Comment 13•21 years ago
|
||
*** Bug 207132 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 14•21 years ago
|
||
I thought of another way to fix this - store the uri of the message we're
replying to in the database for the unsent messages folder. Then, when we're
sending the message, we can pull the uri out of the database and mark the
original message replied, forwarded, etc.
Assignee | ||
Comment 15•21 years ago
|
||
this patch implements what I described in my previous comment, and does some
other code cleanup.
Assignee | ||
Comment 16•21 years ago
|
||
fix checked in, r/sr=mscott
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment 17•21 years ago
|
||
Great, if it was tested and it works.
David, could you also implement the second way of marking the messages, while
you are at it? Some users wish that the messages are marked immediately after
the reply is composed, not wait until it is really send. Your implementation
could handle this too. When you write the message URI in the database, at this
point you know that there is a reply prepared. You can mark the message replied
to immediately (as replied). And there can be a pref for switching these 2 modes
of operation. I think there isn't much work involved in this hack. Please, make
this if you have time. Thanks.
Comment 18•21 years ago
|
||
Does the uri in the database automatically change if its mail is deleted or moved?
If not, we might run into problems, because:
what happens, if the original mail is moved to another folder before the Reply
has been sent?
Assignee | ||
Comment 19•21 years ago
|
||
re the comment about making it optionally mark a message replied at message
composition or send later time, that's a low priority for me (which is not to
say that someone else couldn't implement it). Re the question about the message
moving between send later and the actual send, that might not work in some
cases, but that's an edge case that would be very difficult to fix.
Comment 20•21 years ago
|
||
If the message is not found (was moved), it can search for it in all other
folders (of the same account). Just extract the message id from the URI and find
the message. I think not too difficult, maybe slow. Deleted messages are no problem.
Comment 21•21 years ago
|
||
David Bienvenu, can the fix from this bug be applied to solve the same problem
for messages saved as drafts (bug 128996)?
Assignee | ||
Comment 22•21 years ago
|
||
yes, it might.
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•