Bug 28081

Summary: Make card number a label in patron search results
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, jonathan.druart, lucas
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
In the Patrons module, in patron search results, the user can now click the patron card number to select the checkbox for that row.
Version(s) released in:
21.05.00
Circulation function:
Attachments: Bug 28081: Make card number a label in patron search results
Bug 28081: Make card number a label in patron search results
Bug 28081: Make card number a label in patron search results

Description Owen Leonard 2021-04-02 18:23:22 UTC
I would like to modify patron search results in the patrons module so that each patron card number is wrapped in a label tag which controls the state of the corresponding checkbox.
Comment 1 Lucas Gass (lukeg) 2021-04-02 19:41:21 UTC
+1, good idea.
Comment 2 Owen Leonard 2021-04-05 16:37:54 UTC
Created attachment 119185 [details] [review]
Bug 28081: Make card number a label in patron search results

This patch modifies the output of the AJAX patron search results so that
the number in the card number column is wrapped in a <label> tag which
triggers the corresponding checkbox.

To test, apply the patch and go to the Patrons module.

 - Perform a patron search which will return more than one result or
   click on a letter to browse.
 - In the search results, confirm that clicking any card number triggers
   the corresponding checkbox.
Comment 3 David Nind 2021-04-05 20:24:10 UTC
Created attachment 119189 [details] [review]
Bug 28081: Make card number a label in patron search results

This patch modifies the output of the AJAX patron search results so that
the number in the card number column is wrapped in a <label> tag which
triggers the corresponding checkbox.

To test, apply the patch and go to the Patrons module.

 - Perform a patron search which will return more than one result or
   click on a letter to browse.
 - In the search results, confirm that clicking any card number triggers
   the corresponding checkbox.

Signed-off-by: David Nind <david@davidnind.com>
Comment 4 Jonathan Druart 2021-04-14 12:09:51 UTC
Created attachment 119566 [details] [review]
Bug 28081: Make card number a label in patron search results

This patch modifies the output of the AJAX patron search results so that
the number in the card number column is wrapped in a <label> tag which
triggers the corresponding checkbox.

To test, apply the patch and go to the Patrons module.

 - Perform a patron search which will return more than one result or
   click on a letter to browse.
 - In the search results, confirm that clicking any card number triggers
   the corresponding checkbox.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 5 Jonathan Druart 2021-04-14 13:36:30 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 6 Fridolin Somers 2021-04-19 12:54:22 UTC
Enhancement not pushed to 20.11.x