Tmp release 24.3.9

NEWS.md

@@ -1,3 +1,6 @@
+## 24.3.9 2025-01-17
+* Support DCB re-request (CIRC-2198)
+
 ## 24.3.8 2025-01-14
 * Fix TLR fetching issue for search slips (CIRC-2197)
 

pom.xml

@@ -2,7 +2,7 @@
   <modelVersion>4.0.0</modelVersion>
   <artifactId>mod-circulation</artifactId>
   <groupId>org.folio</groupId>
-  <version>24.3.9-SNAPSHOT</version>
+  <version>24.3.10-SNAPSHOT</version>
   <licenses>
     <license>
       <name>Apache License 2.0</name>

ramls/request.json

@@ -437,6 +437,10 @@
     "itemLocationCode": {
       "description": "Allow specifying item location when creating title-level requests",
       "type": "string"
+    },
+    "isDcbReRequestCancellation": {
+      "description": "Indicates whether the request was cancelled during a DCB transaction update",
+      "type": "boolean"
     }
   },
   "additionalProperties": false,

src/main/java/org/folio/circulation/domain/Request.java

@@ -29,6 +29,7 @@
 import static org.folio.circulation.domain.representations.RequestProperties.REQUEST_LEVEL;
 import static org.folio.circulation.domain.representations.RequestProperties.REQUEST_TYPE;
 import static org.folio.circulation.domain.representations.RequestProperties.STATUS;
+import static org.folio.circulation.support.json.JsonPropertyFetcher.getBooleanProperty;
 import static org.folio.circulation.support.json.JsonPropertyFetcher.getDateTimeProperty;
 import static org.folio.circulation.support.json.JsonPropertyFetcher.getIntegerProperty;
 import static org.folio.circulation.support.json.JsonPropertyFetcher.getProperty;
@@ -427,6 +428,10 @@ public boolean hasLoan() {
     return loan != null;
   }
 
+  public boolean getDcbReRequestCancellationValue() {
+    return getBooleanProperty(requestRepresentation, "isDcbReRequestCancellation");
+  }
+
   public enum Operation {
     CREATE, REPLACE, MOVE;
   }

src/main/java/org/folio/circulation/resources/RequestNoticeSender.java

@@ -120,10 +120,12 @@ public Result<RequestAndRelatedRecords> sendNoticeOnRequestCancelled(
     log.debug("sendNoticeOnRequestCancelled:: parameters records: {}", () -> records);
     Request request = records.getRequest();
 
-    if (request.hasItemId()) {
-      sendCancellationNoticeForRequestWithItemId(request);
-    } else {
-      sendCancellationNoticeForRequestWithoutItemId(request);
+    if (!request.getDcbReRequestCancellationValue()) {
+      if (request.hasItemId()) {
+        sendCancellationNoticeForRequestWithItemId(request);
+      } else {
+        sendCancellationNoticeForRequestWithoutItemId(request);
+      }
     }
 
     return succeeded(records);

src/test/java/org/folio/circulation/resources/RequestNoticeSenderTest.java

@@ -0,0 +1,36 @@
+package org.folio.circulation.resources;
+
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.times;
+
+import org.folio.circulation.domain.Request;
+import org.folio.circulation.domain.RequestAndRelatedRecords;
+import org.folio.circulation.domain.notice.ImmediatePatronNoticeService;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
+import org.mockito.InjectMocks;
+import org.mockito.Mock;
+import org.mockito.Mockito;
+import org.mockito.junit.jupiter.MockitoExtension;
+
+import api.support.builders.RequestBuilder;
+import io.vertx.core.json.JsonObject;
+
+@ExtendWith(MockitoExtension.class)
+class RequestNoticeSenderTest {
+
+  @Mock
+  private ImmediatePatronNoticeService immediatePatronNoticeService;
+  @InjectMocks
+  private RequestNoticeSender requestNoticeSender;
+
+  @Test
+  void shouldNotSendNotificationWhenIsDcbCancellationTrue() {
+    JsonObject representation = new RequestBuilder().create();
+    representation.put("isDcbReRequestCancellation", true);
+    requestNoticeSender.sendNoticeOnRequestCancelled(
+      new RequestAndRelatedRecords(Request.from(representation)));
+    Mockito.verify(immediatePatronNoticeService, times(0)).acceptNoticeEvent(any());
+    Mockito.verify(immediatePatronNoticeService, times(0)).sendNotice(any(), any());
+  }
+}