Bug 12062

Summary: Untranslatable "Reserve not found"
Product: Koha Reporter: paxed <pasi.kallinen>
Component: I18N/L10NAssignee: Bernardo Gonzalez Kriegel <bgkriegel>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: bgkriegel, f.demians, jonathan.druart, kyle, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12043
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 12062 - Untranslatable "Reserve not found"
Bug 12062 - Untranslatable "Reserve not found"
[SIGNED-OFF] Bug 12062 - Untranslatable "Reserve not found"
Bug 12062 - Follow up to fix more untraslatable cases
Bug 12062 - Follow up to fix more untraslatable cases
Bug 12062 - Follow up to fix more untraslatable cases
Bug 12062 - Follow up to fix more untraslatable cases
[PASSED QA] Bug 12062 - Untranslatable "Reserve not found"
[PASSED QA] Bug 12062 - Follow up to fix more untraslatable cases
Bug 12062: Follow up - adds missing )

Description paxed 2014-04-10 08:41:44 UTC
circ/hold-transfer-slip.pl:    $slip = "Reserve not found";

The same file also has the title hardcoded as "Koha -- Circulation: Transfers"
Comment 1 Bernardo Gonzalez Kriegel 2014-04-22 11:48:28 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2014-04-22 12:58:19 UTC
This patch really ought to update the text to use the correct term: 'hold' instead of 'reserve.'

While you're at it, the page title ought to conform to other pages:

Koha &rsaquo; Circulation &rsaquo; Transfers

Not a big deal, but consistency is good.
Comment 3 Bernardo Gonzalez Kriegel 2014-04-22 13:54:46 UTC Comment hidden (obsolete)
Comment 4 Bernardo Gonzalez Kriegel 2014-04-22 13:59:34 UTC
(In reply to Owen Leonard from comment #2)
> Not a big deal, but consistency is good.

Of course :)
New patch to test.
Comment 5 Owen Leonard 2014-04-22 17:46:58 UTC Comment hidden (obsolete)
Comment 6 Kyle M Hall (khall) 2014-04-25 13:54:17 UTC
printslip.tt is used by circ/transfer-slip.pl, members/printslip.pl and opac/sco/printslip.pl as well as circ/hold-transfer-slip.pl

The fixed title will not work here. You'll need to add a flag and title for each of these scripts to the template so each can have a separate translatable title.
Comment 7 Bernardo Gonzalez Kriegel 2014-04-29 19:38:04 UTC Comment hidden (obsolete)
Comment 8 Bernardo Gonzalez Kriegel 2014-04-29 19:39:40 UTC Comment hidden (obsolete)
Comment 9 paxed 2014-04-30 08:14:27 UTC
<title>Koha &rsaquo; Members &rsaquo; Print receipt for [%- borrowernumber -%]</title>

The [%- will eat away the space between the "for" and the borrowernumber.
Comment 10 Bernardo Gonzalez Kriegel 2014-04-30 11:00:05 UTC Comment hidden (obsolete)
Comment 11 Bernardo Gonzalez Kriegel 2014-04-30 11:00:45 UTC
(In reply to paxed from comment #9)
> <title>Koha &rsaquo; Members &rsaquo; Print receipt for [%- borrowernumber
> -%]</title>
> 
> The [%- will eat away the space between the "for" and the borrowernumber.

Thanks, now is fixed.
Comment 12 Jonathan Druart 2014-05-22 15:01:34 UTC
Not sure it is pertinent, but bug 12043 impacts the same templates.
Comment 13 Chris Cormack 2014-07-03 05:23:55 UTC Comment hidden (obsolete)
Comment 14 Katrin Fischer 2014-07-06 17:03:41 UTC
The QA script points out a problem with these patches:
 FAIL	koha-tmpl/intranet-tmpl/prog/en/modules/circ/printslip.tt
   OK	  tt_valid
   FAIL	  valid_template
		parse error - /home/katrin/kohaclone/koha-tmpl/intranet-tmpl/prog/en/modules/circ/printslip.tt line 3: unexpected end of directive
  [% IF ( caller = 'hold-transfer' %]

I am fixing this in a follow up.
Comment 15 Katrin Fischer 2014-07-06 17:20:28 UTC
Created attachment 29515 [details] [review]
[PASSED QA] Bug 12062 - Untranslatable "Reserve not found"

This patch rewrites transfer slip code to make some
strings translatable.
Also simplifies some code and parameters.

To test:
1) Produce a transfer slip, no changes must be noted.

2) Update PO file, new strings
msgid "%s %s%s%sNo hold found%s %s %s "
msgid "%s%s%sNo hold found%s "
msgid "Koha &rsaquo; Circulation &rsaquo; Transfers"

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Comment 16 Katrin Fischer 2014-07-06 17:20:33 UTC
Created attachment 29516 [details] [review]
[PASSED QA] Bug 12062 - Follow up to fix more untraslatable cases

This patch modifies staff printslip.tt to take into account
caller. Also fixes similar problem on opac printslip.tt

To test:
staff
1. Print hold transfer slip
2. Print transfer slip
3. Print member slip
opac
4. Print self checkout slip

5. Update translation files, check new strings are present

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Comment 17 Katrin Fischer 2014-07-06 17:20:37 UTC
Created attachment 29517 [details] [review]
Bug 12062: Follow up - adds missing )

With this patch, patches pass QA script and unit tests.
Tested printing different slips successfully.
Comment 18 Tomás Cohen Arazi (tcohen) 2014-07-07 14:02:07 UTC
Patches pushed to master.

Thanks Bernardo and Katrina!