Remove unnecessary advanced search path parameter

We can use the current path as URL instead of passing it every time.
Passing the `page: 1` parameter is also redundant since by default the
index goes to the first page and the search form does not send any page
parameter.
This commit is contained in:
Javi Martín
2020-12-28 13:58:17 +01:00
parent 142f9b300b
commit dda79a9224
4 changed files with 4 additions and 5 deletions

View File

@@ -55,7 +55,7 @@
</div> </div>
<% end %> <% end %>
<%= render("shared/advanced_search", search_path: budget_investments_path(@budget)) %> <%= render("shared/advanced_search") %>
<% if unfeasible_or_unselected_filter %> <% if unfeasible_or_unselected_filter %>
<ul class="no-bullet submenu"> <ul class="no-bullet submenu">

View File

@@ -52,7 +52,7 @@
</div> </div>
</div> </div>
<%= render "shared/advanced_search", search_path: debates_path(page: 1) %> <%= render "shared/advanced_search" %>
<%= render "shared/order_links", i18n_namespace: "debates.index" %> <%= render "shared/order_links", i18n_namespace: "debates.index" %>

View File

@@ -73,8 +73,7 @@
<% end %> <% end %>
<% unless params[:retired].present? || params[:selected].present? %> <% unless params[:retired].present? || params[:selected].present? %>
<%= render "shared/advanced_search", <%= render "shared/advanced_search" %>
search_path: proposals_path(page: 1) %>
<% end %> <% end %>
<% unless params[:selected].present? %> <% unless params[:selected].present? %>

View File

@@ -3,7 +3,7 @@
</div> </div>
<div class="row advanced-search-form"> <div class="row advanced-search-form">
<%= form_tag search_path, id: "advanced_search_form", method: :get do %> <%= form_tag request.path, id: "advanced_search_form", method: :get do %>
<div id="js-advanced-search" data-advanced-search-terms="<%= @advanced_search_terms.present? %>" style="display: none"> <div id="js-advanced-search" data-advanced-search-terms="<%= @advanced_search_terms.present? %>" style="display: none">
<div class="small-12 column"> <div class="small-12 column">