Have you had any luck submitting tickets to Rails Trac site? I keep getting an error.

Why are you so keen to get rid of the extra action in favour of a load of conditional code?

* update has edit
* create has new
* why shouldn’t destroy have delete?

I would much rather have one extra action and do away with all the conditional stuff and hacks.

First, drop in my handy dandy hack that allows you to use arrays for method requirements: http://buckymatters.com/simply_restful_hack.rb It looks like a lot, but I assure you it’s only an additional 12 lines consisting mostly of if statements.

Controller:

def destroy
  if request.delete?
    User.find(params[:id]).destroy

    respond_to do |type|
      type.html { redirect_to('/') }
      type.js   { render(:update) { |page| page.redirect_to('/') } }
    end
  end
end

Routes:

map.resources :users
map.resources :users, :member => {:destroy => [:delete, :get]}

You could also use an AJAX link here, but this way requires you to always
confirm it.

View:


  Cancel Account

I obviously take arguing too seriously, but boy did I get rid of that extra action. Maybe it’ll even make it to the core one day, but I doubt it.

By the way, this was my first core modification.

In response to:

bq. “However, if you think of the pre-action forms (new & edit) as merely a way to define the parameters of the action. Then it doesn’t really work for delete (or show, for that matter). Do you need parameters for the delete or show actions (other than the id)?”

There may be occasions when you want to e.g. log a reason for deletion etc. That is what Iove about this method, it caters for many scenarios.

If you fall back to /products/24 when javascript is disabled you will just get the show page as the routes have no way of telling that you wanted destroy. That is why the fallback url needs to be different i.e. /products/24;delete, you can then POST to destroy with method set to delete and the routes know what you want.

Make sense?

Rabbit is right, it is simply a “request” to “delete” which gives you a form that then posts to “destroy”, just as “new” is a post to “create” and “edit” is a post to “update”.