Bug 9283

Summary: Change structure of export checkouts form
Product: Koha Reporter: Owen Leonard <oleonard>
Component: CirculationAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: enhancement    
Priority: P5 - low CC: gmcharlt, jonathan.druart, kyle.m.hall, m.de.rooy
Version: Main   
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:    
Bug Blocks: 9294    
Attachments: Export checkouts proposed revised interface
Bug 9283 - Change structure of export checkouts form
Bug 9283 - Change structure of export checkouts form
Bug 9283 - Change structure of export checkouts form

Description Owen Leonard 2012-12-13 20:13:47 UTC
Created attachment 14103 [details]
Export checkouts proposed revised interface

The export checkouts form on the circulation page uses a YUI-generated menu button to offer export format choices. I think it would be simple to create this menu in HTML. I think it would also make the action of the form clearer.
Comment 1 Owen Leonard 2012-12-13 20:20:18 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2012-12-14 11:26:45 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2012-12-14 11:27:57 UTC
Thanks for this ergonomic improvement!
Comment 4 Marcel de Rooy 2013-02-08 10:33:16 UTC
QA: Looking at this one..
Comment 5 Marcel de Rooy 2013-02-08 10:45:36 UTC
Created attachment 15158 [details] [review]
Bug 9283 - Change structure of export checkouts form

This patch changes the structure of the export checkouts form
so that it is a little more linear. This change moves generation of the
export options from JavaScript to the markup, eliminating an instance
of dependence on YUI menus.

To test, enable checkout exports by specifying a value for
ExportWithCsvProfile or  ExportRemoveFields. Load a patron with
checkouts in circulation. Try the various checkout export options. Each
should function correctly.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 6 Marcel de Rooy 2013-02-08 10:47:33 UTC
QA Comment: Looks even better now. Presentation moved to template instead of javascript/yui. Works as advertised.

Passed QA
Comment 7 Jared Camins-Esakov 2013-02-08 14:48:52 UTC
This patch has been pushed to master.