From 78d0fb001532507826fedb3def4fe17d2e7c17f1 Mon Sep 17 00:00:00 2001
From: Nicolas Florentin <nicolas@sleede.com>
Date: Thu, 7 Dec 2023 09:33:56 +0100
Subject: [PATCH] makes member searches accent insensitive

---
 app/views/users/index.html.erb            |  2 +-
 app/views/users/manage.html.erb           |  2 +-
 config/initializers/ransack.rb            |  8 ++++++++
 spec/controllers/users_controller_spec.rb | 12 +++++++-----
 4 files changed, 17 insertions(+), 7 deletions(-)
 create mode 100644 config/initializers/ransack.rb

diff --git a/app/views/users/index.html.erb b/app/views/users/index.html.erb
index 47b7322e..bf73c674 100644
--- a/app/views/users/index.html.erb
+++ b/app/views/users/index.html.erb
@@ -11,7 +11,7 @@
   <div class="col-md-12">
     <%= search_form_for(@search, class: "navbar-form navbar-left", url: users_path) do |f| %>
       <div class="form-group">
-        <%= f.search_field :member_search_cont, value: params.dig(:q, :member_search_cont), class: "form-control" %>
+        <%= f.search_field :member_search_unaccent_cont, value: params.dig(:q, :member_search_unaccent_cont), class: "form-control" %>
       </div>
       <button class="btn btn-default" type="submit">
         <%= t 'global.search' %>
diff --git a/app/views/users/manage.html.erb b/app/views/users/manage.html.erb
index 8619f9d1..a60fe081 100644
--- a/app/views/users/manage.html.erb
+++ b/app/views/users/manage.html.erb
@@ -11,7 +11,7 @@
   <div class="col-md-12">
     <%= search_form_for(@search, class: "navbar-form navbar-left", url: manage_users_path) do |f| %>
       <div class="form-group">
-        <%= f.search_field :member_search_cont, value: params.dig(:q, :member_search_cont), class: "form-control" %>
+        <%= f.search_field :member_search_unaccent_cont, value: params.dig(:q, :member_search_unaccent_cont), class: "form-control" %>
       </div>
       <button class="btn btn-default" type="submit">
         <%= t 'global.search' %>
diff --git a/config/initializers/ransack.rb b/config/initializers/ransack.rb
new file mode 100644
index 00000000..145aa549
--- /dev/null
+++ b/config/initializers/ransack.rb
@@ -0,0 +1,8 @@
+Ransack.configure do |config|
+  config.add_predicate 'unaccent_cont',
+                       arel_predicate: 'matches',
+                       formatter: proc { |s| ActiveSupport::Inflector.transliterate("%#{s}%") },
+                       validator: proc { |s| s.present? },
+                       compounds: true,
+                       type: :string
+end
\ No newline at end of file
diff --git a/spec/controllers/users_controller_spec.rb b/spec/controllers/users_controller_spec.rb
index a677d6d1..b715a4f9 100644
--- a/spec/controllers/users_controller_spec.rb
+++ b/spec/controllers/users_controller_spec.rb
@@ -124,7 +124,7 @@
         user = Fabricate(:user, username: 'foo', email: 'foo@email.com')
         member = Fabricate(:member, user: user, organization: test_organization, member_uid: 1000)
 
-        get :index, params: { q: { member_search_cont: 1000 } }
+        get :index, params: { q: { member_search_unaccent_cont: 1000 } }
 
         expect(assigns(:members)).to include(member)
       end
@@ -133,7 +133,7 @@
         user = Fabricate(:user, username: 'foo', email: 'foo@email.com')
         member = Fabricate(:member, user: user, organization: test_organization, member_uid: 1000, tags: ["Boss"])
 
-        get :index, params: { q: { member_search_cont: "Bos" } }
+        get :index, params: { q: { member_search_unaccent_cont: "Bos" } }
 
         expect(assigns(:members)).to include(member)
       end
@@ -151,8 +151,10 @@
         user = Fabricate(:user, username: 'fôô', email: 'test@email.com')
         member = Fabricate(:member, user: user, organization: test_organization)
 
-        get :index, params: { q: { member_search_cont: "foo" } }
+        get :index, params: { q: { member_search_unaccent_cont: "foo" } }
+        expect(assigns(:members)).to include(member)
 
+        get :index, params: { q: { member_search_unaccent_cont: "föö" } }
         expect(assigns(:members)).to include(member)
       end
     end
@@ -200,7 +202,7 @@
         user = Fabricate(:user, phone: 123456789)
         member = Fabricate(:member, user: user, organization: test_organization)
 
-        get :manage, params: { q: { member_search_cont: 123456789 } }
+        get :manage, params: { q: { member_search_unaccent_cont: 123456789 } }
 
         expect(assigns(:members)).to include(member)
       end
@@ -209,7 +211,7 @@
         user = Fabricate(:user)
         member = Fabricate(:member, user: user, organization: test_organization, member_uid: 1000, tags: ["Boss"])
 
-        get :index, params: { q: { member_search_cont: "Bos" } }
+        get :index, params: { q: { member_search_unaccent_cont: "Bos" } }
 
         expect(assigns(:members)).to include(member)
       end