Bug 29389

Summary: Add holding branch to holds queue report
Product: Koha Reporter: Nick Clemens <nick>
Component: Hold requestsAssignee: Nick Clemens <nick>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: gmcharlt
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:
22.11.00
Bug Depends on: 29015    
Bug Blocks:    
Attachments: Bug 29389: Add holding branch to the holds queue report
Bug 29389: Add holding branch to the holds queue report
Bug 29389: Add holding branch to the holds queue report
Bug 29389: Add holding branch to the holds queue report
Bug 29389: Add holding branch to the holds queue report

Description Nick Clemens 2021-11-02 11:57:07 UTC
When viewing 'All' branches for the holds queue, it would be nice to be able to see which branch the item is located in
Comment 1 Nick Clemens 2021-11-02 12:01:14 UTC
Created attachment 127212 [details] [review]
Bug 29389: Add holding branch to the holds queue report

This patch adds the holding branch to the hold queue report.
It is added as hidden by default, when running per branch it is not needed

To test:
1 - Place some holds in your system
2 - perl misc/cronjobs/holds/build_holdsqueue.pl
3 - View the holds queue
4 - Note there is no holding branch column
5 - Apply patch
6 - Reload
7 - Note the column is not there
8 - Click column setting gear
9 - Check the column and confirm it appears
10 - Confirmthe filter works
Comment 2 Katrin Fischer 2021-11-02 14:41:48 UTC
Thx :)
Comment 3 Owen Leonard 2021-11-03 11:52:30 UTC
I don't think the new column should be first. Title is almost always first in tables like this and I think we should keep it consistent.

The new column should be called "Current library" to be consistent with the detail page and holds awaiting pickup.
Comment 4 Nick Clemens 2022-09-21 18:39:33 UTC
Created attachment 140828 [details] [review]
Bug 29389: Add holding branch to the holds queue report

This patch adds the holding branch to the hold queue report.
It is added as hidden by default, when running per branch it is not needed

To test:
 1 - Place some holds in your system
 2 - perl misc/cronjobs/holds/build_holdsqueue.pl
 3 - View the holds queue
 4 - Note there is no 'Current library' column
 5 - Apply patch
 6 - Reload
 7 - Note the column is not there
 8 - Click column setting gear
 9 - Check the column and confirm it appears
10 - Confirm the filter works
Comment 5 Nick Clemens 2022-09-21 18:47:29 UTC
Created attachment 140829 [details] [review]
Bug 29389: Add holding branch to the holds queue report

This patch adds the holding branch to the hold queue report.
It is added as hidden by default, when running per branch it is not needed

To test:
 1 - Place some holds in your system
 2 - perl misc/cronjobs/holds/build_holdsqueue.pl
 3 - View the holds queue
 4 - Note there is no 'Current library' column
 5 - Apply patch
 6 - Reload
 7 - Note the column is not there
 8 - Click column setting gear
 9 - Check the column and confirm it appears
10 - Confirm the filter works
Comment 6 Owen Leonard 2022-09-21 20:30:15 UTC
Created attachment 140835 [details] [review]
Bug 29389: Add holding branch to the holds queue report

This patch adds the holding branch to the hold queue report.
It is added as hidden by default, when running per branch it is not needed

To test:
 1 - Place some holds in your system
 2 - perl misc/cronjobs/holds/build_holdsqueue.pl
 3 - View the holds queue
 4 - Note there is no 'Current library' column
 5 - Apply patch
 6 - Reload
 7 - Note the column is not there
 8 - Click column setting gear
 9 - Check the column and confirm it appears
10 - Confirm the filter works

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 7 Katrin Fischer 2022-10-01 23:11:56 UTC
Created attachment 141201 [details] [review]
Bug 29389: Add holding branch to the holds queue report

This patch adds the holding branch to the hold queue report.
It is added as hidden by default, when running per branch it is not needed

To test:
 1 - Place some holds in your system
 2 - perl misc/cronjobs/holds/build_holdsqueue.pl
 3 - View the holds queue
 4 - Note there is no 'Current library' column
 5 - Apply patch
 6 - Reload
 7 - Note the column is not there
 8 - Click column setting gear
 9 - Check the column and confirm it appears
10 - Confirm the filter works

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

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 8 Tomás Cohen Arazi 2022-10-03 13:09:54 UTC
Pushed to master for 22.11.

Nice work everyone, thanks!