Bug 19877

Summary: Move template JavaScript to the footer: UNIMARC editor plugins, part 6
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: jonathan.druart, nick, sandboxes, testopia
Version: MainKeywords: Academy
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 19874    
Bug Blocks: 17858, 19878    
Attachments: Bug 19877: Move template JavaScript to the footer: UNIMARC editor plugins, part 6
Bug 19877: Move template JavaScript to the footer: UNIMARC editor plugins, part 6
Bug 19877: Move template JavaScript to the footer: UNIMARC editor plugins, part 6

Description Owen Leonard 2017-12-22 15:52:45 UTC
Most UNIMARC plugin templates already have JavaScript at the bottom of the template instead of the top, but they should still be modified to be consistent with other templates which have been updated as part of Bug 17858.

This patch modifies more UNIMARC editor plugins than ever before.
Comment 1 Owen Leonard 2017-12-22 16:37:04 UTC
Created attachment 70134 [details] [review]
Bug 19877: Move template JavaScript to the footer: UNIMARC editor plugins, part 6

This patch updates more UNIMARC plugin templates than ever before, so
that JavaScript is included in the footer following the same pattern as
other templates which have been modified to fix Bug 17858.

To test you do not need a UNIMARC system.

Apply the patch and for each plugin, configure a MARC subfield (e.g.
100$a) to use that plugin.

From the MARC edit page, trigger the plugin and confirm that changes
made in the popup window are saved to the corresponding field in the
editor.

- unimarc_field_128a.tt
- unimarc_field_128b.tt
- unimarc_field_128c.tt
- unimarc_field_130.tt
- unimarc_field_135a.tt
- unimarc_field_140.tt
- unimarc_field_141.tt
Comment 2 Biblibre Sandboxes 2018-03-14 13:53:12 UTC
Patch tested with a sandbox, by delaye <stephane.delaye@biblibre.com>
Comment 3 Biblibre Sandboxes 2018-03-14 13:53:33 UTC
Created attachment 72864 [details] [review]
Bug 19877: Move template JavaScript to the footer: UNIMARC editor plugins, part 6

This patch updates more UNIMARC plugin templates than ever before, so
that JavaScript is included in the footer following the same pattern as
other templates which have been modified to fix Bug 17858.

To test you do not need a UNIMARC system.

Apply the patch and for each plugin, configure a MARC subfield (e.g.
100$a) to use that plugin.

From the MARC edit page, trigger the plugin and confirm that changes
made in the popup window are saved to the corresponding field in the
editor.

- unimarc_field_128a.tt
- unimarc_field_128b.tt
- unimarc_field_128c.tt
- unimarc_field_130.tt
- unimarc_field_135a.tt
- unimarc_field_140.tt
- unimarc_field_141.tt

Signed-off-by: delaye <stephane.delaye@biblibre.com>
Comment 4 Katrin Fischer 2018-03-25 18:48:10 UTC
Created attachment 73253 [details] [review]
Bug 19877: Move template JavaScript to the footer: UNIMARC editor plugins, part 6

This patch updates more UNIMARC plugin templates than ever before, so
that JavaScript is included in the footer following the same pattern as
other templates which have been modified to fix Bug 17858.

To test you do not need a UNIMARC system.

Apply the patch and for each plugin, configure a MARC subfield (e.g.
100$a) to use that plugin.

From the MARC edit page, trigger the plugin and confirm that changes
made in the popup window are saved to the corresponding field in the
editor.

- unimarc_field_128a.tt
- unimarc_field_128b.tt
- unimarc_field_128c.tt
- unimarc_field_130.tt
- unimarc_field_135a.tt
- unimarc_field_140.tt
- unimarc_field_141.tt

Signed-off-by: delaye <stephane.delaye@biblibre.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Jonathan Druart 2018-03-26 20:36:22 UTC
Pushed to master for 18.05, thanks to everybody involved!
Comment 6 Nick Clemens 2018-04-23 22:51:02 UTC
Awesome work all, enhancement, not backported for stable