Bug 27200 - say that MailNotification UnreadMailCount can be bigger then UnreadMails array
Summary: say that MailNotification UnreadMailCount can be bigger then UnreadMails array
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-spec (show other bugs)
Version: git master
Hardware: Other All
: lowest enhancement
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL: http://git.collabora.co.uk/?p=user/ni...
Whiteboard:
Keywords: patch
Depends on:
Blocks: 13349
  Show dependency treegraph
 
Reported: 2010-03-19 07:55 UTC by Nicolas Dufresne
Modified: 2010-03-31 11:25 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

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.