From e40b8ecbcfed06b85313f86b5daba15d6c97e65b Mon Sep 17 00:00:00 2001 From: Eric Allen Date: Sun, 1 Feb 2009 15:25:09 -0500 Subject: [PATCH] Need to save todo after updating state from project. Fixes #834. --- app/controllers/todos_controller.rb | 1 + test/functional/todos_controller_test.rb | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/app/controllers/todos_controller.rb b/app/controllers/todos_controller.rb index 57befc38..ca1e37c1 100644 --- a/app/controllers/todos_controller.rb +++ b/app/controllers/todos_controller.rb @@ -260,6 +260,7 @@ class TodosController < ApplicationController @project_changed = @original_item_project_id != @todo.project_id if (@project_changed && !@original_item_project_id.nil?) then @todo.update_state_from_project + @todo.save! @remaining_undone_in_project = current_user.projects.find(@original_item_project_id).not_done_todo_count end determine_down_count diff --git a/test/functional/todos_controller_test.rb b/test/functional/todos_controller_test.rb index 3d2bda83..0e21f83c 100644 --- a/test/functional/todos_controller_test.rb +++ b/test/functional/todos_controller_test.rb @@ -516,7 +516,7 @@ class TodosControllerTest < Test::Rails::TestCase assert_equal "project_hidden", todo.state # clear project from todo: the todo should be unhidden - xhr :post, :update, :id => 1, :_source_view => 'todo', "project_name"=>"None", "todo"=>{} + xhr :post, :update, :id => 5, :_source_view => 'todo', "project_name"=>"None", "todo"=>{} todo.reload() assert_equal "active", todo.state end