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):

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

我觉得创建动作是正确的，其他地方有问题。像强参数类型的问题？有没有关于我的文件中的其他地方可能导致出现这样的错误的建议？

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添加新列。

希望有所帮助！

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

1 个答案: