出货订单修改海外仓属性值判断逻辑修正

yudao-module-order/yudao-module-order-core/src/main/java/cn/iocoder/yudao/module/order/service/order/impl/OrderServiceImpl.java

@@ -1903,7 +1903,7 @@ public class OrderServiceImpl extends AbstractService<OrderMapper, OrderDO> impl
             if (vo.getTransportId() == TransportTypeShortEnum.SEA_CONTAINER.getValue()) {
                 throw exception(ORDER_PRE_INSTALLATION_UNPACKAGED_CONTAINER_NOT_UPDATE);
             } else {
-                // TODO 当前不是海运就是空运
+                // 当前不是海运就是空运
                 throw exception(ORDER_SORTING_NOT_SHIPMENT_NOT_UPDATE);
             }
         }
@@ -1911,7 +1911,6 @@ public class OrderServiceImpl extends AbstractService<OrderMapper, OrderDO> impl
         if (updateReqVO.getTransportId() != vo.getTransportId()) {
             throw exception(NOT_UPDATE_TRANSPORT);
         }
-        // TODO 这里需要优化查询,这里查询了后面调用不建议重复查询，后面做优化
         LogisticsInfoDto logisticsInfoDto = warehouseLineMapper.getStartInfoAndDestInfoByLineId(updateReqVO.getLineId());
         LogisticsInfoDto orgLogisticsInfoDto = warehouseLineMapper.getStartInfoAndDestInfoByLineId(vo.getLineId());
         // 不允许修改始发地
@@ -1923,8 +1922,7 @@ public class OrderServiceImpl extends AbstractService<OrderMapper, OrderDO> impl
             throw exception(NOT_UPDATE_DEST_COUNTRY);
         }
         // 海外仓业务变动申请
-        if (!vo.getType().contains("2") && updateReqVO.getType().contains("2")) {
-            // TODO 订单如果转海外仓需要单独申请， 订单如果是海外仓改为非海外仓呢？
+        if ((!vo.getType().contains("2") && updateReqVO.getType().contains("2")) || (vo.getType().contains("2") && !updateReqVO.getType().contains("2"))) {
             throw exception(ORDER_OVERSEAS_WAREHOUSE_UPDATE_NEED_APPLY);
         }
         // 当是否控货从是修改成否，需校验订单是否有放货记录状态≠已取消，有则弹出提示“订单已放货，无法修改为不控货”
@@ -1940,7 +1938,7 @@ public class OrderServiceImpl extends AbstractService<OrderMapper, OrderDO> impl
         }
         // 当是否控货从否修改成是，校验订单是否有提货记录，有则弹出提示“订单已有提货记录，无法修改成控货”
         if (!vo.getIsCargoControl() && updateReqVO.getIsCargoControl()) {
-            // TODO 校验订单是否有提货记录
+            // 校验订单是否有提货记录
             long pickupCount = orderPickupMapper.selectCount(new LambdaQueryWrapper<OrderPickupDO>()
                     .eq(OrderPickupDO::getOrderId, vo.getOrderNo()));
             if (pickupCount > 0) {
@@ -2170,7 +2168,7 @@ public class OrderServiceImpl extends AbstractService<OrderMapper, OrderDO> impl
                 }
             }
             if (isShipment) {
-                // TODO 出货阶段的修改，需要将当前订单状态值标出，放首位
+                //  出货阶段的修改，需要将当前订单状态值标出，放首位
                 ApplyInfoVO applyInfoVO = new ApplyInfoVO();
                 applyInfoVO.setName(statusName);
                 applyInfoList.addFirst(applyInfoVO);