This project is archived and is in readonly mode.
form_for in current rails master is broken (really)
Reported by z3cko | March 16th, 2010 @ 08:05 PM | in 3.0.2
= form_for :staticpage, :url => { :action => "update" } do |f|
outputs:
<form action="/staticpages" method="post">
the action gets stripped away.
seems to be something with the last commits, since it was working
two days ago.
Comments and changes to this ticket
-
z3cko March 16th, 2010 @ 08:06 PM
- Title changed from form_for in current rails master is broken (no to form_for in current rails master is broken (really)
-
Jeremy Kemper March 16th, 2010 @ 08:18 PM
- Milestone cleared.
- State changed from new to open
- Assigned user set to Carl Lerche
-
z3cko March 16th, 2010 @ 09:03 PM
i just confirmed that this issue also exists with erb, so it's definitely rails related and not haml related.
<%= form_for(:staticpage, @page, :url => { :action => "create" }) do |f| %> <%= f.label :title %> <% end %>
produces
<form action="/staticpages" method="post"> <label for="staticpage_title">Title</label> </form>
-
Yehuda Katz (wycats) March 17th, 2010 @ 10:19 PM
- State changed from open to resolved
I can't reproduce on master.
-
z3cko March 19th, 2010 @ 01:38 PM
sorry, but i can reproduce this on the current master.
nothing changed, same outcome. -
Denis Odorcic March 19th, 2010 @ 02:51 PM
I can't reproduce either, just tried with a fresh app. Perhaps you have some different routes setup that are triggering this?
-
z3cko March 21st, 2010 @ 11:37 AM
more specific: if you are trying to override the action that was intended,
i.e. update2 instead of update, the form_for helper does not accept the extra :url => {:action => "update2"}i hope the issue is more understandable now and can be reproduced.
-
Jeremy Kemper March 21st, 2010 @ 03:32 PM
- State changed from resolved to open
-
José Valim March 21st, 2010 @ 06:28 PM
@z3cko can you please provide a failing test case in Rails test suite or isolate the issue in an application?
-
Yehuda Katz (wycats) March 29th, 2010 @ 02:47 AM
- State changed from open to needs-more-info
-
Santiago Pastorino April 3rd, 2010 @ 09:58 PM
- State changed from needs-more-info to resolved
Works on the last version, thanks for your bug report.
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>