Merge pull request #143 from BenMorganMY/fix-inertia-errors

Call to_hash On inertia_errors for Session Serialization

Author: bknoles

lib/inertia_rails/controller.rb

@@ -109,7 +109,7 @@ def inertia_location(url)
 
     def capture_inertia_errors(options)
       if (inertia_errors = options.dig(:inertia, :errors))
-        session[:inertia_errors] = inertia_errors
+        session[:inertia_errors] = inertia_errors.to_hash
       end
     end
   end

spec/dummy/app/controllers/inertia_test_controller.rb

@@ -1,3 +1,7 @@
+class MyError
+  def to_hash() { uh: 'oh' } end
+end
+
 class InertiaTestController < ApplicationController
   layout 'conditional', only: [:with_different_layout]
 
@@ -43,6 +47,10 @@ def redirect_with_inertia_errors
     redirect_to empty_test_path, inertia: { errors: { uh: 'oh' } }
   end
 
+  def redirect_with_inertia_error_object
+    redirect_to empty_test_path, inertia: { errors: MyError.new }
+  end
+
   def redirect_back_with_inertia_errors
     redirect_back(
       fallback_location: empty_test_path,

spec/dummy/config/routes.rb

@@ -23,6 +23,7 @@
   get 'share_multithreaded_error' => 'inertia_multithreaded_share#share_multithreaded_error'
   get 'redirect_with_inertia_errors' => 'inertia_test#redirect_with_inertia_errors'
   post 'redirect_with_inertia_errors' => 'inertia_test#redirect_with_inertia_errors'
+  post 'redirect_with_inertia_error_object' => 'inertia_test#redirect_with_inertia_error_object'
   post 'redirect_back_with_inertia_errors' => 'inertia_test#redirect_back_with_inertia_errors'
   get 'error_404' => 'inertia_test#error_404'
   get 'error_500' => 'inertia_test#error_500'

spec/inertia/response_spec.rb

@@ -27,6 +27,15 @@
           expect(response.headers['Location']).to eq(empty_test_url)
           expect(session[:inertia_errors]).to include({ uh: 'oh' })
         end
+
+        it 'serializes :inertia_errors to the session' do
+          post redirect_with_inertia_error_object_path,
+            headers: { 'X-Inertia' => true }
+
+          expect(response.status).to eq 302
+          expect(response.headers['Location']).to eq(empty_test_url)
+          expect(session[:inertia_errors]).to include({ uh: 'oh' })
+        end
       end
     end
   end