API: Eager load page associations

In order to prevent N+1 queries in the pages API we eager load all necessary associations
AlchemyCMS · Nov 15, 2019 · 5b2f6dd · 5b2f6dd
1 parent feaa50f
commit 5b2f6dd
Showing 1 changed file with 19 additions and 0 deletions.
diff --git a/app/controllers/alchemy/api/pages_controller.rb b/app/controllers/alchemy/api/pages_controller.rb
@@ -13,6 +13,7 @@ def index
       else
         @pages = Page.accessible_by(current_ability, :index)
       end
+      @pages = @pages.includes(*page_includes)
       if params[:page_layout].present?
         Alchemy::Deprecation.warn <<~WARN
           Passing page_layout parameter to Alchemy::Api::PagesController#index is deprecated.
@@ -91,5 +92,23 @@ def per_page_value
     def page_value
       params[:page] ? params[:page].to_i : nil
     end
+
+    def page_includes
+      [
+        :tags,
+        {
+          elements: [
+            {
+              nested_elements: [
+                :contents,
+                :tags
+              ]
+            },
+            :contents,
+            :tags
+          ]
+        }
+      ]
+    end
   end
 end