From 3d12c2b5916fcc66d6383fa4c6e7d9c48e2bb908 Mon Sep 17 00:00:00 2001
From: Nathan Gray <nathangray.bsc@gmail.com>
Date: Tue, 27 Aug 2013 16:29:13 +0000
Subject: [PATCH] Add namespace to drop event bindings to easily remove only
 the ones the widget added.

---
 etemplate/js/et2_widget_file.js | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/etemplate/js/et2_widget_file.js b/etemplate/js/et2_widget_file.js
index 467440fd97..e9042d2adf 100644
--- a/etemplate/js/et2_widget_file.js
+++ b/etemplate/js/et2_widget_file.js
@@ -193,11 +193,13 @@ var et2_file = et2_inputWidget.extend(
 		{
 			var widget = this.getRoot().getWidgetById(this.options.drop_target);
 			var drop_target = widget && widget.getDOMNode() || document.getElementById(this.options.drop_target);
-			//$j(drop_target).off("."+this.id);
+			$j(drop_target).off("."+this.id);
 		}
 		
 		this.options.drop_target = new_target;
 		
+		if(!this.options.drop_target) return;
+		
 		// Set up new drop target
 		var widget = this.getRoot().getWidgetById(this.options.drop_target);
 		var drop_target = widget && widget.getDOMNode() || document.getElementById(this.options.drop_target);
@@ -210,7 +212,7 @@ var et2_file = et2_inputWidget.extend(
 					return false;
 			};
 			$j(drop_target)
-				.on("drop", jQuery.proxy(function(event) {
+				.on("drop."+this.id, jQuery.proxy(function(event) {
 					event.stopPropagation();
 					event.preventDefault();
 					this.input.removeClass("ui-state-active");
@@ -220,7 +222,7 @@ var et2_file = et2_inputWidget.extend(
 					}
 					return false;
 				}, this))
-				.on("dragenter",function(e) {
+				.on("dragenter."+this.id,function(e) {
 					// Accept the drop if at least one mimetype matches
 					// Individual files will be rejected later
 					var mime_ok = false;
@@ -235,16 +237,16 @@ var et2_file = et2_inputWidget.extend(
 					// Returning true cancels, return false to allow
 					return self.disabled || !mime_ok;
 				})
-				.on("dragleave", function(e) {
+				.on("dragleave."+this.id, function(e) {
 					self.input.removeClass("ui-state-active");
 					// Returning true cancels, return false to allow
 					return self.disabled
 				})
-				.on("dragover", function(e) {
+				.on("dragover."+this.id, function(e) {
 					// Returning true cancels, return false to allow
 					return self.disabled;
 				})
-				.on("dragend", false);
+				.on("dragend."+this.id, false);
 		}
 		else
 		{