Closed
Bug 727951
Opened 13 years ago
Closed 13 years ago
removing expired news articles broken
Categories
(MailNews Core :: Networking: NNTP, defect)
MailNews Core
Networking: NNTP
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 13.0
People
(Reporter: Bienvenu, Assigned: Bienvenu)
References
()
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
patch
|
jcranmer
:
review+
|
Details | Diff | Splinter Review |
the url we run to get rid of expired articles is failing. Patch upcoming
Attachment #597934 -
Flags: review?(Pidgeot18)
Assignee | ||
Updated•13 years ago
|
Status: NEW → ASSIGNED
OS: Windows 7 → Windows 2000
Hardware: x86_64 → All
Summary: removing expired news articles borken → removing expired news articles broken
Comment 1•13 years ago
|
||
I'd feel a lot better about this if we could test it.
Updated•13 years ago
|
Keywords: regression
OS: Windows 2000 → All
Comment 2•13 years ago
|
||
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?"...
Assignee | ||
Comment 3•13 years ago
|
||
(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 4•13 years ago
|
||
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+
Assignee | ||
Comment 5•13 years ago
|
||
fixed on trunk - forgot to address the comments; I'll do that later today
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•13 years ago
|
Target Milestone: --- → Thunderbird 13.0
Assignee | ||
Comment 6•13 years ago
|
||
http://hg.mozilla.org/comm-central/rev/c7d25333f38a pushed for review comments.
You need to log in
before you can comment on or make changes to this bug.
Description
•