开发者

Rails 3 - Ajax Put request gives 404, yet it works

开发者 https://www.devze.com 2023-04-02 02:41 出处:网络
I am updating a task status from my deal overview page where i am displaying my list of tasks for that deal

I am updating a task status from my deal overview page where i am displaying my list of tasks for that deal

It works, the task gets updated. However, i get a nasty 404 error.

Any ideas?

js/coffeescript

jQuery ->
    $(".complete_task").live "click", ->
    task_id = $(this).attr("value")
    alert "checkbox clicked" + task_id
    $('ul#tasks').sortable "refresh"
    $.ajax 
      url: "tasks/" + task_id
      data: "&task[is_completed]=2"
      type: "PUT"
      success: ->
        $(this).addClass "done"
开发者_开发问答

My firebug output

http://img707.imageshack.us/img707/7899/screenshot20110901at102.png

I'm open to suggestions if this is not the "right way" to do it. Though the requirement is that it must be done with ajax.

Thanks in advance!

EDIT: Requested Information

Routing Error

<h1>Routing Error</h1>

No route matches {:controller=>"tasks", :action=>"show", :id=>#<Deal id: 1, title: "4919 Torrey Pines Run", user_id: nil, created_at: "2011-09-02 00:08:24", updated_at: "2011-09-02 00:08:24", account_id: 1>, :format=>#<Task id: 39, title: "new tdsfsdf", position: 25, user_id: nil, deal_id: 1, created_at: "2011-09-02 02:49:09", updated_at: "2011-09-02 02:50:31", is_completed: 2, user_who_completed: nil>}

routes

  resources :accounts do
resources :users
resources :deals do
  post :sort, on: :member
  resources :tasks do
    post :sort, on: :collection 
  end
end

end

respond_to block

def update
 respond_to do |format|
   if @task.update_attributes(params[:task])
     format.html { redirect_to([@task.deal, @task], :notice => 'Task was successfully updated.') }
     format.xml  { head :ok }
     format.js   { render :nothing => true } 

   else
     format.html { render :action => "edit" }
     format.xml  { render :xml => @task.errors, :status => :unprocessable_entity }
     format.js   { render :nothing => true } 

   end
 end

end

rake/grep

MacBook-Pro-2:accountable jbeasley6651$ rake routes | grep tasks
 sort_account_deal_tasks POST   /accounts/:account_id/deals/:deal_id/tasks/sort(.:format)     {:action=>"sort", :controller=>"tasks"}
  account_deal_tasks GET    /accounts/:account_id/deals/:deal_id/tasks(.:format)          {:action=>"index", :controller=>"tasks"}
                     POST   /accounts/:account_id/deals/:deal_id/tasks(.:format)          {:action=>"create", :controller=>"tasks"}
   new_account_deal_task GET    /accounts/:account_id/deals/:deal_id/tasks/new(.:format)      {:action=>"new", :controller=>"tasks"}
  edit_account_deal_task GET    /accounts/:account_id/deals/:deal_id/tasks/:id/edit(.:format) {:action=>"edit", :controller=>"tasks"}
       account_deal_task GET    /accounts/:account_id/deals/:deal_id/tasks/:id(.:format)      {:action=>"show", :controller=>"tasks"}
                     PUT    /accounts/:account_id/deals/:deal_id/tasks/:id(.:format)      {:action=>"update", :controller=>"tasks"}
                     DELETE /accounts/:account_id/deals/:deal_id/tasks/:id(.:format)      {:action=>"destroy", :controller=>"tasks"}


You need to add the account to your redirect, e.g.

# adding it as @account, change to however you're referencing it.
# params[:account_id] should also work.
format.html { redirect_to([@account, @task.deal, @task], ...) }

Also, as noted in the comments, your Ajax request isn't asking for javascript, but rather HTML. You'd need to do what Kristian PD says and put dataType: script to the options to specify JS, e.g.

$.ajax 
  url: "tasks/" + task_id
  data: "&task[is_completed]=2"
  type: "PUT"
  dataType: "script"
  success: ->
    $(this).addClass "done"


Try

 $.ajax 
      url: "tasks/" + task_id
      dataType: "script"
      data: "&task[is_completed]=2"
      type: "PUT"
      success: ->
        $(this).addClass "done"

I don't think it's rendering your JS response and it's trying to use the HTML redirect (which also has problems as noted by numbers)

0

精彩评论

暂无评论...
验证码 换一张
取 消