Add search page module and remove controller extension

README.md

@@ -135,6 +135,9 @@ In order to render the search results, you'll need a page layout that represents
 page layout as `searchresults: true`. The search form will pick this page as result page.
 
 #### Search Results Page
+
+Add a search layout to the `page_layout.yml` and mark it with a `searchresults` flag. These flag is used to find the 
+correct page path for search form.
 
 ```yaml
 # page_layouts.yml
@@ -143,40 +146,12 @@ page layout as `searchresults: true`. The search form will pick this page as res
   unique: true
 ```
 
-Tip: For maximum flexibility you could also add an element that represents the search results. This lets your editors to
-place additional elements (maybe a header image or additional text blocks) on the search result page.
-
-```yaml
-# page_layouts.yml
-- name: search
-  searchresults: true
-  unique: true
-  elements:
-    - searchresults
-  autogenerate:
-    - searchresults
-
-# elements.yml
-- name: searchresults
-  unique: true
-  ingredients:
-    - role: search_string
-      hint: The is only for presentational purposes in the Alchemy preview
-      default: lorem
-      type: Text
-```
-
-and then use the view helpers to render the search form on the page layout partial and the search results on the element
-view partial.
+and then use the view helpers to render the search form on the page layout partial.
 
 ```erb
 <!-- app/views/alchemy/page_layouts/_search.html.erb -->
-<%= render_elements %>
-
-<!-- app/views/alchemy/elements/_searchresults.html.erb -->
-<%= element_view_for(searchresults) do |el| -%>
-  <%= render_search_results %>
-<%- end -%>
+<% @search_results = Alchemy::Search::Page.perform_search(params, ability: current_ability) %>
+<%= render_search_results %>
 ```
 
 ### View Helpers

app/helpers/alchemy/pg_search/search_helper.rb

@@ -29,7 +29,8 @@ def render_search_form(options = {})
           class: "fulltext_search",
           id: "search",
         }
-        render "alchemy/search/form", options: default_options.merge(options), search_result_page: search_result_page
+        search_result_page = Alchemy::Search::SearchPage.search_result_page
+        render "alchemy/search/form", options: default_options.merge(options), search_result_page:
       end
 
       # Renders the search results partial within +app/views/alchemy/search/_results.html+

app/services/alchemy/search/search_page.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module Alchemy
+  module Search
+    module SearchPage
+      def self.perform_search(params, ability: nil)
+        search_results = Alchemy.search_class.search(params[:query], ability:)
+        search_results = search_results&.page(params[:page])&.per(paginate_per) if paginate_per.present?
+        search_results
+      end
+
+      def self.paginate_per
+        Alchemy::PgSearch.config[:paginate_per]
+      end
+
+      def self.search_result_page
+        @search_result_page ||= begin
+          page_layouts = PageLayout.all.select do |page_layout|
+            page_layout.key?(:searchresults) && page_layout[:searchresults].to_s.casecmp(true.to_s).zero?
+          end
+
+          if page_layouts.nil?
+            raise "No searchresults page layout found. Please add page layout with `searchresults: true` into your `page_layouts.yml` file."
+          end
+
+          page = Page.published.find_by(
+            page_layout: page_layouts.first["name"],
+            language_id: Language.current.id,
+          )
+          if page.nil?
+            logger.warn "\n++++++\nNo published search result page found. Please create one or publish your search result page.\n++++++\n"
+          end
+          page
+        end
+      end
+    end
+  end
+end

lib/alchemy/pg_search/engine.rb

@@ -11,10 +11,6 @@ class Engine < ::Rails::Engine
           require_dependency(c)
         end
 
-        # We need to have the search methods present in all Alchemy controllers
-        Alchemy::PagesController.send(:include, Alchemy::PgSearch::ControllerMethods)
-        Alchemy::Admin::PagesController.send(:include, Alchemy::PgSearch::ControllerMethods)
-
         # In development environment, this runs on every code reload, so avoid multiple reindexing jobs
         unless Alchemy.publish_targets.map(&:name).include? 'Alchemy::PgSearch::IndexPageJob'
           # reindex the page after it was published

spec/dummy/app/views/alchemy/page_layouts/_search.html.erb

@@ -1,2 +1,3 @@
+<% @search_results = Alchemy::Search::SearchPage.perform_search(params, ability: current_ability) %>
 <%= render_search_form %>
-<%= render_elements %>
+<%= render_search_results %>

spec/dummy/config/alchemy/elements.yml

@@ -45,9 +45,6 @@
       type: Picture
       searchable: false
 
-- name: searchresults
-  unique: true
-
 - name: content_test
   ingredients:
     - role: without_searchable

spec/dummy/config/alchemy/page_layouts.yml

@@ -9,7 +9,3 @@
 - name: search
   searchresults: true
   unique: true
-  elements:
-  - searchresults
-  autogenerate:
-  - searchresults

spec/features/fulltext_search_feature_spec.rb

@@ -2,7 +2,7 @@
 
 RSpec.describe "Fulltext search" do
   let!(:search_page) do
-    create(:alchemy_page, :public, name: "Suche", page_layout: "search", autogenerate_elements: true)
+    create(:alchemy_page, :public, name: "Suche", page_layout: "search")
   end
 
   it "form has correct path in the form tag" do

spec/services/alchemy/search/search_page_spec.rb

@@ -0,0 +1,60 @@
+require "spec_helper"
+
+RSpec.describe Alchemy::Search::SearchPage do
+  let(:paginate_per) { 10 }
+  before do
+    Alchemy::PgSearch.config = {
+      paginate_per:,
+    }
+  end
+
+  context "#perform_search" do
+    let(:query) {"foo"}
+    let(:params) { {query:}}
+    subject { described_class.perform_search(params, ability: nil) }
+
+    it "calls the Alchemy.search_class" do
+      expect(Alchemy.search_class).to receive(:search).with(query, ability: nil)
+      subject
+    end
+
+    context "pagination" do
+      let(:query) {"page"}
+
+      before do
+        12.times do
+          create(:alchemy_page, :public)
+        end
+        Alchemy.search_class.rebuild
+      end
+
+      it "response with 10 documents" do
+        expect(subject.length).to eq(10)
+      end
+
+      context "without pagination" do
+        let(:paginate_per) { nil }
+
+        it "response with all documents" do
+          expect(subject.length).to eq(12)
+        end
+      end
+    end
+  end
+
+  context '#paginate_per' do
+    subject { described_class.paginate_per }
+
+    it 'should be 10 if no configuration is set' do
+      expect(subject).to eq(10)
+    end
+
+    context "with configuration" do
+      let(:paginate_per) { 50 }
+
+      it 'should be 50 if no configuration is set' do
+        expect(subject).to eq(50)
+      end
+    end
+  end
+end