From 1b8d4c6d52a7b6c14b3ebf656287e99afcd44628 Mon Sep 17 00:00:00 2001
From: Eric Davis <edavis@littlestreamsoftware.com>
Date: Thu, 10 Jul 2008 17:44:29 -0700
Subject: [PATCH] Moved methods from HourlyDeliverable to Deliverable. #1439

---
 app/models/deliverable.rb        |  9 +++++----
 app/models/hourly_deliverable.rb | 12 ------------
 2 files changed, 5 insertions(+), 16 deletions(-)

diff --git a/app/models/deliverable.rb b/app/models/deliverable.rb
index 2164d16..ef5effc 100644
--- a/app/models/deliverable.rb
+++ b/app/models/deliverable.rb
@@ -128,14 +128,15 @@ class Deliverable < ActiveRecord::Base
     return self.budget - self.spent
   end
   
-  # Number of hours used.  Virtual accessor that is overriden by subclasses.
+  # Number of hours used.
   def hours_used
-    0
+    return 0 unless self.issues.size > 0
+    return self.issues.collect(&:time_entries).flatten.collect(&:hours).sum
   end
   
-  # Amount spent on members.  Virtual accessor that is overriden by subclasses.
+  # Amount spent on members.
   def members_spent
-    []
+    return MemberSpent.find_all_by_deliverable(self)
   end
   
   # Amount of the budget remaining
diff --git a/app/models/hourly_deliverable.rb b/app/models/hourly_deliverable.rb
index e466cdd..d966c28 100644
--- a/app/models/hourly_deliverable.rb
+++ b/app/models/hourly_deliverable.rb
@@ -19,18 +19,6 @@ class HourlyDeliverable < Deliverable
     return total
   end
   
-  # Number of hours used by Members on the Deliverable's issues
-  def hours_used
-    return 0 unless self.issues.size > 0
-    return self.issues.collect(&:time_entries).flatten.collect(&:hours).sum
-
-  end
-  
-  # Amount of time and money spent by the Members on the Deliverable's issues.
-  def members_spent
-    return MemberSpent.find_all_by_deliverable(self)
-  end
-  
   def profit # :nodoc:
     if read_attribute(:profit_percent).nil?
       return super