link_to destroy action results in get request

Question

I'm trying to make a link to delete a Project...

  <% @projects.each do |project| %>
    <h2><%= project.name %> (<%= project.videos.size %>)</h2> 
      <%= link_to("Delete!", {:controller=>'projects', :action=>'destroy', :id=>project.hashed_id}, {method: :delete} ) %>
  <% end %>

No matter what I do I get a get request so I end up on the projects/show view. Urrrgh!

My controller:

  def destroy
    @project = Project.where( :hashed_id=> params[:id]).first.destroy()
    flash[:notice]= "Project deleted"
    redirect_to(:action=>'index')
  end

What am I missing?

PS: I found this, but the solution doesn't work for me.

Update

$ rake routes
      Prefix Verb   URI Pattern                  Controller#Action
        root GET    /                            pages#home
    projects GET    /projects(.:format)          projects#index
             POST   /projects(.:format)          projects#create
 new_project GET    /projects/new(.:format)      projects#new
edit_project GET    /projects/:id/edit(.:format) projects#edit
     project GET    /projects/:id(.:format)      projects#show
             PATCH  /projects/:id(.:format)      projects#update
             PUT    /projects/:id(.:format)      projects#update
             DELETE /projects/:id(.:format)      projects#destroy

Update 2 Here's the error I get:

enter image description here

Did you try `link_to("Delete!", project_path(project), action: :delete)` — Fred, Aug 17 '13 at 04:44
OK, no problem. The NoMethodError you're getting is because `@project` is nil. That means, the first place to check is your `show` action does return a non-nil `@project`. — Benjamin Tan Wei Hao, Aug 17 '13 at 12:07

score 1 · Accepted Answer · edited May 23 '17 at 12:05

1

Paste your output of rake routes. I suspect that the routes are not matching, that's why it falls to #show instead.

Make sure you have jquery-rails installed too.

See Delete / Destroy is not working in rails 3 with jQuery.

In general, make sure that you have included all the necessary javascript files.

edited May 23 '17 at 12:05

Community

1
1

answered Aug 17 '13 at 02:38

Benjamin Tan Wei Hao

9,621
3
30
56

Just added it to the question above. – emersonthis Aug 17 '13 at 10:43
Why is my problem connected to JavaScript? (Im a rails n00b) – emersonthis Aug 17 '13 at 12:08
Also what's the best way to check? – emersonthis Aug 17 '13 at 12:09
It has to do with unobstrusive javascript. The best way to check is to paste your `Gemfile` and `application.js` and also to make sure you are including the js. – Benjamin Tan Wei Hao Aug 17 '13 at 12:10
I'm sorry but what is uour hashed_id for? You usually should be able to do something like link_to(project, method: :delete). or link_to(project_path(project), method: :delete) – Marc-Alexandre Bérubé Aug 17 '13 at 12:50
I'm using a hashed ID which comes from a 3rd party API that I'm integrating with. Id rather show only the hashed_id to the client. – emersonthis Aug 17 '13 at 13:07
1

@BenjaminTan it WAS a JavaScript issue. I had commented our the application.js tag because the file was empty, but evidently that does more than I realized because when I uncommented it, the delete request works as expected! – emersonthis Aug 17 '13 at 23:12

score 0 · Answer 2 · answered Aug 17 '13 at 05:08

0

Do something like this:

Controller:

def destroy
  Flower.find(params[:id]).destroy
  redirect_to :action => 'index'
end

Views:

<%= link_to 'Destroy', flower_path(flower), method: :delete, data: { confirm: 'Are you sure?' } %>

answered Aug 17 '13 at 05:08

Sami

1,041
3
16
32

Hi Sami, I tried it with project_path(project) instead of the :action/:controller syntax and I *STILL* get a get request. – emersonthis Aug 17 '13 at 10:46
can you show your error message, latest controller's destroy method and the link to tag. – Sami Aug 17 '13 at 10:53
Error message added above. – emersonthis Aug 17 '13 at 11:09
it could not find name in your projects – Sami Aug 17 '13 at 12:16
It could not find name because it's rendering the show view but the controller failed to pass a project object – emersonthis Aug 17 '13 at 13:04

score 0 · Answer 3 · answered Aug 17 '13 at 15:45

0

Try this:

<%= link_to 'Destroy', project_path(project), method: :delete %>

It should work. If doesn't, paste the server log in comments or try to check your generated path.

Or check at your rails c:

app.project_path(Project.first)

Good luck!

answered Aug 17 '13 at 15:45

Carlos André Oliveira

340
1
4

link_to destroy action results in get request

3 Answers3