mirror of
https://github.com/meilisearch/meilisearch.git
synced 2025-01-18 08:48:32 +08:00
Resolve code modification suggestions
This commit is contained in:
parent
5012cc3a32
commit
1c0a5cd136
@ -240,9 +240,7 @@ impl FacetCondition {
|
||||
let value = items.next().unwrap();
|
||||
let (result, svalue) = pest_parse(value);
|
||||
|
||||
// TODO we must normalize instead of lowercase.
|
||||
let svalue = svalue.to_lowercase();
|
||||
|
||||
Ok(Operator(fid, Equal(result.ok(), svalue)))
|
||||
}
|
||||
|
||||
|
@ -98,7 +98,6 @@ impl<'t, 'u, 'i> Facets<'t, 'u, 'i> {
|
||||
self.index.put_string_faceted_documents_ids(self.wtxn, field_id, &string_documents_ids)?;
|
||||
self.index.put_number_faceted_documents_ids(self.wtxn, field_id, &number_documents_ids)?;
|
||||
|
||||
// Store the
|
||||
write_into_lmdb_database(
|
||||
self.wtxn,
|
||||
*self.index.facet_id_f64_docids.as_polymorph(),
|
||||
|
@ -816,7 +816,6 @@ fn extract_facet_values(value: &Value) -> (Vec<f64>, Vec<String>) {
|
||||
output_numbers.push(float);
|
||||
},
|
||||
Value::String(string) => {
|
||||
// TODO must be normalized and not only lowercased.
|
||||
let string = string.trim().to_lowercase();
|
||||
output_strings.push(string);
|
||||
},
|
||||
|
Loading…
Reference in New Issue
Block a user