【问题标题】:Rails creating new record instead of updatingRails 创建新记录而不是更新
【发布时间】:2014-10-16 18:33:22
【问题描述】:

更新记录时 Rails 应用程序出现问题。 每次我这样做时,Rails 都会创建一条新记录,而不是编辑选择要更新的记录。

我注意到,无论我浏览编辑还是创建,表单都会使用Create Flyer 按钮呈现。所以我不确定问题是否与 Rails 内部相关。

我在此处附上了我认为相关的所有内容。如果有任何其他信息,请告诉我。

对象本身是具有以下值的传单:

ActiveRecord::Schema.define(version: 20141016141700) do

  create_table "flyers", force: true do |t|
    t.string   "event_name"
    t.string   "location"
    t.string   "genre"
    t.string   "date"
    t.string   "frequency"
    t.string   "tags"
    t.datetime "created_at"
    t.datetime "updated_at"
  end

end

flyers_controller.rb

class FlyersController < ApplicationController

  def index
    @flyers = Flyer.all
  end

  def new
    @flyer = Flyer.new
  end

  def create
    Flyer.create flyer_params
    redirect_to '/flyers'
  end

  def edit
    find_params
  end

  def update
    find_params
    @flyer.update flyer_params
    redirect_to '/flyers'
  end

private 

  def flyer_params
    params[:flyer].permit(:event_name, :location, :genre, :tags, :date, :frequency)
  end

  def find_params
    @flyer = Flyer.find params[:id]
  end

end

edit.html.erbnew.html.erb 都使用以下渲染:

<%= form_for Flyer.new do |f| %>

    <%= f.label :event_name %>
    <%= f.text_field :event_name %>

    <%= f.label :location %>
    <%= f.text_field :location %>

    <%= f.label :genre%>
    <%= f.text_field :genre %>

    <%= f.label :tags %>
    <%= f.text_field :tags %>

    <%= f.label :date %>
    <%= f.text_field :date %>

    <%= f.label :frequency %>
    <%= f.text_field :frequency %>


    <%= f.submit %>

<% end %>

flyers_feature_spec.rb

require 'rails_helper'

describe 'flyer upload page' do
    context 'no flyers uploaded' do
        it 'should display a notification' do
            visit '/flyers'
            expect(page).to have_content 'No flyers added'
        end
    end

    context 'adding a flyer' do
        it 'should be listed on the index' do
            visit '/flyers'
            click_link 'Add New Flyer'
            fill_in 'Event name', with: 'Hospitality'
            fill_in 'Location', with: 'Brighton'
            fill_in 'Genre', with: 'Drum and bass, obvs'
            fill_in 'Tags', with: 'liquid'
            fill_in 'Date', with: '12/12/14'
            fill_in 'Frequency', with: 'Weekly'

            click_button 'Create Flyer'
            expect(current_path).to eq '/flyers'
            expect(page). to have_content 'Hospitality'
        end
    end


    context 'with existing flyers' do

        before do
            Flyer.create(event_name: 'Hospitality')
        end

        describe 'editing a flyer' do
            it 'should update flyer details' do
                visit '/flyers'
                click_link 'Edit'
                fill_in 'Event name', with: 'Hospitality v2'
                click_button 'Update'

                expect(page).to have_content 'Hospitality v2'
            end
        end

    end
end

【问题讨论】:

  • 你在创建调用rails console中的方法时看到了什么?

标签: ruby-on-rails ruby rails-activerecord


【解决方案1】:

在控制器中添加edit方法:

@flyer = Flyer.find(params[:id])

并将视图的第一行更改为:

<%= form_for @flyer do |f| %>

附言您应该同时更改 edit.html.erb 和 new.html.erb,因此如果 new 由于验证而失败,它将保留正确填充的字段而不是清空它们。

【讨论】:

  • 正如@makhan 正确指出的那样,您观察到这一点的原因是因为您在表单中使用了Flyer.new。这意味着表单将始终用于 Flyer 的新实例。用@flyer 替换它就可以了,因为@flyer = Flyer.new 是从新方法调用时,@flyer = Flyer.find(params[:id]) 是从编辑方法调用时
  • 太棒了。就是这样。伙计们干杯
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 2023-01-12
  • 1970-01-01
  • 2012-07-25
  • 2021-03-30
  • 1970-01-01
  • 1970-01-01
  • 2011-12-10
相关资源
最近更新 更多