There seem to be to many 'shows' at times

[mkdru-moved-to-drupal.org.git] / mkdru.client.js

diff --git a/mkdru.client.js b/mkdru.client.js
index 5c2d0d0..d30fa48 100644 (file)
--- a/mkdru.client.js
+++ b/mkdru.client.js
@@ -137,16 +137,12 @@ mkdru.contains = function (hash, key, value) {
 }
 
 mkdru.pz2Term = function (data) {
-  // if signaled, prepare container client-side
-  if (mkdru.facetContainerSelector) {
-    $(mkdru.facetContainerSelector).html(Drupal.theme('mkdruFacetContainer',
-      data, mkdru.facets));
-  }
   // map all facets against selected, for simple rendering 
   var hash = $.deparam.fragment();
   for (var key in hash) {
-    if (key.indexOf('limit') == 0) //always wrap in array
-      hash[key.substr(6)] = hash[key].split(';');
+    if (key.indexOf('limit') == 0 && hash[key]) 
+      //always wrap in array
+      hash[key.substr(6)] = hash[key].split(/;+/);
     delete hash[key];
   }
   for (var facet in mkdru.facets) {
@@ -155,15 +151,16 @@ mkdru.pz2Term = function (data) {
       var term = terms[i];
       var value = facet == "source" ? term.id : term.name;
       if (mkdru.contains(hash, facet, value)) { //enabled
-        term.hashQuery = mkdru.removeLimit(facet, value);
+        term.toggleLink = mkdru.removeLimit(facet, value);
         term.selected = true;
       } else { //disabled
-        term.hashQuery = mkdru.addLimit(facet, value);
+        term.toggleLink = mkdru.addLimit(facet, value);
         term.selected = false;
       }
     }
     $('.mkdru-facet-' + facet).html(
-        Drupal.theme('mkdruFacet', terms, facet, mkdru.facets[facet].max, hash));
+        Drupal.theme('mkdruFacet', terms, facet, mkdru.facets[facet].max, 
+          hash[facet]));
   }
 };
 
@@ -330,11 +327,12 @@ mkdru.search = function () {
         filter = 'pz:id=' + mkdru.state.limit_source;
       }
       else {
-        var limits = mkdru.state['limit_' + facet].split(';');
+        var limits = mkdru.state['limit_' + facet].split(/;+/);
         for (var i = 0; i < limits.length; i++) {
           // ex. query + and au="{limit_author}"
-          query += ' and ' + mkdru.facets[facet]['limiter'] + '="'
-                  + mkdru.state['limit_' + facet] + '"';
+          if (limits[i]) 
+            query += ' and ' + mkdru.facets[facet]['limiter'] + '="'
+                  + limits[i] + '"';
         }
       }
     }
@@ -399,6 +397,12 @@ $(document).ready(function () {
     mkdru.defaultState['limit_' + key] = null;
   }
 
+  // if signaled, prepare facet container client-side
+  if (mkdru.facetContainerSelector) {
+    $(mkdru.facetContainerSelector).html(Drupal.theme('mkdruFacetContainer',
+      mkdru.facets));
+  }
+
   mkdru.pz2 = new pz2( { "onshow": mkdru.pz2Show,
               "showtime": 500, //each timer (show, stat, term, bytarget) can be specified this way
               "pazpar2path": mkdru.pazpar2Path,
@@ -411,6 +415,7 @@ $(document).ready(function () {
               "showResponseType": mkdru.showResponseType,
               "onrecord": mkdru.pz2Record,
               "autoInit": false } );
+  pz2.showFastCount = 0;
 
   // initialise state to hash string or defaults
   mkdru.stateFromHash();