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