From 918561d6675c1a22460b9924d8de67f376070d65 Mon Sep 17 00:00:00 2001 From: Eric Davis Date: Wed, 1 Sep 2010 18:19:43 -0700 Subject: [PATCH] [#4420] Handle edge cases if a Retainer has no start or end date to extend. --- app/models/retainer_deliverable.rb | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/app/models/retainer_deliverable.rb b/app/models/retainer_deliverable.rb index 5b330ee..8598196 100644 --- a/app/models/retainer_deliverable.rb +++ b/app/models/retainer_deliverable.rb @@ -135,6 +135,8 @@ class RetainerDeliverable < HourlyDeliverable end def extend_period_to_new_end_date + return if end_date_change[0].nil? # No previous end date, so it will not have budgets + old_end_date = end_date_change[0] last_labor_budgets = labor_budgets.all(:conditions => {:year => old_end_date.year, :month => old_end_date.month}) last_overhead_budgets = overhead_budgets.all(:conditions => {:year => old_end_date.year, :month => old_end_date.month}) @@ -145,6 +147,8 @@ class RetainerDeliverable < HourlyDeliverable end def extend_period_to_new_start_date + return if start_date_change[0].nil? # No previous start date, so it will not have budgets + old_start_date = start_date_change[0] first_labor_budgets = labor_budgets.all(:conditions => {:year => old_start_date.year, :month => old_start_date.month}) first_overhead_budgets = overhead_budgets.all(:conditions => {:year => old_start_date.year, :month => old_start_date.month})