Summary: | [1.0] check that ContactSearch has a sane RequestableChannelClasses entry now | ||
---|---|---|---|
Product: | Telepathy | Reporter: | Simon McVittie <smcv> |
Component: | tp-glib | Assignee: | Telepathy bugs list <telepathy-bugs> |
Status: | RESOLVED FIXED | QA Contact: | Telepathy bugs list <telepathy-bugs> |
Severity: | normal | ||
Priority: | medium | Keywords: | patch |
Version: | git master | ||
Hardware: | Other | ||
OS: | All | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Bug Depends on: | |||
Bug Blocks: | 68660 | ||
Attachments: |
[tp-glib] allow CMs to put 'TargetEntityType=None' in contact search requests
[gabble] search-manager: use properties constants [gabble] search-manager: use tp_asv_new() [gabble] search-manager: allow user to pass TargetEntityType=NONE when requesting |
Description
Simon McVittie
2014-04-03 20:04:26 UTC
Created attachment 98967 [details] [review] [tp-glib] allow CMs to put 'TargetEntityType=None' in contact search requests Created attachment 98968 [details] [review] [gabble] search-manager: use properties constants Created attachment 98969 [details] [review] [gabble] search-manager: use tp_asv_new() Created attachment 98970 [details] [review] [gabble] search-manager: allow user to pass TargetEntityType=NONE when requesting Comment on attachment 98967 [details] [review] [tp-glib] allow CMs to put 'TargetEntityType=None' in contact search requests Review of attachment 98967 [details] [review]: ----------------------------------------------------------------- Looks good, although a regression test would be nice. ::: telepathy-glib/capabilities.c @@ +920,5 @@ > continue; > > + entity_type = tp_vardict_get_uint32 (fixed, > + TP_PROP_CHANNEL_TARGET_ENTITY_TYPE, &valid); > + /* If TargetEntityType is missing we threat it as NONE */ treat A threat is something rather different :-) Comment on attachment 98968 [details] [review] [gabble] search-manager: use properties constants Review of attachment 98968 [details] [review]: ----------------------------------------------------------------- Sure Comment on attachment 98969 [details] [review] [gabble] search-manager: use tp_asv_new() Review of attachment 98969 [details] [review]: ----------------------------------------------------------------- Yes, for next Comment on attachment 98970 [details] [review] [gabble] search-manager: allow user to pass TargetEntityType=NONE when requesting Review of attachment 98970 [details] [review]: ----------------------------------------------------------------- Looks good for 'next'. Are there any other CMs that implement ContactSearch? If not, this can be closed when these are merged. Fixed & merged. |
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.