Build ID: 2001-08-31-03, All OSs. Summary: Brand new profile, news://poisonivy.mcom.com/xrated is coming up with 'all read' status. Steps to Reproduce: 1. Make sure you have a clean profile, or at least have deleted your news files (rc, msf, etc) 2. Using poisonivy.mcom.com, subscribe to 'xrated' (an auth group) 3. Enter your authentication info (Seth, if you don't know user/pass, AIM me or mail me). 4. Get all of the messages in the group (105 at time of this bug report). Actual Results: On a brand new profile, all of these postings are marked as read. Expected Results: Newsgroup should be totally unread, as it's the first time you've seen it (according to your profile).

This happened even on a migrated 4.7x profile (win32), but doesn't happen on any other newsgroups... Interesting to note, when you restart Mozilla this usually gets the right unread counts and status markings back.

The screenshot I posted was after restarting Mozilla, as soon as I clicked on the newsgroup, the unread count in the folder pane vanished (even though you can see that the message pane is totally unread). Note that we're also incorrect in the taskbar/status bar.

not an emojo stopper.

When you create new profile are you just exiting from a profile that has same newsgroup and leaving quick launch open?

John Lar, no quick launch was used, and I just ran mozilla.exe with - profilemanager to create a brand new profile.

yes, I see this too, and on netscape.champions both groups require authentication.

So far, only seeing this on NNTP servers running Netscape-Collabra software. I log into other NNTP servers NOT running this software and expected read/unread behaviour is normal.

Do you read groups that require authentication on those non-Collabra servers?

Yes Dan, I do log into one particular non-collabra server that requires authentication and the read/unread anomaly doesn't exist.

part of the problem (there might be more) is that when we update the unread counts (issuing GROUP commands) we end up setting the read set to include everything. I'll continue debugging.

This sounds pretty bad. I'm chanigng my mind.

wacky, the value in the newsrc looks fine. I'll continue to debug.

I've noticed that if you are already authenticated on one group on a given server (such as netscape.champion.beta), if you subsequently move on to another auth group (such as netscape.champions) the messages appear marked as UNread. Just as they should be.

yes, I'm seeing similar weirdness to what steve mann reports. actually, I do see some newsrc weirdness. instead of setting the newsrc line to be "1-oldest" we're somehow setting it to "1-youngest", which makes all articles read. not sure why that's happening yet.

fix in hand. damn, this bug was a pain to debug.

the problem is I was calling CleanupNewsgroupList() and I didn't need to at BeginReadXOver(). CleanupNewsgroupList calls FinishXOVERLINE() which will adjust the unread set. I was smoking the good stuff when I slapped that call to CleanupNewsgroupList() to release m_newsgroupList. (CleanupNewsgroupList() sets it to null.) that's not necessary, since we create a new nsINNTPNewsgroupList, which would cause m_newsgroupList to get released. cleanup should only be called after we've done work.

Comment on attachment 50191 [details] [diff] [review] the fix. r=mscott

Comment on attachment 50191 [details] [diff] [review] the fix. this has sr=bienvenu, over aim.

fixed on the trunk. awaiting verification and then PDT approval for the branch. stephend, if you have cycles, can you verify this on tomorrow's trunk builds?

Verified FIXED on the trunk: Mac OS 9.1 - 2001-09-21-04 Windows 2K - 2001-09-21-03 RedHat 7.1 - 2001-09-21-06 I did both migrated 4.7x profiles, and new profiles, and this is indeed FIXED.

check it in - PDT+

As of 2001092103 it's fixed. Deleted n.champions, re-subscribed and behaviour is back to normal. Thanks !!

This should stay open until Seth lands this on the trunk.

s/trunk/branch

fixed on both trunk and branch.

This has now been VERIFIED FIXED using the latest 9-23-xx BRANCH builds on Mac OS 9.1, RedHat Linux 7.1 and Windows 2000.