Permalink
Browse files

Added support for descending year values in DateHelper#select_year, l…

…ike select_year(Date.today, :start_year => 2005, :end_year => 1900), which would count down from 2005 to 1900 instead of the other way #1274 [nwoods@mail.com]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@1320 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
  • Loading branch information...
1 parent f9103e1 commit 202d5c0751ac9efde2235102fb2099f4cbe93fd6 @dhh dhh committed May 19, 2005
View
@@ -1,5 +1,7 @@
*SVN*
+* Added support for descending year values in DateHelper#select_year, like select_year(Date.today, :start_year => 2005, :end_year => 1900), which would count down from 2005 to 1900 instead of the other way #1274 [nwoods@mail.com]
+
* Fixed that FormHelper#checkbox should return a checked checkbox if the value is the same as checked_value #1286 [Florian Weber]
* Fixed Form.disable in Prototype #1317 [Wintermute]
@@ -190,16 +190,20 @@ def select_month(date, options = {})
end
# Returns a select tag with options for each of the five years on each side of the current, which is selected. The five year radius
- # can be changed using the <tt>:start_year</tt> and <tt>:end_year</tt> keys in the +options+. The <tt>date</tt> can also be substituted
- # for a year given as a number. Example:
+ # can be changed using the <tt>:start_year</tt> and <tt>:end_year</tt> keys in the +options+. Both ascending and descending year
+ # lists are supported by making <tt>:start_year</tt> less than or greater than <tt>:end_year</tt>. The <tt>date</tt> can also be
+ # substituted for a year given as a number. Example:
#
- # select_year(Date.today, :start_year => 1992, :end_year => 2007)
+ # select_year(Date.today, :start_year => 1992, :end_year => 2007) # ascending year values
+ # select_year(Date.today, :start_year => 2005, :end_year => 1900) # descending year values
def select_year(date, options = {})
year_options = []
y = date ? (date.kind_of?(Fixnum) ? (y = (date == 0) ? Date.today.year : date) : date.year) : Date.today.year
- default_start_year, default_end_year = y-5, y+5
- (options[:start_year] || default_start_year).upto(options[:end_year] || default_end_year) do |year|
+ start_year, end_year = (options[:start_year] || y-5), (options[:end_year] || y+5)
+ step_val = start_year < end_year ? 1 : -1
+
+ start_year.step(end_year, step_val) do |year|
year_options << ((date && (date.kind_of?(Fixnum) ? date : date.year) == year) ?
"<option value=\"#{year}\" selected=\"selected\">#{year}</option>\n" :
"<option value=\"#{year}\">#{year}</option>\n"
@@ -269,4 +273,4 @@ def to_datetime_select_tag(options = {})
end
end
end
-end
+end
@@ -124,6 +124,15 @@ def test_select_year_with_type_discarding
2003, :prefix => "date_year", :discard_type => true, :start_year => 2003, :end_year => 2005)
end
+ def test_select_year_descending
+ expected = %(<select name="date[year]">\n)
+ expected << %(<option value="2005" selected="selected">2005</option>\n<option value="2004">2004</option>\n<option value="2003">2003</option>\n)
+ expected << "</select>\n"
+
+ assert_equal expected, select_year(Time.mktime(2005, 8, 16), :start_year => 2005, :end_year => 2003)
+ assert_equal expected, select_year(2005, :start_year => 2005, :end_year => 2003)
+ end
+
def test_select_hour
expected = %(<select name="date[hour]">\n)
expected << %(<option value="00">00</option>\n<option value="01">01</option>\n<option value="02">02</option>\n<option value="03">03</option>\n<option value="04">04</option>\n<option value="05">05</option>\n<option value="06">06</option>\n<option value="07">07</option>\n<option value="08" selected="selected">08</option>\n<option value="09">09</option>\n<option value="10">10</option>\n<option value="11">11</option>\n<option value="12">12</option>\n<option value="13">13</option>\n<option value="14">14</option>\n<option value="15">15</option>\n<option value="16">16</option>\n<option value="17">17</option>\n<option value="18">18</option>\n<option value="19">19</option>\n<option value="20">20</option>\n<option value="21">21</option>\n<option value="22">22</option>\n<option value="23">23</option>\n)

0 comments on commit 202d5c0

Please sign in to comment.