Refactor estimates and invoices rendering to use local variables for the abilty for other plugins to display filtered lists

This commit is contained in:
2026-01-31 12:45:47 -05:00
parent f1bdf59697
commit 92460392b9
3 changed files with 8 additions and 9 deletions

View File

@@ -27,12 +27,12 @@
<div class="splitcontent">
<div class="splitcontentleft">
<h4><%=t(:estimates)%>:</h4>
<%= render partial: 'estimates/list', locals: {customer: @customer} %>
<%= render partial: 'estimates/list', locals: {estimates: @customer.estimates} %>
</div>
<div class="splitcontentleft">
<h4><%=t(:label_invoices)%>:</h4>
<%= render partial: 'invoices/list', locals: {customer: @customer} %>
<%= render partial: 'invoices/list', locals: {invoices: @customer.invoices} %>
</div>
</div>

View File

@@ -1,6 +1,6 @@
<% if @customer.present? %>
<% if estimates.empty? %>
<% @customer.estimates.order(id: :desc).each do |estimate| %>
<% estimates.sort.reverse.each do |estimate| %>
<div class="row">
<b><%= link_to "##{estimate.doc_number}", estimate_path(estimate), target: :_blank %></b> <%= estimate.txn_date %>
</div>

View File

@@ -1,24 +1,23 @@
<% if @customer.present? %>
<% if invoices.empty? %>
<%= form_with(url: invoice_path, method: :get) do |form| %>
<% if @customer.invoices.count > 1 %>
<% if invoices.count > 1 %>
<div class="form-check">
<%= check_box_tag "select-all-invoices", "1", false, id: "select-all-invoices" %>
<%= label_tag "select-all-invoices", t(:label_select_all) %>
</div>
<hr>
<% end %>
<% @customer.invoices.order(id: :desc).each do |invoice| %>
<% invoices.sort.reverse.each do |invoice| %>
<div class="row">
<%= check_box_tag "invoice_ids[]", invoice.id, false, class: "invoice-checkbox" %>
<b><%= link_to "##{invoice.doc_number}", invoice_path(invoice), target: :_blank %></b> <%= invoice.txn_date %>
</div>
<% end %>
<% if @customer.invoices.count > 1 %>
<% if invoices.count > 1 %>
<%= form.submit t(:button_bulk_pdf) %>
<% end %>
<% end %>