From a62cad5cdb935915debaf45531822bb83f8062a1 Mon Sep 17 00:00:00 2001
From: Ian Walls <mail.sekjal@gmail.com>
Date: Wed, 28 May 2014 11:49:30 -0400
Subject: [PATCH] Add support for dynamically updated facets

Some facet interfaces are enriched by updating the facets after each
new Ajax request is performed.  This commit adds support for this.

The key is to use Event Delegation (learn.jquery.com/events/event-delegation)
to bind the listener to an unchanging parent element (the form itself) rather
than directly to the inputs.  This way, inputs can change with the Ajax
requests, and the binding is not broken.

Implementation requires a custom postAJAX function to be specified in the
plugin configuration, and for the updated facets to be returned in some format
that can be processed by that function.
---
 jquery.facets.js | 41 ++++++++++++-----------------------------
 1 file changed, 12 insertions(+), 29 deletions(-)

diff --git a/jquery.facets.js b/jquery.facets.js
index 8389562..2b93f65 100644
--- a/jquery.facets.js
+++ b/jquery.facets.js
@@ -54,35 +54,18 @@
       plugin.data("settings",settings);
 
       //bind each input to the appropriate bind type
-      plugin.find(":input").each(function(){
-
-        //find out if we should not bind to this input
-        var excludeBindTypes = plugin.data('settings').excludeBindTypes;
-        var shouldBind = true;
-        for(var k = 0; k < excludeBindTypes.length; k++) { 
-          if($(this).is(excludeBindTypes[k])) { 
-            shouldBind = false;
-            break;
-          }   
-        }   
-
-        if(!shouldBind)
-          return true;
-
-        //use default bindType
-        var bindType = plugin.data('settings').bindType;
-
-        //see if this input has a special bindType
-        var bindTypes = plugin.data('settings').bindTypes;
-        for(var k = 0; k < bindTypes.length; k++) { 
-          if($(this).is(bindTypes[k]['selector'])) { 
-            bindType = bindTypes[k]['bindType'];
-            break; //will get first bindType of selector it matches
-          }   
-        }   
-
-        $(this).bind(bindType, {'plugin': plugin}, methods.ajaxReq);  
-      }); 
+      var excludeBindTypes = plugin.data('settings').excludeBindTypes;
+      var bindTypes = plugin.data('settings').bindTypes;
+      var excludedBindTypesList = excludeBindTypes;
+      for(var k=0; k < bindTypes.length; k++) {
+            // add this selector to the default's exclude
+            excludedBindTypesList.push(bindTypes[k]['selector']);
+
+            // Also bind to the selector on the particular type
+            plugin.on(bindTypes[k]['bindType'], bindTypes[k]['selector'], {'plugin' : plugin}, methods.ajaxReq);
+      }	
+      var bindTypeList = ':input:not(' + excludedBindTypesList.join(', ') + ')';
+      plugin.on(plugin.data('settings').bindType, bindTypeList, {'plugin': plugin}, methods.ajaxReq);
 
       if(plugin.data('settings').hash) 
         methods.hashInit.apply(plugin);