544489 - When mails are selected in "Grouped By Sort" mode, group headings are included in "total" count

Status: RESOLVED FIXED

(Thunderbird :: Folder and Message Lists, defect)

Description

[Umang]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2) Gecko/20100115 Firefox/3.6
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.7) Gecko/20100111 Thunderbird/3.0.1

In "Grouped By Sort" mode, when emails are selected the total doesn't tally properly.

Reproducible: Always

Steps to Reproduce:
1. Group by sort
2. Search for mail from a sender (should have mails in two or three groups)
3. Ctrl+A

Actual Results:  
If there are three mails in three groups, the total is shown as 12 instead of 9, because TB is counting the group heading as an email also.

Expected Results:  
1. I do not expect the group heading to be selectable at all.
2. The "Total" tally should not count the group heading at all.

Using Thunderbird 3, Ubuntu Karmic Koala. Downloaded TB from Ubuntuzilla PPA (which packages Official Mozilla Builds of TB).

Comment 1

[Nathan Tuggy (:tuggyne)]

Confirming based on Vista x86 (Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1.7) Gecko/20100111 Lightning/1.0b2pre Thunderbird/3.0.1 ThunderBrowse/3.2.8.1 ID:20100111101938).

Comment 2

[Jason Joseph]

Confirming! This is only marginally annoying because I obsess over the total number of messages sitting in my unified inboxes.

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2.7) Gecko/20100713 Lightning/1.0b2 Thunderbird/3.1.1

Also happens on the same version of Thunderbird on my Windows 7 Professional machine.

Comment 3

[Jason Joseph]

Also even if you aren't selecting items, totals are shown at the bottom of the Thunderbird window. This total seems to show correctly on individual inboxes/folders but for unified views thats when the totals seem incorrect.

Comment 4

[alta88]

Attachment: selectedMessages.patch

Always use folderDisplay.selectedMessages to get the real message count, and not any treeview derived row count.

Comment 5

[Jason Joseph]

Wow someone is finally fixing this, thanks!

Comment 6

[Magnus Melin [:mkmelin]]

Comment on attachment 8657580 [details] [diff] [review]
selectedMessages.patch

Review of attachment 8657580 [details] [diff] [review]:
-----------------------------------------------------------------

What do you think about instead making gFolderDisplay.selectedCount work as intended?
It would be good to decrease usage of global functions like GetNumSelectedMessages instead of increasing it.

Comment 7

[alta88]

Yes, that would surely be better; it's never clear how much refactoring should be done, even in clear cases like this.  The function itself can't be killed due to a few mailnews/ uses, but will be removed everywhere else.

Comment 8

[alta88]

Attachment: selectedMessages.patch

Comment 9

[alta88]

Attachment: selectedMessages.patch

unbitrot.  this patch should be done before the 2 in bug 1192838.

Comment 10

[alta88]

Attachment: selectedMessages.patch

meaning this one...

Comment 11

[Magnus Melin [:mkmelin]]

Comment on attachment 8664007 [details] [diff] [review]
selectedMessages.patch

Review of attachment 8664007 [details] [diff] [review]:
-----------------------------------------------------------------

Great! r=mkmelin
Please have it sent to try if you haven't already

Comment 12

[alta88]

there are no mozmill errors locally.  anyway, is there a procedure to request try?  is push-needed used or watched?  otherwise, i'll ask you for the try.. ;)

Comment 13

[Magnus Melin [:mkmelin]]

No real procedure AFAIK, but if you'd like access to try, file a bug for permissions.

https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=95f65a06f86f

Comment 14

[alta88]

try looks good.

Comment 15

[aleth [:aleth]]

https://hg.mozilla.org/comm-central/rev/c11a88082846ec6ab926d36586e684600c3812b7
Bug 544489 - When mails are selected in "Grouped By Sort" mode, group headings are included in "total" count. r=mkmelin