Bug 17353

Summary: Add phone number column to checkout search
Product: Koha Reporter: Nick Clemens <nick>
Component: CirculationAssignee: Nick Clemens <nick>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: david, dcook, gmcharlt, jonathan.druart, josef.moravec, katrin.fischer, kyle.m.hall, martin.renvoize
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This enhancement adds patrons phone numbers to the checkout search results. It is hidden by default and is displayed by configuring the columns for circulation tables (Administration > Additional parameters > Configure columns > Circulation > table_borrowers > untick the phone column).
Version(s) released in:
Bug Depends on:    
Bug Blocks: 22215, 30068    
Attachments: Bug 17353 - Add phone number column to checkout search
Bug 17353 - Add phone number column to checkout search
Bug 17353 - Add phone number column to checkout search
Bug 17353: Add phone number column to checkout search
Bug 17353: (follow-up) Adjust config variable for moremember.tt too
Bug 17353: Add phone number column to checkout search
Bug 17353: (follow-up) Adjust config variable for moremember.tt too

Description Nick Clemens 2016-09-26 17:42:59 UTC

    
Comment 1 Nick Clemens 2016-09-26 17:45:08 UTC Comment hidden (obsolete)
Comment 2 Josef Moravec 2016-09-27 05:30:36 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2016-09-27 14:40:57 UTC
Is it a very useful information to show for everybody?
Would not it better to hide it by default (and use the columns configuration tool)?
Comment 4 Katrin Fischer 2016-10-16 12:36:52 UTC
I like the idea of using the Columns Configuration tool here.

I also wondered about the phone number displaying here - can you explain why the library/libraries have asked for it? I notice we don't show the phone number in the more verbose patron search.
Comment 5 Nick Clemens 2019-01-08 18:03:12 UTC Comment hidden (obsolete)
Comment 6 Owen Leonard 2019-01-09 14:26:51 UTC
Created attachment 83743 [details] [review]
Bug 17353: Add phone number column to checkout search

This patch adds borrowers.phone as 'Primary phone' in the checkout
search results. It also ads the table to columns configuration

To test:
1 - Perform a checkout search
2 - Note phone is not displayed
3 - Apply patch
4 - Note phone does not display
5 - Visit columns configuration
6 - Unhide the 'phone' column
7 - Note phone displays
8 - Ensure page functions as before patch

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 7 Nick Clemens 2019-01-10 20:43:10 UTC
Created attachment 83775 [details] [review]
Bug 17353: (follow-up) Adjust config variable for moremember.tt too
Comment 8 Katrin Fischer 2019-01-13 22:14:34 UTC
Small favor: can we please use a clearer name for the table, that is consistent with the others? This uses: table_borrowers. Most use *-table and hyphens. Also... patron vs. borrower.
Comment 9 Katrin Fischer 2019-01-13 22:15:28 UTC
Created attachment 83858 [details] [review]
Bug 17353: Add phone number column to checkout search

This patch adds borrowers.phone as 'Primary phone' in the checkout
search results. It also ads the table to columns configuration

To test:
1 - Perform a checkout search
2 - Note phone is not displayed
3 - Apply patch
4 - Note phone does not display
5 - Visit columns configuration
6 - Unhide the 'phone' column
7 - Note phone displays
8 - Ensure page functions as before patch

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 10 Katrin Fischer 2019-01-13 22:15:33 UTC
Created attachment 83859 [details] [review]
Bug 17353: (follow-up) Adjust config variable for moremember.tt too

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 11 Katrin Fischer 2019-01-13 22:15:53 UTC
see comment 8
Comment 12 Nick Clemens 2019-01-28 14:11:38 UTC
Awesome work all!

Pushed to master for 19.05
Comment 13 Martin Renvoize 2019-02-01 12:40:47 UTC
Enhancement, will not be backported to 18.11.x series.