Rails 5 jQuery .ajax方法数据格式

Question

升级到Rails 5之后，我遇到了jQuery .ajax请求的问题。 我相信这与（有些）新的强大参数有关。 我收到以下错误：

ArgumentError (When assigning attributes, you must pass a hash as an argument.):

我想我在我的.ajax请求中错误地设置了我的数据包（使用Coffeescript）。

或者可能是我的可信参数无法正常工作。

使用byebug，看起来我的控制器的第一行是可以的，但它在第二行失败： @prospect.update(prospect_params[:status]) 。 这是byebug中的@_params变量：

1: @_params = <ActionController::Parameters {"prospect"=><ActionController::Parameters {"id"=>"16", "status"=>"inactive"} permitted: false>, "controller"=>"prospects", "action"=>"update_status"} permitted: false>

这是我的控制器和Coffescript的片段

调节器

  # PUT /prospects/update_status
  def update_status
    @prospect = Prospect.find(params[:id])
    @prospect.update(prospect_params[:status])

    respond_to do |format|
      if @prospect.update_attributes(prospect_params)
        format.json { head :no_content }
      else
        format.json { render json: @prospect.errors, status: :unprocessable_entity }
      end
    end
  end

...

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

  # Only allow a trusted parameter "white list" through.
  def prospect_params
    params.require(:prospect).permit(:name, :status, :priority, :id)
  end

CoffeeScript的

    # Update prospects active list when a toggle switch is clicked
    $("#prospects_list").on "change", ".js-switch", ->
        prospect_id = $(this).parent('form').find('input[name="prospect[id]"]').val()
        if @checked
            status = 'active'
        else
            prospect_status = 'inactive'
        # Get parent TR
        tr = $(this).closest('tr')
        # Update prospect using Ajax
        $.ajax
            url: '/prospects/update_status'
            type: 'POST'
            data: { prospect: {id: prospect_id, status: prospect_status } }
            dataType: 'json'
            success: (data) ->
                return
            false

Answer 1

您传递参数的值。

查看you must pass a hash as an argument.的错误消息you must pass a hash as an argument.

@prospect.update(prospect_params[:status])
# here is you pass a value of status key, that's not a hash
# {"id"=>"16", "status"=>"inactive"} 
# prospect_params[:status] => "inactive"

使用只：

@prospect.update(prospect_params)
# that's pass a hash
# prospect_params => {"id"=>"16", "status"=>"inactive"}

要么：

@prospect.update(status: prospect_params[:status])

Answer 2

首先问自己是否确实需要一个特定的路由来更新状态，或者这应该是对规范更新路由的请求：

PATCH /prospects/:id

在这种情况下，您应该更改您的AJAX处理程序：

# Update prospects active list when a toggle switch is clicked
$("#prospects_list").on "change", ".js-switch", ->
    prospect_id = $(this).parent('form').find('input[name="prospect[id]"]').val()
    if @checked
      status = 'active'
    else
      prospect_status = 'inactive'
    # Get parent TR
    tr = $(this).closest('tr')
    # Update prospect using Ajax
    $.ajax
      url: '/prospects/' + prospect_id
      type: 'PATCH' # Not POST!
      data: { prospect: { status: prospect_status } }
      dataType: 'json'
      success: (data) ->
          return
      false

如果由于某种原因想要一个特定的方法，你可以通过将：id移动到路径而不是params来修复它。

# config/routes.rb
resources :prospects do
  member do
    patch :update_status
  end
end

---

# PATCH /prospects/:id/update_status
def update_status
  @prospect = Prospect.find(params[:id])
  respond_to do |format|
    if @prospect.update_attributes(prospect_params)
      format.json { head :no_content }
    else
      format.json { render json: @prospect.errors, status: :unprocessable_entity }
    end
  end
end

# ...

def prospect_params
  params.require(:prospect).permit(:name, :status, :priority)
end

---

# coffeescript
url: '/prospects/' + prospect_id  +'/update_status'

使用@prospect.update(prospect_params[:status])两次更新记录并绕过强参数，以便引发异常。

Answer 3

感谢Зелёный和max让我朝着正确的方向前进。 这就是我用过的东西。 我只需要对控制器进行一些小改动

调节器

# PUT /prospects/update_status
def update_status
  @prospect = Prospect.find(prospect_params[:id])
  respond_to do |format|
    if @prospect.update(status: prospect_params[:status])
      format.json { head :no_content }
    else
      format.json { render json: @prospect.errors, status: :unprocessable_entity }
    end
  end
end