Вызванный в цикле form_for перегружает идентификаторы и неправильно связывает поля и метки - PullRequest
1 голос
/ 18 февраля 2011

Rails любит присваивать всем моим полям одинаковые идентификаторы, когда они создаются в цикле, и это вызывает проблемы.

<% current_user.subscriptions.each do |s| %>
  <div class="subscription_listing">
    <%= link_to_function s.product.name, "toggle_delay(this)"%>
    in <%= s.calc_time_to_next_arrival %> days.
    <div class="modify_subscription">
        <%= form_for s, :url => change_subscription_path(s) do |f| %>
            <%= label_tag(:q, "Days to delay:") %>
            <%= text_field_tag(:query) %>
            <%= check_box_tag(:always) %>
            <%= label_tag(:always, "Apply delay to all future orders") %>
            <%= submit_tag("Change") %>
        <% end %>
        <%= link_to 'Destroy', s, :confirm => 'Are you sure?', :method => :delete %>
    </div>
  </div>
<% end %>

Производит

<div class="subscription_listing">
    <a href="#" onclick="toggle_delay(this); return false;">Pasta</a>
    in 57 days.
    <div class="modify_subscription">
        <form accept-charset="UTF-8" action="/subscriptions/7/change" class="edit_subscription" id="edit_subscription_7" method="post"><div style="margin:0;padding:0;display:inline"><input name="utf8" type="hidden" value="&#x2713;" /><input name="_method" type="hidden" value="put" /><input name="authenticity_token" type="hidden" value="s5LJffuzmbEMkSrez8b3KLVmDWN/PGmDryXhp25+qc4=" /></div>

            <label for="q">Days to delay:</label>
            <input id="query" name="query" type="text" />
            <input id="always" name="always" type="checkbox" value="1" />
            <label for="always">Apply delay to all future orders</label>
            <input name="commit" type="submit" value="Change" />
</form>     <a href="/subscriptions/7" data-confirm="Are you sure?" data-method="delete" rel="nofollow">Destroy</a>
    </div>

  </div>
  <div class="subscription_listing">
    <a href="#" onclick="toggle_delay(this); return false;">Gummy Bears</a>
    in 57 days.
    <div class="modify_subscription">
        <form accept-charset="UTF-8" action="/subscriptions/8/change" class="edit_subscription" id="edit_subscription_8" method="post"><div style="margin:0;padding:0;display:inline"><input name="utf8" type="hidden" value="&#x2713;" /><input name="_method" type="hidden" value="put" /><input name="authenticity_token" type="hidden" value="s5LJffuzmbEMkSrez8b3KLVmDWN/PGmDryXhp25+qc4=" /></div>
            <label for="q">Days to delay:</label>
            <input id="query" name="query" type="text" />

            <input id="always" name="always" type="checkbox" value="1" />
            <label for="always">Apply delay to all future orders</label>
            <input name="commit" type="submit" value="Change" />
</form>     <a href="/subscriptions/8" data-confirm="Are you sure?" data-method="delete" rel="nofollow">Destroy</a>
    </div>
  </div>

И это проблема, потому чтотеперь независимо от того, какое «Применить задержку ко всем будущим заказам», я выбираю его, всегда очень услужливо проверяю первое поле для меня.Как я могу переопределить идентификатор, не делая ничего уродливого и не похожего на рельсы?

1 Ответ

1 голос
/ 18 февраля 2011

Вы должны иметь возможность переопределить идентификатор, передав его в качестве опции

text_field_tag(:query, '', :id=>'unique id')
check_box_tag(:always, '1', false, :id=>'unique_id')

Например,

<% current_user.subscriptions.each_with_index do |s,i| %>
  <div class="subscription_listing">
    <%= link_to_function s.product.name, "toggle_delay(this)"%>
    in <%= s.calc_time_to_next_arrival %> days.
    <div class="modify_subscription">
        <%= form_for s, :url => change_subscription_path(s) do |f| %>
            <%= label_tag("query_#{i}", "Days to delay:") %>
            <%= text_field_tag(:query, '', :id=>"query_#{i}") %>
            <%= check_box_tag(:always, '1', false, :id=>"always_#{i}") %>
            <%= label_tag("always_#{i}", "Apply delay to all future orders") %>
            <%= submit_tag("Change") %>
        <% end %>
        <%= link_to 'Destroy', s, :confirm => 'Are you sure?', :method => :delete %>
    </div>
  </div>
<% end %>
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...