diff --git a/src/main/java/org/folio/circulation/resources/foruseatlocation/HoldByBarcodeRequest.java b/src/main/java/org/folio/circulation/resources/foruseatlocation/HoldByBarcodeRequest.java index b51e16072e..5fba844c3a 100644 --- a/src/main/java/org/folio/circulation/resources/foruseatlocation/HoldByBarcodeRequest.java +++ b/src/main/java/org/folio/circulation/resources/foruseatlocation/HoldByBarcodeRequest.java @@ -8,9 +8,9 @@ import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; import org.folio.Environment; -import org.folio.circulation.domain.Item; import org.folio.circulation.domain.Loan; import org.folio.circulation.domain.ServicePoint; +import org.folio.circulation.domain.representations.ItemSummaryRepresentation; import org.folio.circulation.support.BadRequestFailure; import org.folio.circulation.support.HttpFailure; import org.folio.circulation.support.results.Result; @@ -92,15 +92,13 @@ static Supplier forUseAtLocationIsNotEnabledFailure() { public JsonObject responseBody() { JsonObject body = new JsonObject(); - body.put("loan", loan.asJson()); - JsonObject itemAsJson = new JsonObject(); - Item item = getLoan().getItem(); - itemAsJson.put("title", item.getTitle()); - JsonObject location = new JsonObject(); - location.put("id", item.getLocation().getId()); - location.put("name", item.getLocation().getName()); - itemAsJson.put("location", location); - body.getJsonObject("loan").put("item", itemAsJson); + JsonObject loanJson = loan.asJson(); + JsonObject itemJson = new ItemSummaryRepresentation().createItemSummary(loan.getItem()); + + loanJson.put("item", itemJson); + body.put("loan", loanJson); + return body; } + } diff --git a/src/main/java/org/folio/circulation/resources/foruseatlocation/HoldByBarcodeResource.java b/src/main/java/org/folio/circulation/resources/foruseatlocation/HoldByBarcodeResource.java index 8908524f0d..4fea1988af 100644 --- a/src/main/java/org/folio/circulation/resources/foruseatlocation/HoldByBarcodeResource.java +++ b/src/main/java/org/folio/circulation/resources/foruseatlocation/HoldByBarcodeResource.java @@ -161,7 +161,7 @@ private Result setActionToHeld(Result> updateLoan(HoldByBarcodeRequest request, LoanRepository loanRepository) { - log.info("updateLoan:: parameters: loan {}", request.getLoan().asJson()); + log.debug("updateLoan:: parameters: loan {}", request.getLoan().asJson()); return loanRepository.updateLoan(request.getLoan()) .thenApply(loanResult -> loanResult.map(request::withLoan)); }