创建对论坛帖子的回复时出现 ActiveModel::ForbiddenAttributesError

Question

我正在使用 Rails 5.0.0。当我尝试创建对论坛帖子的回复时，我收到以下错误。

Started POST "/discussions/7/replies" for 108.252.220.249 at 2018-03-19 15:54:44 +0000
Cannot render console from 108.252.220.249! Allowed networks: 127.0.0.1,::1, 127.0.0.0/127.255.255.255
Processing by RepliesController#create as JS
Parameters: {"utf8"=>"✓", "reply"=>{"reply"=>""}, "commit"=>"Submit Reply", "discussion_id"=>"7"}
User Load (0.6ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = ? ORDER BY "users"."id" ASC LIMIT ?  [["id", 1], ["LIMIT", 1]]
Discussion Load (0.2ms)  SELECT  "discussions".* FROM "discussions" WHERE "discussions"."id" = ? LIMIT ?  [["id", 7], ["LIMIT", 1]]
(0.1ms)  begin transaction
(0.1ms)  rollback transaction
Completed 401 Unauthorized in 10ms (ActiveRecord: 1.0ms)



ActiveModel::ForbiddenAttributesError (ActiveModel::ForbiddenAttributesError):

这是我的回复控制器文件的一部分。该错误使创建操作看起来有问题，所以这就是我在这篇文章中包含的内容。此外，我的架构有一个“回复”表，其中有一列名为“回复”，这是用户想要提交的实际文本作为他们的回复。对不起，令人困惑的命名方案。

我觉得创建操作是正确的，但其他地方有问题。像强参数类型的问题？对于我的文件中的其他位置可能导致出现此类错误是否有任何建议？

class RepliesController < ApplicationController
before_action :authenticate_user!
before_action :set_reply, only: [:edit, :update, :show, :destroy]
before_action :set_discussion, only: [:create, :edit, :show, :update, :destroy]

def create
    #create a reply within the discussion and save userid to the reply
    @reply = @discussion.replies.create(params[:reply]).permit(:reply, :discussion_id)
    @reply.user_id = current_user.id

    respond_to do |format|
        if @reply.save
            format.html {redirect_to discussion_path(@discussion)}
            format.js #render create.js.erb

        else
            format.html{redirect_to discussion_path(@discussion), notice: 'Reply did not save. Try again'}
            format.js
        end
    end
end

...

private
def set_discussion
    @discussion = Discussion.find(params[:discussion_id])
end

def set_reply
    @reply = Reply.find(params[:id])
end

def reply_params
    params.require(:reply).permit(:reply)
end

结束

Answer 1

问题出在这一行：

 @reply = @discussion.replies.create(params[:reply]).permit(:reply, :discussion_id)

你应该使用类似的东西：

 @reply = @discussion.replies.create(reply_params)

我建议您更新表单以包含discussion_id，并将其添加到reply_params。这样您就可以使用上面建议的行创建记录。

为此，您需要在表单中添加类似于以下内容的行：

<%= f.hidden_field :discussion_id, @discussion.id %>

但是，值得考虑的是，如果用户有恶意，他们仍然可以在页面上编辑此内容。因此，如果修复讨论对安全性至关重要，您可以处理分配服务器端（正如您目前正在做的那样 - 请参阅下面的更新）。

希望对您有所帮助 - 如果您有任何问题，请告诉我您的进展情况。

---

更新

当您使用@discussion.replies.create... 时，将自动分配discussion_id。这意味着以下应该可以正常工作：

@reply = @discussion.replies.create(reply_params)

强参数用于防止因批量分配引起的问题，因此您实际上只在这里分配了一个属性，您可以进一步简化它：

@reply = @discussion.replies.create(params[:reply][:reply])

但是，我建议您使用第一个选项，因为随着项目的扩展，您只需将新列添加到 reply_params。

希望有帮助！