Merged in r2-2966-numeric-family-number (pull request #6896)

R2-2966 - Cannot search for family records by all-numeric Family Numbers

Approved-by: Pavel Nabutovsky

Author: dhernandez-quoin

app/services/search_filter_service.rb

@@ -5,12 +5,14 @@
 # Transform a params hash into SearchFilters::SearchFiklter objects
 class SearchFilterService
   EXCLUDED = %w[format controller action page per order order_by fields id_search].freeze
+  FILTERABLE_MODELS = [Child, Incident, TracingRequest, RegistryRecord, Family].freeze
 
   def self.build_filters(params, permitted_field_names)
     service = SearchFilterService.new
     filter_params = service.select_filter_params(params, permitted_field_names)
     filter_params = DestringifyService.destringify(filter_params.to_h, true)
-    service.build_filters(filter_params)
+    id_params = service.select_id_params(params)
+    service.build_filters(filter_params.merge(id_params))
   end
 
   # rubocop:disable Metrics/AbcSize
@@ -46,7 +48,16 @@ def build_filters(params)
   # rubocop:enable Metrics/PerceivedComplexity
 
   def select_filter_params(params, permitted_field_names)
-    filter_params = params.except(*EXCLUDED)
+    excluded_params = EXCLUDED + filterable_id_fields
+    filter_params = params.except(*excluded_params)
     filter_params.select { |key, _| permitted_field_names.any? { |name| key.match?(/#{name}[0-5]?$/) } }
   end
+
+  def select_id_params(params)
+    params.select { |key, _| filterable_id_fields.include?(key) }
+  end
+
+  def filterable_id_fields
+    @filterable_id_fields = FILTERABLE_MODELS.map(&:filterable_id_fields).flatten
+  end
 end

spec/mailers/record_action_mailer_spec.rb

@@ -416,7 +416,7 @@
       end
     end
 
-    describe 'transition request with diferent locale' do
+    describe 'transition request with different locale' do
       before do
         @user1.locale = 'ar-LB'
         @user1.save(validate: false)
@@ -432,7 +432,12 @@
 
       it 'renders the body' do
         expect(mail.text_part.body.encoded)
-          .to match('مستخدم بريميرو user2 من مؤسسة Test Agency يطلب أن تنقل ملكية هذا السجل من نوع Case ومعرّف')
+          .to match(
+            %(مستخدم بريميرو user2 من الهيئة Test Agency يطلب أن تنقل ملكية هذا السجل من نوع Case ومعرّف
+              <a href=\"https://localhost:3000/v2/cases/#{@case.id}\">#{@case.short_id}</a>
+              بحيث يتمكنون من تقديم خدمات إدارة للشخص في منطقتهم. إذا كان التحويل مقبولا من طرفك، يرجى النقر على رابط معرف نوع السجل
+              Case في هذا الايميل لفتح نوع السجل Case في بريميرو والبدء بعملية التحويل.).squish
+            )
       end
     end