diff --git a/app/helpers/contracts_helper.rb b/app/helpers/contracts_helper.rb index 4af0e28..b511a0a 100644 --- a/app/helpers/contracts_helper.rb +++ b/app/helpers/contracts_helper.rb @@ -23,15 +23,19 @@ module ContractsHelper if contract.closed? && !contract.includes_deliverable_id?(selected_key) html else - options = contract.deliverables.collect do |deliverable| - deliverable_option(deliverable, selected_key) - end - - html << content_tag(:optgroup, options.join("\n"), :label => h(contract.name)) + html << content_tag(:optgroup, + deliverable_options_for_contract(contract, selected_key).join("\n"), + :label => h(contract.name)) end end end + def deliverable_options_for_contract(contract, selected_key) + contract.deliverables.collect do |deliverable| + deliverable_option(deliverable, selected_key) + end + end + def deliverable_option(deliverable, selected_key) option_attributes = {} option_attributes[:value] = h(deliverable.id)