From 3f19f4cec0e00a085be86aa40aebd56f18507ffb Mon Sep 17 00:00:00 2001
From: Ingo Schommer <ingo@silverstripe.com>
Date: Fri, 11 May 2012 12:40:57 +0200
Subject: [PATCH] MINOR Removing special case handling for chosen.js from
 UploadField, using generic change listener (incl. some iframe scoping fixes)

---
 javascript/UploadField.js | 27 +++++++++------------------
 1 file changed, 9 insertions(+), 18 deletions(-)

diff --git a/javascript/UploadField.js b/javascript/UploadField.js
index 7f244f79b..576392392 100644
--- a/javascript/UploadField.js
+++ b/javascript/UploadField.js
@@ -287,27 +287,18 @@
 				var editform = this.closest('.ss-uploadfield-item').find('.ss-uploadfield-item-editform');
 				var disabled;
 				var iframe = editform.find('iframe');
-				var inputs = iframe.contents().find('.ss-uploadfield-edit-iframe input');
-				var ulSelects =iframe.contents().find('.ss-uploadfield-edit-iframe .chzn-drop ul.chzn-results');
-				
-				for(var i=0;i<inputs.length;i++){
-					$(inputs[i]).change(function(){
+
+				// Mark the row as changed if any of its form fields are edited
+				iframe.contents().ready(function() {
+					// Need to use the iframe's own jQuery, as custom event triggers
+					// (e.g. from TreeDropdownField) can't be captured by the parent jQuery object.
+					var iframe_jQuery = iframe.get(0).contentWindow.jQuery;
+					iframe_jQuery(iframe_jQuery.find(':input')).bind('change', function(e){
 						editform.removeClass('edited'); 
 						editform.addClass('edited'); 
 					});
-
-				}	 
-				for(var i=0;i<ulSelects.length;i++){
-					var current= $(ulSelects[i]).find('li.result-selected');
-					$(ulSelects[i]).children('li').click(function(){
-						if(this !== current[0]){
-							editform.removeClass('edited'); 
-							editform.addClass('edited'); 
-						}
-					});
-				}	
-
-
+				});
+				
 				if (editform.hasClass('loading')) {
 					// TODO Display loading indication, and register an event to toggle edit form 
 				} else {