3.1: params_wrapper raises error on anynomous controllers

[dchelimsky]

Given this code:

Class.new(ApplicationController)

I get the following error:

+/Users/david/.rvm/gems/ruby-1.9.2-p180@rspec-dev/gems/activesupport-3.1.0.beta1/lib/active_support/whiny_nil.rb:48:in `method_missing': undefined method `sub' for nil:NilClass (NoMethodError)
+   from /Users/david/.rvm/gems/ruby-1.9.2-p180@rspec-dev/gems/actionpack-3.1.0.beta1/lib/action_controller/metal/params_wrapper.rb:140:in `_default_wrap_model'

This is caused by model_name = self.name.sub(/Controller$/, '').singularize on

rails/actionpack/lib/action_controller/metal/params_wrapper.rb

Line 143 in 8b0262f

model_name = self.name.sub(/Controller$/, '').singularize

I've added a failing test case, and will offer up a solution in the next day or so.

[alindeman]

What's the expected behavior here? Should param wrapping simply be ignored?

[josevalim]

@alindeman we should just not try to wrap things automatically then. you would have to be explicit.

[alindeman]

Awesome @dchelimsky, would love to get this merged for beta2. Adding a comment because GitHub doesn't notify pull request participants simply when new commits are added.