before_filter not cancelling action

Henry Collingridge picture Henry Collingridge · Jun 15, 2011 · Viewed 22.8k times · Source

I'm having trouble getting before filters to work in a Rails app I have recently upgraded from 1.9(?) to 2.3.11. To try and debug it, I have put a before_filter in a controller:

before_filter :false_filter

and the following in application_controller.rb:

def false_filter
  puts "false filter running"
  false
end

I then call the method from either cucumber/webrat or a browser, and while the filter is getting called (I can see the puts outputting the message), the filter chain isn't getting terminated.

I'm wondering if there's some boilerplate code that hasn't been generated. Can anyone suggest where to look?

Answer

mu is too short picture mu is too short · Jun 15, 2011

Nothing pays any attention to a before-filter's return value. If you want to stop processing, you have to render something from your filter or redirect to somewhere else, from the fine guide:

If a before filter renders or redirects, the action will not run. If there are additional filters scheduled to run after that filter they are also cancelled.

The same text appears in the 5.2.0 guide.

This behavior does make sense, if the filter chain doesn't complete (i.e. stops filtering part way through) then you'd end up calling controller methods with things not set up the way they were expecting them to be and that would just cause pain, suffering, and confusion and that wouldn't be at all friendly or fun.