diff --git a/src/main/java/de/metas/ui/web/pickingV2/productsToPick/process/ProductsToPick_4EyesReview_ProcessAll.java b/src/main/java/de/metas/ui/web/pickingV2/productsToPick/process/ProductsToPick_4EyesReview_ProcessAll.java index d9623b5d7..0cac7183d 100644 --- a/src/main/java/de/metas/ui/web/pickingV2/productsToPick/process/ProductsToPick_4EyesReview_ProcessAll.java +++ b/src/main/java/de/metas/ui/web/pickingV2/productsToPick/process/ProductsToPick_4EyesReview_ProcessAll.java @@ -68,7 +68,12 @@ protected ProcessPreconditionsResolution checkPreconditionsApplicable() return ProcessPreconditionsResolution.rejectWithInternalReason("not all rows were approved"); } - if (!getRowsNotAlreadyProcessed().stream().allMatch(this::isEligibleForProcessing)) + final List rows = getRowsNotAlreadyProcessed(); + if (rows.isEmpty()) + { + return ProcessPreconditionsResolution.rejectWithInternalReason("no unprocessed rows"); + } + if (!rows.stream().allMatch(this::isEligibleForProcessing)) { return ProcessPreconditionsResolution.rejectWithInternalReason("not all rows eligible for processing"); } @@ -86,6 +91,7 @@ protected String doIt() final List pickingCandidates = processAllPickingCandidates(); deliverAndInvoice(pickingCandidates); + return MSG_OK; }