Always return nil for empty values (#11)

Don't sometimes return []any{} instead. Keep the API consistent.

Author: erikdubbelboer

filter/converter.go

@@ -53,14 +53,18 @@ func (c *Converter) Convert(query []byte, startAtParameterIndex int) (conditions
 		return "", nil, fmt.Errorf("startAtParameterIndex must be greater than 0")
 	}
 
+	if len(query) == 0 {
+		return c.emptyCondition, nil, nil
+	}
+
 	var mongoFilter map[string]any
 	err = json.Unmarshal(query, &mongoFilter)
 	if err != nil {
 		return "", nil, err
 	}
 
 	if len(mongoFilter) == 0 {
-		return c.emptyCondition, []any{}, nil
+		return c.emptyCondition, nil, nil
 	}
 
 	conditions, values, err = c.convertFilter(mongoFilter, startAtParameterIndex)

filter/converter_test.go

@@ -198,7 +198,7 @@ func TestConverter_Convert(t *testing.T) {
 			nil,
 			`{}`,
 			`FALSE`,
-			[]any{},
+			nil,
 			nil,
 		}, {
 			"empty or conditions",