Make things simple and they are easier to understand. #1924

2009-01-21 15:29:00 -08:00
parent 075310da6b
commit aed49c68da
1 changed files with 1 additions and 1 deletions
--- 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