From 44dfe1250c81fdbf16cc6a10208ea91502ec9dfe Mon Sep 17 00:00:00 2001
From: Qiuyi LI <qiuyi.li@gmail.com>
Date: Thu, 26 Jun 2025 16:02:47 +0200
Subject: [PATCH] Hotfix: Use continue instead of break when parsing orders in
 AddGiftJob

---
 .../org/jeecg/modules/business/domain/job/AddGiftJob.java    | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/domain/job/AddGiftJob.java b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/domain/job/AddGiftJob.java
index d07a64cf5..7180ec357 100644
--- a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/domain/job/AddGiftJob.java
+++ b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/domain/job/AddGiftJob.java
@@ -177,7 +177,8 @@ public class AddGiftJob implements Job {
                         .filter(orderItem -> !orderItem.getStatus().equalsIgnoreCase(OBSOLETE_STATUS_CODE))
                         .collect(groupingBy(orderItem -> giftSkuSet.contains(orderItem.getErpCode())));
                 if (orderItemMap.isEmpty()) {
-                    break;
+                    log.info("No item in order {}, skipping", order.getPlatformOrderId());
+                    continue;
                 }
                 for (OrderItem orderItem : orderItemMap.get(Boolean.FALSE)) {
                     String erpCode = orderItem.getErpCode();
@@ -196,7 +197,7 @@ public class AddGiftJob implements Job {
                         }
                     }
                 }
-                log.debug("Order {} 's new gift map : ", newGiftMap);
+                log.info("Order {} 's new gift map : {}", order.getPlatformOrderId(), newGiftMap);
                 HashSet<Triple<String, String, Integer>> oldSkuData = new HashSet<>();
                 HashMap<String, Integer> oldGiftMap = new HashMap<>();
                 List<OrderItem> oldGifts = orderItemMap.get(Boolean.TRUE) == null ? new ArrayList<>() : orderItemMap.get(Boolean.TRUE);