From 29c35f5bd10806f2704a91b54e0670697a1d3498 Mon Sep 17 00:00:00 2001 From: Radoslaw Jablonski Date: Wed, 27 Apr 2011 13:52:00 +0300 Subject: [PATCH] obexd: Remove grouping by phone number in call-history results Now call history entries are distinguished also by call date (previously only contact-id was compared, so calls with different date were grouped into one entry in pull results). This change is needed for headsets with PBAP support that are confused when number of "new missed calls" is different than number of returned entries by pull call-history response (by example Nokia BH-903 hangs when this happens). --- obexd/plugins/phonebook-tracker.c | 30 +++++++++++++++++++++++++++--- 1 file changed, 27 insertions(+), 3 deletions(-) diff --git a/obexd/plugins/phonebook-tracker.c b/obexd/plugins/phonebook-tracker.c index 2fced7a99..d72c04e2b 100644 --- a/obexd/plugins/phonebook-tracker.c +++ b/obexd/plugins/phonebook-tracker.c @@ -1208,13 +1208,36 @@ static void set_call_type(struct phonebook_contact *contact, contact->datetime = iso8601_utc_to_localtime(datetime); } -static struct phonebook_contact *find_contact(GSList *contacts, const char *id) +static gboolean contact_matches(struct contact_data *c_data, const char *id, + const char *datetime) +{ + char *localtime; + int cmp_ret; + + if (g_strcmp0(c_data->id, id) != 0) + return FALSE; + + /* id is equal and not call history entry => contact matches */ + if (c_data->contact->calltype == CALL_TYPE_NOT_A_CALL) + return TRUE; + + /* for call history entries have to compare also timestamps of calls */ + localtime = iso8601_utc_to_localtime(datetime); + cmp_ret = g_strcmp0(c_data->contact->datetime, localtime); + g_free(localtime); + + return (cmp_ret == 0) ? TRUE : FALSE; +} + +static struct phonebook_contact *find_contact(GSList *contacts, const char *id, + const char *datetime) { GSList *l; for (l = contacts; l; l = l->next) { struct contact_data *c_data = l->data; - if (g_strcmp0(c_data->id, id) == 0) + + if (contact_matches(c_data, id, datetime)) return c_data->contact; } @@ -1609,7 +1632,8 @@ static int pull_contacts(const char **reply, int num_fields, void *user_data) /* Trying to find contact in recently added contacts. It is needed for * contacts that have more than one telephone number filled */ - contact = find_contact(data->contacts, reply[CONTACTS_ID_COL]); + contact = find_contact(data->contacts, reply[CONTACTS_ID_COL], + reply[COL_DATE]); /* If contact is already created then adding only new phone numbers */ if (contact) { -- 2.47.3