jro_apps/redmine Commit - r9655:dbdc9b9da756

Display dates using user's locale if available (#2190)....

Etienne Massip -

r9655:dbdc9b9da756

parent child

Context file:

r9655:dbdc9b9da756

Collapse all files

app/views/settings/_display.html.erb +3 -3

             <%= form_tag({:action => 'edit', :tab => 'display'}) do %>
             <div class="box tabular settings">
             <p><%= setting_select :ui_theme, Redmine::Themes.themes.collect {|t| [t.name, t.id]}, :blank => :label_default, :label => :label_theme %></p>
             <p><%= setting_select :default_language, lang_options_for_select(false) %></p>
             <p><%= setting_select :start_of_week, [[day_name(1),'1'], [day_name(6),'6'], [day_name(7),'7']], :blank => :label_language_based %></p>
+            <% locale = User.current.language.blank? ? ::I18n.locale : User.current.language %>
+            <p><%= setting_select :date_format, Setting::DATE_FORMATS.collect {|f| [::I18n.l(Date.today, :locale => locale, :format => f), f]}, :blank => :label_language_based %></p>
-            <p><%= setting_select :date_format, Setting::DATE_FORMATS.collect {|f| [Date.today.strftime(f), f]}, :blank => :label_language_based %></p>
+            <p><%= setting_select :time_format, Setting::TIME_FORMATS.collect {|f| [::I18n.l(Time.now, :locale => locale, :format => f), f]}, :blank => :label_language_based %></p>
-            <p><%= setting_select :time_format, Setting::TIME_FORMATS.collect {|f| [Time.now.strftime(f), f]}, :blank => :label_language_based %></p>
             <p><%= setting_select :user_format, @options[:user_format] %></p>
             <p><%= setting_check_box :gravatar_enabled %></p>
             <p><%= setting_select :gravatar_default, [["Wavatars", 'wavatar'], ["Identicons", 'identicon'], ["Monster ids", 'monsterid'], ["Retro", 'retro'], ["Mystery man", 'mm']], :blank => :label_none %></p>
             </div>
             <%= submit_tag l(:button_save) %>
             <% end %>

lib/redmine/i18n.rb +8 -3

             module Redmine
               module I18n
                 def self.included(base)
                   base.extend Redmine::I18n
                 end
                 def l(*args)
                   case args.size
                   when 1
                     ::I18n.t(*args)
                   when 2
                     if args.last.is_a?(Hash)
                       ::I18n.t(*args)
                     elsif args.last.is_a?(String)
                       ::I18n.t(args.first, :value => args.last)
                     else
                       ::I18n.t(args.first, :count => args.last)
                     end
                   else
                     raise "Translation string with multiple values: #{args.first}"
                   end
                 end
                 def l_or_humanize(s, options={})
                   k = "#{options[:prefix]}#{s}".to_sym
                   ::I18n.t(k, :default => s.to_s.humanize)
                 end
                 def l_hours(hours)
                   hours = hours.to_f
                   l((hours < 2.0 ? :label_f_hour : :label_f_hour_plural), :value => ("%.2f" % hours.to_f))
                 end
                 def ll(lang, str, value=nil)
                   ::I18n.t(str.to_s, :value => value, :locale => lang.to_s.gsub(%r{(.+)\-(.+)$}) { "#{$1}-#{$2.upcase}" })
                 end
                 def format_date(date)
                   return nil unless date
-                  Setting.date_format.blank? ? ::I18n.l(date.to_date) : date.strftime(Setting.date_format)
+                  options = {}
+                  options[:format] = Setting.date_format unless Setting.date_format.blank?
+                  options[:locale] = User.current.language unless User.current.language.blank?
+                  ::I18n.l(date.to_date, options)
                 end
                 def format_time(time, include_date = true)
                   return nil unless time
+                  options = {}
+                  options[:format] = (Setting.time_format.blank? ? :time : Setting.time_format)
+                  options[:locale] = User.current.language unless User.current.language.blank?
                   time = time.to_time if time.is_a?(String)
                   zone = User.current.time_zone
                   local = zone ? time.in_time_zone(zone) : (time.utc? ? time.localtime : time)
-                  (include_date ? "#{format_date(local)} " : "") +
+                  (include_date ? "#{format_date(local)} " : "") + ::I18n.l(local, options)
-                    (Setting.time_format.blank? ? ::I18n.l(local, :format => :time) : local.strftime(Setting.time_format))
                 end
                 def day_name(day)
                   ::I18n.t('date.day_names')[day % 7]
                 end
                 def month_name(month)
                   ::I18n.t('date.month_names')[month]
                 end
                 def valid_languages
                   @@valid_languages ||= Dir.glob(File.join(Rails.root, 'config', 'locales', '*.yml')).collect {|f| File.basename(f).split('.').first}.collect(&:to_sym)
                 end
                 def find_language(lang)
                   @@languages_lookup = valid_languages.inject({}) {|k, v| k[v.to_s.downcase] = v; k }
                   @@languages_lookup[lang.to_s.downcase]
                 end
                 def set_language_if_valid(lang)
                   if l = find_language(lang)
                     ::I18n.locale = l
                   end
                 end
                 def current_language
                   ::I18n.locale
                 end
               end
             end

test/unit/lib/redmine/i18n_test.rb +27 -1

             # Redmine - project management software
             # Copyright (C) 2006-2012  Jean-Philippe Lang
             #
             # This program is free software; you can redistribute it and/or
             # modify it under the terms of the GNU General Public License
             # as published by the Free Software Foundation; either version 2
             # of the License, or (at your option) any later version.
             #
             # This program is distributed in the hope that it will be useful,
             # but WITHOUT ANY WARRANTY; without even the implied warranty of
             # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
             # GNU General Public License for more details.
             #
             # You should have received a copy of the GNU General Public License
             # along with this program; if not, write to the Free Software
             # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
             require File.expand_path('../../../../test_helper', __FILE__)
             class Redmine::I18nTest < ActiveSupport::TestCase
               include Redmine::I18n
               include ActionView::Helpers::NumberHelper
               def setup
-                @hook_module = Redmine::Hook
+                User.current.language = nil
               end
               def test_date_format_default
                 set_language_if_valid 'en'
                 today = Date.today
                 Setting.date_format = ''
                 assert_equal I18n.l(today), format_date(today)
               end
               def test_date_format
                 set_language_if_valid 'en'
                 today = Date.today
                 Setting.date_format = '%d %m %Y'
                 assert_equal today.strftime('%d %m %Y'), format_date(today)
               end
+              def test_date_format_default_with_user_locale
+                set_language_if_valid 'es'
+                today = now = Time.parse('2011-02-20 14:00:00')
+                Setting.date_format = '%d %B %Y'
+                User.current.language = 'fr'
+                assert_equal "20 f\u00E9vrier 2011", format_date(today)
+                User.current.language = nil
+                assert_equal '20 Febrero 2011', format_date(today)
+              end
               def test_date_and_time_for_each_language
                 Setting.date_format = ''
                 valid_languages.each do |lang|
                   set_language_if_valid lang
                   assert_nothing_raised "#{lang} failure" do
                     format_date(Date.today)
                     format_time(Time.now)
                     format_time(Time.now, false)
                     assert_not_equal 'default', ::I18n.l(Date.today, :format => :default),
                                      "date.formats.default missing in #{lang}"
                     assert_not_equal 'time',    ::I18n.l(Time.now, :format => :time),
                                      "time.formats.time missing in #{lang}"
                   end
                   assert l('date.day_names').is_a?(Array)
                   assert_equal 7, l('date.day_names').size
                   assert l('date.month_names').is_a?(Array)
                   assert_equal 13, l('date.month_names').size
                 end
               end
               def test_time_for_each_zone
                 ActiveSupport::TimeZone.all.each do |zone|
                   User.current.stubs(:time_zone).returns(zone.name)
                   assert_nothing_raised "#{zone} failure" do
                     format_time(Time.now)
                   end
                 end
               end
               def test_time_format
                 set_language_if_valid 'en'
                 now = Time.parse('2011-02-20 15:45:22')
                 with_settings :time_format => '%H:%M' do
                   with_settings :date_format => '' do
                     assert_equal '02/20/2011 15:45', format_time(now)
                     assert_equal '15:45', format_time(now, false)
                   end
                   with_settings :date_format => '%Y-%m-%d' do
                     assert_equal '2011-02-20 15:45', format_time(now)
                     assert_equal '15:45', format_time(now, false)
                   end
                 end
               end
               def test_time_format_default
                 set_language_if_valid 'en'
                 now = Time.parse('2011-02-20 15:45:22')
                 with_settings :time_format => '' do
                   with_settings :date_format => '' do
                     assert_equal '02/20/2011 03:45 pm', format_time(now)
                     assert_equal '03:45 pm', format_time(now, false)
                   end
                   with_settings :date_format => '%Y-%m-%d' do
                     assert_equal '2011-02-20 03:45 pm', format_time(now)
                     assert_equal '03:45 pm', format_time(now, false)
                   end
                 end
               end
+              def test_time_format_default_with_user_locale
+                set_language_if_valid 'en'
+                User.current.language = 'fr'
+                now = Time.parse('2011-02-20 15:45:22')
+                with_settings :time_format => '' do
+                  with_settings :date_format => '' do
+                    assert_equal '20/02/2011 15:45', format_time(now)
+                    assert_equal '15:45', format_time(now, false)
+                  end
+                  with_settings :date_format => '%Y-%m-%d' do
+                    assert_equal '2011-02-20 15:45', format_time(now)
+                    assert_equal '15:45', format_time(now, false)
+                  end
+                end
+              end
               def test_time_format
                 set_language_if_valid 'en'
                 now = Time.now
                 Setting.date_format = '%d %m %Y'
                 Setting.time_format = '%H %M'
                 assert_equal now.strftime('%d %m %Y %H %M'), format_time(now)
                 assert_equal now.strftime('%H %M'), format_time(now, false)
               end
               def test_utc_time_format
                 set_language_if_valid 'en'
                 now = Time.now
                 Setting.date_format = '%d %m %Y'
                 Setting.time_format = '%H %M'
                 assert_equal now.strftime('%d %m %Y %H %M'), format_time(now.utc)
                 assert_equal now.strftime('%H %M'), format_time(now.utc, false)
               end
               def test_number_to_human_size_for_each_language
                 valid_languages.each do |lang|
                   set_language_if_valid lang
                   assert_nothing_raised "#{lang} failure" do
                     size = number_to_human_size(257024)
                     assert_match /251/, size
                   end
                 end
               end
               def test_valid_languages
                 assert valid_languages.is_a?(Array)
                 assert valid_languages.first.is_a?(Symbol)
               end
               def test_locales_validness
                 lang_files_count = Dir["#{Rails.root}/config/locales/*.yml"].size
                 assert_equal lang_files_count, valid_languages.size
                 valid_languages.each do |lang|
                   assert set_language_if_valid(lang)
                 end
                 set_language_if_valid('en')
               end
               def test_valid_language
                 to_test = {'fr' => :fr,
                            'Fr' => :fr,
                            'zh' => :zh,
                            'zh-tw' => :"zh-TW",
                            'zh-TW' => :"zh-TW",
                            'zh-ZZ' => nil }
                 to_test.each {|lang, expected| assert_equal expected, find_language(lang)}
               end
               def test_fallback
                 ::I18n.backend.store_translations(:en, {:untranslated => "Untranslated string"})
                 ::I18n.locale = 'en'
                 assert_equal "Untranslated string", l(:untranslated)
                 ::I18n.locale = 'fr'
                 assert_equal "Untranslated string", l(:untranslated)
                 ::I18n.backend.store_translations(:fr, {:untranslated => "Pas de traduction"})
                 ::I18n.locale = 'en'
                 assert_equal "Untranslated string", l(:untranslated)
                 ::I18n.locale = 'fr'
                 assert_equal "Pas de traduction", l(:untranslated)
               end
               def test_utf8
                 set_language_if_valid 'ja'
                 str_ja_yes  = "\xe3\x81\xaf\xe3\x81\x84"
                 i18n_ja_yes = l(:general_text_Yes)
                 if str_ja_yes.respond_to?(:force_encoding)
                   str_ja_yes.force_encoding('UTF-8')
                   assert_equal "UTF-8", i18n_ja_yes.encoding.to_s
                 end
                 assert_equal str_ja_yes, i18n_ja_yes
               end
             end

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages