linux-bluetooth.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Fix query for VCARD listing (work phone issue)
@ 2010-07-22 12:06 Radoslaw Jablonski
  2010-07-22 12:14 ` Johan Hedberg
  0 siblings, 1 reply; 2+ messages in thread
From: Radoslaw Jablonski @ 2010-07-22 12:06 UTC (permalink / raw)
  To: linux-bluetooth; +Cc: Radoslaw Jablonski

Fixed tracker query for VCARD listing - previously contacts with only
work phone filled were missing in query result (and in vcard listing).
Now if mobile/home phone number is not available, then trying to get
work phone number.
---
 plugins/phonebook-tracker.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/plugins/phonebook-tracker.c b/plugins/phonebook-tracker.c
index ee072d3..41d7fde 100644
--- a/plugins/phonebook-tracker.c
+++ b/plugins/phonebook-tracker.c
@@ -72,6 +72,10 @@
 	"WHERE { "							\
 		"?c a nco:PersonContact . "				\
 	"OPTIONAL { ?c nco:hasPhoneNumber ?h . } "			\
+	"OPTIONAL { "							\
+		"?c nco:hasAffiliation ?a . "				\
+		"?a nco:hasPhoneNumber ?h . "				\
+	"} "								\
 	"}"
 
 #define MISSED_CALLS_QUERY						\
-- 
1.7.0.4


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] Fix query for VCARD listing (work phone issue)
  2010-07-22 12:06 [PATCH] Fix query for VCARD listing (work phone issue) Radoslaw Jablonski
@ 2010-07-22 12:14 ` Johan Hedberg
  0 siblings, 0 replies; 2+ messages in thread
From: Johan Hedberg @ 2010-07-22 12:14 UTC (permalink / raw)
  To: Radoslaw Jablonski; +Cc: linux-bluetooth

Hi Radek,

On Thu, Jul 22, 2010, Radoslaw Jablonski wrote:
> Fixed tracker query for VCARD listing - previously contacts with only
> work phone filled were missing in query result (and in vcard listing).
> Now if mobile/home phone number is not available, then trying to get
> work phone number.
> ---
>  plugins/phonebook-tracker.c |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/plugins/phonebook-tracker.c b/plugins/phonebook-tracker.c
> index ee072d3..41d7fde 100644
> --- a/plugins/phonebook-tracker.c
> +++ b/plugins/phonebook-tracker.c
> @@ -72,6 +72,10 @@
>  	"WHERE { "							\
>  		"?c a nco:PersonContact . "				\
>  	"OPTIONAL { ?c nco:hasPhoneNumber ?h . } "			\
> +	"OPTIONAL { "							\
> +		"?c nco:hasAffiliation ?a . "				\
> +		"?a nco:hasPhoneNumber ?h . "				\
> +	"} "								\
>  	"}"
>  
>  #define MISSED_CALLS_QUERY						\

Thanks for the patch. It's now upstream.

Johan

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-07-22 12:14 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-07-22 12:06 [PATCH] Fix query for VCARD listing (work phone issue) Radoslaw Jablonski
2010-07-22 12:14 ` Johan Hedberg

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).