Andreas/tracks
1
0
Fork 0
mirror of https://github.com/TracksApp/tracks.git synced 2025-12-26 12:08:47 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix error in last patch

Browse source 
seems that our test suite does not test updating recurring todos
This commit is contained in:
Reinier Balt 2009-02-05 22:41:37 +01:00
parent b8fd942eb4
commit d41c369f09
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
app/controllers/recurring_todos_controller.rb
View file

@ -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