126626 - getNumUnread(true) and getTotalMessages(true) return incorrect values for servers

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[[e-mail]]

There's a later comment // it's legal for counts to be negative if we don't know
which stops subfolder negative counts from being considered. However the folder
itself might have a negative count. In this case getNumUnread could return 0
when a subfolder has a positive count. This stops next message navigation from
working. The fix is to ensure that the initial count is at least zero, this then
avoids the need for a test of the return from the deep search.

Comment 1

[[e-mail]]

Attachment: Proposed patch

Fix problem by moving negative test

Comment 2

[Vidar Haarr (not reading bugmail)]

*random acts of poking* ;)

Comment 3

[David :Bienvenu]

Comment on attachment 70498 [details] [diff] [review]
Proposed patch

sr=bienvenu

Comment 4

[[e-mail]]

Fix updated for bitrot and checked in.
Also applied to the total count, for consistency.

Comment 5

[[e-mail]]

Attachment: Updated for bitrot and checked in

Comment 6

[[e-mail]]

So I should actually close it as such :-)