Рельсы образуют не спасающий город

Question

Я пытаюсь создать форму рельсов для клиента. Они должны выбрать город для списка существующих городов. Но это не сохраняется.

index.html.erb

<%= form_for @customer do |f| %>
  <div class="field">
    <%= f.label :first_name %><br>
    <%= f.text_field :first_name %>
  </div>
  <div class="field">
    <%= f.label :last_name %><br>
    <%= f.text_field :last_name %>
  </div>
  <div class="field">
    <%= f.label :phone_number %><br>
    <%= f.text_field :phone_number %>
  </div>
  <div class="field">
    <%= f.label :city %><br>
    <%= f.select :city_id, City.all.map(&:name) %>
  </div>
  <div class="actions">
    <%= f.submit %>
  </div>
<% end %>

city.rb

class City < ActiveRecord::Base
  validates_presence_of :name
  has_many :customers
end

customer.rb

class Customer < ActiveRecord::Base
  validates_presence_of :first_name
  validates_presence_of :last_name
  validates_uniqueness_of :phone_number
  belongs_to :city
end

customer_controller.orb

  def create
    @customer = Customer.find_or_initialize_by(customer_params)

    respond_to do |format|
      if @customer.save
        format.html { redirect_to @customer, notice: 'Customer was successfully created.' }
        format.json { render :show, status: :created, location: @customer }
      else
        format.html { render :new }
        format.json { render json: @customer.errors, status: :unprocessable_entity }
      end
    end
  end

  def customer_params
      params.require(:customer).permit(:first_name, :last_name, :phone_number, :city)
    end

Answer 1

Попробуйте изменить customer_params чтобы позволить city_id атрибут, в настоящее время вы определили только city:

def customer_params
  params.require(:customer).permit(:first_name, :last_name, :phone_number, :city_id)
end

Попробуйте также добавить атрибут id города в тэг выбора формы, вы просто отображаете имя, вы можете легко добиться этого, используя pluck:

<%= f.select :city_id, City.all.pluck(:name, :id) %>

Я бы порекомендовал вам создать переменную экземпляра в вашем контроллере и затем использовать ее.

Answer 2

Я смог исправить это с помощью options_for_select

<%= f.select :city_id, options_for_select(City.all.map{ |city| [city.name, city.id]}) %>

Answer 3

В вашей форме вы используете city_id

<div class="field">
  <%= f.label :city %><br>
  <%= f.select :city_id, City.all.map(&:name) %>
</div>

Но в вашем контроллере вы используете city

def customer_params
  params.require(:customer).permit(:first_name, :last_name, :phone_number, :city)
end

Другая ошибка может заключаться в том, что вы используете имя модели города в качестве значения параметра вместо идентификатора модели и, как результат, и эта строка (вместо числового идентификатора) передается в find_or_initialize_by,