From: Mike Taylor <mike@indexdata.com>
Date: Fri, 4 Apr 2014 14:53:26 +0000 (+0100)
Subject: Differences between various facets are now driven by a table rather
X-Git-Tag: 1.0.0~1032
X-Git-Url: http://git.indexdata.com/?p=mkws-moved-to-github.git;a=commitdiff_plain;h=011aa84e21fa9f9ff46ad16fe8d7afd28d9b7372

Differences between various facets are now driven by a table rather
than hardwired switching.
---

diff --git a/src/mkws-widget-termlists.js b/src/mkws-widget-termlists.js
index 3762600..6175100 100644
--- a/src/mkws-widget-termlists.js
+++ b/src/mkws-widget-termlists.js
@@ -2,6 +2,12 @@ mkws.registerWidgetType('Termlists', function() {
     var that = this;
     var M = mkws.M;
 
+    var facetConfig = {
+	xtargets: [ "Sources",  16, false ],
+	subject:  [ "Subjects", 10, true ],
+	author:   [ "Author",   10, true ]
+    }
+
     this.team.queue("termlists").subscribe(function(data) {
 	// no facets: this should never happen
 	var facets = that.config.facets;
@@ -18,14 +24,12 @@ mkws.registerWidgetType('Termlists', function() {
 	acc.push('<div class="title">' + M('Termlists') + '</div>');
 
 	for (var i = 0; i < facets.length; i++) {
-	    if (facets[i] == "xtargets") {
-		addSingleFacet(acc, "Sources",  data.xtargets, 16, null);
-	    } else if (facets[i] == "subject") {
-		addSingleFacet(acc, "Subjects", data.subject,  10, "subject");
-	    } else if (facets[i] == "author") {
-		addSingleFacet(acc, "Authors",  data.author,   10, "author");
+	    var name = facets[i]
+	    var ref = facetConfig[name];
+	    if (!ref) {
+		alert("bad facet configuration: '" + name + "'");
 	    } else {
-		alert("bad facet configuration: '" + facets[i] + "'");
+		addSingleFacet(acc, ref[0], data[name], ref[1], ref[2] ? name : null);
 	    }
 	}