Merge pull request #628 from bettercodepaul/1722-suche-des-vereins-bei-neuem-vereinsmitglied

McYum · web-flow · commit 3069fdabc648 · 2026-04-20T20:16:14.000+02:00
1722 suche des vereins bei neuem vereinsmitglied, 1952 Fehlender Text bei NoPermissionException
diff --git a/bogenliga/src/app/components/sidebar/sidebar.component.ts b/bogenliga/src/app/components/sidebar/sidebar.component.ts
@@ -13,15 +13,15 @@ import { LigaContextService } from '@shared/services/liga-context/liga-context.s
 import { Subscription } from 'rxjs';
 import { distinctUntilChanged } from 'rxjs/operators';
 import { LigaDO } from '@verwaltung/types/liga-do.class';
-import {slugifyLigaName} from "@shared/functions/slug-utils";
+import {slugifyLigaName} from '@shared/functions/slug-utils';
 import {isNullOrUndefined, isUndefined} from '@shared/functions';
 import { AnalyticsService } from '@shared/services';
-import {SelectedLigaDataprovider} from '../../modules/shared/data-provider/SelectedLigaDataprovider'
+import {SelectedLigaDataprovider} from '../../modules/shared/data-provider/SelectedLigaDataprovider';
 
 
 
 const ID_PATH_PARAM = 'id';
-export var ligaID: number;
+export let ligaID: number;
 
 @Component({
   selector: 'bla-sidebar',
@@ -65,7 +65,7 @@ export class SidebarComponent implements OnInit, OnDestroy {
     // Liga-Kontext beobachten
     this.ligaSub = this.ligaContext.currentLiga$
       .pipe(distinctUntilChanged((a, b) => a?.id === b?.id))
-      .subscribe(liga => {
+      .subscribe((liga) => {
         this.currentLiga = liga ?? null;
       });
   }
@@ -96,33 +96,35 @@ export class SidebarComponent implements OnInit, OnDestroy {
   public getRoute(route: string, detailType: string): string {
     let result: string = route;
     this.URLRoute = this.router.url;
-    if (this.URLRoute.startsWith("/ligatabelle") ||this.URLRoute.startsWith("/home") ) {
+    if (this.URLRoute.startsWith('/ligatabelle') || this.URLRoute.startsWith('/home') ) {
       const lastSlashIndex = this.URLRoute.lastIndexOf('/');
-      switch(lastSlashIndex){
+      switch (lastSlashIndex) {
         case 0:
-          //forget liga ID if no liga ID is part of the URL-route
+          // forget liga ID if no liga ID is part of the URL-route
           this.ligaID = undefined;
           break;
         case result.length:
-          !(this.URLRoute.substring(lastSlashIndex + 1).toString() === "ligaid") ?
-            this.ligaID = parseInt(this.URLRoute.substring(lastSlashIndex + 1)) : undefined;
+          const currentPathParam = this.URLRoute.substring(lastSlashIndex + 1);
+          if (currentPathParam !== 'ligaid') {
+            this.ligaID = parseInt(currentPathParam, 10);
+          }
           break;
       }
 
     }
 
     if (detailType === 'undefined') {
       return(route);
-    } else if (detailType === 'verein'){
+    } else if (detailType === 'verein') {
       result = result + '/' + this.currentUserService.getVerein();
     } else {
       result = result;
     }
 
     // Für die Home-Seite wollen wir NICHT automatisch eine Liga-ID anhängen,
     // damit man immer zurück auf die echte Startseite kommt.
-    if(this.ligaID != undefined && route.startsWith("/ligatabelle")){
-      result =  result + '/'+ this.ligaID.toString();
+    if (this.ligaID !== undefined && route.startsWith('/ligatabelle')) {
+      result =  result + '/' + this.ligaID.toString();
     }
 
     this.selectedLigaDataprovider.setSelectedLigaID(this.ligaID);
@@ -191,6 +193,12 @@ export class SidebarComponent implements OnInit, OnDestroy {
   }
 
   isSelected(itemroute: string): boolean {
-    return (this.router.url.indexOf(itemroute) >= 0);
+    const currentPath = this.getCurrentPath();
+    return currentPath === itemroute || currentPath.startsWith(`${itemroute}/`);
+  }
+
+  private getCurrentPath(): string {
+    const url = this.router.url || '';
+    return url.split('?')[0].split('#')[0];
   }
 }
diff --git a/bogenliga/src/app/modules/verwaltung/components/dsb-mitglied/dsb-mitglied-detail/dsb-mitglied-detail.component.html b/bogenliga/src/app/modules/verwaltung/components/dsb-mitglied/dsb-mitglied-detail/dsb-mitglied-detail.component.html
@@ -135,14 +135,24 @@
           <span> *</span>
         </label>
         <div class="col-sm-9">
+          <input type="text"
+                 class="form-control mb-2"
+                 id="dsbMitgliedVereinSuche"
+                 name="dsbMitgliedVereinSuche"
+                 [(ngModel)]="vereinSearchTerm"
+                 (ngModelChange)="onVereinSearch($event)"
+                 placeholder="{{ 'SELECTIONLIST.SEARCH_PLACEHOLDER' | translate }}"
+                 autocomplete="off"
+                 data-cy="detail-vereine-dsb-search"
+                 [readonly]="isReadOnly">
           <select class="form-control"
                   id="dsbMitgliedVerein"
                   name="dsbMitgliedVerein"
                   #dsbMitgliedVerein="ngModel"
                   [(ngModel)]="currentVerein"
                   data-cy="detail-vereine-dsb"
                   [disabled]="isReadOnly">
-            <option [ngValue]="verein" *ngFor="let verein of vereine"> {{verein.name}}</option>
+            <option [ngValue]="verein" *ngFor="let verein of filteredVereine"> {{verein.name}}</option>
           </select>
           <div class="invalid-feedback">
             {{ 'MANAGEMENT.DSBMITGLIEDER_DETAIL.FORM.VEREIN.ERROR' | translate }}
diff --git a/bogenliga/src/app/modules/verwaltung/components/dsb-mitglied/dsb-mitglied-detail/dsb-mitglied-detail.component.ts b/bogenliga/src/app/modules/verwaltung/components/dsb-mitglied/dsb-mitglied-detail/dsb-mitglied-detail.component.ts
@@ -48,9 +48,9 @@ export class DsbMitgliedDetailComponent extends CommonComponentDirective impleme
   public ButtonType = ButtonType;
   public currentMitglied: DsbMitgliedDO = new DsbMitgliedDO();
   public currentVerein: VereinDO = new VereinDO();
-  // public vereine: Array<VereinDO> = [new VereinDO()];
-  public vereine: VereinDO[];
-  // public currentVerein: VereinDO;
+  public vereine: VereinDO[] = [];
+  public filteredVereine: VereinDO[] = [];
+  public vereinSearchTerm = '';
 
   public dsbMitgliedNationalitaet: string[];
   public loadingVereine = true;
@@ -373,6 +373,7 @@ export class DsbMitgliedDetailComponent extends CommonComponentDirective impleme
 
       }
     });
+    this.applyVereinFilter();
   }
 
   private handleFailure(response: BogenligaResponse<DsbMitgliedDO>) {
@@ -432,13 +433,43 @@ export class DsbMitgliedDetailComponent extends CommonComponentDirective impleme
         if (this.currentUserService.hasPermission(UserPermission.CAN_CREATE_VEREIN_DSBMITGLIEDER)) {
           response.payload = response.payload.filter((entry) => this.currentUserService.getVerein() === entry.id);
         }
-        // this.currentVerein = response.payload[0];
         this.vereine = response.payload;
+        this.applyVereinFilter();
         this.loadingVereine = false;
         this.vereineLoaded = true;
       })
       .catch((response: BogenligaResponse<VereinDTO[]>) => {
         this.vereine = response.payload;
+        this.applyVereinFilter();
       });
   }
+
+  public onVereinSearch(searchTerm: string): void {
+    this.vereinSearchTerm = searchTerm;
+    this.applyVereinFilter();
+  }
+
+  private applyVereinFilter(): void {
+    const normalizedSearchTerm = (this.vereinSearchTerm || '').trim().toLocaleLowerCase();
+
+    if (!normalizedSearchTerm) {
+      this.filteredVereine = [...this.vereine];
+      return;
+    }
+
+    this.filteredVereine = this.vereine.filter((verein: VereinDO) => this.matchesVereinSearch(verein, normalizedSearchTerm));
+
+    if (this.currentVerein?.id && !this.filteredVereine.some((verein: VereinDO) => verein.id === this.currentVerein.id)) {
+      const selectedVerein = this.vereine.find((verein: VereinDO) => verein.id === this.currentVerein.id);
+      if (selectedVerein) {
+        this.filteredVereine = [selectedVerein, ...this.filteredVereine];
+      }
+    }
+  }
+
+  private matchesVereinSearch(verein: VereinDO, normalizedSearchTerm: string): boolean {
+    return [verein.name, verein.identifier, verein.regionName]
+      .filter((value: string) => !!value)
+      .some((value: string) => value.toLocaleLowerCase().includes(normalizedSearchTerm));
+  }
 }
diff --git a/bogenliga/src/app/modules/verwaltung/components/veranstaltung/veranstaltung-detail/veranstaltung-detail.component.ts b/bogenliga/src/app/modules/verwaltung/components/veranstaltung/veranstaltung-detail/veranstaltung-detail.component.ts
@@ -71,6 +71,7 @@ const NOTIFICATION_DELETE_MANNSCHAFT = 'mannschaft_detail_delete';
 const NOTIFICATION_FINISH_VERANSTALTUNG = 'veranstaltung_detail_finish';
 const NOTIFICATION_CREATE_PLATZHALTER = 'platzhalter_create';
 const NOTIFICATION_CREATE_PLATZHALTER_FAILURE = 'platzhalter_create_failure';
+const NOTIFICATION_CREATE_PLATZHALTER_PERMISSION_FAILURE = 'platzhalter_create_permission_failure';
 
 
 
@@ -517,7 +518,7 @@ export class VeranstaltungDetailComponent extends CommonComponentDirective imple
     this.saveLoading = true;
 
     const platzhalterId = 99;
-    const platzhalterNummer = "1";
+    const platzhalterNummer = '1';
 
 
     this.selectedMannschaft.vereinId = platzhalterId;
@@ -531,7 +532,6 @@ export class VeranstaltungDetailComponent extends CommonComponentDirective imple
             && !isNullOrUndefined(response.payload)
             && !isNullOrUndefined(response.payload.id)) {
             console.log('Saved with id: ' + response.payload.id);
-            console.log('Wir sind der Sturm, der über das Ziel hinwegfegt, niemand kann uns aufhalten!');
             const notification: Notification = {
               id:          NOTIFICATION_CREATE_PLATZHALTER,
               title:       'MANAGEMENT.VERANSTALTUNG_DETAIL.NOTIFICATION.PLATZHALTER_SAVE.TITLE',
@@ -555,16 +555,26 @@ export class VeranstaltungDetailComponent extends CommonComponentDirective imple
         }
     )
         .catch((response) => {
+      let notificationId = NOTIFICATION_CREATE_PLATZHALTER_FAILURE;
+      let title = 'MANAGEMENT.VERANSTALTUNG_DETAIL.NOTIFICATION.PLATZHALTER_FAILURE.TITLE';
+      let description = 'MANAGEMENT.VERANSTALTUNG_DETAIL.NOTIFICATION.PLATZHALTER_FAILURE.DESCRIPTION';
+
+      if (response.error && response.error.status === 403) {
+        notificationId = NOTIFICATION_CREATE_PLATZHALTER_PERMISSION_FAILURE;
+        title = 'Keine Berechtigung';
+        description = 'Sie haben keine Berechtigung für diese Aktion. Die Änderungen wurden nicht gespeichert.';
+      }
+
       const notification: Notification = {
-        id:          NOTIFICATION_CREATE_PLATZHALTER_FAILURE,
-        title: 'MANAGEMENT.VERANSTALTUNG_DETAIL.NOTIFICATION.PLATZHALTER_FAILURE.TITLE',
-        description: 'MANAGEMENT.VERANSTALTUNG_DETAIL.NOTIFICATION.PLATZHALTER_FAILURE.DESCRIPTION',
+        id:          notificationId,
+        title:       title,
+        description: description,
         severity:    NotificationSeverity.ERROR,
         origin:      NotificationOrigin.USER,
         type:        NotificationType.OK,
         userAction:  NotificationUserAction.PENDING
       };
-      this.notificationService.observeNotification(NOTIFICATION_CREATE_PLATZHALTER_FAILURE)
+      this.notificationService.observeNotification(notificationId)
           .subscribe((myNotification) => {
             if (myNotification.userAction === NotificationUserAction.ACCEPTED) {
               this.saveLoading = false;
diff --git a/bogenliga/src/app/modules/verwaltung/components/verein/verein-detail/mannschafts-detail/mannschaft-detail.component.ts b/bogenliga/src/app/modules/verwaltung/components/verein/verein-detail/mannschafts-detail/mannschaft-detail.component.ts
diff --git a/bogenliga/src/app/modules/verwaltung/services/dsb-mannschaft-data-provider.service.ts b/bogenliga/src/app/modules/verwaltung/services/dsb-mannschaft-data-provider.service.ts
