活动模型::禁止属性创建对论坛帖子的回复时出错

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。

希望对您有所帮助！