void client_set_state(struct client *cl, enum client_state st)
{
+ int was_active = 0;
+ if (client_is_active(cl))
+ was_active = 1;
cl->state = st;
- /* no need to check for all client being non-active if this one
- already is. Note that session_active_clients also LOCKS session */
- if (!client_is_active(cl) && cl->session)
+ /* If client is going from being active to inactive and all clients
+ are now idle we fire a watch for the session . The assumption is
+ that session is not mutex locked if client is already active */
+ if (was_active && !client_is_active(cl) && cl->session)
{
int no_active = session_active_clients(cl->session);
if (no_active == 0)
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;
struct session_database *sdb = client_get_database(cl);
const char *opt_facet_term_sort = session_setting_oneval(sdb, PZ_TERMLIST_TERM_SORT);
const char *opt_facet_term_count = session_setting_oneval(sdb, PZ_TERMLIST_TERM_COUNT);
+ /* Disable when no count is set */
+ if (opt_facet_term_count == 0 || opt_facet_term_count[0] == '\0') {
+ yaz_log(YLOG_LOG, "Resetting ZOOM facets option to empty.");
+ ZOOM_connection_option_set(link, "facets", "");
+ return 0;
+ }
struct session *session = client_get_session(cl);
struct conf_service *service = session->service;
int num = service->num_metadata;
WRBUF wrbuf = wrbuf_alloc();
int first = 1;
+ yaz_log(YLOG_DEBUG, "Facet settings, sort: %s count: %s", opt_facet_term_sort, opt_facet_term_count);
for (index = 0; index < num; index++) {
struct conf_metadata *conf_meta = &service->metadata[index];
if (conf_meta->termlist) {
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);
}
}
}
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;
}