Launchpad

Maybe more simplistic but… don’t you think it would be simpler to do set
::ActionController::Base.page_cache_directory = RAILS_ROOT + “/public/cache/…/”

on application.rb based on subdomains ?

Thanks

“Once you set it at initialize”

Indeed but I putted it in application.rb
I had to manage caching on multilingual website with some contents having the same url.
It’s set in a before_filter method and it worked fine for me.

Yes it is risky to put to change ActionController::Base.page_cache_directory for each request.

I have like this solution, but it did’nt work for me. It throws error. I have placed your code in application.rb

While starting the server it shows this error.
usr/lib/ruby/gems/1.8/gems/activesupport-2.2.2/lib/active_support/core_ext/module/aliasing.rb:34:in `alias_method’: undefined method `cache_page_with_site_specific_change’ for class `ApplicationController’ (NameError)

Please help me fix this.
Thanks in advance ;)

::ActionController::Base.page_cache_directory = RAILS_ROOT + “/public/cache/…/”

don’t forget the “::” (both) at the beginning

Hi Nate,

Thanks for your comment.
I have slightly changed you function. But the requirement is same. Cache path has to change based on the request.
I have placed the following piece of code in directly in application.rb

alias_method_chain :cache_page, :site_specific_change private def cache_page_with_site_specific_change(content = nil, options = nil) return unless @site.get_property(“cache_page”) == “true” path = case options when Hash url_for(options.merge(:only_path => true, :skip_relative_url_root => true, :format => params[:format])) when String options else request.path end path = “#{SITES_ROOT}/#{@site.short_name}/public/cache/#{@site.name}/#{path}” cache_page_without_subdomain(content, path) end

Code is not formatted properly.

alias_method_chain :cache_page, :site_specific_change private def cache_page_with_site_specific_change(content = nil, options = nil) return unless @site.get_property(“cache_page”) == “true” path = case options when Hash url_for(options.merge(:only_path => true, :skip_relative_url_root => true, :format => params[:format])) when String options else request.path end path = “#{SITES_ROOT}/#{@site.short_name}/public/cache/#{@site.name}/#{path}” cache_page_without_subdomain(content, path) end

I was having the same problem as Senthil, getting the " undefined method `cache_page_with_site_specific_change’ for class `ApplicationController’ (NameError)" error.

Adding a before_filter to my ApplicationController may not be the ideal solution, but it’s working.

The RewriteRules were what really helped me out. Thanks!

Thanks very much for this Nate – a neat solution. I’ve adapted it for multi-domains, and added in a fix: the method above overlooks the case where path is nil or /, and fails in Rails’ cache_page method as the generated path is a directory.

Also, the alias_method_chain should be after the method definition — some of your readers seem to be getting errors because of this.

  # From http://launchpad.rocketjumpindustries.com/posts/5-defining-a-dynamic-page-cache-loction-by-subdomain-in-rails
  #
  # Inserts the currently requested host as a directory in front of the
  # cached file location. Assuming you’ve moved the default page cache
  # storage location to /public/cache, a request to www.example.com would be
  # cached as:
  #
  # RAILS_ROOT/public/cache/www.example.com/index.html
  # (page_cache_directory/host/request_path)
  #
  def cache_page_with_domain(content = nil, options = nil)
    path = "/#{request.host}/"
    path << case options
      when Hash
        url_for(options.merge(:only_path => true, :skip_relative_url_root => true, :format => params[:format]))
      when String
        options
      else
        if request.path.empty? || request.path == '/'
          '/index'
        else
          request.path
        end
    end
    cache_page_without_domain(content, path)
  end
  alias_method_chain :cache_page, :domain

</code>