Migration from Rails 2.3.5 to Rails 8.1 successful.

Merging dev branch.

1 files changed, 22 insertions, 17 deletions

diff --git a/app/controllers/content_controller.rb b/app/controllers/content_controller.rb
index 1b13456..8d33105 100644
--- a/app/controllers/content_controller.rb
+++ b/app/controllers/content_controller.rb
@@ -1,30 +1,31 @@
 class ContentController < ApplicationController
-  
+
   # Public
-  
-  before_filter :find_page
-  
+
+  before_action :find_page
+
   # This is the method that renders most of the the public content. It recieves
-  # a :locale and a :page_path parameter through the params hash. It looks up 
+  # a :locale and a :page_path parameter through the params hash. It looks up
   # the node with the corresponding unique_name attribute. The method doesn't
-  # return a node though, the node is really a proxy object for pages. It 
+  # return a node though, the node is really a proxy object for pages. It
   # returns the most recent page associated to this node instead.
   def render_page
-    
+
     expires_in 20.minutes, :public => true
-    
+
     if @page and @page.public?
       render(
-        :file => @page.valid_template,
+        :template => @page.valid_template,
         :layout => true
       )
     else
-      render( 
-        :file => File.join(RAILS_ROOT, 'public', '404.html'),
-        :status => 404
+      render(
+        :file => Rails.root.join('public', '404.html').to_s,
+        :status => 404,
+        :layout => false
       )
     end
-    
+
   end
 
   def render_gallery
@@ -32,13 +33,17 @@ class ContentController < ApplicationController
       @images = @page.assets.images
       render :file => "content/gallery"
     else
-      render :nothing => true, :status => 404
+      head :not_found
     end
   end
-  
+
   private
     def find_page
-      path = params[:page_path].join('/')
-      @page = Node.find_page(path)
+      path = params[:page_path].is_a?(Array) ? params[:page_path].join('/') : params[:page_path]
+      if path =~ /^[a-zA-Z\:\/\/\.\-\d_]+$/
+        @page = Node.find_page(path)
+      else
+        @page = nil
+      end
     end
 end