From 20e91ede4e68271026947699a317f9d4fa1bf885 Mon Sep 17 00:00:00 2001 From: Sylvain Date: Mon, 11 Mar 2019 12:47:46 +0100 Subject: [PATCH] Limit members search to 50 results to speed up queries --- CHANGELOG.md | 1 + app/services/members/list_service.rb | 1 + 2 files changed, 2 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 23ec96aee..e64c30a1f 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -5,6 +5,7 @@ - Fix a bug: application in unavailable if a SSO is active - Fix a security issue: dependency bootstrap < 4.3.1 has an XSS vulnerability as described in [CVE-2019-8331](https://blog.getbootstrap.com/2019/02/13/bootstrap-4-3-1-and-3-4-1/) - Fixed missing translations in authentication providers form +- Limit members search to 50 results to speed up queries - [TODO DEPLOY] `bundle install` ## v2.8.3 2019 January 29 diff --git a/app/services/members/list_service.rb b/app/services/members/list_service.rb index 428675304..9fe939826 100644 --- a/app/services/members/list_service.rb +++ b/app/services/members/list_service.rb @@ -38,6 +38,7 @@ class Members::ListService 'FROM "subscriptions" ' \ 'WHERE "user_id" = "users"."id")') .where("users.is_active = 'true' AND roles.name = 'member'") + .limit(50) query.downcase.split(' ').each do |word| members = members.where('lower(f_unaccent(profiles.first_name)) ~ :search OR ' \ 'lower(f_unaccent(profiles.last_name)) ~ :search',