From: Dennis Schafroth <dennis@indexdata.com>
Date: Mon, 26 Jul 2010 14:22:03 +0000 (+0200)
Subject: Support for target facets.
X-Git-Tag: v1.4.4~32
X-Git-Url: http://lists.indexdata.dk/?a=commitdiff_plain;h=7e46fe0e2db894bc60bff710725c6854a261adc4;p=pazpar2-moved-to-github.git

Support for target facets.

Fix: Set "@attr" for all attributes (not just 1).
Add some more debug around facets.
---

diff --git a/src/client.c b/src/client.c
index b5df1c1..c7c0443 100644
--- a/src/client.c
+++ b/src/client.c
@@ -400,9 +400,9 @@ int client_report_facets(struct client *cl, ZOOM_resultset rs) {
     ZOOM_facet_field *facets = ZOOM_resultset_facets(rs);
     int facet_num;
     struct session *se = client_get_session(cl);
-    yaz_log(YLOG_DEBUG, "client_report_facets %p %p", rs, cl->resultset);
-
     facet_num = ZOOM_resultset_facets_size(rs);
+    yaz_log(YLOG_DEBUG, "client_report_facets: %d", facet_num);
+
     for (facet_idx = 0; facet_idx < facet_num; facet_idx++) {
         const char *name = ZOOM_facet_field_name(facets[facet_idx]);
         size_t term_idx;
@@ -560,6 +560,7 @@ static int client_set_facets_request(struct client *cl, ZOOM_connection link) {
     struct session *session = client_get_session(cl);
     struct conf_service *service = session->service;
     int num = service->num_metadata;
+    yaz_log(YLOG_DEBUG, "Facet settings, sort: %s count: %s", opt_facet_term_sort, opt_facet_term_count);
     WRBUF wrbuf = wrbuf_alloc();
     int first = 1;
     for (index = 0; index < num; index++) {
@@ -572,10 +573,10 @@ static int client_set_facets_request(struct client *cl, ZOOM_connection link) {
             wrbuf_printf(wrbuf, "@attr 1=%s ", conf_meta->name);
 
             if (opt_facet_term_sort && opt_facet_term_sort[0] != '\0') {
-                wrbuf_printf(wrbuf, " 2=%s ", opt_facet_term_sort);
+                wrbuf_printf(wrbuf, " @attr 2=%s ", opt_facet_term_sort);
             }
             if (opt_facet_term_count && opt_facet_term_count[0] != '\0') {
-                wrbuf_printf(wrbuf, " 3=%s ", opt_facet_term_count);
+                wrbuf_printf(wrbuf, " @attr 3=%s ", opt_facet_term_count);
             }
         }
     }
@@ -589,14 +590,16 @@ static int client_set_facets_request(struct client *cl, ZOOM_connection link) {
 
 int client_has_facet(struct client *cl, const char *name) {
     ZOOM_facet_field facet_field;
-    if (!cl || !cl->resultset || !name)
+    if (!cl || !cl->resultset || !name) {
+        yaz_log(YLOG_DEBUG, "client has facet: Missing %p %p %s", cl, (cl ? cl->resultset: 0), name);
         return 0;
+    }
     facet_field = ZOOM_resultset_get_facet_field(cl->resultset, name);
     if (facet_field) {
-        yaz_log(YLOG_DEBUG, "target has facets for %s", name);
+        yaz_log(YLOG_DEBUG, "client: has facets for %s", name);
         return 1;
     }
-    yaz_log(YLOG_DEBUG, "target: No facets for %s", name);
+    yaz_log(YLOG_DEBUG, "client: No facets for %s", name);
     return 0;
 }