From a981595bce6f132a299994ce624b8ec7605bc0d0 Mon Sep 17 00:00:00 2001
From: Reinier Balt <lrbalt@gmail.com>
Date: Fri, 25 Jul 2008 20:39:22 +0200
Subject: [PATCH] fix error where editing start date or end date for a
 recurring todo does not show the calendar

---
 app/controllers/recurring_todos_controller.rb | 4 +++-
 app/views/recurring_todos/_edit_form.html.erb | 8 ++++----
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/app/controllers/recurring_todos_controller.rb b/app/controllers/recurring_todos_controller.rb
index 1f9d6939..e3d652ba 100644
--- a/app/controllers/recurring_todos_controller.rb
+++ b/app/controllers/recurring_todos_controller.rb
@@ -32,8 +32,10 @@ class RecurringTodosController < ApplicationController
 
     # we needed to rename the recurring_period selector in the edit form 
     # because the form for a new recurring todo and the edit form are on the
-    # same page.
+    # same page. Same goed for start_from and end_date
     params['recurring_todo']['recurring_period']=params['recurring_edit_todo']['recurring_period']
+    params['recurring_todo']['end_date']=params['recurring_todo_edit_end_date']
+    params['recurring_todo']['start_from']=params['recurring_todo_edit_start_from']
     
     # update project
     if params['recurring_todo']['project_id'].blank? && !params['project_name'].nil?
diff --git a/app/views/recurring_todos/_edit_form.html.erb b/app/views/recurring_todos/_edit_form.html.erb
index aea4be4d..23e69a32 100644
--- a/app/views/recurring_todos/_edit_form.html.erb
+++ b/app/views/recurring_todos/_edit_form.html.erb
@@ -75,13 +75,13 @@
       <div id="recurring_timespan">
         <br/>
         <label for="recurring_todo[start_from]">Starts on </label><%= 
-        text_field_tag("recurring_todo[start_from]", format_date(@recurring_todo.start_from), "size" => 12, "class" => "Date", "onfocus" => "Calendar.setup", "tabindex" => 6, "autocomplete" => "off") %><br/>
+        text_field_tag("recurring_todo_edit_start_from", format_date(@recurring_todo.start_from), "size" => 12, "class" => "Date", "onfocus" => "Calendar.setup", "tabindex" => 6, "autocomplete" => "off") %><br/>
         <br/>
         <label for="recurring_todo[ends_on]">Ends on:</label><br/>
         <%= radio_button_tag('recurring_todo[ends_on]', 'no_end_date', @recurring_todo.ends_on == 'no_end_date')%> No end date<br/>
         <%= radio_button_tag('recurring_todo[ends_on]', 'ends_on_number_of_times', @recurring_todo.ends_on == 'ends_on_number_of_times')%> Ends after <%= text_field_tag("recurring_todo[number_of_occurences]", @recurring_todo.number_of_occurences, "size" => 3, "tabindex" => 7) %> times<br/>
         <%= radio_button_tag('recurring_todo[ends_on]', 'ends_on_end_date', @recurring_todo.ends_on == 'ends_on_end_date')%> Ends on <%= 
-        text_field_tag('recurring_todo[end_date]', format_date(@recurring_todo.end_date), "size" => 12, "class" => "Date", "onfocus" => "Calendar.setup", "tabindex" => 8, "autocomplete" => "off") %><br/>
+        text_field_tag('recurring_todo_edit_end_date', format_date(@recurring_todo.end_date), "size" => 12, "class" => "Date", "onfocus" => "Calendar.setup", "tabindex" => 8, "autocomplete" => "off") %><br/>
     </div></div>
     <div id="recurring_edit_daily" style="display:<%= @recurring_todo.recurring_period == 'daily' ? 'block' : 'none' %> ">
       <label>Recurrence</label><br/>
@@ -142,6 +142,6 @@
       </div>
     </div>
   <% end %>
-  <%= calendar_setup( "recurring_todo_start_from"  ) %>
-  <%= calendar_setup( "recurring_todo_end_date"  ) %>
+  <%= calendar_setup( "recurring_todo_edit_start_from"  ) %>
+  <%= calendar_setup( "recurring_todo_edit_end_date"  ) %>
 </div>