Nick Quaranto

Added some tests and refactored Tomasz's implementation. You'll have to apply his patch then mine to get this working cleanly.

That was a minus 1. Seems like LH likes to eat my formatting.

1. This seems like a strange use case, I can see it maybe for CI servers and the like, but it doesn't seem helpful enough for normal development.

Looks like this is an issue with Rails 3.0 as well, and since the Response class is now under ActionDispatch it needed a new patch. Here's patches for both 2-3-...

Looks like ActionController already supports this. From actionpack/test/controller/rack_test.rb:

  def test_streaming_block
    @response.body = Proc.new do |r...

Verified with 2-3-stable, used the same example and it breaks with both a proc and lambda.

2893 master % curl http://localhost:3000/lines/show
This is line 0
c...

-1, I don't like the alias_method to has_#{name} and it seems that keeping a similar api to the dirty methods would be better. For example, if a Post has many C...

+1 to this, definitely seeing it happen here.

% rails1.9 -v
Rails 2.3.3
% rails1.9 plugintest
% cd plugintest
% ruby1.9 script/plugin install git://github.com/...

I love this idea, but I'm not necessarily sure if this should be in the core. I think this extended out to more parts of the framework could be a great way to b...

+1 for this. During the process of converting Clearance to a Rails engine, we had to hack into the routes loading process so it was possible to override routes ...