Bug 18209

Summary: Patron's card manage.pl page is not fully translatable
Product: Koha Reporter: Blou <philippe.blouin>
Component: Label/patron card printingAssignee: Blou <philippe.blouin>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: francois.charbonnier, jonathan.druart, julian.maurice, katrin.fischer, kyle, mtj, philippe.blouin, veron
Version: master   
Hardware: All   
OS: All   
Whiteboard:
Change sponsored?: --- Patch complexity: Trivial patch
Bot Control: --- When did the bot last check this:
Who signed the patch off: Text to go in the release notes:
Attachments: Bug 18209 - Patron's card manage.pl page is not fully translatable
Bug 18209 - Patron's card manage.pl page is not fully translatable
Bug 18209 - Patron's card manage.pl page is not fully translatable
Bug 18209 - Patron's card manage.pl page is not fully translatable
Bug 18209 - Patron's card manage.pl page is not fully translatable
Bug 18209: Add default, 'Actions' and fix case

Description Blou 2017-03-03 21:24:16 UTC
The headers of the tables displayed in patroncards/manage.tt are hardcoded in the .pl and thus cannot be translated.

This simple fix reuse the translate_card_element tool already in the code to make the strings translatable.
Comment 1 Blou 2017-03-03 21:26:20 UTC
Created attachment 60817 [details] [review]
Bug 18209 - Patron's card manage.pl page is not fully translatable

The headers of the tables displayed in patroncards/manage.tt are hardcoded in the .pl and thus cannot be translated.

This simple fix reuse the translate_card_element tool already in the code to make the strings translatable.
Comment 2 Marc Véron 2017-03-07 13:51:02 UTC
Created attachment 60873 [details] [review]
Bug 18209 - Patron's card manage.pl page is not fully translatable

The headers of the tables displayed in patroncards/manage.tt are hardcoded in the .pl and thus cannot be translated.

This simple fix reuse the translate_card_element tool already in the code to make the strings translatable.

Applied patch and verified that table headers on manage.pl display the same as before.
Signed-off-by: Marc Véron <veron@veron.ch>
Comment 3 Jonathan Druart 2017-03-07 17:49:00 UTC
Philippe,
This is done differently in labels/label-manage.tt
We should stay consistent and use the same way to make these strings translatable in both template files.
Comment 4 Blou 2017-03-08 21:50:30 UTC
Created attachment 60935 [details] [review]
Bug 18209 - Patron's card manage.pl page is not fully translatable

The headers of the tables displayed in patroncards/manage.tt are hardcoded in the .pl and thus cannot be translated.

This simple fix reuse the translate_card_element tool already in the code to make the strings translatable.

Applied patch and verified that table headers on manage.pl display the same as before.
Signed-off-by: Marc Véron <veron@veron.ch>
Comment 5 Marc Véron 2017-03-09 09:38:11 UTC
Created attachment 60936 [details] [review]
Bug 18209 - Patron's card manage.pl page is not fully translatable

The headers of the tables displayed in patroncards/manage.tt are hardcoded in the .pl and thus cannot be translated.

This simple fix reuse the translate_card_element tool already in the code to make the strings translatable.

Applied patch and verified that table headers on manage.pl display the same as before.
Signed-off-by: Marc Véron <veron@veron.ch>

Re-tested, works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>
Comment 6 Jonathan Druart 2017-03-09 14:26:11 UTC
Created attachment 60953 [details] [review]
Bug 18209 - Patron's card manage.pl page is not fully translatable

The headers of the tables displayed in patroncards/manage.tt are hardcoded in the .pl and thus cannot be translated.

This simple fix reuse the translate_card_element tool already in the code to make the strings translatable.

Applied patch and verified that table headers on manage.pl display the same as before.
Signed-off-by: Marc Véron <veron@veron.ch>

Re-tested, works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 7 Jonathan Druart 2017-03-09 14:26:15 UTC
Created attachment 60954 [details] [review]
Bug 18209: Add default, 'Actions' and fix case

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 8 Kyle M Hall 2017-03-31 14:10:33 UTC
Pushed to master for 17.05, thanks Phillipe, Jonathan!
Comment 9 Katrin Fischer 2017-04-02 17:31:43 UTC
These patches have been pushed to 16.11.x and will be in 16.11.07.
Comment 10 Julian Maurice 2017-04-21 10:21:37 UTC
Pushed to 3.22.x for 3.22.20
Comment 11 Mason James 2017-05-03 03:58:11 UTC
Pushed to 16.05.x, for 16.05.12 release