Remove unnecessary check.

[mkws-moved-to-github.git] / src / mkws-widget-termlists.js

diff --git a/src/mkws-widget-termlists.js b/src/mkws-widget-termlists.js
index b9e7b11..69d97a5 100644 (file)
--- a/src/mkws-widget-termlists.js
+++ b/src/mkws-widget-termlists.js
@@ -2,38 +2,51 @@ mkws.registerWidgetType('Termlists', function() {
     var that = this;
     var M = mkws.M;
 
+    var facetConfig = {
+       xtargets: [ "Sources",  16, false ],
+       subject:  [ "Subjects", 10, true ],
+       author:   [ "Authors",  10, true ]
+    }
+
     this.team.queue("termlists").subscribe(function(data) {
        // no facets: this should never happen
        var facets = that.config.facets;
-       if (!facets || facets.length == 0) {
-           alert("onTerm called even though we have no facets: " + $.toJSON(data));
-           $(that.node).hide();
-           return;
-       }
 
        // display if we first got results
        $(that.node).show();
 
        var acc = [];
        acc.push('<div class="title">' + M('Termlists') + '</div>');
+       for (var i = 0; i < facets.length; i++) {
+           var name = facets[i]
+           var ref = facetConfig[name];
+           if (!ref) {
+               alert("bad facet configuration: '" + name + "'");
+           } else {
+               acc.push('<div class="mkwsFacet mkwsFacet' + ref[0] + ' mkwsTeam_' + that.team.name() + '">');
+               acc.push('</div>');
+           }
+       }
+       $(that.node).html(acc.join(''));
+
 
        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];
+           var caption = ref[0];
+           if (!ref) {
+               alert("bad facet configuration: '" + name + "'");
            } else {
-               alert("bad facet configuration: '" + facets[i] + "'");
+               var output = makeSingleFacet(ref[0], data[name], ref[1], ref[2] ? name : null);
+               that.team.findnode('.mkwsFacet' + caption).html(output);
            }
        }
 
-       $(that.node).html(acc.join(''));
 
-       function addSingleFacet(acc, caption, data, max, pzIndex) {
+       function makeSingleFacet(caption, data, max, pzIndex) {
            var teamName = that.team.name();
-           acc.push('<div class="facet mkwsFacet' + caption + ' mkwsTeam_' + teamName + '">');
+
+           var acc = [];
            acc.push('<div class="termtitle">' + M(caption) + '</div>');
            for (var i = 0; i < data.length && i < max; i++) {
                acc.push('<div class="term">');
@@ -52,7 +65,7 @@ mkws.registerWidgetType('Termlists', function() {
                         + ' <span>' + data[i].freq + '</span>');
                acc.push('</div>');
            }
-           acc.push('</div>');
+           return acc.join('');
        }
     });