[Patches] [PATCH] Bug 7282 Invalid language selection

[koha-patchbot at kohaaloha.com]

From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Demians?= <f.demians at tamil.fr>
Date: Thu, 1 Dec 2011 17:52:22 +0100
Subject: [PATCH] Bug 7282 Invalid language selection
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Frère Sébastien Marie refactored patch to ensure that without cookie, language
selection is based on browser preferences.

To reproduce the bug:

- Activate en and fr-FR for OPAC
- Clear your browser cookies
- Select your language preference in your browser: fr, fr-fr, en
- Load OPAC main page
- OPAC is displayed in English, rather than French as asked by browser
  preferences

Apply the patch and test:

- Clear your browser cookies
- Load OPAC main page
- Pages are displayed in French
---
 C4/Templates.pm |   20 +++++---------------
 1 files changed, 5 insertions(+), 15 deletions(-)

diff --git a/C4/Templates.pm b/C4/Templates.pm
index 73d94c2..8b90c20 100644
--- a/C4/Templates.pm
+++ b/C4/Templates.pm
@@ -277,8 +277,11 @@ sub themelanguage {
     my @languages = split(",", C4::Context->preference(
         $is_intranet ? 'language' : 'opaclanguages'));
     my $lang;
-    $lang = getlanguagecookie($query);
-    unless ($lang) {
+    if ( $query->cookie('KohaOpacLanguage') ) {
+        $lang = $query->cookie('KohaOpacLanguage') ;
+        $lang =~ s/[^a-zA-Z_-]*//; #sanitzie
+    }
+    else {
         my $http_accept_language = $ENV{ HTTP_ACCEPT_LANGUAGE };
         $lang = accept_language( $http_accept_language, 
             getTranslatedLanguages($interface,'prog') );
@@ -317,18 +320,5 @@ sub setlanguagecookie {
     );
 }
 
-sub getlanguagecookie {
-    my ($query) = @_;
-    my $lang;
-    if ($query->cookie('KohaOpacLanguage')){
-        $lang = $query->cookie('KohaOpacLanguage') ;
-    }else{
-        $lang = $ENV{HTTP_ACCEPT_LANGUAGE};
-        
-    }
-    $lang =~ s/[^a-zA-Z_-]*//; #sanitzie
-    return $lang;
-}
-
 1;
 
-- 
1.7.6.1