From 9eb6f1f7568ff22971beda0a0237e5c07c823498 Mon Sep 17 00:00:00 2001
From: JoenioCosta <JoenioCosta@3f533792-8f58-4932-b0fe-aaf55b0a4547>
Date: Sat, 7 Jun 2008 21:35:56 +0000
Subject: [PATCH] ActionItem373: fixing page reload after select language on dropdown

---
 app/helpers/language_helper.rb    |  3 +--
 test/unit/language_helper_test.rb | 21 +++++++++++++++++++++
 2 files changed, 22 insertions(+), 2 deletions(-)

diff --git a/app/helpers/language_helper.rb b/app/helpers/language_helper.rb
index 8d75ead..3955e28 100644
--- a/app/helpers/language_helper.rb
+++ b/app/helpers/language_helper.rb
@@ -14,7 +14,7 @@ module LanguageHelper
     if options[:element] == 'dropdown'
       select_tag('lang', 
         options_for_select(Noosfero.locales.map{|code,name| [name, code]}, current),
-        :onchange => remote_function(:update => 'wrap', :url => { :action => :index }, :with => "'lang='+value"),
+        :onchange => "document.location='?' + name + '=' + value;",
         :help => _('The language you choose here is the language used for options, buttons, etc. It does not affect the language of the content created by other users.')
       )
     else
@@ -27,7 +27,6 @@ module LanguageHelper
       end.join(' &mdash; ')
       content_tag('div', languages, :id => 'language-chooser', :help => _('The language you choose here is the language used for options, buttons, etc. It does not affect the language of the content created by other users.'))
     end
-
   end
 
 end
diff --git a/test/unit/language_helper_test.rb b/test/unit/language_helper_test.rb
index b4a2239..4d02857 100644
--- a/test/unit/language_helper_test.rb
+++ b/test/unit/language_helper_test.rb
@@ -36,6 +36,27 @@ class LanguageHelperTest < Test::Unit::TestCase
 
   end
 
+  should 'generate drodown language chooser correcly' do
+    Noosfero.expects(:locales).returns({ 'en' => 'English', 'pt_BR' => 'Português Brasileiro', 'fr' => 'Français', 'it' => 'Italiano' }).at_least_once
+
+    self.expects(:language).returns('pt_BR')
+    result = self.language_chooser(:element => 'dropdown')
+    assert_match /<option value="en" selected="selected">English<\/option>/, result
+
+    #assert_match /<strong>Português Brasileiro<\/strong>/, result
+    #assert_no_match /<strong>English<\/strong>/, result
+    #assert_no_match /<strong>Français<\/strong>/, result
+    #assert_no_match /<strong>Italiano<\/strong>/, result
+
+    #self.expects(:language).returns('fr')
+    #result = self.language_chooser
+    #assert_no_match /<strong>Português Brasileiro<\/strong>/, result
+    #assert_no_match /<strong>English<\/strong>/, result
+    #assert_match /<strong>Français<\/strong>/, result
+    #assert_no_match /<strong>Italiano<\/strong>/, result
+
+  end
+
   protected
 
   def _(s)
--
libgit2 0.21.2