@@ -37,7 +37,7 @@ class AllowanceTasksController < ApplicationController end # Toggle the active state of a task - def task_toggle + def update task=AllowanceTask.find params[:id].to_i active = (params[:active] == 'true') task.deleted = !active app/controllers/allowance_tasks_controller.rb @@ -19,10 +19,8 @@ <tr id="task_<%= t.id %>" <%= t.deleted ? 'class="deleted"' : '' %>> <td> <%= content_tag :input, '', :type => :checkbox, - :onclick => remote_function(:url => { - :controller => :allowance, - :action => :task_toggle, - :id => t.id}, + :onclick => remote_function(:url => allowance_task_path(t), + :method => :put, :with => "'active='+this.checked"), :checked => 'checked' %> </td> app/views/allowance_tasks/index.html.erb @@ -88,7 +88,7 @@ class AllowanceTasksControllerTest < Test::Unit::TestCase def test_deactivation login_as :quentin assert_difference 'AllowanceTask.count_active', -1 do - xhr :post, :task_toggle, :id => 1, :active => 'false' + xhr :post, :update, :id => 1, :active => 'false' assert_response :success assert_template 'allowance_tasks/deactivate' end @@ -97,7 +97,7 @@ class AllowanceTasksControllerTest < Test::Unit::TestCase def test_activation login_as :quentin assert_difference 'AllowanceTask.count_active' do - xhr :post, :task_toggle, :id => 4, :active => 'true' + xhr :post, :update, :id => 4, :active => 'true' assert_response :success assert_template 'allowance_tasks/activate' end test/functional/allowance_tasks_controller_test.rb e32f222b60251c00a9a2324129c21096b0a1c068 Dustin Sallings dustin@spy.net http://github.com/dustin/money/commit/4f527c1471b3c6fa3fb72a35562a01d2c0a53b8b 4f527c1471b3c6fa3fb72a35562a01d2c0a53b8b 2008-05-24T15:51:19-07:00 2008-05-24T15:51:19-07:00 Use PUT for enabling and disabling tasks. 7ad7807050c4e94f274a9c2a6a0524dcceca4aae Dustin Sallings dustin@spy.net