aboutsummaryrefslogtreecommitdiffstats
path: root/addressbook/gui/component/select-names
diff options
context:
space:
mode:
authorNot Zed <NotZed@Ximian.com>2004-03-04 12:15:33 +0800
committerMichael Zucci <zucchi@src.gnome.org>2004-03-04 12:15:33 +0800
commitf096c882c0d012b85bc8d3cb9104a2334488a667 (patch)
tree0722694d5b43949c7b97b7b3766df01592832554 /addressbook/gui/component/select-names
parente194e841885b83da5cf1df123da7c414acc493ee (diff)
downloadgsoc2013-evolution-f096c882c0d012b85bc8d3cb9104a2334488a667.tar
gsoc2013-evolution-f096c882c0d012b85bc8d3cb9104a2334488a667.tar.gz
gsoc2013-evolution-f096c882c0d012b85bc8d3cb9104a2334488a667.tar.bz2
gsoc2013-evolution-f096c882c0d012b85bc8d3cb9104a2334488a667.tar.lz
gsoc2013-evolution-f096c882c0d012b85bc8d3cb9104a2334488a667.tar.xz
gsoc2013-evolution-f096c882c0d012b85bc8d3cb9104a2334488a667.tar.zst
gsoc2013-evolution-f096c882c0d012b85bc8d3cb9104a2334488a667.zip
** See #54991.
2004-03-04 Not Zed <NotZed@Ximian.com> ** See #54991. * gui/component/select-names/e-select-names-completion.c (e_select_names_completion_start_query): dont try to unref a null query. (name_style_query): use a GString to build the query, so we can use e_sexp_encode_string to properly encode the sub terms. svn path=/trunk/; revision=24957
Diffstat (limited to 'addressbook/gui/component/select-names')
-rw-r--r--addressbook/gui/component/select-names/e-select-names-completion.c37
1 files changed, 20 insertions, 17 deletions
diff --git a/addressbook/gui/component/select-names/e-select-names-completion.c b/addressbook/gui/component/select-names/e-select-names-completion.c
index b80611b9ff..e3fa256dc8 100644
--- a/addressbook/gui/component/select-names/e-select-names-completion.c
+++ b/addressbook/gui/component/select-names/e-select-names-completion.c
@@ -39,6 +39,8 @@
#include <addressbook/util/eab-destination.h>
#include <addressbook/gui/merging/eab-contact-compare.h>
+#include <e-util/e-sexp.h>
+
typedef struct {
EBook *book;
guint book_view_tag;
@@ -261,7 +263,8 @@ name_style_query (ESelectNamesCompletion *comp, const gchar *field)
gchar *cpy = g_strdup (comp->priv->query_text), *c;
gchar **strv;
gchar *query;
- gint i, count=0;
+ gint i;
+ GString *out = g_string_new("");
for (c = cpy; *c; ++c) {
if (*c == ',')
@@ -269,23 +272,23 @@ name_style_query (ESelectNamesCompletion *comp, const gchar *field)
}
strv = g_strsplit (cpy, " ", 0);
+ if (strv[0] && strv[1])
+ g_string_append(out, "(and ");
for (i=0; strv[i]; ++i) {
- gchar *old;
- ++count;
- g_strstrip (strv[i]);
- old = strv[i];
- strv[i] = g_strdup_printf ("(beginswith \"%s\" \"%s\")", field, old);
- g_free (old);
+ if (i==0)
+ g_string_append(out, "(beginswith ");
+ else
+ g_string_append(out, " (beginswith ");
+ e_sexp_encode_string(out, field);
+ g_strstrip(strv[i]);
+ e_sexp_encode_string(out, strv[i]);
+ g_string_append(out, ")");
}
+ if (strv[0] && strv[1])
+ g_string_append(out, ")");
- if (count == 1) {
- query = strv[0];
- strv[0] = NULL;
- } else {
- gchar *joined = g_strjoinv (" ", strv);
- query = g_strdup_printf ("(and %s)", joined);
- g_free (joined);
- }
+ query = out->str;
+ g_string_free(out, FALSE);
g_free (cpy);
g_strfreev (strv);
@@ -1025,12 +1028,12 @@ e_select_names_completion_start_query (ESelectNamesCompletion *comp, const gchar
that the search is over. */
if (!comp->priv->pending_completion_seq)
e_select_names_completion_done (E_SELECT_NAMES_COMPLETION (comp));
+
+ e_book_query_unref (query);
} else {
g_free (comp->priv->query_text);
comp->priv->query_text = NULL;
}
- e_book_query_unref (query);
-
} else {
comp->priv->waiting_query = g_strdup (query_text);