Rails DELETE http方法呈现页面

Question

我正在尝试向我的模型添加删除功能。 这就是我想出来的，但即使我不需要渲染页面来删除某些内容，Rails也会渲染并找不到“delete.html.erb”的文件

我正在使用Ruby 2.0dev和Rails 4.0

我的删除链接：

<%= link_to "Delete", reservation_delete_path(item), :class => "btn btn-small btn-danger", method: :delete, data: {confirm: 'Are you sure?'} %></td>

我的路线档案：

match 'reservations/delete/:id' => 'reservations#delete', via: :delete, :as => 'reservation_delete'

我的控制器：

def delete
  @current = Reservations.find(params[:id])
  if current_user
    if @current.user_id == current_user.id
      @current.destroy!
      redirect_to reservations_path
    else
      redirect_to reservations_path
    end
  else
    redirect_to reservations_path
  end
end

Answer 1

对于每个条件，无需复制重定向3次。 您可以简化删除方法：

def delete
  @current = Reservations.find(params[:id])

  if current_user && @current.user_id == current_user.id
    @current.destroy!
  end

  redirect_to reservations_path
end

在您的问题中，如果current_user不可用，则您没有重定向，因此正在运行隐式渲染。

Answer 2

你的设置不是惯用的，而且你没有包含的代码，所以任何事情都可能出错。 例如，那不能是你的整个路线文件; 没有指定索引/显示/编辑/删除按钮所在的任何页面。 另一个例子：你的行动被命名为delete而不是destroy 。 无论如何，我可以向你展示一个有效的例子，并且更具规范性：

车型/ reservation.rb：

class Reservation < ActiveRecord::Base
end

控制器/ reservations_controller.rb：

class ReservationsController < ApplicationController
  def index
    @reservations = Reservation.all
  end

  def destroy
    @reservation = Reservation.find(params[:id])
    @reservation.destroy

    redirect_to reservations_url
  end
end

意见/预订/ index.html.erb：

<% @reservations.each do |reservation| %>
  <%= link_to 'Destroy', reservation, method: :delete, data: { confirm: 'Are you sure?' } %>
<% end %>

（这将只显示删除相应预订的链接...如果您想查看更多信息，您必须坚持<%= reservation.name %>或其他任何内容）

配置/ routes.rb文件：

Howdy::Application.routes.draw do
  resources :reservations, only: [:index, :destroy]
  root 'reservations#index'
end

（我的应用名称是你的）

您正在进行一些用户身份验证，因此请相应添加。 如果您在执行操作之前从一个执行特殊用户身份验证的控制器继承，这可能就是为什么它试图渲染delete.html.erb

Answer 3

看起来你错过了那些重定向的返回，这实际上导致Rails执行重定向并尝试渲染视图。

return redirect_to reservations_path

Answer 4

两件事情：

在routes文件中指定delete（destroy）操作是资源的一部分。 要以“rails”的方式执行此操作，您可以考虑让路由文件看起来更像：

resources: :reservations, only: [:delete]

...然后删除链接更像是：

<%= link_to 'Delete', delete_reservation_path(item), :class => 'btn btn-small btn-danger', method: :delete, data: {confirm: 'Are you sure?'} %>

...然后在您的控制器中，您可以：

def destroy
  @current = Reservations.find(params[:id])
  if current_user
    if @current.user_id == current_user.id
      @current.destroy!
      redirect_to reservations_path
    else
      redirect_to reservations_path
    end
  else
    redirect_to reservations_path
  end
end

...或者您实际上可以为删除操作创建一个rjs模板来执行一些奇特的javascript工作，或者您可以简单地渲染索引操作的视图（更快地加载重定向）。

Answer 5

当你开始提出&& gate时我的建议是检查是否有现有的解决方案。 在这种情况下，您可能正在寻找CanCan gem中可用的功能。

可以可以

基本上你在控制器操作之前对用户进行load_and_authorize ，并通过Ability模型检查它们。 您也可以获得视图助手

if can? :destroy, reservation 
  ... do awesome stuff here ...

从长远来看，这将是一个更好的解决方案。