Merge branch 'dev6.8.5-cancelorder' into test6.8.5

@@ -2,7 +2,6 @@ package com.yohoufo.order.service.impl;
 
 
 import com.google.common.base.Verify;
 import com.google.common.base.Verify;
 import com.yoho.error.ServiceError;
 import com.yoho.error.ServiceError;
-import com.yoho.error.exception.ServiceException;
 import com.yohobuy.ufo.model.order.common.OrderStatus;
 import com.yohobuy.ufo.model.order.common.OrderStatus;
 import com.yohoufo.common.alarm.EventBusPublisher;
 import com.yohoufo.common.alarm.EventBusPublisher;
 import com.yohoufo.common.alarm.SmsAlarmEvent;
 import com.yohoufo.common.alarm.SmsAlarmEvent;
@@ -30,11 +29,13 @@ import java.math.BigDecimal;
 import java.util.Objects;
 import java.util.Objects;
 import java.util.function.BiConsumer;
 import java.util.function.BiConsumer;
 
 
+import static com.yohoufo.order.utils.ServiceExceptions.throwServiceException;
+
 /**
 /**
  * @author LUOXC
  * @author LUOXC
  * @date 2019/1/8 14:07
  * @date 2019/1/8 14:07
  */
  */
-@Slf4j
+@Slf4j(topic = "orderCloseLog")
 class BuyerOrderCancelHandler {
 class BuyerOrderCancelHandler {
 
 
     private final int uid;
     private final int uid;
@@ -65,8 +66,6 @@ class BuyerOrderCancelHandler {
     private BuyerRefundCouponEvent.BizCase refundCouponCase;
     private BuyerRefundCouponEvent.BizCase refundCouponCase;
 
 
 
 
-
-
     private AlarmConfig failAlarm;
     private AlarmConfig failAlarm;
 
 
     BuyerOrderCancelHandler(int uid, long orderCode) {
     BuyerOrderCancelHandler(int uid, long orderCode) {
@@ -132,12 +131,18 @@ class BuyerOrderCancelHandler {
     }
     }
 
 
     public void cancel() {
     public void cancel() {
+        log.info("cancel order {} from {} to {}", orderCode, expectStatus, targetStatus);
         verify();
         verify();
         // 获取买家订单
         // 获取买家订单
         BuyerOrder buyerOrder = buyerOrderMapper.selectByOrderCodeUid(orderCode, uid);
         BuyerOrder buyerOrder = buyerOrderMapper.selectByOrderCodeUid(orderCode, uid);
+        if (Objects.isNull(buyerOrder)) {
+            log.info("cancel order {} fail, order is not exist", orderCode);
+            throwServiceException(ServiceError.ORDER_NOT_EXIST);
+        }
         // 当前状态不可取消订单
         // 当前状态不可取消订单
         if (expectStatus.getCode() != buyerOrder.getStatus()) {
         if (expectStatus.getCode() != buyerOrder.getStatus()) {
-            throw new ServiceException(ServiceError.ORDER_SERVICE_ERROR);
+            log.info("cancel order {} fail, order status is not expect", orderCode);
+            throwServiceException("当前状态不可取消订单");
         }
         }
         // 更新订单状态成功
         // 更新订单状态成功
         if (updateBuyerOrderStatus()) {
         if (updateBuyerOrderStatus()) {
@@ -145,6 +150,9 @@ class BuyerOrderCancelHandler {
             transfer();
             transfer();
             refundPayGoodsMoney(buyerOrder);
             refundPayGoodsMoney(buyerOrder);
             noticeSeller();
             noticeSeller();
+        } else {
+            log.info("cancel order {} fail, order status has changed", orderCode);
+            throwServiceException("当前状态不可取消订单");
         }
         }
     }
     }
 
 
@@ -180,6 +188,7 @@ class BuyerOrderCancelHandler {
                 .uid(uid)
                 .uid(uid)
                 .orderCode(orderCode)
                 .orderCode(orderCode)
                 .build();
                 .build();
+        log.info("refund coupon by order {} request is {}", orderCode, buyerRefundCouponEvent);
         EventBusPublisher.publishEvent(buyerRefundCouponEvent);
         EventBusPublisher.publishEvent(buyerRefundCouponEvent);
     }
     }
 
 
@@ -193,6 +202,7 @@ class BuyerOrderCancelHandler {
                 .transferCase(transferCase)
                 .transferCase(transferCase)
                 .swdType(sellerWalletDetailType)
                 .swdType(sellerWalletDetailType)
                 .build();
                 .build();
+        log.info("transfer by order {} request is {}", orderCode, transferMoneyRequest);
         transferService.transfer(transferMoneyRequest);
         transferService.transfer(transferMoneyRequest);
     }
     }
 
 
@@ -215,9 +225,11 @@ class BuyerOrderCancelHandler {
                 .refundCase(goodsMoneyRefundCase)
                 .refundCase(goodsMoneyRefundCase)
                 .amount(goodsMoney);
                 .amount(goodsMoney);
         try {
         try {
+            log.info("refund pay goods money by order {} request is {} ", orderCode, paymentRequest);
             goodsMoneyRefundConsumer.accept(paymentRequest, billLogEventBuilder.skup(getSkup()));
             goodsMoneyRefundConsumer.accept(paymentRequest, billLogEventBuilder.skup(getSkup()));
-        } catch (Exception ex) {
-            log.warn("pay refund fail,req {}", paymentRequest, ex);
+            log.info("refund pay goods money by order {} success", orderCode);
+        } catch (Exception e) {
+            log.info("refund pay goods money by order {} fail request is {}", orderCode, paymentRequest, e);
             BillLogEvent billLogEvent = billLogEventBuilder.tradeStatus(BillTradeStatus.FAIL.getCode()).build();
             BillLogEvent billLogEvent = billLogEventBuilder.tradeStatus(BillTradeStatus.FAIL.getCode()).build();
             EventBusPublisher.publishEvent(billLogEvent);
             EventBusPublisher.publishEvent(billLogEvent);
 
 
@@ -235,10 +247,11 @@ class BuyerOrderCancelHandler {
     private void noticeSeller() {
     private void noticeSeller() {
         if (Objects.nonNull(sellerNoticeSender)) {
         if (Objects.nonNull(sellerNoticeSender)) {
             try {
             try {
+                log.info("notice seller buyer order has closed by order {}", orderCode);
                 SellerOrderGoods sog = getSellerOrderGoods();
                 SellerOrderGoods sog = getSellerOrderGoods();
                 sellerNoticeSender.notice(sog, orderCode, sellerNoticeSenderTimes);
                 sellerNoticeSender.notice(sog, orderCode, sellerNoticeSenderTimes);
-            } catch (Exception ex) {
-                log.warn("seller deliver notice fail,times {} order code is {}", sellerNoticeSenderTimes, orderCode, ex);
+            } catch (Exception e) {
+                log.info("notice seller buyer order has closed by order {} fail", orderCode, e);
             }
             }
         }
         }
     }
     }

+package com.yohoufo.order.utils;
+
+import com.yoho.error.ServiceError;
+import com.yoho.error.exception.ServiceException;
+
+/**
+ * @author LUOXC
+ * @date 2019/1/9 11:14
+ */
+public class ServiceExceptions {
+
+    public static void throwServiceException(String message) {
+        ServiceException serviceException = new ServiceException(ServiceError.ORDER_SERVICE_ERROR);
+        serviceException.setParams(message);
+        throw serviceException;
+    }
+
+    public static void throwServiceException(ServiceError serviceError) {
+        throw new ServiceException(serviceError);
+    }
+
+}