From c3c472b3b1cd1912047cc4bb28d71c373869a38b Mon Sep 17 00:00:00 2001
From: Eric Davis <edavis@littlestreamsoftware.com>
Date: Mon, 30 Aug 2010 10:09:13 -0700
Subject: [PATCH] [#4420] Extract the Deliverable finances form to a partial.

---
 app/views/deliverables/_finance_form.html.erb | 59 ++++++++++++++++++
 app/views/deliverables/_form.html.erb         | 60 +------------------
 2 files changed, 60 insertions(+), 59 deletions(-)
 create mode 100644 app/views/deliverables/_finance_form.html.erb

diff --git a/app/views/deliverables/_finance_form.html.erb b/app/views/deliverables/_finance_form.html.erb
new file mode 100644
index 0000000..a672579
--- /dev/null
+++ b/app/views/deliverables/_finance_form.html.erb
@@ -0,0 +1,59 @@
+<% form.inputs :name => l(:text_deliverable_finances), :id => 'deliverable-finances' do %>
+
+  <li class="numeric optional">
+    <%= content_tag(:label, l(:field_labor)) %>
+    <table id="deliverable-labor" class="deliverable_finance_table">
+
+      <% form.fields_for :labor_budgets do |labor_budget| %>
+      <tr>
+        <%# TODO: Select field for the Time Entry Activity in a td %>
+        <td>
+          <select><option>TODO: Release 3</option></select>
+        </td>
+        <td>
+          <p class="inline-hints"><%= labor_budget.label(:hours, l(:text_short_hours)) %></p>
+          <%= labor_budget.text_field(:hours, :value => format_deliverable_value_fields(labor_budget.object.hours), :size => 10) %>
+        </td>
+        <td>
+          <p class="inline-hints"><%= labor_budget.label(:budget, l(:text_dollar_sign)) %></p>
+          <%= labor_budget.text_field(:budget, :value => format_deliverable_value_fields(labor_budget.object.budget), :size => 10) %>
+        </td>
+        <%# TODO: Green Add button for multiple records %>
+        <td>
+          Todo: Add button (Release 3)
+        </td>
+      </tr>
+      <% end %>
+    </table>
+  </li>
+
+  <li class="numeric optional">
+    <%= content_tag(:label, l(:field_overhead)) %>
+    <table id="deliverable-overhead" class="deliverable_finance_table">
+      
+      <% form.fields_for :overhead_budgets do |overhead_budget| %>
+      <tr>
+        <%# TODO: Select field for the Time Entry Activity in a td %>
+        <td>
+          <select><option>TODO: Release 3</option></select>
+        </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),:size => 10) %>
+        </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), :size => 10) %>
+        </td>
+        <%# TODO: Green Add button for multiple records %>
+        <td>
+          Todo: Add button (Release 3)
+        </td>
+      </tr>
+      <% end %>
+    </table>
+  </li>
+
+  <%= form.input :total, :input_html => {:size => 10}, :wrapper_html => {:class => 'deliverable_total_input'}, :hint => l(:text_dollar_sign) %>
+
+<% end %>
diff --git a/app/views/deliverables/_form.html.erb b/app/views/deliverables/_form.html.erb
index a9cb9a2..46ed70c 100644
--- a/app/views/deliverables/_form.html.erb
+++ b/app/views/deliverables/_form.html.erb
@@ -28,65 +28,7 @@
   <% end %>
 <% end %>
 
-<% form.inputs :name => l(:text_deliverable_finances), :id => 'deliverable-finances' do %>
-
-  <li class="numeric optional">
-    <%= content_tag(:label, l(:field_labor)) %>
-    <table id="deliverable-labor" class="deliverable_finance_table">
-
-      <% form.fields_for :labor_budgets do |labor_budget| %>
-      <tr>
-        <%# TODO: Select field for the Time Entry Activity in a td %>
-        <td>
-          <select><option>TODO: Release 3</option></select>
-        </td>
-        <td>
-          <p class="inline-hints"><%= labor_budget.label(:hours, l(:text_short_hours)) %></p>
-          <%= labor_budget.text_field(:hours, :value => format_deliverable_value_fields(labor_budget.object.hours), :size => 10) %>
-        </td>
-        <td>
-          <p class="inline-hints"><%= labor_budget.label(:budget, l(:text_dollar_sign)) %></p>
-          <%= labor_budget.text_field(:budget, :value => format_deliverable_value_fields(labor_budget.object.budget), :size => 10) %>
-        </td>
-        <%# TODO: Green Add button for multiple records %>
-        <td>
-          Todo: Add button (Release 3)
-        </td>
-      </tr>
-      <% end %>
-    </table>
-  </li>
-
-  <li class="numeric optional">
-    <%= content_tag(:label, l(:field_overhead)) %>
-    <table id="deliverable-overhead" class="deliverable_finance_table">
-      
-      <% form.fields_for :overhead_budgets do |overhead_budget| %>
-      <tr>
-        <%# TODO: Select field for the Time Entry Activity in a td %>
-        <td>
-          <select><option>TODO: Release 3</option></select>
-        </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),:size => 10) %>
-        </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), :size => 10) %>
-        </td>
-        <%# TODO: Green Add button for multiple records %>
-        <td>
-          Todo: Add button (Release 3)
-        </td>
-      </tr>
-      <% end %>
-    </table>
-  </li>
-
-  <%= form.input :total, :input_html => {:size => 10}, :wrapper_html => {:class => 'deliverable_total_input'}, :hint => l(:text_dollar_sign) %>
-
-<% end %>
+<%= render :partial => 'finance_form', :locals => {:form => form, :resource => resource} %>
 
 </div>