ActionBar: check confirm after checks
authorSven Schöling <s.schoeling@linet-services.de>
Thu, 13 Oct 2016 12:07:20 +0000 (14:07 +0200)
committerMoritz Bunkus <m.bunkus@linet-services.de>
Tue, 28 Feb 2017 09:04:33 +0000 (10:04 +0100)
So muss im Falle dass checks meckern nicht doppelt bestätigt werden

js/kivi.ActionBar.js

index 6e7cc16..44f0d6e 100644 (file)
@@ -13,7 +13,6 @@ namespace('kivi', function(k){
        $(e).click(function(event) {
          var $hidden, key, func, check;
          if (data.disabled) return;
-         if (data.confirm && !confirm(data.confirm)) return;
          if (data.checks) {
            for (var i=0; i < data.checks.length; i++) {
              check = data.checks[i];
@@ -22,6 +21,7 @@ namespace('kivi', function(k){
              if (!func()) return;
            }
          }
+         if (data.confirm && !confirm(data.confirm)) return;
          for (key in params) {
            $hidden = $('<input type=hidden>')
            $hidden.attr('name', key)
@@ -35,7 +35,6 @@ namespace('kivi', function(k){
        $(e).click(function(event) {
          var func;
          if (data.disabled) return;
-         if (data.confirm && !confirm(data.confirm)) return;
          if (data.checks) {
            for (var i=0; i < data.checks.length; i++) {
              check = data.checks[i];
@@ -44,6 +43,7 @@ namespace('kivi', function(k){
              if (!func()) return;
            }
          }
+         if (data.confirm && !confirm(data.confirm)) return;
          func = kivi.get_function_by_name(data.function[0]);
          func.apply(document, data.function.slice(1))
        });