Bug 11571

Summary: Bug 7934 introduces a regression in the translate script
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: I18N/L10NAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: f.demians, fridolin.somers, gmcharlt
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 7934    
Bug Blocks:    
Attachments: Bug 11571: FIX regression introduced by bug 7934
Bug 11571: FIX regression introduced by bug 7934
[SIGNED OFF] Bug 11571 - FIX regression introduced by bug 7934
[PASSED QA] Bug 11571 - FIX regression introduced by bug 7934

Description Jonathan Druart 2014-01-16 14:46:39 UTC
Bug 7934 breaks the -f option introduced by bug 9161.
Comment 1 Jonathan Druart 2014-01-16 14:53:00 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2014-01-17 08:13:20 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2014-01-17 08:14:05 UTC
Second patch replaces 'not' with 'now' in the commit message...
Comment 4 Jonathan Druart 2014-01-24 15:22:56 UTC
Since it is a regression, easier to test and makes me waste a lot of time (on testing translation), I consider this bug as major.
Comment 5 Jonathan Druart 2014-01-24 15:23:36 UTC
s/easier/easy
Comment 6 Mark Tompsett 2014-01-24 16:05:29 UTC Comment hidden (obsolete)
Comment 7 Jonathan Druart 2014-01-24 16:32:10 UTC
> -- verify the output and the execution time is now corrected.
s/is/are
...
Comment 8 Katrin Fischer 2014-01-25 09:05:35 UTC
Created attachment 24729 [details] [review]
[PASSED QA] Bug 11571 - FIX regression introduced by bug 7934

Bug 7934 breaks the -f option introduced in bug 9161.

While the generation of the command tmpl_process3.pl, a space is
missing if the -x option is given.
The tmpl_process3.pl is called like:
/home/koha/src/misc/translator/tmpl_process3.pl -q update -i
/home/koha/src/koha-tmpl/intranet-tmpl/prog/en/ -s
/home/koha/src/misc/translator/po/fr-FR-i-staff-t-prog-v-3006000.po -r
-x 'help'-f pay.tt

Revised test plan:
1) cd ./misc/translator
2) put a warn at LangInstaller.pm line 375.
3) time ./translate update fr-FR -f pay.tt
-- note the execution time and the output. The options in the
    command contain "-x 'help'-f pay.tt"
    The -f param is not passed to the script.
    The execution time is strangely long.
5) git reset --hard origin/master
6) apply this patch
7) put a warn at LangInstaller.pm line 375.
8) time ./translate update fr-FR -f pay.tt
-- verify the output and the execution time is now corrected.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes all tests and QA script.
Comment 9 Galen Charlton 2014-01-26 15:53:15 UTC
Pushed to master.  Thanks, Jonathan!
Comment 10 Fridolin Somers 2014-06-03 09:51:36 UTC
I've choosen to not backport Bug 7934 to 3.14.x so this this resolved. 
This is now in 3.16.x