Bug 23441

Summary: Export should not include the 'actions' column in Z3950 results
Product: Koha Reporter: Martin Renvoize <martin.renvoize>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: fridolin.somers, katrin.fischer, maryse.simard
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:
19.11.00
Bug Depends on: 23307    
Bug Blocks:    
Attachments: Bug 23441: Export should not include the 'actions' column in Z3950 results.
Bug 23441: Export should not include the 'actions' column in Z3950 results.
Bug 23441: Export should not include the 'actions' column in Z3950 results.

Description Martin Renvoize 2019-08-09 08:06:11 UTC
As mentioned in bug 23307 

It would be nice if the "Actions" column could be dropped from the export (it's about 200 characters wide and there's no way to turn it off and it adds no value to the export) and if the "Additional fields" column could be included in options with the "Column settings" button if AdditionalFieldsInZ3950ResultSearch is configured.
Comment 1 Owen Leonard 2019-08-09 11:46:59 UTC Comment hidden (obsolete)
Comment 2 Maryse Simard 2019-08-09 19:18:02 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2019-08-10 11:45:01 UTC
Created attachment 92125 [details] [review]
Bug 23441: Export should not include the 'actions' column in Z3950 results.

This patch adds the "noExport" class to the table header for the Z39.50
search results "Actions" column. This class prevents the column from
being included in DataTables export operations.

To test, apply the patch and perform a Z39.50 search from Cataloging. On
the search results page, click the "Export or print" button. Choose one
of the export options and verify that in the resulting data there is no
"Actions" column.

Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Martin Renvoize 2019-08-13 10:33:52 UTC
Nice work!

Pushed to master for 19.11.00
Comment 5 Fridolin Somers 2019-09-02 12:17:01 UTC
Depends on Bug 23307 not in 19.05.x