[MODFISTO-513] - Used similar logic in other methods

folio-org · Dec 24, 2024 · 1bd77aa · 1bd77aa
1 parent ec16f13
commit 1bd77aa
Showing 1 changed file with 4 additions and 10 deletions.
diff --git a/src/main/java/org/folio/rest/exception/HttpException.java b/src/main/java/org/folio/rest/exception/HttpException.java
@@ -30,9 +30,10 @@ public HttpException(int code, String message) {
   public HttpException(int code, String message, Throwable cause) {
     super(message, cause);
     this.code = code;
+    var ec = code == 409 ? CONFLICT : GENERIC_ERROR_CODE;
     Parameter causeParam = new Parameter().withKey("cause").withValue(cause.getMessage());
     Error error = new Error()
-      .withCode(ErrorCodes.GENERIC_ERROR_CODE.getCode())
+      .withCode(ec.getCode())
       .withMessage(message)
       .withParameters(List.of(causeParam));
     this.errors = new Errors()
@@ -43,17 +44,10 @@ public HttpException(int code, String message, Throwable cause) {
   public HttpException(int code, Throwable cause) {
     super(cause.getMessage(), cause);
     this.code = code;
+    var ec = code == 409 ? CONFLICT : GENERIC_ERROR_CODE;
     this.errors = new Errors()
-      .withErrors(List.of(new Error().withCode(ErrorCodes.GENERIC_ERROR_CODE.getCode()).withMessage(cause.getMessage())))
-      .withTotalRecords(1);
-  }
-
-  public HttpException(int code, ErrorCodes errCodes) {
-    super(errCodes.getDescription());
-    this.errors = new Errors()
-      .withErrors(List.of(new Error().withCode(errCodes.getCode()).withMessage(errCodes.getDescription())))
+      .withErrors(List.of(new Error().withCode(ec.getCode()).withMessage(cause.getMessage())))
       .withTotalRecords(1);
-    this.code = code;
   }
 
   public HttpException(int code, Error error) {