Merge pull request #647 from bettercodepaul/swt2#2186

fix: verständliche Fehlermeldung bei voller Veranstaltung (swt2#2186)

Author: Cenk7

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

@@ -47,7 +47,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";
@@ -361,9 +361,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 '{}',",
 
@@ -642,8 +642,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-common/src/main/java/de/bogenliga/application/common/errorhandling/ErrorCode.java

@@ -28,6 +28,7 @@ public enum ErrorCode {
     // precondition checks
     INVALID_ARGUMENT_ERROR(ErrorCategory.BUSINESS, "INVALID_ARGUMENT_ERROR"),
     MATCH_INKONSISTENTE_SCHUETZEN(ErrorCategory.BUSINESS, "MATCH_INKONSISTENTE_SCHUETZEN"),
+    VERANSTALTUNG_MAX_CAPACITY_ERROR(ErrorCategory.BUSINESS, "VERANSTALTUNG_MAX_CAPACITY_ERROR"),
     // database
     ENTITY_NOT_FOUND_ERROR(ErrorCategory.BUSINESS, "ENTITY_NOT_FOUND_ERROR"),
     ENTITY_CONFLICT_ERROR(ErrorCategory.BUSINESS, "ENTITY_CONFLICT_ERROR"),