1772550 - Marking a message as spam only moves it to Junk sometimes

Status: RESOLVED DUPLICATE of bug 1773605

(MailNews Core :: Networking: IMAP, defect)

Description

[Steve Shockley]

Steps to reproduce:

I recently upgraded from TB 91.x to the beta channel. Since then, marking messages as junk behaves differently and inconsistently.

Actual results:

Open an IMAP folder (not Inbox, if that matters)
Highlight a junk message. It doesn't matter if it's already marked as Junk by TB.
Press J

Sometimes the message is marked deleted/read/junk and moved to Junk. Other times it's just marked read/junk and not moved or marked deleted. Occasionally it's marked deleted and moved, but not marked read.

Expected results:

In 91.x and before, pressing J would mark the message as junk/read, mark it as deleted, then purge the message from the folder without affecting other messages marked as deleted.

I did a little more testing, and it looks like moving a message has different behavior now. Previously, moving a message would mark it deleted and then purge it. Now it just marks it as deleted. I prefer the old behavior. Regardless, marking a message as junk still behaves inconsistently.

Comment 1

[gene smith]

I'm seeing something else when I try to do a move between folders on same server. I have to go to the destination folder and then back to the source folder before the move is actually triggered. I'm not sure if this is caused by some uncommitted changes I have running or what. I need to back out some stuff, including maybe the charter change, bug 1771409

Or maybe this is also what the reporter is seeing? (Pressing J also triggers a MOVE to Junk folder.)

Otherwise I don't immediately see what is reported regarding expunging. I probably need more info on the imap server that the problem is seen on. "Move" function can depend on a few factors: Is MOVE supported and, if not, is UIDPLUS supported in imap capability. An IMAP:5 log may be helpful to show more details: https://wiki.mozilla.org/MailNews:Logging
I tried it on a server that supports MOVE and one that doesn't support MOVE but simulates a MOVE using UID EXPUNGE with UIDPLUS capability. I don't think I have a server that only doesn't support UIDPLUS and doesn't support MOVE (i.e., COPY only).

Comment 2

[gene smith]

Backed out uncommitted changes and still see the problem. URL onlinemove only occurs when user triggers a move on the first select of a folder. After that you have to move off of the source folder and then back to the source folder before an onlinemove URL occurs again. Also, if you shutdown with a move triggered but before onlinemove URL occurs and then start back up, the onlinemove URL occurs and the messages is moved to the destinatrion. (Didn't know URLs would be saved/queued between sessions!) This really has nothing to do with that the message is Junk. In fact, the URLs that mark the message as junk occur correctly when "J" is pressed but the onlinemove URL doesn't occur after the first time source folder is selected.
Haven't tried backing out charter changes but don't see how they would affect this (but who knows).

Comment 3

[gene smith]

Works OK with 100.0b4 and bad on next beta 101.0b1. Haven't determined yet the change that is causing the bug that I see with move. FWIW, I also see the same problem with copy. Also, bad beta 101.0b1 does not have the charter change (bug 1771409).
I only see one other imap related change in the release notes but it is at 101.0b4 (bug 1747173) so it doesn't seem to be causing the problem.

Comment 4

[gene smith]

I'm unable to duplicate that message is not marked as junk. However, the failure to move to Junk folder is the same as bug 1773605 so I'm marking this as a duplicate.