query_reviewer breaks send_file in Rails 3.0

[jshraibman-mdsol]

The problem is this code ( https://github.com/nesquena/query_reviewer/blob/master/lib/query_reviewer/controller_extensions.rb#L42 )

        if response.body.is_a?(String) && response.body.match(/<\/body>/i)

The problem is that as soon as you call response.body the response is ruined and a blank page is shown to the user. You might want to replace response.body.is_a?(String) with response.instance_eval{ @body.is_a?(String) }

[nesquena]

Would you be able to send a quick pull request? happy to patch it in.

[jshraibman-mdsol]

When I try to run rake test I get:

[jshraibman@Joseph-Shraibmans-MacBook-Pro master ~/work/query_reviewer]$ rake test rake aborted! ERROR: 'rake/rdoctask' is obsolete and no longer supported. Use 'rdoc/task' (available in RDoc 2.4.2+) instead. /Users/jshraibman/work/query_reviewer/Rakefile:3:in `<top (required)>' (See full trace by running task with --trace)