refactor: improve filter query handling in utility functions

- Added nil return for empty condition strings in GetFilterQueryFromConditionString and GetFilterFromConditionString functions to enhance error handling.
- Improved clarity and maintainability by ensuring consistent behavior when handling empty input conditions.
- Updated comments for better understanding of the changes made.
This commit is contained in:
Marvin Zhang
2025-03-17 14:20:19 +08:00
parent 4dace3ce8e
commit 657c73ad2f

View File

@@ -42,6 +42,9 @@ func GetFilterQueryFromListParams(params *GetListParams) (q bson.M, err error) {
}
func GetFilterQueryFromConditionString(condStr string) (q bson.M, err error) {
if condStr == "" {
return nil, nil
}
conditions, err := GetFilterFromConditionString(condStr)
if err != nil {
return nil, err
@@ -50,6 +53,9 @@ func GetFilterQueryFromConditionString(condStr string) (q bson.M, err error) {
}
func GetFilterFromConditionString(condStr string) (f *entity.Filter, err error) {
if condStr == "" {
return nil, nil
}
var conditions []*entity.Condition
if err := json.Unmarshal([]byte(condStr), &conditions); err != nil {
return nil, err