action caching sets wrong content-type when :cache_path is a string

This bug occurs also with a Proc object that returns a string.

The gist-url with the patch was deleted, so I post an easy workaround for this bug. Simply set the content type yourself somewhere inside the macro, like:

caches_action :index, :if => Proc.new { |c| result = c.cache_me?; c.headers["Content-Type"] = 'text/xml; charset=UTF-8' if result; result },

:cache_path => Proc.new { |c| File.join('abc', c.some_method) }

Aw damn, I've been cleaning up my gists and deleted the patch because I thought it was applied. If anyone needs it back, shout now. I was pretty sure this was fixed soon after this was reported.

Christoph, can you make a failing test for your case?

Ok, I made the failing tests.

I choose to attach a whole rails project instead of a single file. Please look at docs/README for some explanation.

I am not an experienced tests writer, yet it seems to me, that I found two other bugs while writing the tests. Please look at the comments in test/functional/xml_caching_controller_test.rb therefore.

I would like to investigate that a little further, but I will be buzy for the next seven days.