From bbbcc95f1ce602c30dfa8a00fc9c0b246c56b948 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Oliver=20G=C3=BCnther?= <mail@oliverguenther.de>
Date: Thu, 23 Aug 2018 15:58:09 +0200
Subject: [PATCH] [28325] Fix double submit on destroy modal

https://community.openproject.com/wp/28325
---
 .../modals/wp-destroy-modal/wp-destroy.modal.html    |  1 +
 .../modals/wp-destroy-modal/wp-destroy.modal.ts      | 12 ++++++++++++
 2 files changed, 13 insertions(+)

diff --git a/frontend/src/app/components/modals/wp-destroy-modal/wp-destroy.modal.html b/frontend/src/app/components/modals/wp-destroy-modal/wp-destroy.modal.html
index a81ad8de7b..c551115653 100644
--- a/frontend/src/app/components/modals/wp-destroy-modal/wp-destroy.modal.html
+++ b/frontend/src/app/components/modals/wp-destroy-modal/wp-destroy.modal.html
@@ -66,6 +66,7 @@
       <div class="form--space -left-spacing">
         <button class="button -danger"
                 [textContent]="text.confirm"
+                [attr.disabled]="busy || undefined"
                 (click)="confirmDeletion($event)">
         </button>
         <button class="button"
diff --git a/frontend/src/app/components/modals/wp-destroy-modal/wp-destroy.modal.ts b/frontend/src/app/components/modals/wp-destroy-modal/wp-destroy.modal.ts
index 7f02beea52..3a1c97eee3 100644
--- a/frontend/src/app/components/modals/wp-destroy-modal/wp-destroy.modal.ts
+++ b/frontend/src/app/components/modals/wp-destroy-modal/wp-destroy.modal.ts
@@ -51,6 +51,7 @@ export class WpDestroyModal extends OpModalComponent implements OnInit {
   // Single work package
   public singleWorkPackage:WorkPackageResource;
   public singleWorkPackageChildren:WorkPackageResource[];
+  public busy = false;
 
   public text:any = {
     label_visibility_settings: this.I18n.t('js.label_visibility_settings'),
@@ -106,12 +107,23 @@ export class WpDestroyModal extends OpModalComponent implements OnInit {
 
 
   public confirmDeletion($event:JQueryEventObject) {
+    if (this.busy) {
+      return false;
+    }
+
+    this.busy = true;
     this.WorkPackageService.performBulkDelete(this.workPackages.map(el => el.id), true)
       .then(() => {
+        this.busy = false;
         this.closeMe($event);
         this.wpTableFocus.clear();
         this.$state.go('work-packages.list');
+      })
+      .catch(() => {
+        this.busy = false;
       });
+
+    return false;
   }
 
   public children(workPackage:WorkPackageResource) {