c8h4/aux-search
1
0
Fork 0
Code Issues Pull requests Packages Releases Activity

space should present an OR operation in query (#150)

Browse source 
filtering part of the query was messed up when I added search over multiple
fields.

fixes #147
fixes #149
This commit is contained in:
Rok Garbas 2020-08-21 21:23:21 +02:00 committed by GitHub
parent 818074f626
commit 91527f1deb
Failed to generate hash of commit
2 changed files with 51 additions and 68 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/Page/Options.elm
View file

@ -165,8 +165,8 @@ viewResultItem channel show item =
-- DEBUG: |> List.append -- DEBUG: |> List.append
-- DEBUG: [ tr [] -- DEBUG: [ tr []
-- DEBUG: [ td [ colspan 1 ] -- DEBUG: [ td [ colspan 1 ]
-- DEBUG: [ p [] [ text <| "score: " ++ String.fromFloat item.score ] -- DEBUG: [ div [] [ text <| "score: " ++ String.fromFloat (Maybe.withDefault 0 item.score) ]
-- DEBUG: , p [] -- DEBUG: , div []
-- DEBUG: [ text <| -- DEBUG: [ text <|
-- DEBUG: "matched queries: " -- DEBUG: "matched queries: "
-- DEBUG: , ul [] -- DEBUG: , ul []

41
src/Search.elm
View file

@ -737,23 +737,21 @@ filter_by_type type_ =
filter_by_query : filter_by_query :
List String List String
-> String -> String
-> List (List ( String, Json.Encode.Value )) -> ( String, Json.Encode.Value )
filter_by_query fields queryRaw = filter_by_query fields queryRaw =
let let
query = query =
queryRaw queryRaw
|> String.trim |> String.trim
in in
query ( "bool"
|> String.replace "." " " , Json.Encode.object
[ ( "should"
, Json.Encode.list Json.Encode.object
(query
|> String.words |> String.words
|> List.indexedMap |> List.indexedMap
(\i query_word -> (\i query_word ->
let
isLast =
List.length (String.words query) == i + 1
in
if isLast then
[ ( "bool" [ ( "bool"
, Json.Encode.object , Json.Encode.object
[ ( "should" [ ( "should"
@ -766,7 +764,7 @@ filter_by_query fields queryRaw =
, Json.Encode.object , Json.Encode.object
[ ( "query", Json.Encode.string query_word ) [ ( "query", Json.Encode.string query_word )
, ( "fuzziness", Json.Encode.string "1" ) , ( "fuzziness", Json.Encode.string "1" )
, ( "_name", Json.Encode.string <| "filter_queries_" ++ String.fromInt (i + 1) ++ "_should_match" ) , ( "_name", Json.Encode.string <| "filter_queries_" ++ String.fromInt i ++ "_should_match" )
] ]
) )
] ]
@ -793,25 +791,11 @@ filter_by_query fields queryRaw =
] ]
) )
] ]
)
else )
List.map
(\field ->
( "match_bool_prefix"
, Json.Encode.object
[ ( field
, Json.Encode.object
[ ( "query", Json.Encode.string query_word )
, ( "_name"
, Json.Encode.string <| "filter_queries_" ++ String.fromInt (i + 1) ++ "_prefix"
) )
] ]
) )
]
)
)
fields
)
makeRequestBody : makeRequestBody :
@ -849,10 +833,9 @@ makeRequestBody query from sizeRaw sort type_ sort_field query_fields should_que
, Json.Encode.object , Json.Encode.object
[ ( "filter" [ ( "filter"
, Json.Encode.list Json.Encode.object , Json.Encode.list Json.Encode.object
(List.append [ [ filter_by_type type_ ]
[ [ filter_by_type type_ ] ] , [ filter_by_query query_fields query ]
(filter_by_query query_fields query) ]
)
) )
, ( "should" , ( "should"
, Json.Encode.list Json.Encode.object should_queries , Json.Encode.list Json.Encode.object should_queries