From 28921345e8ecc18008e2d19bd21c5aef3b1a2e3d Mon Sep 17 00:00:00 2001
From: gazebo <gazeboxu@gmail.com>
Date: Sun, 1 Jul 2018 10:11:28 +0800
Subject: [PATCH] - more error and warn output.

---
 business/elm/controller/order.go | 4 ++--
 business/freshfood/freshfood.go  | 2 +-
 business/jd/controller/order.go  | 4 ++--
 business/tasks/configrefresh.go  | 4 ++--
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/business/elm/controller/order.go b/business/elm/controller/order.go
index fd3900813..f2204496d 100644
--- a/business/elm/controller/order.go
+++ b/business/elm/controller/order.go
@@ -48,7 +48,7 @@ func (o *OrderController) NewOrder(msg *elmapi.CallbackMsg, orderId string) *elm
 
 	created, _, err := db.ReadOrCreate(rec, "OrderId")
 	if err != nil {
-		globals.SugarLogger.Warnf("error when calling ReadOrCreate, error:%v", err)
+		globals.SugarLogger.Warnf("error when calling ReadOrCreate, error:%v, rec:%v", err, rec)
 		return errResponseDBError
 	}
 	if created || rec.Type != msg.Type {
@@ -74,7 +74,7 @@ func (o *OrderController) OrderStatusChanged(msg *elmapi.CallbackMsg, orderId st
 
 	err := db.Read(rec, "OrderId")
 	if err != nil {
-		globals.SugarLogger.Warnf("error when calling ReadOrCreate, error:%v", err)
+		globals.SugarLogger.Warnf("error when calling ReadOrCreate, error:%v, rec:%v", err, rec)
 	} else {
 		if rec.Type != msg.Type {
 			err = globals2.FreshFoodAPI.ELMOrderStatus(orderId, msg.Type, utils.GetCurTimeStr())
diff --git a/business/freshfood/freshfood.go b/business/freshfood/freshfood.go
index b897730b7..e14c6fd46 100644
--- a/business/freshfood/freshfood.go
+++ b/business/freshfood/freshfood.go
@@ -128,7 +128,7 @@ func (f *FreshFoodAPI) AccessFreshFood(apiStr string, params url.Values) error {
 		}
 	}
 	if err != nil || response.StatusCode != 200 {
-		globals.SugarLogger.Errorf("Call %s error:%v, response: %v", fullURL, err, response)
+		globals.SugarLogger.Errorf("Call %s error:%v, url:%s, response: %v", fullURL, err, fullURL, response)
 		if err != nil {
 			return err
 		}
diff --git a/business/jd/controller/order.go b/business/jd/controller/order.go
index 398f79f88..30c45b6fe 100644
--- a/business/jd/controller/order.go
+++ b/business/jd/controller/order.go
@@ -124,7 +124,7 @@ func (c *OrderController) OrderStatus(msg *jdapi.CallbackOrderMsg) *jdapi.Callba
 						addOrderMsg(msg)
 					}
 				} else {
-					globals.SugarLogger.Warnf("duplicated jd order msg %v", msg)
+					globals.SugarLogger.Warnf("duplicated jd order msg:%v", msg)
 				}
 			}
 		} else {
@@ -205,7 +205,7 @@ func normalOrderStatus(msg *jdapi.CallbackOrderMsg) (err error) {
 
 		err = db.Read(rec, "JdOrderId")
 		if err != nil {
-			globals.SugarLogger.Warnf("error when accessing db rec:%v, err:%v", rec, err)
+			globals.SugarLogger.Warnf("error when accessing db err:%v, rec:%v", err, rec)
 		} else {
 			if rec.OrderStatus == int(utils.Str2Int64(msg.StatusID)) {
 				globals.SugarLogger.Infof("Duplicate message msg:%v", msg)
diff --git a/business/tasks/configrefresh.go b/business/tasks/configrefresh.go
index 1a410b948..cacc366e5 100644
--- a/business/tasks/configrefresh.go
+++ b/business/tasks/configrefresh.go
@@ -45,7 +45,7 @@ func RefreshConfig(configKey string, expiresTime time.Duration, configGetter fun
 			handleType := 0
 			if err := db.Read(curConfig, "Thirdparty"); err != nil {
 				if err != orm.ErrNoRows {
-					globals.SugarLogger.Error("db error:%v", err)
+					globals.SugarLogger.Errorf("db error:%v, curConfig:%v", err, curConfig)
 				} else {
 					curConfig.Token = configGetter()
 					handleType = 2
@@ -70,7 +70,7 @@ func RefreshConfig(configKey string, expiresTime time.Duration, configGetter fun
 					num, err = db.Insert(curConfig)
 				}
 				if err != nil || num == 0 {
-					globals.SugarLogger.Errorf("db error:%v, num:%d", err, num)
+					globals.SugarLogger.Errorf("db error:%v, num:%d, curConfig:%v", err, num, curConfig)
 				} else {
 					configSetter(curConfig.Token)
 				}