267855 - Delete mail marked as junk" does nothing when Grouped By Sort is on

Status: RESOLVED FIXED

(Thunderbird :: Mail Window Front End, defect)

Description

[John Keiser (jkeiser)]

I have my mail in an IMAP folder (secure, if it matters) and I am trying the new
Grouped By Sort (I sort by date descending so the latest is at the top). 
However, when I do that Delete Mail Marked as Junk does not work.

Repro Steps:
1. Set up an IMAP folder with Grouped By Sort, date descending (not sure if IMAP
or date descending matter but Grouped By does)
2. Mark a message junk (it was marked automatically for me by the junk mail
controls)
3. Use Tools > Delete Mail Marked As Junk

Actual Results: first time you try it, it expands all the groups, second time
does nothing, but never deletes any mail
Expected Results: deletes mail marked as junk

Note that when I remove Grouped By Sort it *does* delete the junk mail,
indicating it is definitely related to the confluence of the two features.

Comment 1

[Mike Fedyk]

Confirmed on win2k with TB 0.9

Comment 2

[David :Bienvenu]

taking - I see the problem. Mike, thx for working on reproducing bugs like this
- here's a way to be even more helpful: if you run with the js console turned
on, for a lot of front end bugs, you'll actually see useful information in the
js console that will help us track down the problem - in  this case, 

* Call to xpconnect wrapped JSObject produced this error:  *
[Exception... "Component returned failure code: 0x80550018 [nsIMsgDBView.getURIF
orViewIndex]"  nsresult: "0x80550018 (<unknown>)"  location: "JS frame :: chrome
://messenger/content/mailCommands.js :: deleteJunkInFolder :: line 675"  data: n
o]

if you ever see an Exception in the js console, that's very likely a cause or at
least a symptom of the problem.

Comment 3

[David :Bienvenu]

Attachment: proposed fix

need to catch a couple exceptions.

Comment 4

[Mike Fedyk]

OK, thanks for the tip.

Comment 5

[Marcia Knous [:marcia]]

Has a patch, setting TM to 1.0 for consideration.

Comment 6

[David :Bienvenu]

fixed on trunk and branch

Comment 7

[sairuh (rarely reading bugmail)]

*** Bug 270471 has been marked as a duplicate of this bug. ***

Comment 8

[Mike Cowperthwaite]

*** Bug 273023 has been marked as a duplicate of this bug. ***