Fix for zeitwerk

app/controllers/concerns/occams/reorder_action.rb

@@ -11,6 +11,9 @@ def reorder
     resource_class = self.class.reorder_action_resource
     (params.permit(order: [])[:order] || []).each_with_index do |id, index|
       resource_class.where(id: id).update_all(position: index)
+      if resource_class == ::Occams::Cms::Page
+        Occams::Cms::Page.all.each(&:save!)
+      end
     end
     head :ok
   end

app/controllers/occams/admin/cms/pages_controller.rb

@@ -35,8 +35,14 @@ def edit
     render
   end
 
+  def update_family
+    @page.siblings.each(&:save!) if @page.siblings
+    @page.parent.save! if @page.parent
+  end
+
   def create
     @page.save!
+    update_family
     flash[:success] = I18n.t('occams.admin.cms.pages.created')
     redirect_to action: :edit, id: @page
   rescue ActiveRecord::RecordInvalid
@@ -46,8 +52,7 @@ def create
 
   def update
     @page.save!
-    @page.siblings.each(&:save!) if @page.siblings
-    @page.parent.save! if @page.parent
+    update_family
     flash[:success] = I18n.t('occams.admin.cms.pages.updated')
     redirect_to action: :edit, id: @page
   rescue ActiveRecord::RecordInvalid
@@ -57,6 +62,7 @@ def update
 
   def destroy
     @page.destroy
+    update_family
     flash[:success] = I18n.t('occams.admin.cms.pages.deleted')
     redirect_to action: :index
   end

config/application.rb

@@ -28,11 +28,8 @@ class Application < Rails::Application
       # Please, add to the `ignore` list any other `lib` subdirectories that do
       # not contain `.rb` files, or that should not be reloaded or eager loaded.
       # Common ones are `templates`, `generators`, or `middleware`, for example.
-      config.autoload_paths << "#{config.root}/lib"
-      config.eager_load_paths << "#{config.root}/lib"
+      config.add_autoload_paths_to_load_path = false
       config.autoload_lib(ignore: %w[generators])
-      config.autoloader = :classic
-      # config.add_autoload_paths_to_load_path = false
     end
 
     # Making sure we don't load our dev routes as part of the engine

config/locales/en.yml

@@ -116,11 +116,11 @@ en:
             update: Update Layout
 
         pages:
-          created: Page created
+          created: Page created, siblings, and parent updated
           creation_failure: Failed to create page
           updated: Page, siblings, and parent updated
           update_failure: Failed to update page
-          deleted: Page deleted
+          deleted: Page deleted, siblings, and parent updated
           not_found: Page not found
           layout_not_found: No Layouts found. Please create one.
 

test/controllers/occams/admin/cms/pages_controller_test.rb

@@ -219,7 +219,7 @@ def test_creation
         page = Occams::Cms::Page.last
         assert_equal @site, page.site
         assert_redirected_to action: :edit, id: page
-        assert_equal 'Page created', flash[:success]
+        assert_equal 'Page created, siblings, and parent updated', flash[:success]
       end
     end
   end
@@ -255,7 +255,7 @@ def test_creation_with_files
           page = Occams::Cms::Page.last
           assert_equal @site, page.site
           assert_redirected_to action: :edit, id: page
-          assert_equal 'Page created', flash[:success]
+          assert_equal 'Page created, siblings, and parent updated', flash[:success]
         end
       end
     end
@@ -333,7 +333,7 @@ def test_destroy
         r :delete, occams_admin_cms_site_page_path(site_id: @site, id: @page)
         assert_response :redirect
         assert_redirected_to action: :index
-        assert_equal 'Page deleted', flash[:success]
+        assert_equal 'Page deleted, siblings, and parent updated', flash[:success]
       end
     end
   end

test/system/zeitwerk_compliance_test.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class ZeitwerkComplianceTest < ActiveSupport::TestCase
+  test 'eager loads all files without errors' do
+    assert_nothing_raised { Rails.application.eager_load! }
+  end
+end