Bug 28802

Summary: Untranslatable strings in browser.js
Product: Koha Reporter: Fridolin Somers <fridolin.somers>
Component: Staff interfaceAssignee: Fridolin Somers <fridolin.somers>
Status: Pushed to oldoldoldstable --- QA Contact: Tomás Cohen Arazi <tomascohen>
Severity: normal    
Priority: P5 - low CC: gmcharlt, jonathan.druart, kyle, tomascohen, victor, wainuiwitikapark
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:
21.11.00,21.05.03,20.11.10,20.05.16,19.11.22
Bug Depends on: 25321    
Bug Blocks:    
Attachments: Bug 28802: Untranslatable strings in browser.js
Bug 28802: Untranslatable strings in browser.js
Bug 28802: Untranslatable strings in browser.js
Bug 28802: Fix Asset.t

Description Fridolin Somers 2021-08-03 07:10:30 UTC
File koha-tmpl/intranet-tmpl/js/browser.js is not parsed by translation process, which uses koha-tmpl/intranet-tmpl/prog/js/**/*.js
We must move it to prog/js.
Comment 1 Jonathan Druart 2021-08-03 08:33:34 UTC
Caused by:
  commit 5fa99d7d59ffe65a3533c79a637b50f04e904736
  Bug 25321: Move translatable strings out of strings.inc into the corresponding JavaScript
Comment 2 Fridolin Somers 2021-08-03 09:07:04 UTC Comment hidden (obsolete)
Comment 3 Owen Leonard 2021-08-03 11:06:57 UTC Comment hidden (obsolete)
Comment 4 Tomás Cohen Arazi 2021-08-17 13:13:00 UTC
Created attachment 123875 [details] [review]
Bug 28802: Untranslatable strings in browser.js

File koha-tmpl/intranet-tmpl/js/browser.js is not parsed by translation
process, which uses koha-tmpl/intranet-tmpl/prog/js/**/*.js
We must move it to prog/js.

Test plan :
1) Perform a search on staff interface
2) Click on a result
3) Check you see records browser

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 5 Jonathan Druart 2021-08-18 12:50:32 UTC
Shouldn't we actually move all the JS files to koha-tmpl/intranet-tmpl/prog/js?
Comment 6 Jonathan Druart 2021-08-18 13:36:39 UTC
Pushed to master for 21.11, thanks to everybody involved!
Comment 7 Jonathan Druart 2021-08-18 14:26:11 UTC
Created attachment 123962 [details] [review]
Bug 28802: Fix Asset.t
Comment 8 Jonathan Druart 2021-08-18 14:31:43 UTC
(In reply to Jonathan Druart from comment #7)
> Created attachment 123962 [details] [review] [review]
> Bug 28802: Fix Asset.t

Patch pushed to master.
Comment 9 Fridolin Somers 2021-08-19 06:54:36 UTC
(In reply to Jonathan Druart from comment #5)
> Shouldn't we actually move all the JS files to
> koha-tmpl/intranet-tmpl/prog/js?

I think yes but I can not confirm.
Comment 10 Kyle M Hall 2021-08-20 16:41:14 UTC
Pushed to 21.05.x for 21.05.03
Comment 11 Fridolin Somers 2021-08-27 18:57:15 UTC
Pushed to 20.11.x for 20.11.10
Comment 12 Victor Grousset/tuxayo 2021-08-31 04:29:20 UTC
Backported: Pushed to 20.05.x branch for 20.05.16
Comment 13 wainuiwitikapark 2021-09-07 01:54:37 UTC
Should this be backported to 19.11.x for 19.11.22?
Comment 14 Victor Grousset/tuxayo 2021-09-07 22:44:50 UTC
Even if the severity is "normal" I backported it since it's about translation and it applied without issues. I do that with accessibility stuff also. Your call if you feel backporting some additional things those.
Comment 15 wainuiwitikapark 2021-09-08 02:38:37 UTC
Backported to 19.11.x for 19.11.22