Added some better details on the deliverables based on if they are hourly or fixed. #1439
This commit is contained in:
1 parent
1b8d4c6d52
commit
495c5f6ac8
2 files changed
+21
-3
No files matched your search
@@ -134,11 +134,16 @@ class Deliverable < ActiveRecord::Base
|
|||||||
return self.issues.collect(&:time_entries).flatten.collect(&:hours).sum
|
return self.issues.collect(&:time_entries).flatten.collect(&:hours).sum
|
||||||
end
|
end
|
||||||
|
|
||||||
# Amount spent on members.
|
# Grouping of members and how much they spent
|
||||||
def members_spent
|
def members_spent
|
||||||
return MemberSpent.find_all_by_deliverable(self)
|
return MemberSpent.find_all_by_deliverable(self)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
# Amount spent on members.
|
||||||
|
def spent_by_members
|
||||||
|
return self.members_spent.collect(&:spent)
|
||||||
|
end
|
||||||
|
|
||||||
# Amount of the budget remaining
|
# Amount of the budget remaining
|
||||||
def left
|
def left
|
||||||
return self.budget - self.spent
|
return self.budget - self.spent
|
||||||
@@ -177,6 +182,13 @@ class Deliverable < ActiveRecord::Base
|
|||||||
(user == user && user.allowed_to?(:manage_budget, project))
|
(user == user && user.allowed_to?(:manage_budget, project))
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def fixed?
|
||||||
|
return self.class == FixedDeliverable
|
||||||
|
end
|
||||||
|
|
||||||
|
def hourly?
|
||||||
|
return self.class == HourlyDeliverable
|
||||||
|
end
|
||||||
private
|
private
|
||||||
|
|
||||||
def use_issue_status_for_done_ratios?
|
def use_issue_status_for_done_ratios?
|
||||||
|
|||||||
@@ -32,10 +32,16 @@
|
|||||||
<table class="progress-table">
|
<table class="progress-table">
|
||||||
<%= row_with_double_data "Progress: ", number_to_percentage(deliverable.progress, :precision => 0), '' %>
|
<%= row_with_double_data "Progress: ", number_to_percentage(deliverable.progress, :precision => 0), '' %>
|
||||||
<% if allowed_management? %>
|
<% if allowed_management? %>
|
||||||
<%= row_with_double_data "Hours Estimated: ", number_with_precision(deliverable.total_hours, 0), '' %>
|
|
||||||
|
<% if deliverable.hourly? %>
|
||||||
|
<%= row_with_double_data "Hours Estimated: ", number_with_precision(deliverable.total_hours, 0), '' %>
|
||||||
|
<% end %>
|
||||||
|
<% if deliverable.fixed? %>
|
||||||
|
<%= row_with_double_data "Fixed Amount: ", '', number_to_currency(deliverable.fixed_cost, :precision => 0) %>
|
||||||
|
<% end %>
|
||||||
|
|
||||||
<% if deliverable.hours_used > 0 %>
|
<% if deliverable.hours_used > 0 %>
|
||||||
<%= row_with_double_data "Hours Used: ", number_with_precision(deliverable.hours_used,0), number_to_currency(deliverable.spent, :precision => 0) %>
|
<%= row_with_double_data "Hours Used: ", number_with_precision(deliverable.hours_used,0), number_to_currency(deliverable.spent_by_members, :precision => 0) %>
|
||||||
|
|
||||||
<tr><td> </td></tr>
|
<tr><td> </td></tr>
|
||||||
|
|
||||||
|
|||||||
Reference in new issue
Block a user