Collapse two redundant helpers

Fold one-used event_status_icon and one-used booth_status_icon into
status_icon, used twice; since there is no conflict on the status-
driven description.
This commit is contained in:
James Mason 2018-06-01 12:01:58 -07:00 committed by Christian Bruckmayer
parent 940012314d
commit fe3c6c499b
3 changed files with 5 additions and 18 deletions

View file

@ -4,24 +4,11 @@ module FormatHelper
##
# Includes functions related to formatting (like adding classes, colors)
##
def event_status_icon(event)
case event.state
when 'new'
'fa-eye'
when 'unconfirmed'
'fa-check text-muted'
when 'confirmed'
'fa-check text-success'
when 'rejected', 'withdrawn', 'canceled'
'fa-ban'
end
end
def booth_status_icon(booth)
case booth.state
def status_icon(object)
case object.state
when 'new', 'to_reject', 'to_accept'
'fa-eye'
when 'accepted'
when 'unconfirmed', 'accepted'
'fa-check text-muted'
when 'confirmed'
'fa-check text-success'

View file

@ -24,7 +24,7 @@
- show_state = 'new'
- else
- show_state = booth.state
%span{ title: show_state, class: "fa #{booth_status_icon(booth)}" }
%span{ title: show_state, class: "fa #{status_icon(booth)}" }
%td
- if booth.logo_link
= image_tag(booth.picture.thumb.url, width: '20%')

View file

@ -67,7 +67,7 @@
- @events.each do |event|
%tr
%td{style: "padding:20px 8px 20px 8px;"}
%span{ title: event.state.humanize, class: "fa #{event_status_icon(event)}" }
%span{ title: event.state.humanize, class: "fa #{status_icon(event)}" }
%td.col-md-7{style: "padding:20px 8px 20px 8px;"}
= link_to event.title, conference_program_proposal_path(@conference.short_title, event.id)