Bug 13474 - Untranslatable log actions
Summary: Untranslatable log actions
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: I18N/L10N (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial (vote)
Assignee: Aleisha Amohia
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 15658
  Show dependency treegraph
 
Reported: 2014-12-17 09:20 UTC by paxed
Modified: 2017-06-14 22:10 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: String patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 13474: Adding untranslatable log actions to viewlog.tt (1.03 KB, patch)
2016-01-19 20:30 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 13474: Adding untranslatable log actions to viewlog.tt (1.06 KB, patch)
2016-01-20 21:13 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 13474: Adding untranslatable log actions to viewlog.tt (1.06 KB, patch)
2016-01-21 01:05 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 13474: Adding untranslatable log actions to viewlog.tt (1.27 KB, patch)
2016-01-25 17:35 UTC, Marc Véron
Details | Diff | Splinter Review
Bug 13474: Adding untranslatable log actions to viewlog.tt (1.33 KB, patch)
2016-01-27 11:13 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description paxed 2014-12-17 09:20:10 UTC
Log actions "RENEW", "CHANGE PASS", "ADDCIRCMESSAGE" and "DELCIRCMESSAGE" are untranslatable, need to be added to translate_log_action block in viewlog.tt
Comment 1 Aleisha Amohia 2016-01-19 20:30:29 UTC Comment hidden (obsolete)
Comment 2 Mark Tompsett 2016-01-20 20:43:18 UTC
I think this is a good start. However:

mtompset@debian:~/kohaclone$ git grep "[         ]logaction(" | cut -f2 -d":" | cut -f2 -d"," | cut -f2 -d"(" | sed -e "s#^ *##g" | sed -e "s#\"#'#g" | sort -u

'ADD'
'ADDCIRCMESSAGE'
'CHANGE PASS'
'CREATE'
'DELCIRCMESSAGE'
'DELETE'
'ISSUE'
'MODIFY'
'RENEW'
'RETURN'
'Run'
mtompset@debian:~/kohaclone$

[% BLOCK translate_log_action %]
[%    SWITCH action %]
[%        CASE 'ADD'    %]Add
[%        CASE 'DELETE' %]Delete
[%        CASE 'MODIFY' %]Modify
[%        CASE 'ISSUE'  %]Checkout
[%        CASE 'RETURN' %]Return
[%        CASE 'CREATE' %]Create
[%        CASE %][% action %]
[%    END %]
[% END %]

SCRATCH PAD:
'ADDCIRCMESSAGE'
'CHANGE PASS'
'DELCIRCMESSAGE'
'RENEW'
'Run'

There is one more action missing: 'Run'.
Comment 3 Aleisha Amohia 2016-01-20 21:13:23 UTC Comment hidden (obsolete)
Comment 4 Mark Tompsett 2016-01-20 23:55:32 UTC
(In reply to Aleisha Amohia from comment #3)
> Created attachment 47051 [details] [review] [review]
> Bug 13474: Adding untranslatable log actions to viewlog.tt
> 
> Added 'Run'
> 
> Signed-off-by: Aleisha <aleishaamohia@hotmail.com>

Sorry, your case is wrong. The code is literally 'Run', not 'RUN'. Yes, I found that strange too.
Comment 5 Aleisha Amohia 2016-01-21 01:05:47 UTC Comment hidden (obsolete)
Comment 6 Marc Véron 2016-01-25 17:35:11 UTC Comment hidden (obsolete)
Comment 7 Jonathan Druart 2016-01-27 11:13:45 UTC
Created attachment 47331 [details] [review]
Bug 13474: Adding untranslatable log actions to viewlog.tt

Added 'Run'

Signed-off-by: Aleisha <aleishaamohia@hotmail.com>

(Amended to remove tab chars to make qa tools happy)
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 8 Brendan Gallagher 2016-02-24 03:20:23 UTC
Pushed to Master - Should be in the May 2016 Release.  Thanks!
Comment 9 Julian Maurice 2016-03-07 10:54:01 UTC
Patch pushed to 3.22.x, will be in 3.22.5
Comment 10 Frédéric Demians 2016-03-15 08:59:48 UTC
This patch has been pushed to 3.20.x, will be in 3.20.10.