fix

src/main/java/com/gic/jpa/service/impl/OrderServiceImpl.java

@@ -82,6 +82,7 @@ public class OrderServiceImpl extends BaseServiceImpl implements OrderService {
 
                 if (isJSON(order.getAmountPayableInfo()) && isJSON(order.getDiscountAmountInfo())) {
                     copyList.remove(order);
+                    continue;
                 }
                 order.setAmountPayableInfo(formatPayinfo(order.getAmountPayableInfo()));
                 order.setDiscountAmountInfo(formatDiscount(order.getDiscountAmountInfo()));
@@ -124,6 +125,7 @@ public class OrderServiceImpl extends BaseServiceImpl implements OrderService {
 
                 if (isJSON(order.getAmountPayableInfo()) && isJSON(order.getDiscountAmountInfo())) {
                     copyList.remove(order);
+                    continue;
                 }
                 order.setAmountPayableInfo(formatPayinfo(order.getAmountPayableInfo()));
                 order.setDiscountAmountInfo(formatDiscount(order.getDiscountAmountInfo()));
@@ -209,7 +211,7 @@ public class OrderServiceImpl extends BaseServiceImpl implements OrderService {
     @Override
     public void formatLogistics() {
 
-        handlerOrderLogistics();
+//        handlerOrderLogistics();
         handlerOrderRefundLogistics();
     }
 
@@ -227,16 +229,14 @@ public class OrderServiceImpl extends BaseServiceImpl implements OrderService {
             CopyOnWriteArrayList<OrderLogistics> copyList = new CopyOnWriteArrayList(page.getContent());
 
             for (OrderLogistics order : copyList) {
-                if (isJSON(order.getLogisticsInformation()) ) {
-                    copyList.remove(order);
-                }
+
                 order.setLogisticsInformation(formateTraces(order.getLogisticsInformation()));
             }
 
-            if (CollectionUtils.isEmpty(copyList)) {
-                continue;
+            if (!CollectionUtils.isEmpty(copyList)) {
+                orderLogisticsRepository.save(copyList);
             }
-            orderLogisticsRepository.save(copyList);
+
         }
     }
 
@@ -254,9 +254,7 @@ public class OrderServiceImpl extends BaseServiceImpl implements OrderService {
             CopyOnWriteArrayList<OrderRefundLogistics> copyList = new CopyOnWriteArrayList(page.getContent());
 
             for (OrderRefundLogistics order : copyList) {
-                if (isJSON(order.getLogisticsInformation()) ) {
-                    copyList.remove(order);
-                }
+
                 order.setLogisticsInformation(formateTraces(order.getLogisticsInformation()));
             }
 
@@ -284,6 +282,7 @@ public class OrderServiceImpl extends BaseServiceImpl implements OrderService {
         try {
             trace.put("Traces", JSON.parseArray(info));
         }catch (Exception e){
+            e.printStackTrace();
             logger.info("source:{}",info);
             trace.put("Traces", new JSONArray());
         }

src/test/java/com/gic/jpa/OrderJsonFormatTest.java

@@ -28,6 +28,11 @@ public class OrderJsonFormatTest extends  GicJpaDemoApplicationTests{
     }
 
     @Test
+    public void testLogistics() {
+        format.formatLogistics();
+    }
+
+    @Test
     public void formatOrderTest() {
         format.formatOrder();
         format.formatOrderItem();