From d41c369f09a32152db2846bf0f48933f987ad0d8 Mon Sep 17 00:00:00 2001 From: Reinier Balt Date: Thu, 5 Feb 2009 22:41:37 +0100 Subject: [PATCH] fix error in last patch seems that our test suite does not test updating recurring todos --- app/controllers/recurring_todos_controller.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/controllers/recurring_todos_controller.rb b/app/controllers/recurring_todos_controller.rb index 0784a606..e9776006 100644 --- a/app/controllers/recurring_todos_controller.rb +++ b/app/controllers/recurring_todos_controller.rb @@ -30,6 +30,7 @@ class RecurringTodosController < ApplicationController end def update + # TODO: write tests for updating @recurring_todo.tag_with(params[:tag_list]) if params[:tag_list] @original_item_context_id = @recurring_todo.context_id @original_item_project_id = @recurring_todo.project_id @@ -71,7 +72,7 @@ class RecurringTodosController < ApplicationController # make sure that we set weekly_return_xxx to empty (space) when they are # not checked (and thus not present in params["recurring_todo"]) - %w{"monday","tuesday", "wednesday", "thursday", "friday", "saturday", "sunday"}.each do |day| + %w{monday tuesday wednesday thursday friday saturday sunday}.each do |day| params["recurring_todo"]["weekly_return_"+day]=' ' if params["recurring_todo"]["weekly_return_"+day].nil? end