Attachment 49881 Details for Bug 16194 – [PASSED QA] Bug 16194: Do not consider xslt as valid theme dir in LangInstaller.pm

[patch] [PASSED QA] Bug 16194: Do not consider xslt as valid theme dir in LangInstaller.pm

PASSED-QA-Bug-16194-Do-not-consider-xslt-as-valid-.patch (text/plain), 1.65 KB, created by Katrin Fischer on 2016-04-04 21:34:15 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2016-04-04 21:34:15 UTC

Size: 1.65 KB

patch

obsolete

>From 766a23e1670549d377ad0736815f4eba29bdce09 Mon Sep 17 00:00:00 2001
>From: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
>Date: Mon, 4 Apr 2016 10:59:37 -0300
>Subject: [PATCH] [PASSED QA] Bug 16194: Do not consider xslt as valid theme
> dir in LangInstaller.pm
>
>Bug 15527 add an xslt dir, LangInstaller.pm must ingnore that dir.
>
>To test:
>1) Verify the problem on current master
>Update translation for any lang, will see errors
>
>2) Apply the patch
>
>3) Update again, no errors
>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
>Works as expected.
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> misc/translator/LangInstaller.pm | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/misc/translator/LangInstaller.pm b/misc/translator/LangInstaller.pm
>index 5244945..683897d 100644
>--- a/misc/translator/LangInstaller.pm
>+++ b/misc/translator/LangInstaller.pm
>@@ -110,7 +110,7 @@ sub new {
> 
>     # OPAC themes
>     opendir my $dh, $context->config('opachtdocs');
>-    for my $theme ( grep { not /^\.|lib/ } readdir($dh) ) {
>+    for my $theme ( grep { not /^\.|lib|xslt/ } readdir($dh) ) {
>         push @{$self->{interface}}, {
>             name   => "OPAC $theme",
>             dir    => "$opachtdocs/$theme",
>@@ -123,7 +123,7 @@ sub new {
>         # search for strings on staff & opac marc files
>         my $dirs = $context->config('intrahtdocs') . '/prog';
>         opendir $fh, $context->config('opachtdocs');
>-        for ( grep { not /^\.|\.\.|lib$/ } readdir($fh) ) {
>+        for ( grep { not /^\.|\.\.|lib$|xslt/ } readdir($fh) ) {
>             $dirs .= ' ' . "$opachtdocs/$_";
>         }
>         push @{$self->{interface}}, {
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 16194: 49860 | 49862 | 49881