Rails how to have multiple forms by putting form_for inside .each loop?

Question

I have a page where I list out a bunch of loans that a user has and I want a form tag around each one of those forms because they can make a payment on any of the loans that are listed.

Here's what I have that is throwing the error "Undefined method 'loan'..."

<% @loans.each do |loan| %>

    <% form_for loan add_payment_payment_path(loan) , :method => :post do |f| %>                                       
        <%= f.text_field :amount, :placeholder => 'Amount' %>
        <%= f.submit 'Make Payment', :class => 'btn btn-primary'%>
    <% end %>

<% end %>

How can i have a dynamic form_for inside my @loans.each loop?

zkcro · Accepted Answer · 2013-05-17 06:44:57Z

4

You're form_for call is incorrect, it should be:

<% form_for loan, url: add_payment_payment_path(loan), method: :post do |f| %>

You're missing the comma after loan, so it's getting interpreted as a method with add_payment_payment_path(loan) as its argument.

Also note that passing a url to form_for is done via the options hash, using the :url key. It is not a separate parameter, and passing it as such will get you a wrong number of arguments (3 for 2) error.

edited May 17, 2013 at 6:44

answered May 17, 2013 at 6:07

zkcro

4,3441 gold badge26 silver badges22 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Catfish Over a year ago

Ok the page loads now, but the form does not render.

zkcro Over a year ago

What do you mean by doesn't render? Doesn't appear on the page at all? Have you inspected the HTML to see what is being output? This sounds somewhat odd, and is quite possibly a separate issue.

Catfish Over a year ago

I believe it is a separate issue after looking at it again.

fontno · Accepted Answer · 2013-05-17 06:23:08Z

1

You have two syntax errors in your form_for method. Your missing a comma after loan and also you have an extra space after add_payment_payment_path(loan)

<% form_for loan add_payment_payment_path(loan) , :method => :post do |f| %>

Change this to

<% form_for loan, add_payment_payment_path(loan), :method => :post do |f| %>

Remember in Ruby parenthesis are optional in most cases, however, you must keep only one space. Here is the same form_for method with the parenthesis

<% form_for(loan, add_payment_payment_path(loan), :method => :post) do |f| %>

answered May 17, 2013 at 6:23

fontno

6,9027 gold badges38 silver badges44 bronze badges

Collectives™ on Stack Overflow

Rails how to have multiple forms by putting form_for inside .each loop?

2 Answers 2

3 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related