Bug 27200

Summary: say that MailNotification UnreadMailCount can be bigger then UnreadMails array
Product: Telepathy Reporter: Nicolas Dufresne <nicolas>
Component: tp-specAssignee: Telepathy bugs list <telepathy-bugs>
Status: RESOLVED FIXED QA Contact: Telepathy bugs list <telepathy-bugs>
Severity: enhancement    
Priority: lowest Keywords: patch
Version: git master   
Hardware: Other   
OS: All   
URL: http://git.collabora.co.uk/?p=user/nicolas/telepathy-spec.git;a=commitdiff;h=3b2374c959f1169b95975d0183f199eb15a2776f;hp=b8e35eaf7f3f2cc132c92be6dbd6af008824e68e
Whiteboard:
i915 platform: i915 features:
Bug Depends on:    
Bug Blocks: 13349    

Description Nicolas Dufresne 2010-03-19 07:55:43 UTC
Google limits the number of item in the UnreadMails array to 30, but report the total number of unread e-mail correctly. This seems reasonable way to reduce the network overhead by such notification.

The result is that the size of UnreadMails maybe smaller or equal to UnreadMailCount. This behavior should be documented in the spec as being normal. Nothing in the spec currently deny this.
Comment 1 Simon McVittie 2010-03-19 09:13:30 UTC
I think at some stage I made the spec wording explicitly say the opposite, but then reverted it... good thing I did, really :-)

Could you provide a patch please? I think the behaviour you've implemented in Gabble (as proposed here) will confuse people if we don't draw attention to it.
Comment 2 Nicolas Dufresne 2010-03-31 09:33:42 UTC
Oops, title was upside-down.
Comment 4 Nicolas Dufresne 2010-03-31 11:02:24 UTC
Updated my patch base on Simon comments on IRC:

New patch:
http://git.collabora.co.uk/?p=user/nicolas/telepathy-spec.git;a=commitdiff;h=3b2374c959f1169b95975d0183f199eb15a2776f;hp=b8e35eaf7f3f2cc132c92be6dbd6af008824e68e

IRC Log:
<smcv> stormer: the tp:rationale there doesn't make sense to me... it's of the form "thing (rationale: rationale for an unrelated thing)"
<smcv> stormer: adding "This list MAY be shorter than <tp:member-ref>UnreadMailCount</> would indicate" would fix that
<smcv> stormer: (i.e. just before the rationale)
<smcv> stormer: thanks for catching up on this
<smcv> stormer: I'd prefer it if UnreadMailCount's docstring had proper HTML paragraphs, now that it's more than one paragraph

Comment 5 Nicolas Dufresne 2010-03-31 11:25:54 UTC
Will be in 0.19.4.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.