[#6441] Refactor: extract and move method

This commit is contained in:
Eric Davis
2011-08-10 15:15:26 -07:00
parent 2f77619fef
commit edc9e84719
2 changed files with 5 additions and 1 deletions
+1 -1
View File
@@ -26,7 +26,7 @@ module ContractsHelper
def grouped_deliverable_options_for_select(project, selected_key=nil)
project.contracts.all(:include => :deliverables).inject([]) do |html, contract|
if contract.closed? && !contract.deliverable_ids.include?(selected_key.to_i)
if contract.closed? && !contract.includes_deliverable_id?(selected_key)
# skip
else
options = contract.deliverables.collect do |deliverable|
+4
View File
@@ -74,6 +74,10 @@ class Contract < ActiveRecord::Base
self.status == "closed"
end
def includes_deliverable_id?(deliverable_id)
deliverable_ids.include?(deliverable_id.to_i)
end
# ------------------------------------------------------------
# Labor Methods
# ------------------------------------------------------------