From aed49c68dac3837af1645edec7cfa141c34291ee Mon Sep 17 00:00:00 2001
From: Eric Davis <edavis@littlestreamsoftware.com>
Date: Wed, 21 Jan 2009 15:29:00 -0800
Subject: [PATCH] Make things simple and they are easier to understand. #1924

---
 app/models/deliverable.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/models/deliverable.rb b/app/models/deliverable.rb
index 19cb39c..0bc6cf9 100644
--- a/app/models/deliverable.rb
+++ b/app/models/deliverable.rb
@@ -49,7 +49,7 @@ class Deliverable < ActiveRecord::Base
   def progress
     return 0 unless self.issues.size > 0
     
-    total ||=  self.issues.collect(&:estimated_hours).delete_if {|e| e.nil? }.inject {|sum, n| sum + n} || 0
+    total ||=  self.issues.collect(&:estimated_hours).compact.sum || 0
 
     return 0 unless total > 0
     balance = 0.0