Turbolinks legacy module

src/lucky/redirectable.cr

@@ -122,38 +122,16 @@ module Lucky::Redirectable
   def redirect(to path : String, status : Int32 = 302) : Lucky::TextResponse
     # flash messages are not consumed here, so keep them for the next action
     flash.keep
-
-    if ajax? && request.method != "GET"
-      context.response.headers.add "Location", path
-
-      # do not enable form disabled elements for XHR redirects, see https://github.com/rails/rails/pull/31441
-      context.response.headers.add "X-Xhr-Redirect", path
-
-      Lucky::TextResponse.new(context,
-        "text/javascript",
-        %[Turbolinks.clearCache();\nTurbolinks.visit(#{path.to_json}, {"action": "replace"})],
-        status: 200)
-    else
-      if request.headers["Turbolinks-Referrer"]?
-        store_turbolinks_location_in_session(path)
-      end
-      # ordinary redirect
-      context.response.headers.add "Location", path
-      context.response.status_code = status
-      Lucky::TextResponse.new(context, "", "")
-    end
+    context.response.headers.add "Location", path
+    context.response.status_code = status
+    Lucky::TextResponse.new(context, "", "")
   end
 
   # :nodoc:
   def redirect(to page_instead_of_action : Lucky::HTMLPage.class, **unused_args)
     {% raise "You accidentally redirected to a Lucky::HTMLPage instead of a Lucky::Action" %}
   end
 
-  private def store_turbolinks_location_in_session(path : String)
-    cookies.set(:_turbolinks_location, path).http_only(true)
-    # this cookie read at Lucky::RedirectableTurbolinksSupport
-  end
-
   private def allowed_host?(referer : String)
     if referer_host = URI.parse(referer).hostname
       referer_host == request.hostname

src/lucky/redirectable_turbolinks_support.cr

@@ -4,6 +4,35 @@
 # This pipe extracted Lucky::Redirectable, because Lucky::Redirectable included to Lucky::ErrorAction
 # but Lucky::ErrorAction not have pipe support
 module Lucky::RedirectableTurbolinksSupport
+  # Overrides Lucky::Redirectable redirect's method
+  def redirect(to path : String, status : Int32 = 302) : Lucky::TextResponse
+    # flash messages are not consumed here, so keep them for the next action
+    flash.keep
+    if ajax? && request.method != "GET"
+      context.response.headers.add "Location", path
+
+      # do not enable form disabled elements for XHR redirects, see https://github.com/rails/rails/pull/31441
+      context.response.headers.add "X-Xhr-Redirect", path
+
+      Lucky::TextResponse.new(context,
+        "text/javascript",
+        %[Turbolinks.clearCache();\nTurbolinks.visit(#{path.to_json}, {"action": "replace"})],
+        status: 200)
+    else
+      if request.headers["Turbolinks-Referrer"]?
+        store_turbolinks_location_in_session(path)
+      end
+      # ordinary redirect
+      context.response.headers.add "Location", path
+      context.response.status_code = status
+      Lucky::TextResponse.new(context, "", "")
+    end
+  end
+
+  private def store_turbolinks_location_in_session(path : String)
+    cookies.set(:_turbolinks_location, path).http_only(true)
+  end
+
   macro included
     before set_turbolinks_location_header_from_session
   end