Bug 29389 - Add holding branch to holds queue report
Summary: Add holding branch to holds queue report
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Hold requests (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Nick Clemens
QA Contact: Testopia
URL:
Keywords:
Depends on: 29015
Blocks:
  Show dependency treegraph
 
Reported: 2021-11-02 11:57 UTC by Nick Clemens
Modified: 2023-06-08 22:28 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
22.11.00


Attachments
Bug 29389: Add holding branch to the holds queue report (2.60 KB, patch)
2021-11-02 12:01 UTC, Nick Clemens
Details | Diff | Splinter Review
Bug 29389: Add holding branch to the holds queue report (2.62 KB, patch)
2022-09-21 18:39 UTC, Nick Clemens
Details | Diff | Splinter Review
Bug 29389: Add holding branch to the holds queue report (3.07 KB, patch)
2022-09-21 18:47 UTC, Nick Clemens
Details | Diff | Splinter Review
Bug 29389: Add holding branch to the holds queue report (3.12 KB, patch)
2022-09-21 20:30 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 29389: Add holding branch to the holds queue report (3.18 KB, patch)
2022-10-01 23:11 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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!