diff --git a/modules/items/api_sellItem.go b/modules/items/api_sellItem.go
index 4f2afcd8b..048bf773a 100644
--- a/modules/items/api_sellItem.go
+++ b/modules/items/api_sellItem.go
@@ -35,7 +35,7 @@ func (this *apiComp) SellItem(session comm.IUserSession, req *pb.ItemsSellItemRe
 		code = pb.ErrorCode_ConfigurationException
 		return
 	}
-	if req.Amount < item.Amount {
+	if req.Amount > item.Amount {
 		code = pb.ErrorCode_ReqParameterError
 		this.module.Errorf("SellItemCheck over all amount:[%d:%d]", req.Amount, item.Amount)
 		return
@@ -49,17 +49,20 @@ func (this *apiComp) SellItem(session comm.IUserSession, req *pb.ItemsSellItemRe
 		return
 	}
 	item.Amount = item.Amount - req.Amount
-	if item.Amount == 0 {
-		if err = this.module.modelItems.DelUserPack(session.GetUserId(), item.GridId); err != nil {
-			code = pb.ErrorCode_DBError
-			return
-		}
-	} else {
-		if err = this.module.modelItems.UpdateUserPack(session.GetUserId(), item); err != nil {
-			code = pb.ErrorCode_DBError
-			return
-		}
+	if code = this.module.AddItemforGrid(&comm.ModuleCallSource{}, session, req.GridId, -1*int32(req.Amount), true); code != pb.ErrorCode_Success {
+		return
 	}
+	// if item.Amount == 0 {
+	// 	if err = this.module.modelItems.DelUserPack(session.GetUserId(), item.GridId); err != nil {
+	// 		code = pb.ErrorCode_DBError
+	// 		return
+	// 	}
+	// } else {
+	// 	if err = this.module.modelItems.UpdateUserPack(session.GetUserId(), item); err != nil {
+	// 		code = pb.ErrorCode_DBError
+	// 		return
+	// 	}
+	// }
 	session.SendMsg(string(this.module.GetType()), "sellitem", &pb.ItemsSellItemResp{GridId: req.GridId, Amount: req.Amount, Issucc: true})
 	return
 }