Closed Bug 727951 Opened 8 years ago Closed 8 years ago

removing expired news articles broken

Categories

(MailNews Core :: Networking: NNTP, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 13.0

People

(Reporter: Bienvenu, Assigned: Bienvenu)

References

()

Details

(Keywords: regression)

Attachments

(1 file)

Attached patch proposed fixSplinter Review
the url we run to get rid of expired articles is failing. Patch upcoming
Attachment #597934 - Flags: review?(Pidgeot18)
Status: NEW → ASSIGNED
OS: Windows 7 → Windows 2000
Hardware: x86_64 → All
Summary: removing expired news articles borken → removing expired news articles broken
I'd feel a lot better about this if we could test it.
Keywords: regression
OS: Windows 2000 → All
I don't think this is quite enough--I tested it briefly (by accident, natch) and the result I got appears to be that it cleared the newsgroup but it also opened up the file dialog "do you want to open the file mozilla.dev.test.multimedia?"...
URL:
(In reply to Joshua Cranmer [:jcranmer] from comment #2)
> I don't think this is quite enough--I tested it briefly (by accident, natch)
> and the result I got appears to be that it cleared the newsgroup but it also
> opened up the file dialog "do you want to open the file
> mozilla.dev.test.multimedia?"...

Did you try it with the patch in bug 492329 ? That's the only way I've tried it, and I did not get the file dialog.
Comment on attachment 597934 [details] [diff] [review]
proposed fix

I finally did a more purposeful test of this, and it worked this time. The UI for clearing messages kind of sucks (you see no apparent visual cue that anything is happening until the operation finishes, when messages disappear), but that's probably worth another bug.

Some other notes:
1. Lose the comment (or at least the "I think")
2. return rv; probably ought to be return NS_OK;
3. It's probably cleaner to separate the if statement from the rest of the code with blank lines.
Attachment #597934 - Flags: review?(Pidgeot18) → review+
fixed on trunk - forgot to address the comments; I'll do that later today
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 13.0
http://hg.mozilla.org/comm-central/rev/c7d25333f38a pushed for review comments.
Duplicate of this bug: 739934
Duplicate of this bug: 743432
Duplicate of this bug: 749803
You need to log in before you can comment on or make changes to this bug.