From: Moritz Bunkus <m.bunkus@linet-services.de>
Date: Wed, 4 Jan 2017 16:01:12 +0000 (+0100)
Subject: kivi.popup_dialog(): übergebenen close-Handler auch ausführen
X-Git-Tag: release-3.5.4~1772
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=e39679e479c62ee194b943b6b53fd9f6dccd3c36;p=kivitendo-erp.git

kivi.popup_dialog(): übergebenen close-Handler auch ausführen

Die Funktion kivi.popup_dialog() installiert ihren eigenen
close-Handler, um den in dem Moment noch vorhandenen Dialog zu
entfernen (sofern der Dialog aus HTML-Code erzeugt oder von einer URL
nachgeladen wurde) bzw. zu schließen (sofern ein vorher bereits im DOM
existierendes Element angezeigt wurde). Dadurch werden vom Aufrufer
vergebene close-Handler überschrieben.

Da es mehrere Stellen im Programm gibt, an denen ein close-Handler
übergeben wird, wurde kivi.popup_dialog() nun so erweitert, dass es den
übergebenen Handler speichert und zuerst ausführt und erst anschließend
das Entfernen respektive Schließen durchführt.
---

diff --git a/js/kivi.js b/js/kivi.js
index 058c8c567..c935c34cd 100644
--- a/js/kivi.js
+++ b/js/kivi.js
@@ -312,6 +312,7 @@ namespace("kivi", function(ns) {
 
     params            = params        || { };
     var id            = params.id     || 'jqueryui_popup_dialog';
+    var custom_close  = params.dialog ? params.dialog.close : undefined;
     var dialog_params = $.extend(
       { // kivitendo default parameters:
           width:  800
@@ -321,7 +322,15 @@ namespace("kivi", function(ns) {
         // User supplied options:
       params.dialog || { },
       { // Options that must not be changed:
-        close: function(event, ui) { if (params.url || params.html) dialog.remove(); else dialog.dialog('close'); }
+        close: function(event, ui) {
+          if (custom_close)
+            custom_close();
+
+          if (params.url || params.html)
+            dialog.remove();
+          else
+            dialog.dialog('close');
+        }
       });
 
     if (!params.url && !params.html) {