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

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

CanCan

基本上你在控制器操作之前 load_and_authorize 用户并通过Ability模型检查它们。你也可以获得像

这样的助手

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

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

Rails DELETE http方法呈现页面

5 个答案: