From ca84747cb907604616133d498692a686087d9d35 Mon Sep 17 00:00:00 2001
From: Eric Davis <edavis@littlestreamsoftware.com>
Date: Tue, 11 Oct 2011 14:12:52 -0700
Subject: [PATCH] [#6574] Refactor: extract overhead budget form to partial

---
 app/views/deliverables/_finance_form.html.erb | 25 +++----------------
 .../_overhead_budget_form.html.erb            | 20 +++++++++++++++
 2 files changed, 24 insertions(+), 21 deletions(-)
 create mode 100644 app/views/deliverables/_overhead_budget_form.html.erb

diff --git a/app/views/deliverables/_finance_form.html.erb b/app/views/deliverables/_finance_form.html.erb
index c2331d0..9f8db24 100644
--- a/app/views/deliverables/_finance_form.html.erb
+++ b/app/views/deliverables/_finance_form.html.erb
@@ -24,27 +24,10 @@
   <li class="numeric optional">
     <%= content_tag(:label, l(:field_overhead)) %>
     <table id="deliverable-overhead" class="deliverable_finance_table">
-      
-      <% form.fields_for :overhead_budgets, overhead_budgets do |overhead_budget| %>
-        <%= overhead_budget.hidden_field(:year) %>
-        <%= overhead_budget.hidden_field(:month) %>
-      <tr>
-        <td>
-          <%= overhead_budget.label(:time_entry_activity_id, :class => "hidden") %>
-          <%= overhead_budget.select(:time_entry_activity_id, options_from_collection_for_select(@project.non_billable_activities, :id, :name, overhead_budget.object.time_entry_activity_id), {:include_blank => false}, {:class => 'financial'}) %>
-        </td>
-        <td>
-          <p class="inline-hints"><%= overhead_budget.label(:hours, l(:text_short_hours)) %></p>
-          <%= overhead_budget.text_field(:hours, :value => format_deliverable_value_fields(overhead_budget.object.hours),:class => 'financial') %>
-        </td>
-        <td>
-          <p class="inline-hints"><%= overhead_budget.label(:budget, l(:text_dollar_sign)) %></p>
-          <%= overhead_budget.text_field(:budget, :value => format_deliverable_value_fields(overhead_budget.object.budget), :class => 'financial') %>
-        </td>
-        <td>
-          <%= release(3, "Green Add button for multiple records") %>
-        </td>
-      </tr>
+      <tbody>
+      <% form.fields_for :overhead_budgets, overhead_budgets.sort_by(&:id) do |overhead_budget| %>
+        <%= render :partial => 'overhead_budget_form', :locals => {:overhead_budget => overhead_budget} %>
+      </tbody>
       <% end %>
     </table>
   </li>
diff --git a/app/views/deliverables/_overhead_budget_form.html.erb b/app/views/deliverables/_overhead_budget_form.html.erb
new file mode 100644
index 0000000..73468c3
--- /dev/null
+++ b/app/views/deliverables/_overhead_budget_form.html.erb
@@ -0,0 +1,20 @@
+      <tr>
+        <td>
+          <%= overhead_budget.hidden_field(:year) %>
+          <%= overhead_budget.hidden_field(:month) %>
+
+          <%= overhead_budget.label(:time_entry_activity_id, :class => "hidden") %>
+          <%= overhead_budget.select(:time_entry_activity_id, options_from_collection_for_select(@project.non_billable_activities, :id, :name, overhead_budget.object.time_entry_activity_id), {:include_blank => false}, {:class => 'financial'}) %>
+        </td>
+        <td>
+          <p class="inline-hints"><%= overhead_budget.label(:hours, l(:text_short_hours)) %></p>
+          <%= overhead_budget.text_field(:hours, :value => format_deliverable_value_fields(overhead_budget.object.hours),:class => 'financial') %>
+        </td>
+        <td>
+          <p class="inline-hints"><%= overhead_budget.label(:budget, l(:text_dollar_sign)) %></p>
+          <%= overhead_budget.text_field(:budget, :value => format_deliverable_value_fields(overhead_budget.object.budget), :class => 'financial') %>
+        </td>
+        <td>
+          <%= release(3, "Green Add button for multiple records") %>
+        </td>
+      </tr>