- 1
- 2
- 3
- 4
- 5
if html_content.product_id == 11 || html_content.product_id == 8 || html_content.product_id == 15 || html_content.product_id == 20
@terms_and_conditions = Page.find(212).contents
else
@terms_and_conditions = Page.find(213).contents
end
А вот эта проверка идентификатора продукта вобще по всему коду разбросана :-)
рефакторится где то так
def terms_and_conditions(product)
case product.terms_type
when 'type1'
Page.find(212).contents
when 'type2'
Page.find(213).contents
else
'invalid terms'
end
end
<%= terms_and_conditions(product) %>
смена логики на использование terms_type вместо id.
Page.terms_and_conditions medical_details_required?
end
def Page.terms_and_conditions(medical_detail s_required?)
medical_details_required? ? Page.find(212) : Page.find(213)
end