More improvements to locale selection for the data browser

Use the browser language preferences for non-logged in users.

app/controllers/application_controller.rb

@@ -12,6 +12,7 @@ class ApplicationController < ActionController::Base
   attr_accessor :current_user
 
   helper_method :current_user
+  helper_method :preferred_langauges
 
   private
 

app/helpers/browse_helper.rb

@@ -11,21 +11,9 @@ module BrowseHelper
     # don't look at object tags if redacted, so as to avoid giving
     # away redacted version tag information.
     unless object.redacted?
-      available_locales = Locale::List.new(name_locales(object))
+      available_locales = Locale.list(name_locales(object))
 
-      Rails.logger.info "available_locales = #{available_locales.map(&:to_s)}"
-
-      preferred_locales = if current_user
-                            current_user.preferred_languages
-                          else
-                            Locale.new(I18n.locale).candidates
-                          end
-
-      Rails.logger.info "preferred_locales = #{preferred_locales.expand.map(&:to_s)}"
-
-      locale = available_locales.preferred(preferred_locales)
-
-      Rails.logger.info "locale = #{locale}"
+      locale = available_locales.preferred(preferred_languages)
 
       if object.tags.include? "name:#{locale}"
         name = t "printable_name.with_name_html", :name => content_tag(:bdi, object.tags["name:#{locale}"].to_s), :id => content_tag(:bdi, name)

test/helpers/browse_helper_test.rb

@@ -154,7 +154,7 @@ class BrowseHelperTest < ActionView::TestCase
 
   private
 
-  def current_user
-    nil
+  def preferred_languages
+    Locale.list(I18n.locale)
   end
 end