Using a Form_for with an each loop

Question

I'm trying to create a form in Rails that allows a user to select certain photos from a larger list using checkboxes. Unfortunately, I haven't found any similar examples and most posts out there are unhelpful. Any ideas on how to solve this?

 <div>
  <%= form_for :photos, url: trip_path, method: "PUT"  do |f| %>
  <% @photos.each_with_index do |image, index|%>    
    <img src="<%= image.url %>" ><br>
      <span> <%=image.caption%> | <%=image.lat  %> | <%= image.long  %>
        <%= f.hidden_field "url", :value => image.url %>
        <%=check_box_tag('photo')  %>
      </span>
    <% end %>
    <%= f.submit 'Submit'  %>
  <% end %>
</div>

Your snippet contains some egregious formatting/indenting inconsistencies, some of which may bear a material impact on whether or not your code renders correctly. For instance, you've indented your <span> inside of an <img> tag, despite the fact that an <img> tag has no closing tag. For the sake of clarity, please reformat your code and repost. — zeantsoi
– zeantsoi, Commented Jul 28, 2013 at 3:00

Helio Santos · Accepted Answer · 2013-07-28 00:37:52Z

1

API docs states that a form_for

creates a form and a scope around a specific model object

so, you cannot use it with a collection.

A possible way to do it, is to use the form_tag instead of form_for and check_box_tag (which you already have).

answered Jul 28, 2013 at 0:37

Helio Santos

6,8653 gold badges28 silver badges31 bronze badges

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

Comments

zeantsoi · Accepted Answer · 2013-07-28 20:15:52Z

0

The behavior you've depicted is categorically impossible using form_form. However, if you're willing to forgo form_for (and there's no reason why you shouldn't, given your criteria), you can imitate the behavior depicted by nesting a foreach loop – each loop containing a form_for block – within a form_tag:

<div>
    <%= form_tag trip_path, method: "PUT"  do |f| %>
        <% @photos.each do |photo|%>    
            <img src="<%= photo.url %>" ><br>
            <span> <%= photo.caption%> | <%= photo.lat  %> | <%= photo.long  %>
                <%= fields_for "photos[#{photo.id}]", photo do |p| %>
                    <%= p.hidden_field 'url', :value => photo.url %>
                    <%= p.check_box 'photo'
                <% end %>
            </span>
        <% end %>
        <%= f.submit 'Submit'  %>
    <% end %>
</div>

edited Jul 28, 2013 at 20:15

answered Jul 28, 2013 at 2:56

zeantsoi

26.3k7 gold badges71 silver badges61 bronze badges

2 Comments

Pavlov's_Dawg Over a year ago

Hmmm this is still not working for me getting an "undefined method `url' for #<ActionView::Helpers::FormBuilder:0x007fd078fcd5a0" Error message. Photos definitely have a url method though.

zeantsoi Over a year ago

Ah, yes, I had a small typo which I've corrected. Rather than passing p.url as the value for the url hidden field, pass photo.url instead. Please advise as to whether or not that fixes the issue.

Collectives™ on Stack Overflow

Using a Form_for with an each loop

2 Answers 2

Comments

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related