Bug 39003 - Cannot see suspend column in user's hold table on OPAC
Summary: Cannot see suspend column in user's hold table on OPAC
Status: Pushed to stable
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Lucas Gass (lukeg)
QA Contact: Marcel de Rooy
URL:
Keywords:
Depends on: 37972
Blocks:
  Show dependency treegraph
 
Reported: 2025-01-30 16:34 UTC by Lucas Gass (lukeg)
Modified: 2025-02-20 20:52 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
This fixes the OPAC > Summary > Holds table for a logged in patron - the 'Suspend' column is now shown.
Version(s) released in:
25.05.00,24.11.02
Circulation function:


Attachments
Bug 39003: Move SuspendHoldsOpac check from script to template (2.26 KB, patch)
2025-01-30 16:46 UTC, Lucas Gass (lukeg)
Details | Diff | Splinter Review
Bug 39003: Fix the tab template variable in opac-user.pl (1.29 KB, patch)
2025-01-30 19:26 UTC, Lucas Gass (lukeg)
Details | Diff | Splinter Review
Bug 39003: Fix the tab template variable in opac-user.pl (1.29 KB, patch)
2025-01-30 20:57 UTC, Lucas Gass (lukeg)
Details | Diff | Splinter Review
Bug 39003: Fix the tab template variable in opac-user.pl (1.34 KB, patch)
2025-01-30 21:33 UTC, David Nind
Details | Diff | Splinter Review
Bug 39003: Fix the tab template variable in opac-user.pl (1.43 KB, patch)
2025-01-31 10:06 UTC, Marcel de Rooy
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Lucas Gass (lukeg) 2025-01-30 16:34:35 UTC
To recreate:

1. SuspendHoldsOpac to 'allow'
2. Make some holds for a patron 
3. Log in as that patron and go to 'Summary' 
4. Click the 'Holds' tab
5. No suspend column
Comment 1 Lucas Gass (lukeg) 2025-01-30 16:44:03 UTC
I don't see any reason to get the SuspendHoldsOpac value from the script, can check in the template.
Comment 2 Lucas Gass (lukeg) 2025-01-30 16:46:31 UTC
Created attachment 177357 [details] [review]
Bug 39003: Move SuspendHoldsOpac check from script to template

To test:
1. SuspendHoldsOpac to 'allow'
2. Make some holds for a patron
3. Log in as that patron and go to 'Summary'
4. Click the 'Holds' tab
5. No suspend column
6. APPLY PATCH, restart_all
7. Try again, the column should show.
8. Turn off SuspendHoldsOpac, the column should be hidden.
Comment 3 Lucas Gass (lukeg) 2025-01-30 19:26:57 UTC
Created attachment 177359 [details] [review]
Bug 39003: Fix the tab template variable in opac-user.pl

To test:
1. SuspendHoldsOpac to 'allow'
2. Make some holds for a patron
3. Log in as that patron and go to 'Summary'
4. Click the 'Holds' tab
5. No suspend column
6. APPLY PATCH, restart_all
7. Try again, the column should show.
8. Turn off SuspendHoldsOpac, the column should be hidden.
Comment 4 David Nind 2025-01-30 19:40:39 UTC
I'm getting the sha1/fake ancestor error.

git bz apply 39003

Bug 39003 - Cannot see suspend column in user's hold table on OPAC

177359 - Bug 39003: Fix the tab template variable in opac-user.pl

Apply? [(y)es, (n)o, (i)nteractive] y
Applying: Bug 39003: Fix the tab template variable in opac-user.pl
error: sha1 information is lacking or useless (opac/opac-user.pl).
error: could not build fake ancestor
Patch failed at 0001 Bug 39003: Fix the tab template variable in opac-user.pl
Comment 5 Lucas Gass (lukeg) 2025-01-30 20:57:35 UTC
Created attachment 177366 [details] [review]
Bug 39003: Fix the tab template variable in opac-user.pl

To test:
1. SuspendHoldsOpac to 'allow'
2. Make some holds for a patron
3. Log in as that patron and go to 'Summary'
4. Click the 'Holds' tab
5. No suspend column
6. APPLY PATCH, restart_all
7. Try again, the column should show.
8. Turn off SuspendHoldsOpac, the column should be hidden.
Comment 6 Lucas Gass (lukeg) 2025-01-30 20:58:07 UTC
(In reply to David Nind from comment #4)
> I'm getting the sha1/fake ancestor error.
> 
> git bz apply 39003

My fault, it should apply now.
Comment 7 David Nind 2025-01-30 21:33:58 UTC
Created attachment 177367 [details] [review]
Bug 39003: Fix the tab template variable in opac-user.pl

To test:
1. SuspendHoldsOpac to 'allow'
2. Make some holds for a patron
3. Log in as that patron and go to 'Summary'
4. Click the 'Holds' tab
5. No suspend column
6. APPLY PATCH, restart_all
7. Try again, the column should show.
8. Turn off SuspendHoldsOpac, the column should be hidden.

Signed-off-by: David Nind <david@davidnind.com>
Comment 8 David Nind 2025-01-30 21:42:03 UTC
(In reply to Lucas Gass (lukeg) from comment #6)
> (In reply to David Nind from comment #4)
> > I'm getting the sha1/fake ancestor error.
> > 
> > git bz apply 39003
> 
> My fault, it should apply now.

Thanks Lucas!
Comment 9 Marcel de Rooy 2025-01-31 10:06:11 UTC
Created attachment 177377 [details] [review]
Bug 39003: Fix the tab template variable in opac-user.pl

To test:
1. SuspendHoldsOpac to 'allow'
2. Make some holds for a patron
3. Log in as that patron and go to 'Summary'
4. Click the 'Holds' tab
5. No suspend column
6. APPLY PATCH, restart_all
7. Try again, the column should show.
8. Turn off SuspendHoldsOpac, the column should be hidden.

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

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 10 Katrin Fischer 2025-01-31 14:58:20 UTC
Pushed for 25.05!

Well done everyone, thank you!
Comment 11 Paul Derscheid 2025-02-04 21:55:02 UTC
Nice work everyone!

Pushed to 24.11.x for 24.11.02
Comment 12 Alex Buckley 2025-02-20 20:52:42 UTC
not backporting to 24.05.x unless requested - dependency not in 24.05