Merge pull request #2633 from opencloud-eu/fix/search-preserve-value-case

fix(search): preserve value case for non-lowercased bleve fields

services/search/pkg/bleve/backend_test.go

@@ -137,6 +137,17 @@ var _ = Describe("Bleve", func() {
 				assertDocCount(rootResource.ID, "Size:<1000", 0)
 				assertDocCount(rootResource.ID, "Size:>100000", 0)
 			})
+
+			It("preserves value case for fields not explicitly marked lowercase", func() {
+				parentResource.Document.Audio = &libregraph.Audio{
+					Artist: libregraph.PtrString("Some Artist"),
+				}
+				err := eng.Upsert(parentResource.ID, parentResource)
+				Expect(err).ToNot(HaveOccurred())
+
+				assertDocCount(rootResource.ID, `audio.artist:"Some Artist"`, 1)
+				assertDocCount(rootResource.ID, `audio.artist:"some artist"`, 0)
+			})
 		})
 
 		Context("by filename", func() {

services/search/pkg/query/bleve/compiler.go

@@ -10,6 +10,17 @@ import (
 	"github.com/opencloud-eu/opencloud/pkg/kql"
 )
 
+// lowercaseFields lists the bleve fields whose index mapping uses a
+// lowercasing analyzer. Values bound to these fields are pre-lowercased
+// so query-side matching stays consistent with the index.
+// Keep in sync with services/search/pkg/bleve/index.go NewMapping.
+var lowercaseFields = map[string]struct{}{
+	"Name":      {},
+	"Tags":      {},
+	"Favorites": {},
+	"Content":   {},
+}
+
 var _fields = map[string]string{
 	"rootid":    "RootID",
 	"path":      "Path",
@@ -91,7 +102,7 @@ func walk(offset int, nodes []ast.Node) (bleveQuery.Query, int, error) {
 				v = bleveEscaper.Replace(n.Value)
 			}
 
-			if k != "Hidden" {
+			if _, ok := lowercaseFields[k]; ok {
 				v = strings.ToLower(v)
 			}
 

services/search/pkg/query/bleve/compiler_test.go

@@ -227,8 +227,8 @@ func Test_compile(t *testing.T) {
 				},
 			},
 			want: query.NewConjunctionQuery([]query.Query{
-				query.NewQueryStringQuery(`author:john\ smith`),
-				query.NewQueryStringQuery(`author:jane`),
+				query.NewQueryStringQuery(`author:John\ Smith`),
+				query.NewQueryStringQuery(`author:Jane`),
 			}),
 			wantErr: false,
 		},
@@ -249,8 +249,8 @@ func Test_compile(t *testing.T) {
 				},
 			},
 			want: query.NewConjunctionQuery([]query.Query{
-				query.NewQueryStringQuery(`author:john\ smith`),
-				query.NewQueryStringQuery(`author:jane`),
+				query.NewQueryStringQuery(`author:John\ Smith`),
+				query.NewQueryStringQuery(`author:Jane`),
 				query.NewQueryStringQuery(`Tags:bestseller`),
 			}),
 			wantErr: false,