From 8f05d8d546d56631e4ec748a4c24e71f870d778f Mon Sep 17 00:00:00 2001
From: mpostma <postma.marin@protonmail.com>
Date: Thu, 5 Aug 2021 15:43:48 +0200
Subject: [PATCH] fix clippy warnings

---
 Cargo.lock                                             |  2 +-
 .../src/index_controller/dump_actor/loaders/v1.rs      | 10 +++++-----
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/Cargo.lock b/Cargo.lock
index f9ea7184c..c1adeeabc 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -1700,7 +1700,7 @@ dependencies = [
 [[package]]
 name = "meilisearch-tokenizer"
 version = "0.2.3"
-source = "git+https://github.com/meilisearch/Tokenizer.git?tag=v0.2.3#c2399c3f879144ad92e20ae057e14984dfd22781"
+source = "git+https://github.com/meilisearch/tokenizer.git?tag=v0.2.3#c2399c3f879144ad92e20ae057e14984dfd22781"
 dependencies = [
  "character_converter",
  "cow-utils",
diff --git a/meilisearch-http/src/index_controller/dump_actor/loaders/v1.rs b/meilisearch-http/src/index_controller/dump_actor/loaders/v1.rs
index 034e464b1..30923584e 100644
--- a/meilisearch-http/src/index_controller/dump_actor/loaders/v1.rs
+++ b/meilisearch-http/src/index_controller/dump_actor/loaders/v1.rs
@@ -144,17 +144,17 @@ impl From<Settings> for index_controller::Settings<Unchecked> {
             // String
             filterable_attributes: settings.attributes_for_faceting.map(|o| o.map(|vec| vec.into_iter().collect())),
             // we need to convert the old `Vec<String>` into a `BTreeSet<String>`
-            ranking_rules: settings.ranking_rules.map(|o| o.map(|vec| vec.into_iter().filter_map(|criterion| {
+            ranking_rules: settings.ranking_rules.map(|o| o.map(|vec| vec.into_iter().filter(|criterion| {
                 match criterion.as_str() {
-                    "words" | "typo" | "proximity" | "attribute" | "exactness" => Some(criterion),
-                    s if s.starts_with("asc") || s.starts_with("desc") => Some(criterion),
+                    "words" | "typo" | "proximity" | "attribute" | "exactness" => true,
+                    s if s.starts_with("asc") || s.starts_with("desc") => true,
                     "wordsPosition" => {
                         warn!("The criteria `attribute` and `wordsPosition` have been merged into a single criterion `attribute` so `wordsPositon` will be ignored");
-                        None
+                        false
                     }
                     s => {
                         error!("Unknown criterion found in the dump: `{}`, it will be ignored", s);
-                        None
+                        false
                     }
                     }
                 }).collect())),