From c2feee741df6713b26e597d8012d8866943c09c4 Mon Sep 17 00:00:00 2001 From: Neeraj Singh Date: Sat, 17 Jul 2010 16:08:17 -0400 Subject: [PATCH] fixing the ternary operation where the logic is very confusing. [#5136 state:resolved] --- actionpack/lib/action_view/helpers/date_helper.rb | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/actionpack/lib/action_view/helpers/date_helper.rb b/actionpack/lib/action_view/helpers/date_helper.rb index f097b9a..8050669 100644 --- a/actionpack/lib/action_view/helpers/date_helper.rb +++ b/actionpack/lib/action_view/helpers/date_helper.rb @@ -800,7 +800,8 @@ module ActionView start = options.delete(:start) || 0 stop = options.delete(:end) || 59 step = options.delete(:step) || 1 - leading_zeros = options.delete(:leading_zeros).nil? ? true : false + options.reverse_merge!({:leading_zeros => true}) + leading_zeros = options.delete(:leading_zeros) select_options = [] start.step(stop, step) do |i| -- 1.6.5.2