Álvaro Bautista

You're right, I didn't take that into account. Here it is a patch that does.

Well, I did misunderstood. ;)

However, I don't see any problem at all.

If you take a look at the test file for cookie stored sessions [http://github.com/rails/r...

I kind of found a workaround, it consists of wrapping the yield call with with_output_buffer:

def wadus
  with_output_buffer { yield }
end

That implies that a...

I run your example and I got the same result.

In fact, it seems to happen when yield is called within any helper method. For instance, having the following help...

I tried to reproduce it, but it works fine for me. Having the following action:

def nothing
  session[:is_there] = "anything wrong?"
  render :nothing => true
...

My patch was, in fact, not needed.

I'm sorry, I should have refreshed the ticket page before submitting.

I agree, since fetch is already overridden.

It may not be needed, as it is quite straightforward, but I attach a patch for it anyway.

The ActiveSupport::TimeWithZone#localtime method is not covered by any test. It fails when calling getlocal on a DateTime instance. A quick (and simple) workaro...

After a quick look at the AR source code it seems that skipping callbacks in this method is intentional:

# Updates a single attribute and saves the record.  
#...

I found out my mistake. The method that fails is update_attribute. So my previous diff didn't fail because I was using update_attributes.

I attach the right pat...