I have a has_many through association and I'm getting an error because I'm not passing the :ingredient_id
as an array (or something like that, it's just a guess :) )
通过关联,我有一个has_many,我得到了一个错误,因为我没有将:component_id作为数组传递(或者类似的东西,只是猜测)
Error:Couldn't find Ingredient with ID=1 for MealIngredient with ID=
错误:找不到ID=1的食材ID=
Parameters:
{"utf8"=>"✓",
"authenticity_token"=>"oJQaOCcVavXpWTs4Zz88TWrBchkf7315AnxCJgoYGnpzOxqWd3tz1tiSFwZGDPmzVPpzkSDhFGdDY9j5bpoI1Q==",
"meal"=>{"name"=>"",
"meal_ingredients_attributes"=>{"1450070337804"=>{"ingredient_attributes"=>{"id"=>"1",
"name"=>"Banana"},
"ingredient_id"=>"2", # HERE THE ID IS BEING SENT
"quantity"=>"0.0",
"_destroy"=>"false"},
"1450070339919"=>{"ingredient_attributes"=>{"id"=>"2",
"name"=>"Grape"},
"ingredient_id"=>"", # HERE THERE'S NO ID BEING SENT
"quantity"=>"0.0",
"_destroy"=>"false"}}},
"commit"=>"Create Meal"}
This is how I'm handling this nested form:
这就是我处理这个嵌套表单的方式:
Ingredient.rb
Ingredient.rb
class Ingredient < ActiveRecord::Base
has_many :meal_ingredients
has_many :meals, through: :meal_ingredients
accepts_nested_attributes_for :meal_ingredients, :reject_if => :all_blank
end
Meal.rb
Meal.rb
class Meal < ActiveRecord::Base
has_many :meal_ingredients
has_many :ingredients, through: :meal_ingredients
accepts_nested_attributes_for :ingredients, :reject_if => :all_blank, :allow_destroy => true
accepts_nested_attributes_for :meal_ingredients
end
MealIngredient.rb
MealIngredient.rb
class MealIngredient < ActiveRecord::Base
belongs_to :meal
belongs_to :ingredient
accepts_nested_attributes_for :ingredient, :reject_if => :all_blank
end
_form.html.erb
_form.html.erb
<div class = "row">
<div class= "col-md-10 col-md-offset-1">
<%= simple_form_for(@meal) do |f| %>
<%= f.error_notification %>
<div class="form-inputs">
<%= f.input :name %>
</div>
<h3>ingredientes</h3>
<fieldset id="ingredients">
<ol>
<%= f.fields_for :meal_ingredients do |meal_ingredient| %>
<%= render "meal_ingredient_fields", :f => meal_ingredient %>
<% end %>
</ol>
<div class="form-actions">
<div><%= link_to_add_association 'adicionar ingrediente', f, :meal_ingredients, 'data-association-insertion-node' => "#ingredients ol", 'data-association-insertion-method' => "append", :wrap_object => Proc.new {|quantity| quantity.build_ingredient; quantity }, :class => "btn btn-default" %></div>
</div>
</fieldset>
<div class="form-actions">
<div><%= f.button :submit, class: "btn btn-success" %></div>
</div>
<% end %>
</div>
</div>
_meal_ingredient_fields.html.erb
_meal_ingredient_fields.html.erb
<div class = "nested-fields">
<table class= "table">
<thead>
<tr>
<td>
<%= f.label "Nome" %>
</td>
<td>
<%= f.label "Unidade" %>
</td>
<td>
<%= f.label "Carbo" %>
</td>
<td>
<%= f.label "Prot" %>
</td>
<td>
<%= f.label "Gordura" %>
</td>
<td>
<%= f.label "Quantidade" %>
</td>
<td>
<%= f.label "kcal" %>
</td>
</tr>
</thead>
<tbody>
<tr>
<%= f.fields_for :ingredient do |fi| %>
<%= fi.hidden_field :id %> ## EDIT 1: THIS LINE WAS REMOVED
<td scope="row" class="col-md-4">
<%= fi.autocomplete_field :name, autocomplete_ingredient_name_ingredients_path, :id_element => '#ingredient_id', class: "form-control" %>
</td>
<td class="col-md-1">
<%= fi.text_field :unit, required: true, class: "form-control" %>
</td>
<td class="col-md-1">
<%= fi.text_field :carb, required: true, :pattern => '^\d+(\.\d+)*$', title: "Apenas números separados por pontos", class: "form-control " %>
</td>
<td class="col-md-1">
<%= fi.text_field :prot, required: true, :pattern => '^\d+(\.\d+)*$', title: "Apenas números separados por pontos", class: "form-control " %>
</td>
<td class="col-md-1">
<%= fi.text_field :fat, required: true, :pattern => '^\d+(\.\d+)*$', title: "Apenas números separados por pontos", class: "form-control " %>
</td>
<% end %>
<td class="col-md-1">
<%= f.hidden_field :ingredient_id %> ## EDIT 1: FOLLOWING CODE WAS REMOVED: :id => 'ingredient_id'
<%= f.number_field :quantity, required: true, :pattern => '^\d+(\.\d+)*$', title: "Apenas números separados por pontos", class: "form-control" %>
</td>
<td class="col-md-1">
</td>
<td class="col-md-1">
<%= link_to_remove_association "remove item", f, :class => "btn btn-danger" %>
</td>
</tr>
</tbody>
</table>
</div>
Finally, the controller: meals_controller.rb
最后,控制器:meals_controller.rb
class MealsController < ApplicationController
before_action :set_meal, only: [:edit, :update, :show]
def index
@meals = Meal.all
end
def show
end
def new
@meal = Meal.new
@ingredient = @meal.ingredients.build
end
def create
@meal = Meal.new(meal_params)
if @meal.save
flash[:success] = "Refeição criada com sucesso!"
redirect_to meals_path
else
render :new
end
end
def edit
end
def update
if @meal.update(meal_params)
flash[:success] = "Your meal was updated succesfully!"
redirect_to meal_path(@meal)
else
render :edit
end
end
private
def meal_params
params.require(:meal).permit(:name, :picture, :tcarb, :tprot, :tfat, :tkcal, meal_ingredients_attributes: [:ingredient_id, :quantity, ingredient_attributes: [:id, :name, :unit, :carb, :prot, :fat, :_destroy]])
end
def set_meal
@meal = Meal.find(params[:id])
end
end
Please help, this is giving me headaches :D
请帮帮我,这让我头疼
EDIT
编辑
ingredients.js I'm using to gather the current id set by cocoon and apply into the fields when an ingredient is selected in autocomplete.
$(function(){
var num;
var id;
$('#ingredients').on('cocoon:after-insert', function(e, insertedItem) {
$('input[id^="meal_meal_ingredients_attributes_"][id$="_ingredient_attributes_name"]').each(function(){
var current = $(this);
var rx = /meal_meal_ingredients_attributes_(.*)_ingredient_attributes_name/;
num = rx.exec(current.attr('id'))[1];
id = '#meal_meal_ingredients_attributes_'+ num +'_ingredient_';
});
$(id + 'attributes_name').bind('railsAutocomplete.select', function(event, data){
$( id + 'id' ).val( data.item.id);
$( id + 'attributes_unit' ).val( data.item.unit).prop('disabled', true);
$( id + 'attributes_carb' ).val( data.item.carb).prop('disabled', true);
$( id + 'attributes_prot' ).val( data.item.prot).prop('disabled', true);
$( id + 'attributes_fat' ).val( data.item.fat).prop('disabled', true);
});
});
});
1 个解决方案
#1
2
The error looks pretty inconspicuous; I don't think this will answer your question directly, but I'll take a shot anyway to see if it helps:
这个错误看起来很不明显;我不认为这能直接回答你的问题,但我还是要试一试,看看是否有帮助:
{"utf8"=>"✓",
"authenticity_token"=>"oJQaOCcVavXpWTs4Zz88TWrBchkf7315AnxCJgoYGnpzOxqWd3tz1tiSFwZGDPmzVPpzkSDhFGdDY9j5bpoI1Q==",
"meal"=>
{"name"=>"",
"meal_ingredients_attributes"=>
{
"1450070337804"=>
{ "ingredient_attributes"=> {"id"=>"1","name"=>"Banana"},
"ingredient_id"=>"2", # HERE THE ID IS BEING SENT
"quantity"=>"0.0",
"_destroy"=>"false"
},
"1450070339919"=>
{ "ingredient_attributes"=> {"id"=>"2", "name"=>"Grape"},
"ingredient_id"=>"", # HERE THERE'S NO ID BEING SENT
"quantity"=>"0.0",
"_destroy"=>"false"
}
}
},
"commit"=>"Create Meal"
}
I tried to determine whether you're actually making a new ingredient, or calling an existing one.
我试着去判断你是在生产一种新的原料,还是在调用现有的原料。
You're using the following fields:
您正在使用以下字段:
<%= f.fields_for :ingredient do |fi| %>
<%= fi.hidden_field :id %>
<% end %>
<%= f.hidden_field :ingredient_id, :id => 'ingredient_id' %>
The ingredient_id
would suggest you're calling an existing ingredient; the id
field suggests you're trying to create a new one. This is probably where the error arises.
这个成分说明你在调用一个现有的成分;id字段表明您正在尝试创建一个新的。这可能是产生错误的地方。
- Why are you setting
id
for aningredient
?- 为什么要为原料设置id ?
- Why are you then referring to
ingredient_id
formeal_ingredient
?- 那么,您为什么要引用见证成分的id呢?
I would personally get rid of id
for the new ingredient, and make ingredient_id
dependent on whether you have a new ingredient (it's set automatically if you create a new nested record).
我个人会去掉新成分的id,并根据是否有新成分(如果您创建一个新的嵌套记录,它将自动设置)来设置。
I would also ensure that ingredient_attributes
are only accepted if they're all filled in (which I think you have already):
我还将确保,只有当所有的成分都被填满时(我认为您已经填满了),才可以接受这些成分。
#app/models/meal_ingredient.rb
class MealIngredient < ActiveRecord::Base
accepts_nested_attributes_for :ingredient, reject_if: :all_blank
end
You don't need to explicitly define id
for a nested record:
不需要为嵌套记录显式定义id:
<%= f.fields_for :ingredient do |fi| %>
# remove ID, you don't need it
<%= fi.text_field :name %>
To manage the ingredient_id
vs ingredient_attributes
thing, you'll probably have to use JS in the front-end (I'll update an answer with this if required); the backend can be handled by using the following:
要管理连接成分表id和连接表属性,您可能需要在前端使用JS(如果需要,我将用这个更新答案);后端可用以下方法处理:
#app/models/meal_ingredient.rb
class MealIngredient < ActiveRecord::Base
validate :ingredient_set
private
def ingredient_set
errors.add(:ingredient_id, "Need either a new ingredient, or to select one") unless ingredient && ingredient_id.blank?
end
end
Also, it looks like your strong params need to be tweaked:
同时,你的强参数也需要调整:
def meal_params
params.require(:meal).permit(:name, :picture, :tcarb, :tprot, :tfat, :tkcal, meal_ingredients_attributes: [:ingredient_id, :quantity, ingredient_attributes: [:name, :unit, :carb, :prot, :fat, :_destroy]])
end
#1
2
The error looks pretty inconspicuous; I don't think this will answer your question directly, but I'll take a shot anyway to see if it helps:
这个错误看起来很不明显;我不认为这能直接回答你的问题,但我还是要试一试,看看是否有帮助:
{"utf8"=>"✓",
"authenticity_token"=>"oJQaOCcVavXpWTs4Zz88TWrBchkf7315AnxCJgoYGnpzOxqWd3tz1tiSFwZGDPmzVPpzkSDhFGdDY9j5bpoI1Q==",
"meal"=>
{"name"=>"",
"meal_ingredients_attributes"=>
{
"1450070337804"=>
{ "ingredient_attributes"=> {"id"=>"1","name"=>"Banana"},
"ingredient_id"=>"2", # HERE THE ID IS BEING SENT
"quantity"=>"0.0",
"_destroy"=>"false"
},
"1450070339919"=>
{ "ingredient_attributes"=> {"id"=>"2", "name"=>"Grape"},
"ingredient_id"=>"", # HERE THERE'S NO ID BEING SENT
"quantity"=>"0.0",
"_destroy"=>"false"
}
}
},
"commit"=>"Create Meal"
}
I tried to determine whether you're actually making a new ingredient, or calling an existing one.
我试着去判断你是在生产一种新的原料,还是在调用现有的原料。
You're using the following fields:
您正在使用以下字段:
<%= f.fields_for :ingredient do |fi| %>
<%= fi.hidden_field :id %>
<% end %>
<%= f.hidden_field :ingredient_id, :id => 'ingredient_id' %>
The ingredient_id
would suggest you're calling an existing ingredient; the id
field suggests you're trying to create a new one. This is probably where the error arises.
这个成分说明你在调用一个现有的成分;id字段表明您正在尝试创建一个新的。这可能是产生错误的地方。
- Why are you setting
id
for aningredient
?- 为什么要为原料设置id ?
- Why are you then referring to
ingredient_id
formeal_ingredient
?- 那么,您为什么要引用见证成分的id呢?
I would personally get rid of id
for the new ingredient, and make ingredient_id
dependent on whether you have a new ingredient (it's set automatically if you create a new nested record).
我个人会去掉新成分的id,并根据是否有新成分(如果您创建一个新的嵌套记录,它将自动设置)来设置。
I would also ensure that ingredient_attributes
are only accepted if they're all filled in (which I think you have already):
我还将确保,只有当所有的成分都被填满时(我认为您已经填满了),才可以接受这些成分。
#app/models/meal_ingredient.rb
class MealIngredient < ActiveRecord::Base
accepts_nested_attributes_for :ingredient, reject_if: :all_blank
end
You don't need to explicitly define id
for a nested record:
不需要为嵌套记录显式定义id:
<%= f.fields_for :ingredient do |fi| %>
# remove ID, you don't need it
<%= fi.text_field :name %>
To manage the ingredient_id
vs ingredient_attributes
thing, you'll probably have to use JS in the front-end (I'll update an answer with this if required); the backend can be handled by using the following:
要管理连接成分表id和连接表属性,您可能需要在前端使用JS(如果需要,我将用这个更新答案);后端可用以下方法处理:
#app/models/meal_ingredient.rb
class MealIngredient < ActiveRecord::Base
validate :ingredient_set
private
def ingredient_set
errors.add(:ingredient_id, "Need either a new ingredient, or to select one") unless ingredient && ingredient_id.blank?
end
end
Also, it looks like your strong params need to be tweaked:
同时,你的强参数也需要调整:
def meal_params
params.require(:meal).permit(:name, :picture, :tcarb, :tprot, :tfat, :tkcal, meal_ingredients_attributes: [:ingredient_id, :quantity, ingredient_attributes: [:name, :unit, :carb, :prot, :fat, :_destroy]])
end