From: lpsolit%gmail.com <> Date: Tue, 21 Aug 2007 02:05:43 +0000 (+0000) Subject: Bug 335354: editparams.cgi crashes when editing the 'languages' and 'defaultlanguage... X-Git-Tag: bugzilla-3.1.2~60 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5a1fdacb12095b417382c15186e5aaaec7eed3a1;p=thirdparty%2Fbugzilla.git Bug 335354: editparams.cgi crashes when editing the 'languages' and 'defaultlanguage' parameters - Patch by Frédéric Buclin r=wurblzap, mkanat a=LpSolit --- diff --git a/Bugzilla/Config/Common.pm b/Bugzilla/Config/Common.pm index 14406019dd..8435b20a1a 100644 --- a/Bugzilla/Config/Common.pm +++ b/Bugzilla/Config/Common.pm @@ -305,17 +305,23 @@ sub check_image_converter { } sub check_languages { - my @languages = split /[,\s]+/, trim($_[0]); + my ($lang) = @_; + my @languages = split(/[,\s]+/, trim($lang)); if(!scalar(@languages)) { return "You need to specify a language tag." } my $templatedir = bz_locations()->{'templatedir'}; + my %lang_seen; + my @validated_languages; foreach my $language (@languages) { if( ! -d "$templatedir/$language/custom" && ! -d "$templatedir/$language/default") { return "The template directory for $language does not exist"; } + push(@validated_languages, $language) unless $lang_seen{$language}++; } + # Rebuild the list of language tags, avoiding duplicates. + $_[0] = join(', ', @validated_languages); return ""; } diff --git a/Bugzilla/Install/Util.pm b/Bugzilla/Install/Util.pm index 931d9f1b55..9bab77a8a7 100644 --- a/Bugzilla/Install/Util.pm +++ b/Bugzilla/Install/Util.pm @@ -127,7 +127,7 @@ sub template_include_path { my @supported; if (defined $params->{use_languages}) { - @supported = $params->{use_languages}; + @supported = @{$params->{use_languages}}; } else { my @dirs = glob(bz_locations()->{'templatedir'} . "/*"); diff --git a/Bugzilla/Template.pm b/Bugzilla/Template.pm index ece9597788..20fe831125 100644 --- a/Bugzilla/Template.pm +++ b/Bugzilla/Template.pm @@ -87,7 +87,7 @@ sub getTemplateIncludePath { my $cache = Bugzilla->request_cache; my $lang = $cache->{'language'} || ""; $cache->{"template_include_path_$lang"} ||= template_include_path({ - use_languages => trim(Bugzilla->params->{'languages'}), + use_languages => [split(/[\s,]+/, Bugzilla->params->{'languages'})], only_language => $lang }); return $cache->{"template_include_path_$lang"}; }