Use predicate instead of checking underlying structure

Since Rails 5.0 it returns empty array instead of nil.
This commit is contained in:
Artur Beljajev 2019-11-06 14:41:38 +02:00
parent abaa5e2f77
commit 77077ca196
2 changed files with 6 additions and 4 deletions

View file

@ -1,4 +1,5 @@
- panel_class = @domain.errors.messages[:admin_contacts] ? 'panel-danger' : 'panel-default' - admin_contacts_invalid = @domain.errors.include?(:admin_contacts)
- panel_class = admin_contacts_invalid ? 'panel-danger' : 'panel-default'
.panel{class: panel_class} .panel{class: panel_class}
.panel-heading.clearfix .panel-heading.clearfix
= t('.title') = t('.title')
@ -15,7 +16,7 @@
%td= link_to(ac, admin_contact_path(ac)) %td= link_to(ac, admin_contact_path(ac))
%td= ac.code %td= ac.code
%td= ac.email %td= ac.email
- if @domain.errors.messages[:admin_contacts] - if admin_contacts_invalid
%tfoot %tfoot
- @domain.errors.messages[:admin_contacts].each do |x| - @domain.errors.messages[:admin_contacts].each do |x|
%tr %tr

View file

@ -1,4 +1,5 @@
- panel_class = @domain.errors.messages[:tech_contacts] ? 'panel-danger' : 'panel-default' - tech_contacts_invalid = @domain.errors.include?(:tech_contacts)
- panel_class = tech_contacts_invalid ? 'panel-danger' : 'panel-default'
#tech_contacts.panel{class: panel_class} #tech_contacts.panel{class: panel_class}
.panel-heading.clearfix .panel-heading.clearfix
= t('.title') = t('.title')
@ -15,7 +16,7 @@
%td= link_to(tc, admin_contact_path(tc)) %td= link_to(tc, admin_contact_path(tc))
%td= tc.code %td= tc.code
%td= tc.email %td= tc.email
- if @domain.errors.messages[:tech_contacts] - if tech_contacts_invalid
%tfoot %tfoot
- @domain.errors.messages[:tech_contacts].each do |x| - @domain.errors.messages[:tech_contacts].each do |x|
%tr %tr