rails/rails — pull request #4515 by amatsuda (2012-01-18 09:42:01+00:00)
Title: some improvements on ActionView::Helpers::FormHelper refactoring

Here are some tiny improvements on AV::Helpers' great refactoring done in #4488 :rocket:

* removed needless module_eval

* memoized `field_type` and `select_type` at class level

* renamed AV::Helpers::Tags::Base#render to to_s

As for the `render` vs `to_s` method name, I did so because

* what this method does is nothing but just a `to_s`
* we probably can do something like `<%= SomeTag.new obj, method %>` in views, since ERB implicitly calls `to_s`

What do you think, @rafaelfranca?