Better use merge! instead of q = q.merge

This commit is contained in:
Jean-Baptiste Bellet
2023-07-12 09:38:07 +02:00
parent 755a1842c2
commit b3bdba3a9b

View File

@@ -106,13 +106,13 @@ class ProductsReflex < ApplicationReflex
def ransack_query
query = { s: "name desc" }
query = query.merge({ supplier_id_in: @producer_id }) if @producer_id.present?
query.merge!(supplier_id_in: @producer_id) if @producer_id.present?
if @search_term.present?
# rubocop:disable Layout/LineLength
query = query.merge({ name_or_meta_keywords_or_variants_display_as_or_variants_display_name_or_supplier_name_cont: @search_term })
query.merge!(name_or_meta_keywords_or_variants_display_as_or_variants_display_name_or_supplier_name_cont: @search_term)
# rubocop:enable Layout/LineLength
end
query = query.merge({ primary_taxon_id_in: @category_id }) if @category_id.present?
query.merge!(primary_taxon_id_in: @category_id) if @category_id.present?
query
end