Как реализовать главную страницу для добавления товаров в корзину с помощью nested attributes?

Question

[kstepanovdev]

Нужен совет, если сможете помочь — выручайте, а то борода.
Не могу сделать добавление в корзину через единственную кнопку. Rails 5. Суть в чём — обычно для каждого товара генерируется форма с кнопкой и, например, окошком для указания количества товаров.
Мне надо сделать, по сути, единственную форму для всего этого добра, чтобы по нажатию "Add to Cart" в корзину падали те товары, где указанное количество в окошках не ноль.
Я так понимаю, надо юзать nested_attributes, чтобы передавать массив параметров, но пока всё, что у меня получилось — это ничего. Может, кто поможет советом.
Есть три модели, по сути.

Cart
has_many :cart_items

Products
has_many :cart_items
#тут, я так понимаю, нужно это
accepts_nested_attributes_for :cart_items

CartItems
belongs_to :cart
belongs_to :product

Выходит такая форма, которая отправляет в массиве только данные последнего товара, а не всех.

<%= form_for @order_item, remote: true do |f| %> 
  <%= f.fields_for :products do |products_form| %> 
    <% @products.each do |product| %>
      <%= product.name %>
      Unit Price:<%= number_to_currency product.price %> 
      <%= f.number_field :quantity, value: 1, class: "form-control", min: 1 %>
      <%= f.hidden_field :product_id, value: product.id %>
    <% end %>
  <% end %>
  <%= f.submit "Add to Cart", class: "btn btn-primary" %>
<% end %>

Вот такая байда. Если есть мысли и/или ссаные тряпки, то я готов :)

Comments

[Антон Костин]

Немного тыкая пальцем в небо, но мне кажется странным, что вы передаете f.number_field, т.е. значение пойдет в параметры @order_item. Что будет, если попробовать products_form.number_field ?

[kstepanovdev]

Антон Костин: да, это мой фейл. По-разному пробовал передавать параметры, писал в спешке и оставил странный вариант. В общем-то говоря, по-вашему я тоже передавал и выглядит неплохо, что неудивительно, но передаётся только значение последнего продукта, а не всех трёх, что странно. Вот таким образом
Parameters: {"utf8"=>"✓", "order_item"=>{"products"=>{"quantity"=>"1", "product_id"=>"3"}}, "commit"=>"Add to Cart"}

[kstepanovdev]

Антон Костин: https://github.com/kstepanovdev/wtf_cart

[Антон Костин]

kstepanovdev: Загрузил ваш проект. У вас в order_item.rb стоит

class OrderItem < ApplicationRecord
  belongs_to :productS

А нужно без S. Но это мелочи.

Мне кажется тут нужно больше над самой логикой поработать. Сейчас вы передаете значение в параметр "products" у order_item.

Started POST "/order_items" for ::1 at 2016-10-03 21:59:37 +0300
Processing by OrderItemsController#create as JS
  Parameters: {"utf8"=>"✓", "order_item"=>{"products"=>{"quantity"=>"6", "product_id"=>"2"}}, "commit"=>"Add to Cart"}
Unpermitted parameter: products
   (0.2ms)  BEGIN
   (0.2ms)  ROLLBACK

Т.е. для того, что бы решить ошибку, нужно, что бы у order_item был has_many :products и туда же запихнуть accepts_nested_attributes_for. Но тогда у вас возможно будет в некотором роде странная связь - и у order_item может быть много products и у product может быть много order_items.

Ну и последнее - вы пробуете в одно поле order_item'а (products) запихнуть все продукты в виде хеша, у которого не может быть одинаковых ключей. Т.е. вот тут
{"products"=>{"quantity"=>"6", "product_id"=>"2"}} не совсем понятно, как это должно работать с кол-вом товаров > 1

Мне кажется (но я не на столько опытный разработчик), что вам лучше для batch-добавления добавить ещё одну модель, что-то вроде multiple_products и в форме использовать array-значения для полей (соответсвенно у order_item должен быть has_many :multiple_products и accepts_nested_attributes_for :multiple_products

<h3 class="text-center">Products for Sale</h3>

<%= form_for @order_item, remote: true do |f| %> 
  <%= f.fields_for :multiple_products do |products_form| %> 
    <% @products.each do |product| %>
      <%= product.name %>
      Unit Price:<%= number_to_currency product.price %> 
      <%= products_form.number_field "quantity[]", value: 1, class: "form-control", min: 1 %>
      <%= products_form.hidden_field "product_id[]", value: product.id %>     
    <% end %>
  <% end %>
  <%= f.submit "Add to Cart", class: "btn btn-primary" %>
<% end %>

Надеюсь, если не помог, то хотя бы навел на нужную мысль)

[kstepanovdev]

Антон Костин: спасибо за ответ, но, к сожалению, это особо ни к чему не привело :)

[Антон Костин]

kstepanovdev: Понимаю, поэтому и не писал в виде ответа) Рад, что товарищ Givandos помог

Answer 1

[Givandos]

<%= f.fields_for :products do |products_form| %>
<% @products.each do |product| %>

По сути хелпер "fields_for" уже есть метод перебора, так что тут "each" лишний.
Следовательно строка

<%= f.hidden_field :product_id, value: product.id %>

станет попроще:
<%= products_form.hidden_field :product_id %>

Вообще я бы рекомендовал что-то типа такого:

<%= form_for @order, remote: true do |f| %>
    <%= f.fields_for :order_items, @products do |product|
        <%= product.hidden_field :product_id %>
        <%= product.number_field :quantity, value: 1 %>
    <% end %>
<% end %>

Только в модели order не забудьте получить атрибуты дочерней модели - accepts_nested_attributes_for :order_items

В результате в реквесте (кстати, сам реквест должен указывать на контроллер, который будет сохранять модель order) у вас должно быть не просто {"products"=>{"quantity"=>"6", "product_id"=>"2"}}, а что-то типа
{"order_items_attributes"=>[{"quantity"=>"6", "product_id"=>"2"}, {"quantity"=>"3", "product_id"=>"5"}]}

Comments

[kstepanovdev]

Спасибо. Параметры передаются вменяемо. Надеюсь, это мне поможет, хотя это уже не имеет отношения к вопросу. Понятия не имею, как бы я сам догадался об этом. Сам вопрос решен, так что спасибо :)