Почему FormOptionsHelper collection_check_boxes (Rails 4.1.0) создает пустое значение (value="")?

Question

[DeepHill]

Есть две модели Article и Tag связаные HABTM:

article.rb

class Article < ActiveRecord::Base
	has_and_belongs_to_many :tags
end

tag.rb

class Tag < ActiveRecord::Base
	has_and_belongs_to_many :articles
end

Хочу вывести теги в виде чекбоксов в форме редактирования статьи:

_form.html.erb

<p>
    <%= f.label "Tags" %><br>
    <%= f.collection_check_boxes :tag_ids, Tag.all, :id, :name%>
  <p>

генерируется такой код:

html

<p>
    <label for="article_Tags">Tags</label><br>
    <input checked="checked" id="article_tag_ids_1" name="article[tag_ids][]" value="1" type="checkbox"><label for="article_tag_ids_1">Linux</label><input checked="checked" id="article_tag_ids_3" name="article[tag_ids][]" value="3" type="checkbox"><label for="article_tag_ids_3">Ubuntu</label><input checked="checked" id="article_tag_ids_4" name="article[tag_ids][]" value="4" type="checkbox"><label for="article_tag_ids_4">Windows</label><input name="article[tag_ids][]" value="" type="hidden">
  </p>

соответственно в params передается пустое значение:

Parameters:

"tag_ids"=>["3",
 ""]

UPD:
Спасибо за ответ Dimitriy:
Пустое значение используется, чтоб не писать обработку удаления связанных элеметнов, если не выбирите не одного чекбокса.

Answer 1

[DeepHill]

Проблема вот в этой строчке

<input name="article[tag_ids][]" value="" type="hidden">

которую генерирует collection_check_boxes

Спасибо за ответ @Vakiliy:
Пустое значение используется, чтоб не писать обработку удаления связанных элеметнов, если не выбирите не одного чекбокса.

Comments

[Dimitriy]

Что с ней не так то?

[DeepHill]

value=""

[DeepHill]

посмотрите html выше, в результате в params передаеться что-то вроде {"1","2","3",""}

[Dimitriy]

Ну, так и есть, после в params[:article][tag_ids] = ['1','2','3', '']
Вы потом куда используете полученные данные?

[DeepHill]

Не могу понять откуда пустое значение в массиве и как от него избавиться. В контролере я использую что-то вроде @article.tags << Tag.find(id) и это работает но пустое значение вызывает вполне ожидаемою ошибку Couldn't find Tag with 'id'= .

[Dimitriy]

Пустое значение используется, чтоб не писать обработку удаления связанных элеметнов, если не выбирите не одного чекбокса.

[DeepHill]

Не подскажете как его правильно обработать а то что-то я туплю :-(

[Dimitriy]

Так для связи можно просто указать массив, т.е @article.tag_ids = [1,23,], этот способ и используется по умолчанию, единственно что надо добавить tag_ids в разрешенный список.

[Dimitriy]

Просто добавтье к фильтру параметров tag_ids: [] в самом низу контроллера, должен быть метод, примерно вот такой в него и добавтье:
def article_params
params.require(:article).permit(....)

пример выше приводил :)

[DeepHill]

Спасибо @Vakiliy за ответ, про Strong Parameters я в курсе.

[Dimitriy]

Пожалуйста! Раньше была опция, для коллекции, include_hidden: true, сейчас смотрю ее нет в исходниках, так как сам пользуюcь таким спобом, ее отсутсивие и не заметил :)

[DeepHill]

Я все никак не могу привыкнуть что в Rails все продумано на несколько шагов в перед и нет нужды писать свои костыли :-)

Answer 2

[Dimitriy]

Подозреваю, что, в контроллере не хватает параметра для белого списка - tag_ids: []
params.require(:article).permit(...., tag_ids: [])

Answer 3

[Сергей Той]

railscasts.com/episodes/196-nested-model-form-part-1 (устаревший скринкаст, но даёт понимание проблемы).
А решение тут: api.rubyonrails.org/classes/ActiveRecord/NestedAtt...

Comments

[DeepHill]

collection_check_boxes возвращает массив с id тегов что мне и нужно, но в массиве присутствует пустое значение " " {"1", "2", "3", "!"} как мне поможет Nested Attributes вы бы не могли уточнить?

Answer 4

[shilovk]

Нужно задать include_hidden: false и тогда скрытый элемент не будет добавляться в форму.

<%= f.collection_check_boxes :tag_ids, Tag.all, :id, :name, include_hidden: false %>