From 994e9b0332e9b652e60d96f331487cd62844aea1 Mon Sep 17 00:00:00 2001
From: Eric Davis <edavis@littlestreamsoftware.com>
Date: Wed, 1 Sep 2010 18:49:55 -0700
Subject: [PATCH] [#4420] Handle edge case when a retainer has no start or end
 date set.

---
 app/models/retainer_deliverable.rb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/app/models/retainer_deliverable.rb b/app/models/retainer_deliverable.rb
index 8598196..9309457 100644
--- a/app/models/retainer_deliverable.rb
+++ b/app/models/retainer_deliverable.rb
@@ -109,6 +109,7 @@ class RetainerDeliverable < HourlyDeliverable
   private
 
   def shrink_budgets_to_new_period
+    return if beginning_date.nil? || ending_date.nil?
     labor_budgets.all.each do |labor_budget|
       # Purge un-dated budgets, should not be saved at all
       labor_budget.destroy unless labor_budget.year.present?