From be558d5c9a6fb9f962d9716a6a0805702667baff Mon Sep 17 00:00:00 2001 From: Gauthier LO Date: Wed, 10 Jan 2024 11:38:16 +0100 Subject: [PATCH] fix : display shipping invoiced orders with invoiceable purchase fees + replace purchaseOrder status with paidAmount --- .../shippingInvoice/InvoiceController.java | 133 ++++++++++-------- .../client/TransactionController.java | 2 +- .../business/entity/PurchaseOrder.java | 13 +- .../business/mapper/PlatformOrderMapper.java | 2 + .../mapper/xml/PlatformOrderMapper.xml | 31 +++- .../service/IPlatformOrderService.java | 11 +- .../impl/PlatformOrderServiceImpl.java | 5 + .../business/vo/PlatformOrderFront.java | 12 ++ 8 files changed, 142 insertions(+), 67 deletions(-) diff --git a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/controller/admin/shippingInvoice/InvoiceController.java b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/controller/admin/shippingInvoice/InvoiceController.java index c5c4db0c1..22df5e56b 100644 --- a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/controller/admin/shippingInvoice/InvoiceController.java +++ b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/controller/admin/shippingInvoice/InvoiceController.java @@ -468,7 +468,7 @@ public class InvoiceController { ((LoginUser) SecurityUtils.getSubject().getPrincipal()).getUsername(), shopIds); List shopIdList = Arrays.asList(shopIds.split(",")); - List orders = platformOrderService.findUninvoicedOrdersByShopForClient(shopIdList, Collections.singletonList(1)); + List orders = platformOrderService.findUninvoicedShippingOrdersByShopForClient(shopIdList, Collections.singletonList(1)); IPage page = new Page<>(); page.setRecords(orders); @@ -486,11 +486,11 @@ public class InvoiceController { log.info("User : {} is requesting uninvoiced orders for shops : [{}]", ((LoginUser) SecurityUtils.getSubject().getPrincipal()).getUsername(), shopIds); - List> ordersAndStatus = new ArrayList<>(); - List> finalOrderAndStatus = new ArrayList<>(); + // list of orders and their status : // checking shipping data availability List shopIdList = Arrays.asList(shopIds.split(",")); + // fetch order that can be invoice either by shipping or purchase or both List orders = platformOrderService.findUninvoicedOrdersByShopForClient(shopIdList, Collections.singletonList(1)); if(orders.isEmpty()) return Result.OK("No order to invoice."); @@ -498,74 +498,93 @@ public class InvoiceController { List orderIds = orders.stream().map(PlatformOrder::getId).collect(Collectors.toList()); Map> orderContentMap = platformOrderService.fetchOrderData(orderIds); + Map errorMapToOrderId = new HashMap<>(); + List orderFronts = new ArrayList<>(); + for(Map.Entry> entry : orderContentMap.entrySet()) { PlatformOrderFront orderFront = new PlatformOrderFront(); - boolean hasError = false; - //rename shop id by shop name to prevent it to leak in front - entry.getKey().setShopId(shops.get(entry.getKey().getShopId())); - // checks if logistic channel is missing - if(entry.getKey().getLogisticChannelName().isEmpty() && entry.getKey().getInvoiceLogisticChannelName() == null) { - ordersAndStatus.add(MutableTriple.of(entry.getKey(), "Error : Missing logistic channel for order : " + entry.getKey().getPlatformOrderId(), "")); - hasError = true; - } + BeanUtils.copyProperties(entry.getKey(), orderFront); + //rename shop id by shop name to prevent it to leak in front + orderFront.setShopId(shops.get(orderFront.getShopId())); + // set default value of shipping and purchase availability + orderFront.setShippingAvailable("0"); + orderFront.setPurchaseAvailable("0"); List skuIds = entry.getValue().stream().map(PlatformOrderContent::getSkuId).distinct().collect(Collectors.toList()); // finds the first sku that isn't in db List skuIdsFound = skuService.listByIds(skuIds); if(skuIdsFound.size() != skuIds.size()) { - if(ordersAndStatus.stream().noneMatch(order -> order.getLeft().getId().equals(entry.getKey().getId()))) - ordersAndStatus.add(MutableTriple.of(entry.getKey(), "Error : Missing one or more sku in db for order : " + entry.getKey().getPlatformOrderId(), "Error : Missing one or more sku in db for order : " + entry.getKey().getPlatformOrderId())); - else { - ordersAndStatus.get(ordersAndStatus.size() - 1).setMiddle((ordersAndStatus.get(ordersAndStatus.size() - 1).getMiddle() + " and missing one or more sku in db for order : " + entry.getKey().getPlatformOrderId())); - ordersAndStatus.get(ordersAndStatus.size() - 1).setRight((ordersAndStatus.get(ordersAndStatus.size() - 1).getRight() + " and missing one or more sku in db for order : " + entry.getKey().getPlatformOrderId())); + if(!errorMapToOrderId.containsKey(entry.getKey().getPlatformOrderId())) + errorMapToOrderId.put(entry.getKey().getPlatformOrderId(), "Error : Missing one or more sku in db for order : " + entry.getKey().getPlatformOrderId()); + else + errorMapToOrderId.put(entry.getKey().getPlatformOrderId(), errorMapToOrderId.get(entry.getKey().getPlatformOrderId()) + " and missing one or more sku in db"); + + orderFront.setShippingAvailable("-1"); + orderFront.setPurchaseAvailable("-1"); + } + + if(entry.getKey().getShippingInvoiceNumber() == null) { + // checks if logistic channel is missing + if(entry.getKey().getLogisticChannelName().isEmpty() && entry.getKey().getInvoiceLogisticChannelName() == null) { + if(!errorMapToOrderId.containsKey(entry.getKey().getPlatformOrderId())) + errorMapToOrderId.put(entry.getKey().getPlatformOrderId(), "Error : Missing logistic channel for order : " + entry.getKey().getPlatformOrderId()); + else + errorMapToOrderId.put(entry.getKey().getPlatformOrderId(), errorMapToOrderId.get(entry.getKey().getPlatformOrderId()) + " and missing logistic channel"); + orderFront.setShippingAvailable("-1"); + } + // finds the first product with missing weight + String missingWeightProductId = productService.searchFirstEmptyWeightProduct(skuIds); + if(missingWeightProductId != null) { + if(!errorMapToOrderId.containsKey(entry.getKey().getPlatformOrderId())) + errorMapToOrderId.put(entry.getKey().getPlatformOrderId(), "Error : Missing one or more weight for order : " + entry.getKey().getPlatformOrderId()); + else + errorMapToOrderId.put(entry.getKey().getPlatformOrderId(), errorMapToOrderId.get(entry.getKey().getPlatformOrderId()) + " and missing weight"); + orderFront.setShippingAvailable("-1"); + } + } + if(entry.getKey().getPurchaseInvoiceNumber() == null) { + // finds the first sku with missing price + String missingPriceSkuId = skuService.searchFirstMissingPriceSku(skuIds); + if(missingPriceSkuId != null) { + if(!errorMapToOrderId.containsKey(entry.getKey().getPlatformOrderId())) + errorMapToOrderId.put(entry.getKey().getPlatformOrderId(), "Error : Missing one or more sku price for order : " + entry.getKey().getPlatformOrderId()); + else + errorMapToOrderId.put(entry.getKey().getPlatformOrderId(), errorMapToOrderId.get(entry.getKey().getPlatformOrderId()) + " and missing one or more sku price"); + orderFront.setPurchaseAvailable("-1"); } - hasError = true; } - // finds the first product with missing weight - String missingWeightProductId = productService.searchFirstEmptyWeightProduct(skuIds); - if(missingWeightProductId != null) { - if(ordersAndStatus.stream().noneMatch(order -> order.getLeft().getId().equals(entry.getKey().getId()))) - ordersAndStatus.add(MutableTriple.of(entry.getKey(), "Error : Missing one or more weight for order : " + entry.getKey().getPlatformOrderId(), "")); - else - ordersAndStatus.get(ordersAndStatus.size() - 1).setMiddle((ordersAndStatus.get(ordersAndStatus.size() - 1).getMiddle() + " and missing weight for order : " + entry.getKey().getPlatformOrderId())); - hasError = true; + // set purchase order status (-1 = unavailable, 0 = available, 1 = invoiced, 2 = paid) + if(entry.getKey().getProductAvailable() == null) { + orderFront.setProductAvailable("0"); + entry.getKey().setProductAvailable("0"); } - // finds the first sku with missing price - String missingPriceSkuId = skuService.searchFirstMissingPriceSku(skuIds); - if(missingPriceSkuId != null) { - if(ordersAndStatus.stream().noneMatch(order -> order.getLeft().getId().equals(entry.getKey().getId()))) - ordersAndStatus.add(MutableTriple.of(entry.getKey(), "OK", "Error : Missing one or more sku price for order : " + entry.getKey().getPlatformOrderId())); - else - ordersAndStatus.get(ordersAndStatus.size() - 1).setRight( ordersAndStatus.get(ordersAndStatus.size() -1).getRight() + "and missing one or more sku price for order : " + entry.getKey().getPlatformOrderId()); - hasError = true; - } - if(!hasError) ordersAndStatus.add(MutableTriple.of(entry.getKey(), "OK", "OK")); - - if(entry.getKey().getProductAvailable().equals("0") && entry.getKey().getVirtualProductAvailable().equals("1")) - entry.getKey().setProductAvailable("2"); + if(entry.getKey().getProductAvailable().equals("0") && entry.getKey().getVirtualProductAvailable().equals("1") && entry.getKey().getPurchaseInvoiceNumber() == null) + orderFront.setProductAvailable("2"); if(entry.getKey().getPurchaseInvoiceNumber() != null) { PurchaseOrder purchase = purchaseOrderService.getPurchaseByInvoiceNumber(entry.getKey().getPurchaseInvoiceNumber()); - if(purchase.getStatus().equals("1")) - entry.getKey().setProductAvailable("3"); + if(purchase.getPaidAmount().compareTo(BigDecimal.ZERO) == 0) + orderFront.setPurchaseAvailable("1");// invoiced else - entry.getKey().setProductAvailable("4"); + orderFront.setPurchaseAvailable("2");// paid } - BeanUtils.copyProperties(entry.getKey(), orderFront); - finalOrderAndStatus.add(MutableTriple.of(orderFront, ordersAndStatus.get(ordersAndStatus.size() - 1).getMiddle(), ordersAndStatus.get(ordersAndStatus.size() - 1).getRight())); + // set shipping order status (-1 = unavailable, 0 = available, 1 = invoiced, 2 = paid) + if(entry.getKey().getShippingInvoiceNumber() != null) { + ShippingInvoice shippingInvoice = iShippingInvoiceService.getShippingInvoice(entry.getKey().getShippingInvoiceNumber()); + if(shippingInvoice.getPaidAmount().compareTo(BigDecimal.ZERO) == 0) { + orderFront.setShippingAvailable("1"); // invoiced + } + else { + orderFront.setShippingAvailable("2"); // paid + } + } + orderFronts.add(orderFront); } - List errorMessages = new ArrayList<>(); - for(MutableTriple orderAndStatus : finalOrderAndStatus) { - if(!orderAndStatus.getMiddle().equals("OK")) { - errorMessages.add(orderAndStatus.getMiddle()); - } - if(!orderAndStatus.getRight().equals("OK")) { - errorMessages.add(orderAndStatus.getRight()); - } - } + List errorMessages = new ArrayList<>(errorMapToOrderId.values()); + // sorting by order time ascending - finalOrderAndStatus = finalOrderAndStatus.stream().sorted(Comparator.comparing(o -> o.getLeft().getOrderTime())).collect(Collectors.toList()); + orderFronts = orderFronts.stream().sorted(Comparator.comparing(PlatformOrderFront::getOrderTime)).collect(Collectors.toList()); // system notification String errors = SECTION_START; int max_entries = 100; @@ -584,14 +603,14 @@ public class InvoiceController { templateParam.put("errors", errors); templateParam.put("current_page", String.valueOf(current_page)); templateParam.put("total_page", String.valueOf(total_page)); - TemplateMessageDTO message = new TemplateMessageDTO("admin", "admin", "Self Service invoicing estimation Errors", templateParam, "expenses_overview_errors"); + TemplateMessageDTO message = new TemplateMessageDTO("admin", "Gauthier", "Self Service invoicing estimation Errors", templateParam, "expenses_overview_errors"); ISysBaseApi.sendTemplateAnnouncement(message); } } - IPage> page = new Page<>(); - page.setRecords(finalOrderAndStatus); - page.setTotal(finalOrderAndStatus.size()); + IPage page = new Page<>(); + page.setRecords(orderFronts); + page.setTotal(orderFronts.size()); return Result.OK(page); } diff --git a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/controller/client/TransactionController.java b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/controller/client/TransactionController.java index a7b10e04d..236a25b85 100644 --- a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/controller/client/TransactionController.java +++ b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/controller/client/TransactionController.java @@ -92,7 +92,7 @@ public class TransactionController { public Result debit(@RequestParam("clientId") String clientId, @RequestParam("currency") String currency) { List errorMessages = new ArrayList<>(); List shopIds = shopService.listIdByClient(clientId); - List orders = platformOrderService.findUninvoicedOrdersByShopForClient(shopIds, Arrays.asList(1,2,3)); + List orders = platformOrderService.findUninvoicedShippingOrdersByShopForClient(shopIds, Arrays.asList(1,2,3)); if(orders.isEmpty()) return Result.OK("No order to invoice."); Date startDate = orders.stream().map(PlatformOrder::getOrderTime).min(Date::compareTo).get(); diff --git a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/entity/PurchaseOrder.java b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/entity/PurchaseOrder.java index c05fae5e4..4c1821854 100644 --- a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/entity/PurchaseOrder.java +++ b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/entity/PurchaseOrder.java @@ -90,7 +90,12 @@ public class PurchaseOrder implements Serializable { @Excel(name = "最终金额", width = 15) @ApiModelProperty(value = "最终金额") private java.math.BigDecimal finalAmount; - + /** + * paid amount + */ + @Excel(name = "已付金额", width = 15) + @ApiModelProperty(value = "已付金额") + private java.math.BigDecimal paidAmount; /** * 订单发票号 */ @@ -98,12 +103,6 @@ public class PurchaseOrder implements Serializable { @ApiModelProperty(value = "订单发票号") private String invoiceNumber; - /** - * Purchase status - */ - @Excel(name = "status", width = 15) - @ApiModelProperty(value = "status") - private String status; /** * Payment document diff --git a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/mapper/PlatformOrderMapper.java b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/mapper/PlatformOrderMapper.java index 0508b74a5..153bdb06d 100644 --- a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/mapper/PlatformOrderMapper.java +++ b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/mapper/PlatformOrderMapper.java @@ -185,6 +185,7 @@ public interface PlatformOrderMapper extends BaseMapper { void cancelInvoice(@Param("invoiceNumber") String invoiceNumber); void cancelBatchInvoice(@Param("invoiceNumbers") List invoiceNumbers); + List findUninvoicedShippingOrdersByShopForClient(@Param("shopIds") List shopIds, @Param("erpStatuses") List erpStatuses); List findUninvoicedOrdersByShopForClient(@Param("shopIds") List shopIds, @Param("erpStatuses") List erpStatuses); List findUninvoicedOrderIdsByShopForClient(@Param("shopIds") List shopIds, @Param("erpStatuses") List erpStatuses); @@ -209,4 +210,5 @@ public interface PlatformOrderMapper extends BaseMapper { List fetchShippingFeeBillableOrders(); List getPlatformOrdersByInvoiceNumber(@Param("invoiceNumber") String invoiceNumber); + } diff --git a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/mapper/xml/PlatformOrderMapper.xml b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/mapper/xml/PlatformOrderMapper.xml index c9dcd2b91..f0ad180d0 100644 --- a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/mapper/xml/PlatformOrderMapper.xml +++ b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/mapper/xml/PlatformOrderMapper.xml @@ -548,7 +548,7 @@ WHERE erp_status IN (4,5) AND order_time < #{endDate}; - SELECT * FROM platform_order WHERE erp_status IN @@ -575,6 +575,35 @@ #{shopId} ; + INSERT INTO platform_order_delete(id, create_by, create_time, update_by, diff --git a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/service/IPlatformOrderService.java b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/service/IPlatformOrderService.java index 900055a72..7f38de0e7 100644 --- a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/service/IPlatformOrderService.java +++ b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/service/IPlatformOrderService.java @@ -180,11 +180,19 @@ public interface IPlatformOrderService extends IService { void cancelBatchInvoice(List invoiceNumbers); /** - * Find all order that can be invoiced themselves. + * Find all order that can be invoiced (shipping only). * @param shopIds list of shop id * @param erpStatuses list of erp_status * @return list of orders */ + List findUninvoicedShippingOrdersByShopForClient(List shopIds, List erpStatuses); + + /** + * Find all order that can be invoiced (shipping and purchase). + * @param shopIds + * @param erpStatuses + * @return + */ List findUninvoicedOrdersByShopForClient(List shopIds, List erpStatuses); /** * Get ids of all order that can be invoiced by small clients (type 2) themselves. @@ -219,4 +227,5 @@ public interface IPlatformOrderService extends IService { */ List fetchShippingFeeBillableOrders(); List getPlatformOrdersByInvoiceNumber(String invoiceNumber); + } diff --git a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/service/impl/PlatformOrderServiceImpl.java b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/service/impl/PlatformOrderServiceImpl.java index b6ebfc842..61edab317 100644 --- a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/service/impl/PlatformOrderServiceImpl.java +++ b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/service/impl/PlatformOrderServiceImpl.java @@ -399,6 +399,11 @@ public class PlatformOrderServiceImpl extends ServiceImpl findUninvoicedShippingOrdersByShopForClient(List shopIds, List erpStatuses) { + return platformOrderMap.findUninvoicedShippingOrdersByShopForClient(shopIds, erpStatuses); + } + @Override public List findUninvoicedOrdersByShopForClient(List shopIds, List erpStatuses) { return platformOrderMap.findUninvoicedOrdersByShopForClient(shopIds, erpStatuses); diff --git a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/vo/PlatformOrderFront.java b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/vo/PlatformOrderFront.java index 4e27b86d9..29a2cc24f 100644 --- a/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/vo/PlatformOrderFront.java +++ b/jeecg-module-system/jeecg-system-biz/src/main/java/org/jeecg/modules/business/vo/PlatformOrderFront.java @@ -93,4 +93,16 @@ public class PlatformOrderFront { @Excel(name = "有货（1=有，0=没有）", width = 15) @ApiModelProperty(value = "有货（1=有，0=没有）") private String productAvailable; + /** + * 可开物流票（0=不可，1=可） + */ + @Excel(name = "可开物流票（0=不可，1=可）", width = 15) + @ApiModelProperty(value = "可开物流票（0=不可，1=可）") + private String shippingAvailable; + /** + * 可开采购票（0=不可，1=可） + */ + @Excel(name = "可开采购票（0=不可，1=可）", width = 15) + @ApiModelProperty(value = "可开采购票（0=不可，1=可）") + private String purchaseAvailable; }