BSAPP-1926 improve duplicate conflict handling in verein dialogs

Author: McYum

bogenliga/src/app/modules/verwaltung/components/verein/verein-detail/mannschafts-detail/mannschaft-detail.component.ts

@@ -86,6 +86,7 @@ export class MannschaftDetailComponent extends CommonComponentDirective implemen
   // maps the MannschaftsMitgliedDO with the DSBMitgliedId
   private currentMannschaftsMitglied: MannschaftsMitgliedDO = new MannschaftsMitgliedDO();
   private members: Map<number, MannschaftsMitgliedDO> = new Map<number, MannschaftsMitgliedDO>();
+  private duplicateNotificationSubscription;
   private deleteNotification: Notification;
   private deleteSubscription;
   private dsbmitglied: DsbMitgliedDO = new DsbMitgliedDO();
@@ -137,6 +138,14 @@ export class MannschaftDetailComponent extends CommonComponentDirective implemen
         }
       }
     });
+
+    this.duplicateNotificationSubscription = this.notificationService.observeNotification(NOTIFICATION_DUPLICATE_MANNSCHAFT)
+      .subscribe((notification) => {
+        if (notification.userAction === NotificationUserAction.ACCEPTED
+          || notification.userAction === NotificationUserAction.DECLINED) {
+          this.saveLoading = false;
+        }
+      });
   }
 
   /** When a MouseOver-Event is triggered, it will call this inMouseOver-function.
@@ -153,13 +162,17 @@ export class MannschaftDetailComponent extends CommonComponentDirective implemen
 
 
   ngOnDestroy() {
+    if (this.duplicateNotificationSubscription != null) {
+      this.duplicateNotificationSubscription.unsubscribe();
+    }
     if (this.deleteSubscription != null) {
       this.deleteSubscription.unsubscribe();
     }
   }
 
 
   public onSave(ignore: any) {
+    this.notificationService.discardNotification();
     this.saveLoading = true;
     // persist
     this.currentMannschaft.vereinId = this.currentVerein.id; // Set selected verein id// set selected veranstaltung id
@@ -171,6 +184,7 @@ export class MannschaftDetailComponent extends CommonComponentDirective implemen
     }
 
   public onUpdate(ignore: any): void {
+    this.notificationService.discardNotification();
     this.saveLoading = true;
 
     // persist
@@ -711,6 +725,8 @@ export class MannschaftDetailComponent extends CommonComponentDirective implemen
   }
 
   private showDuplicateMannschaftNotification(): void {
+    this.notificationService.discardNotification();
+
     const notification: Notification = {
       id:          NOTIFICATION_DUPLICATE_MANNSCHAFT,
       title:       'MANAGEMENT.VEREIN_DETAIL.NOTIFICATION.DUPLICATE.TITLE',

bogenliga/src/app/modules/verwaltung/components/verein/verein-detail/verein-detail.component.ts

@@ -1,4 +1,4 @@
-import {Component, ElementRef, OnInit, ViewChild} from '@angular/core';
+import {Component, ElementRef, OnDestroy, OnInit, ViewChild} from '@angular/core';
 import {ActivatedRoute, Router} from '@angular/router';
 import {isNullOrUndefined, isUndefined} from '@shared/functions';
 import {
@@ -56,14 +56,16 @@ const NOTIFICATION_COPY_MANNSCHAFT = 'mannschaft_detail_copy';
 const NOTIFICATION_DELETE_MANNSCHAFT_SUCCESS = 'mannschaft_detail_delete_success';
 const NOTIFICATION_DELETE_MANNSCHAFT_FAILURE = 'mannschaft_detail_delete_failure';
 const NOTIFICATION_NO_LICENSE = 'no_license_found';
+const NOTIFICATION_ENTITY_CONFLICT_ERROR = 'ENTITY_CONFLICT_ERROR';
+const NOTIFICATION_DATABASE_ERROR = 'DATABASE_ERROR';
 const PLATZHALTER_ID = 99;
 
 @Component({
   selector:    'bla-verein-detail',
   templateUrl: './verein-detail.component.html',
   styleUrls:   ['./verein-detail.component.scss']
 })
-export class VereinDetailComponent extends CommonComponentDirective implements OnInit {
+export class VereinDetailComponent extends CommonComponentDirective implements OnInit, OnDestroy {
   public regionType = 'KREIS';
   public config = VEREIN_DETAIL_CONFIG;
   public config_table = VEREIN_DETAIL_TABLE_CONFIG;
@@ -80,6 +82,7 @@ export class VereinDetailComponent extends CommonComponentDirective implements O
 
 
   private sessionHandling: SessionHandling;
+  private saveErrorNotificationSubscriptions = [];
 
   @ViewChild('downloadLink')
   private aElementRef: ElementRef;
@@ -103,9 +106,15 @@ export class VereinDetailComponent extends CommonComponentDirective implements O
   ngOnInit() {
     this.loading = true;
     this.notificationService.discardNotification();
+    this.registerSaveErrorReset(NOTIFICATION_ENTITY_CONFLICT_ERROR);
+    this.registerSaveErrorReset(NOTIFICATION_DATABASE_ERROR);
     this.loadRegions(this.regionType); // Request all regions from the backend
   }
 
+  ngOnDestroy() {
+    this.saveErrorNotificationSubscriptions.forEach((subscription) => subscription.unsubscribe());
+  }
+
   /** When a MouseOver-Event is triggered, it will call this inMouseOver-function.
    *  This function calls the checkSessionExpired-function in the sessionHandling class and get a boolean value back.
    *  If the boolean value is true, then the page will be reloaded and due to the expired session, the user will
@@ -136,6 +145,7 @@ export class VereinDetailComponent extends CommonComponentDirective implements O
   }
 
   public onSave(ignore: any): void {
+    this.notificationService.discardNotification();
     this.saveLoading = true;
 
     // persist
@@ -186,6 +196,7 @@ export class VereinDetailComponent extends CommonComponentDirective implements O
   }
 
   public onUpdate(ignore: any): void {
+    this.notificationService.discardNotification();
     this.saveLoading = true;
 
     // persist
@@ -618,6 +629,18 @@ export class VereinDetailComponent extends CommonComponentDirective implements O
     this.loading = false;
   }
 
+  private registerSaveErrorReset(notificationId: string): void {
+    const subscription = this.notificationService.observeNotification(notificationId)
+      .subscribe((notification) => {
+        if (notification.userAction === NotificationUserAction.ACCEPTED
+          || notification.userAction === NotificationUserAction.DECLINED) {
+          this.saveLoading = false;
+        }
+      });
+
+    this.saveErrorNotificationSubscriptions.push(subscription);
+  }
+
   private loadMannschaften() {
     this.loading = true;
     this.mannschaftsDataProvider.findAllByVereinsId(this.currentVerein.id)

bogenliga/src/assets/i18n/de.json

@@ -1048,8 +1048,8 @@
           "DESCRIPTION": "Verein wurde erfolgreich gespeichert."
         },
         "DUPLICATE": {
-          "TITLE": "Mannschaftsnummer in dem Verein bereits vergeben!",
-          "DESCRIPTION": "Möchten sie die Mannschaft trotzdem hinzufügen?"
+          "TITLE": "Doppelte Mannschaft",
+          "DESCRIPTION": "Die Mannschaftsnummer ist in diesem Verein bereits vergeben. Bitte wählen Sie eine andere Nummer."
         },
         "SAVE_FAILURE": {
           "TITLE": "Fehler beim Aktualisieren",
@@ -1902,6 +1902,10 @@
         "TITLE": "Datenbankfehler",
         "DESCRIPTION": "Ihre Anfrage wurde von der Datenbank abgewiesen."
       },
+      "ENTITY_CONFLICT_ERROR": {
+        "TITLE": "Doppelte Eingabe",
+        "DESCRIPTION": "Ein Eintrag mit diesen Werten existiert bereits. Bitte prüfen Sie Ihre Eingaben."
+      },
       "ENTITY_NOT_FOUND_ERROR": {
         "TITLE": "Nicht gefunden",
         "DESCRIPTION": "Die angeforderte Resource wurde nicht gefunden."