chore: use fmt.Errorf(...) instead of errors.New(fmt.Sprintf(...))

Signed-off-by: guoguangwu <guoguangwu@magic-shield.com>
pull/469/head
guoguangwu 2 years ago
parent ca071fdf68
commit e1ea9e33ee

@ -109,7 +109,7 @@ func (e *ExtendMsgSetMongoDriver) InsertOrUpdateReactionExtendMsgSet(ctx context
return utils.Wrap(err, "") return utils.Wrap(err, "")
} }
if set == nil { if set == nil {
return errors.New(fmt.Sprintf("conversationID %s has no set", conversationID)) return fmt.Errorf("conversationID %s has no set", conversationID)
} }
_, err = e.ExtendMsgSetCollection.UpdateOne(ctx, bson.M{"source_id": set.ConversationID, "session_type": sessionType}, bson.M{"$set": updateBson}, opt) _, err = e.ExtendMsgSetCollection.UpdateOne(ctx, bson.M{"source_id": set.ConversationID, "session_type": sessionType}, bson.M{"$set": updateBson}, opt)
return utils.Wrap(err, "") return utils.Wrap(err, "")
@ -126,7 +126,7 @@ func (e *ExtendMsgSetMongoDriver) DeleteReactionExtendMsgSet(ctx context.Context
return utils.Wrap(err, "") return utils.Wrap(err, "")
} }
if set == nil { if set == nil {
return errors.New(fmt.Sprintf("conversationID %s has no set", conversationID)) return fmt.Errorf("conversationID %s has no set", conversationID)
} }
_, err = e.ExtendMsgSetCollection.UpdateOne(ctx, bson.M{"source_id": set.ConversationID, "session_type": sessionType}, bson.M{"$unset": updateBson}) _, err = e.ExtendMsgSetCollection.UpdateOne(ctx, bson.M{"source_id": set.ConversationID, "session_type": sessionType}, bson.M{"$unset": updateBson})
return err return err
@ -149,5 +149,5 @@ func (e *ExtendMsgSetMongoDriver) TakeExtendMsg(ctx context.Context, conversatio
if v, ok := setList[0].ExtendMsgs[clientMsgID]; ok { if v, ok := setList[0].ExtendMsgs[clientMsgID]; ok {
return &v, nil return &v, nil
} }
return nil, errors.New(fmt.Sprintf("cant find client msg id: %s", clientMsgID)) return nil, fmt.Errorf("cant find client msg id: %s", clientMsgID)
} }

Loading…
Cancel
Save