fixed invoice sorting and finished auctions sort

internetee · Sep 15, 2023 · ba81737 · ba81737
1 parent 6bbb9be
commit ba81737
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 8 deletions.
diff --git a/app/controllers/histories_controller.rb b/app/controllers/histories_controller.rb
@@ -3,7 +3,6 @@ class HistoriesController < ApplicationController
 
   def index
     auctions = Auction.where('ends_at <= ?', Time.zone.now)
-                      .order(ends_at: :desc, domain_name: :asc)
                       .search(params)
 
     @pagy, @auctions = pagy(auctions, items: params[:per_page] ||= 20,

diff --git a/app/models/invoice.rb b/app/models/invoice.rb
@@ -70,17 +70,19 @@ def self.search(params = {})
                                                status
                                                number
                                                due_date
-                                               billing_profile_id]) || 'id'
+                                               billing_profile_name]) || 'id'
     sort_direction = params[:direction].presence_in(%w[asc desc]) || 'desc'
 
     query = with_search_scope(params[:search_string]).with_statuses(params[:statuses_contains])
 
     if params[:sort] == 'channel'
       query.left_outer_joins(:payment_orders)
            .select("invoices.*, REPLACE(payment_orders.type, 'PaymentOrders::', '') AS payment_order_channel")
-           .order(Arel.sql("payment_order_channel #{sort_direction}"))
+           .order(Arel.sql("payment_order_channel #{sort_direction} NULLS LAST"))
+    elsif params[:sort] == 'billing_profile_name'
+      query.left_outer_joins(:billing_profile).order("billing_profiles.name #{sort_direction} NULLS LAST")
     else
-      query.order("#{sort_column} #{sort_direction}")
+      query.order("#{sort_column} #{sort_direction} NULLS LAST")
     end
   end
 

diff --git a/app/views/admin/invoices/index.html.erb b/app/views/admin/invoices/index.html.erb
@@ -40,8 +40,7 @@
                 <tr>
                     <th scope="col">
                         <%= t('invoices.billing_profile') %>
-                        <%#= order_buttons('invoices.billing_profile_id') %>
-                        <%= sort_link_to "", "billing_profile_id" %>
+                        <%= sort_link_to "", "billing_profile_name" %>
                     </th>
                     <th scope="col">
                         <%= t('invoices.due_date') %>

diff --git a/app/views/auctions/index.html.erb b/app/views/auctions/index.html.erb
@@ -75,7 +75,7 @@
       <div class="four wide column"></div>
       <div class="four wide column"></div>
       <div class="four wide column" style='text-align: right;'>
-        <%= link_to t('auctions.finished_auctions'), histories_path, style: 'margin-bottom: 20px; display: block; font-size: 18px; color: #03456f;' %>
+        <%= link_to t('auctions.finished_auctions'), "#{histories_path}?direction=desc&sort=domain_name", style: 'margin-bottom: 20px; display: block; font-size: 18px; color: #03456f;' %>
       </div>
     </div>
     <div class="grid one column">

diff --git a/app/views/histories/index.html.erb b/app/views/histories/index.html.erb
@@ -63,6 +63,7 @@
                                 </th>
                                 <th scope="col">
                                   <%= t('auctions.winning_bid') %>
+                                  <%= sort_link_to "", "users_price" %> 
                                 </th>
                                 <th scope="col">
                                   <%= t('auctions.auction_type') %>
@@ -79,7 +80,7 @@
                                     <td><%= auction.ends_at.to_formatted_s(:long) %></td>
                                     <td><%= auction.currently_winning_offer&.price if auction.platform == 'english' %></td>
                                     <td>
-                                        <%= t("auctions.#{auction.platform&.downcase.presence || 'blind'}") %>
+                                        <%= t("auctions.#{auction.platform&.downcase.presence || 'blind (lagacy)'}") %>
                                     </td>
                                 </tr>
                             <% end %>