mkdru.removeLimits() no longer leaves dangling limit_subject
authorJason Skomorowski <jason@indexdata.com>
Thu, 3 Mar 2011 06:52:22 +0000 (07:52 +0100)
committerJason Skomorowski <jason@indexdata.com>
Thu, 3 Mar 2011 06:52:22 +0000 (07:52 +0100)
mkdru.client.js

index f6188ea..5c2d0d0 100644 (file)
@@ -272,7 +272,8 @@ mkdru.addLimit = function (facet, limit) {
 mkdru.removeLimit = function (facet, limit) {
   var newHash = $.deparam.fragment();
   delete newHash['page'];
-  if (!newHash['limit_' + facet].indexOf(';') || !mkdru.facets[facet].multiLimit) { 
+  if (!newHash['limit_' + facet].indexOf(';')
+      || !mkdru.facets[facet].multiLimit) {
     delete newHash['limit_' + facet];
   }
   else {
@@ -280,7 +281,10 @@ mkdru.removeLimit = function (facet, limit) {
     for (var i = 0; i < limits.length; i++) {
       if (limits[i] == limit) {
         limits.splice(i, 1);
-        newHash['limit_' + facet] = limits.join(';');
+        if (limits.length < 1)
+          delete newHash['limit_' + facet];
+        else
+          newHash['limit_' + facet] = limits.join(';');
         break;
       }
     }