Search on the same URL by default

consuldemocracy · Dec 4, 2020 · 39a97a4 · 39a97a4
1 parent e33794e
commit 39a97a4
Show file tree

Hide file tree

Showing 9 changed files with 13 additions and 29 deletions.
diff --git a/app/components/admin/search_component.rb b/app/components/admin/search_component.rb
@@ -1,12 +1,16 @@
 class Admin::SearchComponent < ApplicationComponent
-  attr_reader :url, :label, :form_options
+  attr_reader :label, :form_options
 
-  def initialize(url:, label:, **form_options)
-    @url = url
+  def initialize(label:, url: nil, **form_options)
     @label = label
+    @url = url
     @form_options = form_options
   end
 
+  def url
+    @url || request.path
+  end
+
   private
 
     def search_terms

diff --git a/app/views/admin/debates/index.html.erb b/app/views/admin/debates/index.html.erb
@@ -5,10 +5,7 @@
 <h2><%= t("admin.debates.index.title") %></h2>
 
 <% if @debates.any? %>
-  <%= render Admin::SearchComponent.new(
-    url: admin_debates_path,
-    label: t("admin.shared.debate_search.placeholder")
-  ) %>
+  <%= render Admin::SearchComponent.new(label: t("admin.shared.debate_search.placeholder")) %>
 
   <h3 class="inline-block"><%= page_entries_info @debates %></h3>
 

diff --git a/app/views/admin/local_census_records/index.html.erb b/app/views/admin/local_census_records/index.html.erb
@@ -9,7 +9,6 @@
             class: "button float-right hollow" %>
 
 <%= render Admin::SearchComponent.new(
-  url: admin_local_census_records_path,
   label: t("admin.local_census_records.index.search.placeholder"),
   remote: true
 ) %>

diff --git a/app/views/admin/organizations/search.html.erb b/app/views/admin/organizations/search.html.erb
@@ -1,9 +1,6 @@
 <h2><%= t("admin.organizations.search.title") %></h2>
 
-<%= render Admin::SearchComponent.new(
-  url: search_admin_organizations_path,
-  label: t("admin.organizations.index.search_placeholder")
-) %>
+<%= render Admin::SearchComponent.new(label: t("admin.organizations.index.search_placeholder")) %>
 
 <div id="search-results">
   <% if @organizations.any? %>

diff --git a/app/views/admin/poll/questions/_search.html.erb b/app/views/admin/poll/questions/_search.html.erb
@@ -1,4 +1 @@
-<%= render Admin::SearchComponent.new(
-  url: admin_questions_path,
-  label: t("admin.shared.poll_questions_search.placeholder")
-) %>
+<%= render Admin::SearchComponent.new(label: t("admin.shared.poll_questions_search.placeholder")) %>
diff --git a/app/views/admin/proposals/index.html.erb b/app/views/admin/proposals/index.html.erb
@@ -5,10 +5,7 @@
 <h2><%= t("admin.proposals.index.title") %></h2>
 
 <% if @proposals.any? %>
-  <%= render Admin::SearchComponent.new(
-    url: admin_proposals_path,
-    label: t("admin.shared.proposal_search.placeholder")
-  ) %>
+  <%= render Admin::SearchComponent.new(label: t("admin.shared.proposal_search.placeholder")) %>
 
   <h3><%= page_entries_info @proposals %></h3>
 

diff --git a/app/views/admin/users/index.html.erb b/app/views/admin/users/index.html.erb
@@ -1,7 +1,6 @@
 <h2><%= t("admin.users.index.title") %></h2>
 
 <%= render Admin::SearchComponent.new(
-  url: admin_users_path,
   label: t("admin.users.search.placeholder"),
   remote: true
 ) %>

diff --git a/app/views/management/proposals/index.html.erb b/app/views/management/proposals/index.html.erb
@@ -1,10 +1,7 @@
 <main>
   <h2><%= t("management.proposals.index.title") %></h2>
 
-  <%= render Admin::SearchComponent.new(
-    url: management_proposals_path,
-    label: t("admin.shared.proposal_search.placeholder")
-  ) %>
+  <%= render Admin::SearchComponent.new(label: t("admin.shared.proposal_search.placeholder")) %>
 
   <div class="management-list">
     <div class="proposals-list">

diff --git a/app/views/moderation/users/index.html.erb b/app/views/moderation/users/index.html.erb
@@ -1,9 +1,6 @@
 <h2><%= t("moderation.users.index.title") %></h2>
 
-<%= render Admin::SearchComponent.new(
-  url: moderation_users_path,
-  label: t("moderation.users.index.search_placeholder")
-) %>
+<%= render Admin::SearchComponent.new(label: t("moderation.users.index.search_placeholder")) %>
 
 <% if @users.present? %>
   <h3><%= page_entries_info @users %></h3>