diff --git a/business/jxstore/cms/sku.go b/business/jxstore/cms/sku.go
index 98d37cd92..9c3671788 100644
--- a/business/jxstore/cms/sku.go
+++ b/business/jxstore/cms/sku.go
@@ -289,13 +289,13 @@ func ReorderCategories(ctx *jxcontext.Context, parentID int, categoryIDs []int,
 				catsMap[v].Seq = k
 			}
 			catsMap[v].LastOperator = ctx.GetUserName()
-			globals.SugarLogger.Debugf("ReorderCategories2,[%v]", err)
 			if _, err = dao.UpdateEntity(db, catsMap[v]); err != nil {
+				globals.SugarLogger.Debugf("ReorderCategories2,[%v]", err)
 				dao.Rollback(db)
 				return err
 			}
-			globals.SugarLogger.Debugf("ReorderCategories3,[%v]", err)
 			if err = OnUpdateThing(ctx, db, nil, int64(catsMap[v].ID), model.ThingTypeCategory); err != nil {
+				globals.SugarLogger.Debugf("ReorderCategories3,[%v]", err)
 				dao.Rollback(db)
 				return err
 			}
@@ -309,7 +309,6 @@ func ReorderCategories(ctx *jxcontext.Context, parentID int, categoryIDs []int,
 			CurVendorSync.SyncStoresCategory(ctx, db, nil, nil, false, true, true)
 		}
 	}
-	globals.SugarLogger.Debugf("ReorderCategories1,[%v]", err)
 	return err
 }