* [PATCH] Fixed problem with incoming calls query
@ 2010-07-20 11:04 Rafal Michalski
2010-07-20 11:25 ` Luiz Augusto von Dentz
0 siblings, 1 reply; 4+ messages in thread
From: Rafal Michalski @ 2010-07-20 11:04 UTC (permalink / raw)
To: linux-bluetooth
From: Rafal Michalski <rafal.michalski@comarch.com>
After making some incoming, outgoing, missed calls, history lists of them
(ich, och, mch) are stored on device. In current implementation queries
towards tracker for incoming calls include also missed calls.
So list for incoming calls displayed on paired BT headset/carkit screen
(e.g. BH-903, CK-200) also contains numbers from missed calls list.
To block adding missed calls list numbers to incoming calls there should
be added info that incoming call is in fact only answered call. So macros
additionally include info: "nmo:isAnswered true ." Without this, missed
calls was treated as incoming calls.
---
plugins/phonebook-tracker.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/plugins/phonebook-tracker.c b/plugins/phonebook-tracker.c
index 89c612e..98b6410 100644
--- a/plugins/phonebook-tracker.c
+++ b/plugins/phonebook-tracker.c
@@ -125,6 +125,7 @@
"?call a nmo:Call ; " \
"nmo:from ?c ; " \
"nmo:isSent false . " \
+ "nmo:isAnswered true ." \
"?c a nco:Contact . " \
"OPTIONAL { ?c nco:hasPhoneNumber ?h . } " \
"OPTIONAL { ?c nco:hasEmailAddress ?e . } " \
@@ -144,6 +145,7 @@
"?call a nmo:Call ; " \
"nmo:from ?c ; " \
"nmo:isSent false . " \
+ "nmo:isAnswered true ." \
"?c a nco:Contact . " \
"OPTIONAL { ?c nco:hasPhoneNumber ?h . } " \
"} ORDER BY DESC(nmo:receivedDate(?call))"
--
1.6.3.3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] Fixed problem with incoming calls query
2010-07-20 11:04 Rafal Michalski
@ 2010-07-20 11:25 ` Luiz Augusto von Dentz
0 siblings, 0 replies; 4+ messages in thread
From: Luiz Augusto von Dentz @ 2010-07-20 11:25 UTC (permalink / raw)
To: Rafal Michalski; +Cc: linux-bluetooth
Hi Rafal,
On Tue, Jul 20, 2010 at 2:04 PM, Rafal Michalski
<rafal.michalski@comarch.com> wrote:
> From: Rafal Michalski <rafal.michalski@comarch.com>
>
> After making some incoming, outgoing, missed calls, history lists of them
> (ich, och, mch) are stored on device. In current implementation queries
> towards tracker for incoming calls include also missed calls.
> So list for incoming calls displayed on paired BT headset/carkit screen
> (e.g. BH-903, CK-200) also contains numbers from missed calls list.
>
> To block adding missed calls list numbers to incoming calls there should
> be added info that incoming call is in fact only answered call. So macros
> additionally include info: "nmo:isAnswered true ." Without this, missed
> calls was treated as incoming calls.
> ---
> plugins/phonebook-tracker.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/plugins/phonebook-tracker.c b/plugins/phonebook-tracker.c
> index 89c612e..98b6410 100644
> --- a/plugins/phonebook-tracker.c
> +++ b/plugins/phonebook-tracker.c
> @@ -125,6 +125,7 @@
> "?call a nmo:Call ; " \
> "nmo:from ?c ; " \
> "nmo:isSent false . " \
> + "nmo:isAnswered true ." \
> "?c a nco:Contact . " \
> "OPTIONAL { ?c nco:hasPhoneNumber ?h . } " \
> "OPTIONAL { ?c nco:hasEmailAddress ?e . } " \
> @@ -144,6 +145,7 @@
> "?call a nmo:Call ; " \
> "nmo:from ?c ; " \
> "nmo:isSent false . " \
> + "nmo:isAnswered true ." \
> "?c a nco:Contact . " \
> "OPTIONAL { ?c nco:hasPhoneNumber ?h . } " \
> "} ORDER BY DESC(nmo:receivedDate(?call))"
> --
> 1.6.3.3
>
Actually this is not completely correct, this nmo are a list of
properties so only the last one will have the . , something like this:
diff --git a/plugins/phonebook-tracker.c b/plugins/phonebook-tracker.c
index 89c612e..696b544 100644
--- a/plugins/phonebook-tracker.c
+++ b/plugins/phonebook-tracker.c
@@ -124,7 +124,8 @@
"WHERE { " \
"?call a nmo:Call ; " \
"nmo:from ?c ; " \
- "nmo:isSent false . " \
+ "nmo:isSent false ; " \
+ "nmo:isAnswered true ." \
"?c a nco:Contact . " \
"OPTIONAL { ?c nco:hasPhoneNumber ?h . } " \
"OPTIONAL { ?c nco:hasEmailAddress ?e . } " \
@@ -143,7 +144,8 @@
"WHERE { " \
"?call a nmo:Call ; " \
"nmo:from ?c ; " \
- "nmo:isSent false . " \
+ "nmo:isSent false ; " \
+ "nmo:isAnswered true ." \
"?c a nco:Contact . " \
"OPTIONAL { ?c nco:hasPhoneNumber ?h . } " \
"} ORDER BY DESC(nmo:receivedDate(?call))"
@@ -181,7 +183,7 @@
"nmo:to ?c ; " \
"nmo:isSent true . " \
"?c a nco:Contact . " \
- "OPTIONAL { ?c nco:hasPhoneNumber ?h . } " \
+ "OPTIONAL { ?c nco:hasPhoneNumber ?h . } " \
"} ORDER BY DESC(nmo:sentDate(?call))"
#define COMBINED_CALLS_QUERY \
--
1.6.1
--
Luiz Augusto von Dentz
Computer Engineer
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] Fixed problem with incoming calls query
@ 2010-07-20 12:30 Radoslaw Jablonski
2010-07-20 12:35 ` Hedberg Johan (Nokia-D/Helsinki)
0 siblings, 1 reply; 4+ messages in thread
From: Radoslaw Jablonski @ 2010-07-20 12:30 UTC (permalink / raw)
To: linux-bluetooth
From: Radoslaw Jablonski <ext-jablonski.radoslaw@nokia.com>
After making some incoming, outgoing, missed calls, history lists of them
(ich, och, mch) are stored on device. In current implementation queries
towards tracker for incoming calls include also missed calls.
So list for incoming calls displayed on paired BT headset/carkit screen
(e.g. BH-903, CK-200) also contains numbers from missed calls list.
To block adding missed calls list numbers to incoming calls there should
be added info that incoming call is in fact only answered call. So macros
additionally include info: "nmo:isAnswered true ." Without this, missed
calls was treated as incoming calls.
---
plugins/phonebook-tracker.c | 6 ++++--
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/plugins/phonebook-tracker.c b/plugins/phonebook-tracker.c
index 89c612e..ee072d3 100644
--- a/plugins/phonebook-tracker.c
+++ b/plugins/phonebook-tracker.c
@@ -124,7 +124,8 @@
"WHERE { " \
"?call a nmo:Call ; " \
"nmo:from ?c ; " \
- "nmo:isSent false . " \
+ "nmo:isSent false ; " \
+ "nmo:isAnswered true ." \
"?c a nco:Contact . " \
"OPTIONAL { ?c nco:hasPhoneNumber ?h . } " \
"OPTIONAL { ?c nco:hasEmailAddress ?e . } " \
@@ -143,7 +144,8 @@
"WHERE { " \
"?call a nmo:Call ; " \
"nmo:from ?c ; " \
- "nmo:isSent false . " \
+ "nmo:isSent false ; " \
+ "nmo:isAnswered true ." \
"?c a nco:Contact . " \
"OPTIONAL { ?c nco:hasPhoneNumber ?h . } " \
"} ORDER BY DESC(nmo:receivedDate(?call))"
--
1.6.3.3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] Fixed problem with incoming calls query
2010-07-20 12:30 [PATCH] Fixed problem with incoming calls query Radoslaw Jablonski
@ 2010-07-20 12:35 ` Hedberg Johan (Nokia-D/Helsinki)
0 siblings, 0 replies; 4+ messages in thread
From: Hedberg Johan (Nokia-D/Helsinki) @ 2010-07-20 12:35 UTC (permalink / raw)
To: Radoslaw Jablonski; +Cc: linux-bluetooth
Hi,
On Tue, Jul 20, 2010, Radoslaw Jablonski wrote:
> After making some incoming, outgoing, missed calls, history lists of them
> (ich, och, mch) are stored on device. In current implementation queries
> towards tracker for incoming calls include also missed calls.
> So list for incoming calls displayed on paired BT headset/carkit screen
> (e.g. BH-903, CK-200) also contains numbers from missed calls list.
>
> To block adding missed calls list numbers to incoming calls there should
> be added info that incoming call is in fact only answered call. So macros
> additionally include info: "nmo:isAnswered true ." Without this, missed
> calls was treated as incoming calls.
> ---
> plugins/phonebook-tracker.c | 6 ++++--
> 1 files changed, 4 insertions(+), 2 deletions(-)
Thanks for the patch. It has now been pushed upstream.
Johan
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-07-20 12:35 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-07-20 12:30 [PATCH] Fixed problem with incoming calls query Radoslaw Jablonski
2010-07-20 12:35 ` Hedberg Johan (Nokia-D/Helsinki)
-- strict thread matches above, loose matches on Subject: below --
2010-07-20 11:04 Rafal Michalski
2010-07-20 11:25 ` Luiz Augusto von Dentz
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).