Bug 20193

Summary: Path to Greybox CSS broken after Bug 12904
Product: Koha Reporter: Claire Gravely <claire_gravely>
Component: Label/patron card printingAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Katrin Fischer <katrin.fischer>
Severity: minor    
Priority: P5 - low CC: katrin.fischer, mtompset
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:
Bug Depends on: 12904    
Bug Blocks: 19709, 20160    
Attachments: Bug 20193: Path to Greybox CSS broken after Bug 12904
Bug 20193: Path to Greybox CSS broken after Bug 12904
Bug 20193: Path to Greybox CSS broken after Bug 12904

Description Claire Gravely 2018-02-14 10:28:36 UTC
When trying to export a label batch, the pop up frame to display export options is not being correctly created. Instead the export information is at the bottom of a never-ending page. (Scrolling never reaches it).

To reproduce:
Go to Manage Label batches (label-manage.pl) and/or edit a batch (label-edit-batch.pl) and try to export.
Comment 1 Owen Leonard 2018-02-14 14:19:26 UTC
Created attachment 71608 [details] [review]
Bug 20193: Path to Greybox CSS broken after Bug 12904

Bug 12904 changed the way CSS and JS assets should be linked, but the
Greybox CSS path wasn't changed in the process. This patch adds the
required VERSION template variable to greybox.inc.

To test, apply the patch and test usage of Greybox in the staff client.
For instance:

- Go to Tools -> Labels -> Manage -> Label batches
- Select a batch for export and click the 'Export selected' button
- A Greybox modal should be triggered, and the export process should
  proceed correctly.
Comment 2 Mark Tompsett 2018-02-14 16:43:12 UTC
Created attachment 71621 [details] [review]
Bug 20193: Path to Greybox CSS broken after Bug 12904

Bug 12904 changed the way CSS and JS assets should be linked, but the
Greybox CSS path wasn't changed in the process. This patch adds the
required VERSION template variable to greybox.inc.

To test, apply the patch and test usage of Greybox in the staff client.
For instance:

- Go to Tools -> Labels -> Manage -> Label batches
- Select a batch for export and click the 'Export selected' button
- A Greybox modal should be triggered, and the export process should
  proceed correctly.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Comment 3 Mark Tompsett 2018-02-14 16:46:10 UTC
Another test case is the Translate into other languages step in bug 20160. If you do not apply this, the popup failed to popup. It was quite frustrating. Thank you for this patch, Owen!
Comment 4 Mark Tompsett 2018-02-14 17:34:03 UTC
Also, noticed that the patch that made it into master (the apache conf tweak) was [0-9][0-9]... instead of just (.*), so part of the problems may have been the fix was partially broken by use of a bad regular expression.
Comment 5 Katrin Fischer 2018-02-16 06:35:20 UTC
Created attachment 71698 [details] [review]
Bug 20193: Path to Greybox CSS broken after Bug 12904

Bug 12904 changed the way CSS and JS assets should be linked, but the
Greybox CSS path wasn't changed in the process. This patch adds the
required VERSION template variable to greybox.inc.

To test, apply the patch and test usage of Greybox in the staff client.
For instance:

- Go to Tools -> Labels -> Manage -> Label batches
- Select a batch for export and click the 'Export selected' button
- A Greybox modal should be triggered, and the export process should
  proceed correctly.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 6 Jonathan Druart 2018-02-16 16:02:02 UTC
Pushed to master for 18.05, thanks to everybody involved!