From 466e0056d7ca9480d94a3bb90558bf38889f129e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Demians?= 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