Merge branch 'develop' into #2087-löschung-nach-deadline-backend

Author: bedanc25

bogenliga/bogenliga-application/src/main/java/de/bogenliga/application/services/v1/download/DownloadService.java

@@ -8,6 +8,7 @@
 import de.bogenliga.application.business.wettkampf.api.types.WettkampfDO;
 import de.bogenliga.application.business.wettkampf.api.WettkampfComponent;
 import de.bogenliga.application.springconfiguration.security.permissions.RequiresPermission;
+import de.bogenliga.application.springconfiguration.security.permissions.RequiresOnePermissions;
 import de.bogenliga.application.springconfiguration.security.types.UserPermission;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -18,16 +19,19 @@
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.*;
 import de.bogenliga.application.business.bogenkontrollliste.api.BogenkontrolllisteComponent;
+import de.bogenliga.application.business.dsbmannschaft.api.DsbMannschaftComponent;
 import de.bogenliga.application.business.meldezettel.api.MeldezettelComponent;
 import de.bogenliga.application.business.schusszettel.api.SchusszettelComponent;
 import de.bogenliga.application.business.setzliste.api.SetzlisteComponent;
 import de.bogenliga.application.business.lizenz.api.LizenzComponent;
 import de.bogenliga.application.business.rueckennummern.api.RueckennummernComponent;
 import de.bogenliga.application.common.errorhandling.ErrorCode;
+import de.bogenliga.application.common.errorhandling.exception.BusinessException;
 import de.bogenliga.application.common.errorhandling.exception.TechnicalException;
 import de.bogenliga.application.common.service.ServiceFacade;
 import de.bogenliga.application.common.validation.Preconditions;
 import de.bogenliga.application.services.v1.setzliste.service.SetzlisteService;
+import de.bogenliga.application.springconfiguration.security.permissions.RequiresOnePermissionAspect;
 
 
 /**
@@ -65,6 +69,8 @@ public class DownloadService implements ServiceFacade {
     private final BogenkontrolllisteComponent bogenkontrolllisteComponent;
     private final RueckennummernComponent rueckennummernComponent;
     private final WettkampfComponent wettkampfComponent;
+    private final DsbMannschaftComponent dsbMannschaftComponent;
+    private final RequiresOnePermissionAspect requiresOnePermissionAspect;
 
 
     /**
@@ -76,16 +82,20 @@ public DownloadService(final SetzlisteComponent setzlisteComponent, final Lizenz
                            final MeldezettelComponent meldezettelComponent,
                            final BogenkontrolllisteComponent bogenkontrolllisteComponent,
                            final RueckennummernComponent rueckennummernComponent,
-                           final WettkampfComponent wettkampfComponent) {
+                           final WettkampfComponent wettkampfComponent,
+                           final DsbMannschaftComponent dsbMannschaftComponent,
+                           final RequiresOnePermissionAspect requiresOnePermissionAspect) {
         this.lizenzComponent = lizenzComponent;
         this.setzlisteComponent = setzlisteComponent;
         this.schusszettelComponent = schusszettelComponent;
         this.meldezettelComponent = meldezettelComponent;
         this.bogenkontrolllisteComponent = bogenkontrolllisteComponent;
         this.rueckennummernComponent = rueckennummernComponent;
         this.wettkampfComponent = wettkampfComponent;
+        this.dsbMannschaftComponent = dsbMannschaftComponent;
+        this.requiresOnePermissionAspect = requiresOnePermissionAspect;
     }
-  
+
     /**
      * returns the Setzliste as pdf file for client download
      * <p>
@@ -231,8 +241,8 @@ ResponseEntity<InputStreamResource> downloadbogenkontrolllistePdf(@RequestParam(
      */
     @CrossOrigin(maxAge = 0)
     @GetMapping(
-                    path = "pdf/rueckennummern",
-                    produces = MediaType.APPLICATION_PDF_VALUE)
+            path = "pdf/rueckennummern",
+            produces = MediaType.APPLICATION_PDF_VALUE)
     @RequiresPermission(UserPermission.CAN_READ_DEFAULT)
     public
     ResponseEntity<InputStreamResource> downloadRueckennummernPdf(@RequestParam("mannschaftid") final long mannschaftid) {
@@ -291,11 +301,32 @@ private ResponseEntity<InputStreamResource> generateInputStream(byte[] fileBloB)
     @GetMapping(
             path = "pdf/schuetzenlizenz/{dsbMitgliedId}/{teamId}",
             produces = MediaType.APPLICATION_PDF_VALUE)
-    @RequiresPermission(UserPermission.CAN_READ_WETTKAMPF)
+    @RequiresOnePermissions(perm = {UserPermission.CAN_MODIFY_MY_VEREIN, UserPermission.CAN_MODIFY_MY_VERANSTALTUNG})
     public ResponseEntity<InputStreamResource> downloadLizenz(@PathVariable("dsbMitgliedId") final long dsbMitgliedID,
-    @PathVariable("teamId") final long teamID) {
+                                                              @PathVariable("teamId") final long teamID) {
         LOG.debug("dsbMitgliedID: {}", dsbMitgliedID);
         LOG.debug("teamID: {}", teamID);
+
+        // Data-specific authorization check
+        var mannschaft = dsbMannschaftComponent.findById(teamID);
+        if (mannschaft == null) {
+            throw new BusinessException(ErrorCode.NO_PERMISSION_ERROR,
+                    "Team not found");
+        }
+
+        // Check if user has permission to modify their own club (Sportleiter)
+        boolean isSportleiter = requiresOnePermissionAspect.hasSpecificPermissionSportleiter(
+                UserPermission.CAN_MODIFY_MY_VEREIN, mannschaft.getVereinId());
+
+        // Check if user has permission to modify event (Ligaleiter)
+        boolean isLigaleiter = requiresOnePermissionAspect.hasSpecificPermissionLigaLeiterID(
+                UserPermission.CAN_MODIFY_MY_VERANSTALTUNG, mannschaft.getVeranstaltungId());
+
+        if (!isSportleiter && !isLigaleiter) {
+            throw new BusinessException(ErrorCode.NO_PERMISSION_ERROR,
+                    String.format("User does not have permission to download license for team %d", teamID));
+        }
+
         final byte[] fileBloB = lizenzComponent.getLizenzPDFasByteArray(dsbMitgliedID, teamID);
 
         return generateInputStream(fileBloB);
@@ -314,10 +345,29 @@ public ResponseEntity<InputStreamResource> downloadLizenz(@PathVariable("dsbMitg
     @GetMapping(
             path = "pdf/lizenzen",
             produces = MediaType.APPLICATION_PDF_VALUE)
-    @RequiresPermission(UserPermission.CAN_READ_DEFAULT)
+    @RequiresOnePermissions(perm = {UserPermission.CAN_MODIFY_MY_VEREIN, UserPermission.CAN_MODIFY_MY_VERANSTALTUNG})
     public
     ResponseEntity<InputStreamResource> downloadLizenzenPdf(@RequestParam("mannschaftid") final long mannschaftid) {
 
+        // Data-specific authorization check
+        var mannschaft = dsbMannschaftComponent.findById(mannschaftid);
+        if (mannschaft == null) {
+            throw new BusinessException(ErrorCode.NO_PERMISSION_ERROR,
+                    "Team not found");
+        }
+
+        // Check if user has permission to modify their own club (Sportleiter)
+        boolean isSportleiter = requiresOnePermissionAspect.hasSpecificPermissionSportleiter(
+                UserPermission.CAN_MODIFY_MY_VEREIN, mannschaft.getVereinId());
+
+        // Check if user has permission to modify event (Ligaleiter)
+        boolean isLigaleiter = requiresOnePermissionAspect.hasSpecificPermissionLigaLeiterID(
+                UserPermission.CAN_MODIFY_MY_VERANSTALTUNG, mannschaft.getVeranstaltungId());
+
+        if (!isSportleiter && !isLigaleiter) {
+            throw new BusinessException(ErrorCode.NO_PERMISSION_ERROR,
+                    String.format("User does not have permission to download licenses for team %d", mannschaftid));
+        }
 
         final byte[] fileBloB = lizenzComponent.getMannschaftsLizenzenPDFasByteArray(mannschaftid);
 
@@ -334,16 +384,16 @@ ResponseEntity<InputStreamResource> downloadLizenzenPdf(@RequestParam("mannschaf
      * <pre>{@code Request: GET /v1/download/pdf/Einzelstatistik/?werte=x}</pre>
      *
      * @return pdf as InputStreamRessource
-    */
+     */
     @CrossOrigin(maxAge = 0)
     @GetMapping(
             path = "pdf/Einzelstatistik",
             produces = MediaType.APPLICATION_PDF_VALUE)
     @RequiresPermission(UserPermission.CAN_READ_DEFAULT)
     public
     ResponseEntity<InputStreamResource> downloadEinzelstatistikPdf(@RequestParam("veranstaltungsid") final long veranstaltungsid,
-    @RequestParam("manschaftsid") final long manschaftsid,
-    @RequestParam("jahr") final int jahr)
+                                                                   @RequestParam("manschaftsid") final long manschaftsid,
+                                                                   @RequestParam("jahr") final int jahr)
     {
 
         final byte[] fileBloB = wettkampfComponent.getPDFasByteArray("Einzelstatistik",veranstaltungsid,manschaftsid,jahr);

bogenliga/bogenliga-application/src/main/java/de/bogenliga/application/services/v1/dsbmannschaft/service/DsbMannschaftService.java

@@ -50,7 +50,7 @@ public class DsbMannschaftService implements ServiceFacade {
     private static final String PRECONDITION_MSG_DSBMANNSCHAFT_VEREIN_ID_NEGATIVE = "DsbMannschaft Vereins Id must not be negative";
     private static final String PRECONDITION_MSG_DSBMANNSCHAFT_NUMMER_NEGATIVE = "DsbMannschaft Nummer must not be negative";
     private static final String PRECONDITION_MSG_DSBMANNSCHAFT_BENUTZER_ID_NEGATIVE = "DsbMannschaft Benutzer Id must not be negative";
-    private static final String PRECONDITION_MSG_DSBMANNSCHAFT_VERANSTALTUNG_FULL = "DsbMannschaft Veranstaltung has already reached its maximum capacity";
+    private static final String ERROR_MSG_VERANSTALTUNG_FULL = "Die Veranstaltung hat ihre maximale Kapazität erreicht. Es kann keine weitere Mannschaft hinzugefügt werden.";
     private static final String PRECONDITION_MSG_ID_NEGATIVE = "ID must not be negative.";
     private static final String PRECONDITION_MSG_WRONG_YEAR = "Year has to be a valid year.";
     private static final String PRECONDITION_MSG_VERANSTALTUNG_SIZE_NEGATIV = "DsbMannschaft Veranstaltung size can not be negativ";
@@ -74,6 +74,7 @@ public class DsbMannschaftService implements ServiceFacade {
     private static final String ERROR_MSG_PLATZHALTER_NO_PERMISSION = "Sie haben keine Berechtigung für diese Aktion.";
     private static final String ERROR_MSG_DELETE_MANNSCHAFT_NO_PERMISSION = "Löschen einer Mannschaft ist nur mit entsprechender Berechtigung erlaubt.";
     private static final String ERROR_MSG_UPDATE_MANNSCHAFT_NO_PERMISSION = "Ändern einer Mannschaft ist nur mit entsprechender Berechtigung erlaubt.";
+    private static final String ERROR_MSG_VERANSTALTUNG_LAUFEND = "Die Veranstaltung ist in der Phase 'Laufend'. Teilnehmende Mannschaften können in dieser Phase nicht hinzugefügt, geändert oder entfernt werden.";
 
     MannschaftsmitgliedComponent mannschaftsmitgliedComponent;
 
@@ -337,6 +338,9 @@ public DsbMannschaftDTO create(@RequestBody final DsbMannschaftDTO dsbMannschaft
             final Long userId = UserProvider.getCurrentUserId(principal);
             Preconditions.checkArgument(userId >= 0, PRECONDITION_MSG_DSBMANNSCHAFT_BENUTZER_ID_NEGATIVE);
 
+            // Bei laufender Veranstaltung duerfen keine Mannschaften (auch keine Platzhalter) angelegt werden.
+            assertVeranstaltungNotLaufend(dsbMannschaftDTO.getVeranstaltungId());
+
             // Check size of Veranstaltung and if it is full
             // If Veranstaltung does not exist choose 8 as its default size
             if(dsbMannschaftDTO.getVereinId().equals(PLATZHALTER_VEREIN_ID)) {
@@ -360,9 +364,9 @@ public DsbMannschaftDTO create(@RequestBody final DsbMannschaftDTO dsbMannschaft
                 }
 
 
-                Preconditions.checkArgument(actualMannschaftInVeranstaltungCount.size() < veranstaltungsgroesse
-
-                        , PRECONDITION_MSG_DSBMANNSCHAFT_VERANSTALTUNG_FULL);
+                if (actualMannschaftInVeranstaltungCount.size() >= veranstaltungsgroesse) {
+                    throw new BusinessException(ErrorCode.VERANSTALTUNG_MAX_CAPACITY_ERROR, ERROR_MSG_VERANSTALTUNG_FULL);
+                }
             }
             LOG.debug("Receive 'create' request with verein id '{}', nummer '{}', benutzer id '{}', veranstaltung id '{}',",
 
@@ -534,9 +538,11 @@ public void assignMannschaftToVeranstaltung(@PathVariable("VeranstaltungsId") fi
             throw new NoPermissionException();
         }
 
-
         dsbMannschaftDO.setVeranstaltungId(veranstaltungsId);
         VeranstaltungDO veranstaltungDO = veranstaltungComponent.findById(veranstaltungsId);
+        // Bei laufender Veranstaltung duerfen keine Mannschaften zugeordnet werden
+        // (Pruefung auf dem bereits geladenen DO -> kein erneutes findById).
+        assertVeranstaltungNotLaufend(veranstaltungDO);
         dsbMannschaftDO.setSportjahr(veranstaltungDO.getVeranstaltungSportJahr());
 
         DsbMannschaftDO neueMannschaft = dsbMannschaftComponent.update(dsbMannschaftDO, userId);
@@ -559,6 +565,9 @@ public void unassignMannschaftFromVeranstaltung(@PathVariable("MannschaftId") fi
 
         DsbMannschaftDO dsbMannschaftDO = dsbMannschaftComponent.findById(mannschaftId);
 
+        // Bei laufender Veranstaltung darf die Zuordnung der Mannschaft nicht entfernt werden.
+        assertVeranstaltungNotLaufend(dsbMannschaftDO.getVeranstaltungId());
+
         // Prüfen, ob die Veranstaltung in der Phase "geplant" ist -
         // nur dann können wir löschen, ohne dass Daten verloren gehen könnten...
         // das Sportjahr wird immer parallel gelöscht/gesetzt
@@ -574,6 +583,29 @@ public void unassignMannschaftFromVeranstaltung(@PathVariable("MannschaftId") fi
         }
     }
 
+    /**
+     * Stellt sicher, dass die Veranstaltung nicht in der Phase 'Laufend' ist.
+     * In dieser Phase duerfen teilnehmende Mannschaften weder hinzugefuegt noch
+     * entfernt oder umsortiert werden (serverseitige Absicherung der UI-Sperre
+     * aus Ticket swt2#2157, vgl. swt2#2229). Bei {@code null} (keine Veranstaltung
+     * zugeordnet) passiert nichts.
+     *
+     * @param veranstaltungsId id der zu pruefenden Veranstaltung (darf null sein)
+     * @throws BusinessException wenn die Veranstaltung in der Phase 'Laufend' ist
+     */
+    private void assertVeranstaltungNotLaufend(final Long veranstaltungsId) {
+        if (veranstaltungsId != null && veranstaltungComponent.isVeranstaltungLaufend(veranstaltungsId)) {
+            throw new BusinessException(ErrorCode.ENTITY_CONFLICT_ERROR, ERROR_MSG_VERANSTALTUNG_LAUFEND);
+        }
+    }
+
+    /** Variante fuer eine bereits geladene Veranstaltung (vermeidet erneutes findById). */
+    private void assertVeranstaltungNotLaufend(final VeranstaltungDO veranstaltung) {
+        if (veranstaltungComponent.isVeranstaltungLaufend(veranstaltung)) {
+            throw new BusinessException(ErrorCode.ENTITY_CONFLICT_ERROR, ERROR_MSG_VERANSTALTUNG_LAUFEND);
+        }
+    }
+
     /**
      * I persist a newer version of the dsbMannschaft in the database.
      *
@@ -613,8 +645,9 @@ public DsbMannschaftDTO update(@RequestBody final DsbMannschaftDTO dsbMannschaft
             if(veranstaltungDO != null) {
                 veranstaltungsgroesse = veranstaltungDO.getVeranstaltungGroesse();
             }
-            Preconditions.checkArgument(mannschaftenInZielveranstaltung.size() < veranstaltungsgroesse
-                    , PRECONDITION_MSG_DSBMANNSCHAFT_VERANSTALTUNG_FULL);
+            if (mannschaftenInZielveranstaltung.size() >= veranstaltungsgroesse) {
+                throw new BusinessException(ErrorCode.VERANSTALTUNG_MAX_CAPACITY_ERROR, ERROR_MSG_VERANSTALTUNG_FULL);
+            }
         }
 
 

bogenliga/bogenliga-application/src/main/java/de/bogenliga/application/services/v1/dsbmannschaft/service/MannschaftSortierungService.java

@@ -1,7 +1,11 @@
 package de.bogenliga.application.services.v1.dsbmannschaft.service;
 
+import de.bogenliga.application.business.dsbmannschaft.api.DsbMannschaftComponent;
 import de.bogenliga.application.business.dsbmannschaft.api.DsbMannschaftSortierungComponent;
 import de.bogenliga.application.business.dsbmannschaft.api.types.DsbMannschaftDO;
+import de.bogenliga.application.business.veranstaltung.api.VeranstaltungComponent;
+import de.bogenliga.application.common.errorhandling.ErrorCode;
+import de.bogenliga.application.common.errorhandling.exception.BusinessException;
 import de.bogenliga.application.common.service.ServiceFacade;
 import de.bogenliga.application.common.service.UserProvider;
 import de.bogenliga.application.common.validation.Preconditions;
@@ -25,6 +29,7 @@ public class MannschaftSortierungService implements ServiceFacade {
     private static final String PRECONDITION_MSG_DSBMANNSCHAFT_DO = "DsbMannschaftDO must not be null";
     private static final String PRECONDITION_MSG_DSBMANNSCHAFT_ID = "DsbMannschaft-ID must not be null or less than 0";
     private static final String PRECONDITION_MSG_DSBMANNSCHAFT_SORTIERUNG = "DsbMannschaft-Sortierung must not be null or less than 0";
+    private static final String ERROR_MSG_VERANSTALTUNG_LAUFEND = "Die Veranstaltung ist in der Phase 'Laufend'. Der Tabellenplatz teilnehmender Mannschaften kann in dieser Phase nicht geändert werden.";
 
     private static final Logger LOG = LoggerFactory.getLogger(MannschaftSortierungService.class);
 
@@ -34,16 +39,24 @@ public class MannschaftSortierungService implements ServiceFacade {
      * dependency injection with {@link Autowired}
      */
     private final DsbMannschaftSortierungComponent maSortierungComponent;
+    private final DsbMannschaftComponent dsbMannschaftComponent;
+    private final VeranstaltungComponent veranstaltungComponent;
 
 
     /**
      * Constructor with dependency injection
      *
      * @param maSortierungComponent to handle the database CRUD requests
+     * @param dsbMannschaftComponent to resolve the Veranstaltung of a Mannschaft
+     * @param veranstaltungComponent to read the phase of the Veranstaltung
      */
     @Autowired
-    public MannschaftSortierungService(final DsbMannschaftSortierungComponent maSortierungComponent) {
+    public MannschaftSortierungService(final DsbMannschaftSortierungComponent maSortierungComponent,
+                                       final DsbMannschaftComponent dsbMannschaftComponent,
+                                       final VeranstaltungComponent veranstaltungComponent) {
         this.maSortierungComponent = maSortierungComponent;
+        this.dsbMannschaftComponent = dsbMannschaftComponent;
+        this.veranstaltungComponent = veranstaltungComponent;
     }
 
 
@@ -61,10 +74,27 @@ public MannschaftSortierungDTO update(@RequestBody final MannschaftSortierungDTO
         LOG.debug("Receive 'update' request with Mannschafts-ID '{}' and Mannschafts-Sortierung '{}'.",
                 maSortierungDTO.getId(), maSortierungDTO.getSortierung());
 
+        // Bei laufender Veranstaltung darf der Tabellenplatz nicht geaendert werden.
+        final DsbMannschaftDO existingMannschaft = dsbMannschaftComponent.findById(maSortierungDTO.getId());
+        assertVeranstaltungNotLaufend(existingMannschaft.getVeranstaltungId());
+
         final DsbMannschaftDO newDsbMannschaftDO = MannschaftSortierungDTOMapper.toDO.apply(maSortierungDTO);
         final long userId = UserProvider.getCurrentUserId(principal);
 
         final DsbMannschaftDO updatedDsbMannschaftDO = maSortierungComponent.updateSortierung(newDsbMannschaftDO, userId);
         return MannschaftSortierungDTOMapper.toDTO.apply(updatedDsbMannschaftDO);
     }
+
+    /**
+     * Stellt sicher, dass die Veranstaltung nicht in der Phase 'Laufend' ist.
+     * Bei {@code null} (keine Veranstaltung zugeordnet) passiert nichts.
+     *
+     * @param veranstaltungsId id der zu pruefenden Veranstaltung (darf null sein)
+     * @throws BusinessException wenn die Veranstaltung in der Phase 'Laufend' ist
+     */
+    private void assertVeranstaltungNotLaufend(final Long veranstaltungsId) {
+        if (veranstaltungsId != null && veranstaltungComponent.isVeranstaltungLaufend(veranstaltungsId)) {
+            throw new BusinessException(ErrorCode.ENTITY_CONFLICT_ERROR, ERROR_MSG_VERANSTALTUNG_LAUFEND);
+        }
+    }
 }