Attachment 24973 Details for Bug 11655 – Bug 11655: koha-translate --list shouldn't show 'en'

[patch] Bug 11655: koha-translate --list shouldn't show 'en'

Bug-11655-koha-translate---list-shouldnt-show-en.patch (text/plain), 1.63 KB, created by Chris Cormack on 2014-02-01 03:36:18 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Chris Cormack

Created: 2014-02-01 03:36:18 UTC

Size: 1.63 KB

patch

obsolete

>From 2ca7c4471f54c0b7ff62cc243df52778c85724d0 Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@gmail.com>
>Date: Fri, 31 Jan 2014 16:52:15 -0300
>Subject: [PATCH] Bug 11655: koha-translate --list shouldn't show 'en'
>
>'en' is not a translation and it doesn't make any sense to return it as part of the installed translations or the available ones.
>
>It becomes relevant to remove it as users upgrading their installed translations end up grepping the output to remove 'en' like in:
>
>for lang in $(koha-translate --list | grep -v -x -e 'en') ; do
>  koha-translate --update $lang
>done
>
>instead of simply issuing the koha-translate --list command.
>
>To test:
>- Run
>  ./koha-translate --list
>  ./koha-translate --list --available
>=> It shouldn't return 'en' in the result list.
>
>Regards
>To+
>
>Sponsored-by: Universidad Nacional de Cordoba
>Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
>---
> debian/scripts/koha-translate | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
>diff --git a/debian/scripts/koha-translate b/debian/scripts/koha-translate
>index 78bc23e..0164a72 100755
>--- a/debian/scripts/koha-translate
>+++ b/debian/scripts/koha-translate
>@@ -71,14 +71,15 @@ print_available()
> {
>     # Loop over only one opac theme
>     for i in $( ls $PO_DIR | grep opac-t-prog ); do
>-        echo `basename $i -i-opac-t-prog-v-3006000.po`
>+        echo `basename $i -i-opac-t-prog-v-3006000.po` | \
>+            grep -v -x -e en
>     done
> }
> 
> print_installed()
> {
>     ls $KOHA_INSTALL_DIR/opac/htdocs/opac-tmpl/prog/ | \
>-        grep -v -e images -e itemtypeimg
>+        grep -v -e images -e itemtypeimg -x -e en
> }
> 
> install_lang()
>-- 
>1.8.5.3

Actions: View | Diff | Splinter Review

Attachments on bug 11655: 24965 | 24973 | 25018