So I’m learning the latest Rails release by converting an existing app to it. This app used namespaced controllers for its admin section.
New routes features have really cleaned this up.
Before
map.resources :users,
:controller => 'admin/users',
:name_prefix => 'admin_',
:path_prefix => 'admin'
After
map.namespace(:admin) do |admin|
admin.resources :users
end
Much cleaner.
Then I began to take advantage of Rails 2.0’s smarter #form_for courtesy of
simple_helpful in the non-admin portion of the app.
<% form_for @user do |form| %>
<% end %>
If that user object is a #new_record? that #form_for is going to generate
the following html:
<form action="/users" method="post" class="new_user" id="new_user">
</form>
If that user object is not a #new_record? that #form_for is going to
generate the following html (assuming its id is 1):
<form action="/users/1" method="post" class="edit_user" id="edit_user_1">
<div style="margin:0;padding:0">
<input name="_method" type="hidden" value="put" />
</div>
</form>
Now let’s look at the #new and #edit forms for the admin section of the site.
<% form_for [:admin, @user] do |form| %>
<% end %>
This will generate the same html as above except with ‘/admin’ being prefixed
onto each action attribute value.
[:admin, @user]
Ooo that is ugly.
Some more examples.
redirect_to [:admin, @user] # redirect_to admin_user_url
link_to 'show', [:admin, @user] # link_to 'show', admin_user_url
This anonymous Array syntax seems like such a quick hack, like they forgot
about namespaced controllers. With all the nice interfaces in Rails, this one
really sticks out. I’m willing to bet this becomes deprecated in the near
future.
In the meantime I’m going to stick with the much better looking old style.
<% form_for @user, :url => admin_user_url do |form| %>
<% end %>
However, this sucks because I’d really like to take advantage of these smarter
#form_for, #redirect_to and #link_to versions but I consider consistency
in my code more important. I’d rather see 1 consistent way of using named
routes instead of using them solely in the namespaced controller sections of an
app in order to avoid this new ugly syntax.
This would probably be a lot cleaner in PHP.