Как сохранить данные, отмеченные флажком simple_form? - PullRequest
0 голосов
/ 07 июня 2019

Я начинающий с Ruby on Rails, и я пытаюсь создать небольшое приложение, которое позволит людям заказывать выпечку у плиты на выбранный день.

Когда они выбирают выпечку с помощью флажка, я хотел бы, чтобы выбранная выпечка была сохранена как pastrie_id в бою за столом, но вместо этого у меня есть проблема:

Validation failed: Pastrie must exist.

ОБНОВЛЕНИЕ это работает:

<%= f.association :pastrie, as: :check_boxes, label: 'Pastrie' %>

У меня все еще есть проблема, у меня плохие параметры сохранения:

"fight"=>{"pastrie_id"=>["", "1"]},

Я перепробовал множество решений, найденных в stackoverflow, но, похоже, ничего не работает.

Я использую Rails 5.2.3

Итак, это моя схема:

ActiveRecord::Schema.define(version: 2019_06_06_094318) do

  create_table "cookers", force: :cascade do |t|
    t.string "name"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.string "pastrie"
  end

  create_table "events", force: :cascade do |t|
    t.datetime "date"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

  create_table "fights", force: :cascade do |t|
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.integer "pastrie_id"
    t.integer "event_id"
    t.integer "cooker_id"
    t.index ["cooker_id"], name: "index_fights_on_cooker_id"
    t.index ["event_id"], name: "index_fights_on_event_id"
    t.index ["pastrie_id"], name: "index_fights_on_pastrie_id"
  end

  create_table "pastries", force: :cascade do |t|
    t.string "pastrie_name"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

end

Вот мои модели:

class Event < ApplicationRecord
  has_many :pastries, through: :fights
  has_many :cookers, through: :fights
  has_many :fights
end
class Fight < ApplicationRecord
  belongs_to :pastrie
  belongs_to :event
  belongs_to :cooker, optional: true
end
class Pastrie < ApplicationRecord
  has_many :fights
  has_many :events, through: :fights
end

Это мой контроллер. Что я понимаю: для создания Fight мне нужны event_id и pastrie_id (cooker_id не является обязательным). Итак, сначала я создаю новое событие (и поэтому у меня есть event_id), а затем мне нужно подключить pastrie_id (существующий в моем семени) к моей битве. Но это не работает, если я делаю это:

class FightsController < ApplicationController

  def new
    @fight = Fight.new
    @event = Event.find(params[:event_id])
    @pastries = Pastrie.all
  end

  def create
    @fight = Fight.new(fight_params)
    @event = Event.find(params[:event_id])
    @fight.event = Event.find(params[:event_id])
    @fight.pastrie_id = params[:fight][:pastrie_id]
    @fight.save!
    redirect_to root_path
  end

  def show
    @events = Event.all
  end

  def index
    @fights = Fight.all
    fights_by_event = []
  end

  private

  def fight_params
    params.require(:fight).permit(:event_id, :pastrie_id, :cooker_id)
  end

end

И мой взгляд, когда я создаю свой "бой":

<div class=margin-bottom>
  <h2 class=text-center> Bonjour Linguini, quelles patisseries veux-tu choisir ?</h2>
</div>
<div class="container margin-bottom">
  <%= simple_form_for [@event, @fight] do |f| %>
    <% @pastries.each do |pastrie| %>
      <%= f.label :pastrie_id do %>
      <%= f.check_box :pastrie_id, as: :boolean, checked_value: true, unchecked_value: false %> <span><%= pastrie.pastrie_name%></span>
      <% end %></br>
    <% end %>
  <%= f.submit "Valider", class: "btn btn-primary" %>
  <% end %>
</div>

И это мои маршруты, если вам нужно это:

Rails.application.routes.draw do

  root to: 'pages#home'
  resources :events do
    resources :fights, only: [:new, :show, :create]
  end

  resources :fights, only: [:index]
  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
end

1 Ответ

0 голосов
/ 07 июня 2019

В вашей форме измените

<%= f.check_box :pastrie_id, as: :boolean, checked_value: true, unchecked_value: false %> <span><%= pastrie.pastrie_name%></span>

На

<%= f.check_box :pastrie_id, as: :boolean, checked_value: pastrie.id, unchecked_value: nil %> <span><%= pastrie.pastrie_name%></span>

В первой версии вы отправляете параметр {pastrie_id: true}, который, очевидно, не относится к выпечке,Во второй версии должен быть указан идентификатор установленного флажка (хотя, если он принадлежит только 1 выпечке, возможно, имеет смысл использовать эти переключатели)

...