0

次のフォームとコントローラーを使用しています。新しい通知を作成すると、campus_id を除くすべてが保存されます。

ドロップダウンから別のキャンパス パラメータを選択しましたが、間違ったキャンパス パラメータを指定しているようです。後で同じエントリを編集すると、保存されますか? 何が起こっていて、どうすれば修正できますか?

edit アクションと create アクションに同じフォームが使用されます。(一部です)

キャンパス (has_many) と通知 (belongs_to) に浅いルートを使用していることは注目に値するかもしれません。

ルート.rb

  shallow do
    resources :campus do
      resources :notifications
    end
  end

形:

<%= form_for [@campus,@notification] do |f| %>
  <% if @notification.errors.any? %>
    <div id="error_explanation">
      <h2><%= pluralize(@notification.errors.count, "error") %> prohibited this notification from being saved:</h2>

      <ul>
      <% @notification.errors.full_messages.each do |msg| %>
        <li><%= msg %></li>
      <% end %>
      </ul>
    </div>
  <% end %>

  <div class="field">
    <%= f.label :post %><br>
    <%= f.text_area :post %>
  </div>
  <div class="field">
    <%= f.label :campus %><br>
    <%= f.collection_select(:campus_id, Campus.all.order('name ASC'), :id, :name, prompt: true) %>
  </div>
  <div class="actions">
    <%= f.submit %>
  </div>
<% end %>

これはコントローラーです:

class NotificationsController < ApplicationController
  before_action :set_notification, only: [:show, :edit, :update, :destroy]
  before_action :set_campus, only: [:index, :new, :create]

  def index
    @notifications = @campus.notification
  end

  def show
  end

  def new
    @notification = @campus.notification.new
  end

  def edit
  end

  def create
    @notification = @campus.notification.new(notification_params)

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

  def update
    respond_to do |format|
      if @notification.update(notification_params)
        format.html { redirect_to @notification, notice: 'Notification was successfully updated.' }
        format.json { head :no_content }
      else
        format.html { render action: 'edit' }
        format.json { render json: @notification.errors, status: :unprocessable_entity }
      end
    end
  end

  def destroy
    @notification.destroy
    respond_to do |format|
      format.html { redirect_to campu_notifications_url(1) }
      format.json { head :no_content }
    end
  end

  private
    # Use callbacks to share common setup or constraints between actions.
    def set_notification
      @notification = Notification.find(params[:id])
    end

    def set_campus
      @campus = Campus.find(params[:campu_id])
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def notification_params
      params.require(:notification).permit(:post, :campus_id)
    end
end

ログを見ると、間違ったパラメーターがコミットされていることがわかります。

84.193.153.106 の POST "/campus/1/notifications" を 2014-09-29 18:29:33 +0000 で開始:33 +0000 NotificationsController#create as HTML 処理 NotificationsController#create as HTML パラメータ: {"utf8"=>"_", "authenticity_token"=>"oNSlEFeukwEj2hIAT89wFdIYwjHO5c8lzBlCqMyk31Y=", "notification"=>{"post"= >"sdqfdsfd", "campus_id"=>"3"}, "commit"=>"Create Notification", "campu_id"=>"1"} パラメータ: {"utf8"=>"_", "authenticity_token"= >"onSleEFeukwEj2hiAT89wFdIYwjHO5c8lzBlCqMyk31Y=", "通知"=>{"post"=>"sdqfdsfd", "campus_id"=>"3"}, "commit"=>"Create Notification", "campu_id"=>"1"} Campus Load (0.4ms) SELECT "campus".* FROM "campus" WHERE "campus"."id" = $1 LIMIT 1 [["id", "1"]] Campus Load (0.4ms) SELECT "campus".* FROM "campus" WHERE "campus"."id" = $1 LIMIT 1 [["id", "1"]] (0.1ms) BEGIN (0.1ms) BEGIN SQL (28.6ms) INSERT INTO "notifications" ("campus_id", "created_at", "post", "updated_at" ") VALUES ($1, $2, $3, $4) RETURNING "id" [["campus_id", 1], ["created_at", Mon, 29 Sep 2014 18:29:34 UTC +00:00], ["post "、"sdqfdsfd"], ["updated_at", Mon, 29 Sep 2014 18:29:34 UTC +00:00]] SQL (28.6ms) INSERT INTO "notifications" ("campus_id", "created_at", "post", "updated_at") VALUES ($1, $2, $3, $4) RETURNING "id" [["campus_id", 1], ["created_at", Mon, 29 Sep 2014 18:29:34 UTC +00:00], [ "post", "sdqfdsfd"], ["updated_at", Mon, 29 Sep 2014 18:29:34 UTC +00:00]] (3.5ms) コミット (3.5ms) コミットcreated_at"、月、2014 年 9 月 29 日 18:29:34 UTC +00:00]、["post"、"sdqfdsfd"]、["updated_at"、月、2014 年 9 月 29 日 18:29:34 UTC +00:00 ]] (3.5ms) コミット (3.5ms) コミットcreated_at"、月、2014 年 9 月 29 日 18:29:34 UTC +00:00]、["post"、"sdqfdsfd"]、["updated_at"、月、2014 年 9 月 29 日 18:29:34 UTC +00:00 ]] (3.5ms) コミット (3.5ms) コミット

4

1 に答える 1

2

newあなたとcreateアクションを次のように変更したいかもしれません:

def new
  @notification = @campus.notifications.build
end

def create
  @notification = @campus.notifications.build(notification_params)

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

campus.build_notificationという通知をインスタンス化しますbelongs_to campus。を使用するには、パラメータの一部としてnew渡す必要があります。notification[campus_id]

于 2014-09-29T18:36:54.767 に答える