Closes Taskana#2555 - Fix deleteClassification

ryzheboka · Apr 2, 2024 · a67dc52 · a67dc52
1 parent 6ddb86f
commit a67dc52
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/...ana-core/src/main/java/pro/taskana/classification/internal/ClassificationServiceImpl.java b/...ana-core/src/main/java/pro/taskana/classification/internal/ClassificationServiceImpl.java
@@ -154,6 +154,7 @@ public void deleteClassification(String classificationId)
         if (isReferentialIntegrityConstraintViolation(e)) {
           throw new ClassificationInUseException(classification, e);
         }
+        throw e;
       }
     } finally {
       taskanaEngine.returnConnection();

diff --git a/...a-rest-spring/src/main/java/pro/taskana/classification/rest/ClassificationController.java b/...a-rest-spring/src/main/java/pro/taskana/classification/rest/ClassificationController.java
@@ -191,7 +191,7 @@ public ResponseEntity<ClassificationRepresentationModel> updateClassification(
    * @throws NotAuthorizedException if the user is not authorized to delete a Classification
    */
   @DeleteMapping(path = RestEndpoints.URL_CLASSIFICATIONS_ID)
-  @Transactional(readOnly = true, rollbackFor = Exception.class)
+  @Transactional(rollbackFor = Exception.class)
   public ResponseEntity<ClassificationRepresentationModel> deleteClassification(
       @PathVariable String classificationId)
       throws ClassificationNotFoundException, ClassificationInUseException, NotAuthorizedException {