X-Git-Url: http://git.indexdata.com/?a=blobdiff_plain;f=src%2Fmkws-team.js;h=7a9015e7b312ddc90a39be3677a92bf34ebe11dd;hb=dd7ad194841702eeff011443e418b9e71a89cf39;hp=0824618001c5b35f0ad454b45a92f43668435351;hpb=91bc6158ec7702dc5d670383abf67ecec8761030;p=mkws-moved-to-github.git

diff --git a/src/mkws-team.js b/src/mkws-team.js
index 0824618..7a9015e 100644
--- a/src/mkws-team.js
+++ b/src/mkws-team.js
@@ -40,7 +40,7 @@ function team($, teamName) {
     that.currentPage = function() { return m_currentPage; };
     that.currentRecordId = function() { return m_currentRecordId; };
     that.currentRecordData = function() { return m_currentRecordData; };
-    that.filters = function() { return m_filterSet.list(); };
+    that.filters = function() { return m_filterSet; };
     that.config = function() { return m_config; };
 
     // Accessor methods for individual widgets: writers
@@ -169,40 +169,41 @@ function team($, teamName) {
 
     that.limitTarget = function(id, name) {
 	log("limitTarget(id=" + id + ", name=" + name + ")");
-	m_filterSet.add(filter(id, name));
-	triggerSearch();
+	m_filterSet.add(targetFilter(id, name));
+	if (m_query) triggerSearch();
 	return false;
     };
 
 
     that.limitQuery = function(field, value) {
 	log("limitQuery(field=" + field + ", value=" + value + ")");
-	m_filterSet.add(filter(null, null, field, value));
-	triggerSearch();
+	m_filterSet.add(fieldFilter(field, value));
+        if (m_query) triggerSearch();
 	return false;
     };
 
 
     that.limitCategory = function(id) {
 	log("limitCategory(id=" + id + ")");
-	// ### Add a filter
-	// ### triggerSearch() if there's a query
+	m_filterSet.add(categoryFilter(id));
+        if (m_query) triggerSearch();
 	return false;
     };
 
 
     that.delimitTarget = function(id) {
 	log("delimitTarget(id=" + id + ")");
-	m_filterSet.removeMatching(function(f) { return f.id });
-	triggerSearch();
+	m_filterSet.removeMatching(function(f) { return f.type === 'target' });
+        if (m_query) triggerSearch();
 	return false;
     };
 
 
     that.delimitQuery = function(field, value) {
 	log("delimitQuery(field=" + field + ", value=" + value + ")");
-	m_filterSet.removeMatching(function(f) { return f.field && field == f.field && value == f.value });
-	triggerSearch();
+	m_filterSet.removeMatching(function(f) { return f.type == 'field' &&
+                                                 field == f.field && value == f.value });
+        if (m_query) triggerSearch();
 	return false;
     };
 
@@ -262,40 +263,26 @@ function team($, teamName) {
 
 
 	// Continue to use previous query/sort-order unless new ones are specified
-	if (query) {
-	    m_query = query;
-	}
-	if (sortOrder) {
-	    m_sortOrder = sortOrder;
-	}
-	if (perpage) {
-	    m_perpage = perpage;
-	}
-	if (targets) {
-	    m_filterSet.add(filter(id, id));
-	}
+	if (query) m_query = query;
+	if (sortOrder) m_sortOrder = sortOrder;
+	if (perpage) m_perpage = perpage;
+	if (targets) m_filterSet.add(targetFilter(id, id));
 
 	var pp2filter = m_filterSet.pp2filter();
 	var pp2limit = m_filterSet.pp2limit(limit);
 
 	var params = {};
-	if (pp2limit) {
-	    params.limit = pp2limit;
-	}
-	if (maxrecs) {
-	    params.maxrecs = maxrecs;
-	}
+	if (pp2limit) params.limit = pp2limit;
+	if (maxrecs) params.maxrecs = maxrecs;
 	if (torusquery) {
 	    if (!mkws.config.use_service_proxy)
 		alert("can't narrow search by torusquery when Service Proxy is not in use");
 	    params.torusquery = torusquery;
 	}
 
-	log("triggerSearch(" + m_query + "): filters = " + $.toJSON(m_filterSet.list()) + ", " +
+	log("triggerSearch(" + m_query + "): filters = " + m_filterSet.toJSON() + ", " +
 	    "pp2filter = " + pp2filter + ", params = " + $.toJSON(params));
 
-	// We can use: params.torusquery = "udb=NAME"
-	// Note: that won't work when running against raw pazpar2
 	m_paz.search(m_query, m_perpage, m_sortOrder, pp2filter, undefined, params);
     }