0
@@ -503,8 +503,8 @@ module ActionView
0
       }.freeze unless const_defined?('POSITION')
0
 
0
       def initialize(datetime_or_instance_tag, options = {}, html_options = {})
0
-        @options = options
0
-        @html_options = html_options
0
+        @options = options.dup
0
+        @html_options = html_options.dup
0
 
0
         if datetime_or_instance_tag.is_a?(InstanceTag)
0
           @tag = datetime_or_instance_tag
0
@@ -664,13 +664,14 @@ module ActionView
0
         def default_datetime
0
           return if @options[:include_blank]
0
 
0
-          default = @options[:default]
0
-          case default
0
+          case @options[:default]
0
             when nil
0
               Time.current
0
             when Date, Time
0
-              default
0
+              @options[:default]
0
             else
0
+              default = @options[:default].dup
0
+
0
               # Rename :minute and :second to :min and :sec
0
               default[:min] ||= default[:minute]
0
               default[:sec] ||= default[:second]

0
@@ -1888,6 +1888,153 @@ class DateHelperTest < ActionView::TestCase
0
     assert_dom_equal expected, datetime_select("post", "updated_at", {}, :class => 'selector')
0
   end
0
 
0
+  def test_date_select_should_not_change_passed_options_hash
0
+    @post = Post.new
0
+    @post.updated_at = Time.local(2008, 7, 16, 23, 30)
0
+
0
+    options = { 
0
+      :order => [ :year, :month, :day ],
0
+      :default => { :year => 2008, :month => 7, :day => 16, :hour => 23, :minute => 30, :second => 1 },
0
+      :discard_type => false,
0
+      :include_blank => false,
0
+      :ignore_date => false,
0
+      :include_seconds => true
0
+    }
0
+    date_select(@post, :updated_at, options)
0
+
0
+    # note: the literal hash is intentional to show that the actual options hash isn't modified
0
+    #       don't change this!
0
+    assert_equal({ 
0
+      :order => [ :year, :month, :day ],
0
+      :default => { :year => 2008, :month => 7, :day => 16, :hour => 23, :minute => 30, :second => 1 },
0
+      :discard_type => false,
0
+      :include_blank => false,
0
+      :ignore_date => false,
0
+      :include_seconds => true
0
+    }, options)
0
+  end
0
+
0
+  def test_datetime_select_should_not_change_passed_options_hash
0
+    @post = Post.new
0
+    @post.updated_at = Time.local(2008, 7, 16, 23, 30)
0
+
0
+    options = { 
0
+      :order => [ :year, :month, :day ],
0
+      :default => { :year => 2008, :month => 7, :day => 16, :hour => 23, :minute => 30, :second => 1 },
0
+      :discard_type => false,
0
+      :include_blank => false,
0
+      :ignore_date => false,
0
+      :include_seconds => true
0
+    }
0
+    datetime_select(@post, :updated_at, options)
0
+
0
+    # note: the literal hash is intentional to show that the actual options hash isn't modified
0
+    #       don't change this!
0
+    assert_equal({ 
0
+      :order => [ :year, :month, :day ],
0
+      :default => { :year => 2008, :month => 7, :day => 16, :hour => 23, :minute => 30, :second => 1 },
0
+      :discard_type => false,
0
+      :include_blank => false,
0
+      :ignore_date => false,
0
+      :include_seconds => true
0
+    }, options)
0
+  end
0
+
0
+  def test_time_select_should_not_change_passed_options_hash
0
+    @post = Post.new
0
+    @post.updated_at = Time.local(2008, 7, 16, 23, 30)
0
+
0
+    options = { 
0
+      :order => [ :year, :month, :day ],
0
+      :default => { :year => 2008, :month => 7, :day => 16, :hour => 23, :minute => 30, :second => 1 },
0
+      :discard_type => false,
0
+      :include_blank => false,
0
+      :ignore_date => false,
0
+      :include_seconds => true
0
+    }
0
+    time_select(@post, :updated_at, options)
0
+
0
+    # note: the literal hash is intentional to show that the actual options hash isn't modified
0
+    #       don't change this!
0
+    assert_equal({ 
0
+      :order => [ :year, :month, :day ],
0
+      :default => { :year => 2008, :month => 7, :day => 16, :hour => 23, :minute => 30, :second => 1 },
0
+      :discard_type => false,
0
+      :include_blank => false,
0
+      :ignore_date => false,
0
+      :include_seconds => true
0
+    }, options)
0
+  end
0
+
0
+  def test_select_date_should_not_change_passed_options_hash
0
+    options = { 
0
+      :order => [ :year, :month, :day ],
0
+      :default => { :year => 2008, :month => 7, :day => 16, :hour => 23, :minute => 30, :second => 1 },
0
+      :discard_type => false,
0
+      :include_blank => false,
0
+      :ignore_date => false,
0
+      :include_seconds => true
0
+    }
0
+    select_date(Date.today, options)
0
+
0
+    # note: the literal hash is intentional to show that the actual options hash isn't modified
0
+    #       don't change this!
0
+    assert_equal({ 
0
+      :order => [ :year, :month, :day ],
0
+      :default => { :year => 2008, :month => 7, :day => 16, :hour => 23, :minute => 30, :second => 1 },
0
+      :discard_type => false,
0
+      :include_blank => false,
0
+      :ignore_date => false,
0
+      :include_seconds => true
0
+    }, options)
0
+  end
0
+
0
+  def test_select_datetime_should_not_change_passed_options_hash
0
+    options = { 
0
+      :order => [ :year, :month, :day ],
0
+      :default => { :year => 2008, :month => 7, :day => 16, :hour => 23, :minute => 30, :second => 1 },
0
+      :discard_type => false,
0
+      :include_blank => false,
0
+      :ignore_date => false,
0
+      :include_seconds => true
0
+    }
0
+    select_datetime(Time.now, options)
0
+
0
+    # note: the literal hash is intentional to show that the actual options hash isn't modified
0
+    #       don't change this!
0
+    assert_equal({ 
0
+      :order => [ :year, :month, :day ],
0
+      :default => { :year => 2008, :month => 7, :day => 16, :hour => 23, :minute => 30, :second => 1 },
0
+      :discard_type => false,
0
+      :include_blank => false,
0
+      :ignore_date => false,
0
+      :include_seconds => true
0
+    }, options)
0
+  end
0
+
0
+  def test_select_time_should_not_change_passed_options_hash
0
+    options = { 
0
+      :order => [ :year, :month, :day ],
0
+      :default => { :year => 2008, :month => 7, :day => 16, :hour => 23, :minute => 30, :second => 1 },
0
+      :discard_type => false,
0
+      :include_blank => false,
0
+      :ignore_date => false,
0
+      :include_seconds => true
0
+    }
0
+    select_time(Time.now, options)
0
+
0
+    # note: the literal hash is intentional to show that the actual options hash isn't modified
0
+    #       don't change this!
0
+    assert_equal({ 
0
+      :order => [ :year, :month, :day ],
0
+      :default => { :year => 2008, :month => 7, :day => 16, :hour => 23, :minute => 30, :second => 1 },
0
+      :discard_type => false,
0
+      :include_blank => false,
0
+      :ignore_date => false,
0
+      :include_seconds => true
0
+    }, options)
0
+  end
0
+
0
   protected
0
     def with_env_tz(new_tz = 'US/Eastern')
0
       old_tz, ENV['TZ'] = ENV['TZ'], new_tz