Bug 42305

Summary: group-mixin: should set Members_Changed_Detailed flag
Product: Telepathy Reporter: Guillaume Desmottes <guillaume.desmottes>
Component: tp-glibAssignee: Telepathy bugs list <telepathy-bugs>
Status: RESOLVED FIXED QA Contact: Telepathy bugs list <telepathy-bugs>
Severity: normal    
Priority: medium    
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments: group-mixin: always set the Members_Changed_Detailed flag

Description Guillaume Desmottes 2011-10-27 03:13:39 UTC
The group mixin unconditionally implements MembersChangedDetailed so it should set the Members_Changed_Detailed as well.
Comment 1 Guillaume Desmottes 2011-10-27 03:32:23 UTC
Created attachment 52819 [details] [review]
group-mixin: always set the Members_Changed_Detailed flag

We unconditionally implements the MembersChangedDetailed signal so this flag
should always be set.
Comment 2 Will Thompson 2011-10-28 10:51:11 UTC
Comment on attachment 52819 [details] [review]
group-mixin: always set the Members_Changed_Detailed flag

Review of attachment 52819 [details] [review]:
-----------------------------------------------------------------

I think there was a reason I didn't do this but I can't figure out what it would be. So this looks good.
Comment 3 Guillaume Desmottes 2011-10-31 07:52:57 UTC
Merged; will be in 0.16.2

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.