Merge pull request #35 from UNC-Libraries/fix-click-assign

Supporting already created submit buttons, assigning click handlers to t...
UNC-Libraries · Apr 20, 2015 · 04c57db · 04c57db
2 parents bda2973 + 886e02c
commit 04c57db
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 20 deletions.
diff --git a/jquery.xmleditor.js b/jquery.xmleditor.js
@@ -2211,24 +2211,27 @@ ModifyMenuPanel.prototype.initialize = function (parentContainer) {
 
 		if (self.editor.submitButtonConfigs != null){
 			$.each(self.editor.submitButtonConfigs, function(index, config){
-				if (!('createDomElement' in config) || config.createDomElement){
-					var submitButton = $("<input/>").attr({
+				var submitButton;
+				if (config.id && ('createDomElement' in config) && !config.createDomElement) {
+					submitButton = $("#" + config.id);
+				} else {
+					submitButton = $("<input/>").attr({
 						id : config.id,
 						'type' : 'button',
 						'class' : config.cssClass || submitButtonClass,
 						name : config.name || 'submit',
 						value : config.label || 'Submit'
 					}).appendTo(documentStatusPanel);
+				}
 
-					if (!('responseHandler' in config) && config.url) {
-						config.responseHandler = this.options.submitResponseHandler
-							|| this.swordSubmitResponseHandler;
-					}
-
-					submitButton.click(function() {
-						self.editor.submitXML(config);
-					});
+				if (!('responseHandler' in config) && config.url) {
+					config.responseHandler = this.options.submitResponseHandler
+						|| this.swordSubmitResponseHandler;
 				}
+
+				submitButton.click(function() {
+					self.editor.submitXML(config);
+				});
 			});
 		}
 		documentStatusPanel.appendTo(this.menuColumn);

diff --git a/src/modify_menu_panel.js b/src/modify_menu_panel.js
@@ -21,24 +21,27 @@ ModifyMenuPanel.prototype.initialize = function (parentContainer) {
 
 		if (self.editor.submitButtonConfigs != null){
 			$.each(self.editor.submitButtonConfigs, function(index, config){
-				if (!('createDomElement' in config) || config.createDomElement){
-					var submitButton = $("<input/>").attr({
+				var submitButton;
+				if (config.id && ('createDomElement' in config) && !config.createDomElement) {
+					submitButton = $("#" + config.id);
+				} else {
+					submitButton = $("<input/>").attr({
 						id : config.id,
 						'type' : 'button',
 						'class' : config.cssClass || submitButtonClass,
 						name : config.name || 'submit',
 						value : config.label || 'Submit'
 					}).appendTo(documentStatusPanel);
+				}
 
-					if (!('responseHandler' in config) && config.url) {
-						config.responseHandler = this.options.submitResponseHandler
-							|| this.swordSubmitResponseHandler;
-					}
-
-					submitButton.click(function() {
-						self.editor.submitXML(config);
-					});
+				if (!('responseHandler' in config) && config.url) {
+					config.responseHandler = this.options.submitResponseHandler
+						|| this.swordSubmitResponseHandler;
 				}
+
+				submitButton.click(function() {
+					self.editor.submitXML(config);
+				});
 			});
 		}
 		documentStatusPanel.appendTo(this.menuColumn);