Browse Source

remove return status, shuffle if statement

pull/28/head
Kat 4 years ago
parent
commit
3805a177c9
  1. 10
      app/helpers/tasklist_helper.rb

10
app/helpers/tasklist_helper.rb

@ -33,15 +33,13 @@ module TasklistHelper
def get_next_incomplete_section(form, case_log) def get_next_incomplete_section(form, case_log)
subsections = form.all_subsections.keys subsections = form.all_subsections.keys
return subsections.find { |subsection| is_incomplete?(subsection, case_log, form.questions_for_subsection(subsection).keys) } subsections.find { |subsection| is_incomplete?(subsection, case_log, form.questions_for_subsection(subsection).keys) }
end end
def get_sections_count(form, case_log, status = :all) def get_sections_count(form, case_log, status = :all)
subsections = form.all_subsections.keys subsections = form.all_subsections.keys
if status == :all return subsections.count if status == :all
return subsections.count subsections.count { |subsection| get_subsection_status(subsection, case_log, form.questions_for_subsection(subsection).keys) == status }
end
return subsections.count { |subsection| get_subsection_status(subsection, case_log, form.questions_for_subsection(subsection).keys) == status }
end end
private private
@ -51,6 +49,6 @@ module TasklistHelper
def is_incomplete?(subsection, case_log, questions) def is_incomplete?(subsection, case_log, questions)
status = get_subsection_status(subsection, case_log, questions) status = get_subsection_status(subsection, case_log, questions)
return status == :not_started || status == :in_progress status == :not_started || status == :in_progress
end end
end end

Loading…
Cancel
Save