From ec52286a0f5296df1582210a92b7b29a763843fb Mon Sep 17 00:00:00 2001
From: Morris Jobke <hey@morrisjobke.de>
Date: Fri, 9 Dec 2016 10:47:40 +0100
Subject: [PATCH] Fire callback also on pure closing of prompt dialog

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
---
 core/js/oc-dialogs.js | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/core/js/oc-dialogs.js b/core/js/oc-dialogs.js
index 7476c93ba45..7e059b4bbc0 100644
--- a/core/js/oc-dialogs.js
+++ b/core/js/oc-dialogs.js
@@ -124,6 +124,14 @@ var OCdialogs = {
 				modal = false;
 			}
 			$('body').append($dlg);
+
+			// wrap callback in _.once():
+			// only call callback once and not twice (button handler and close
+			// event) but call it for the close event, if ESC or the x is hit
+			if (callback !== undefined) {
+				callback = _.once(callback);
+			}
+
 			var buttonlist = [{
 					text : t('core', 'No'),
 					click: function () {
@@ -147,7 +155,13 @@ var OCdialogs = {
 			$(dialogId).ocdialog({
 				closeOnEscape: true,
 				modal        : modal,
-				buttons      : buttonlist
+				buttons      : buttonlist,
+				close        : function() {
+					// callback is already fired if Yes/No is clicked directly
+					if (callback !== undefined) {
+						callback(false, input.val());
+					}
+				}
 			});
 			input.focus();
 			OCdialogs.dialogsCounter++;
-- 
2.39.5