Bug 40568 - Various corrections to recalls templates
Summary: Various corrections to recalls templates
Status: Needs Signoff
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on: 40560 40566
Blocks: 40589
  Show dependency treegraph
 
Reported: 2025-07-31 16:46 UTC by Owen Leonard
Modified: 2025-11-08 03:41 UTC (History)
1 user (show)

See Also:
GIT URL:
Initiative type: ---
Sponsorship status: ---
Comma delimited list of Sponsors:
Crowdfunding goal: 0
Patch complexity: Small patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 40568: Various corrections to recalls templates (7.71 KB, patch)
2025-07-31 16:47 UTC, Owen Leonard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2025-07-31 16:46:09 UTC
I've got a few random corrections to make to a couple of recalls templates.
Comment 1 Owen Leonard 2025-07-31 16:47:26 UTC
Created attachment 184937 [details] [review]
Bug 40568: Various corrections to recalls templates

This patch makes various minor changes to recalls templates.

- Correct markup in dropdown menu
- Correct sorting of dates in tables
- Wrap recalls table in "page-section" div.
- Hide "Show old recalls" if there are no recalls.

To test, apply the patch and enable Recalls if necessary.

- If necessary, place some recalls.
- Go to Circulation -> Recalls queue and check the "Actions" menu for
  one of the recalls. The "Mark as overdue" menu item should be
  formatted correctly.
- Check the sorting of date columns on the "Recalls queue" page. It
  should be correct.
- View a patron who has never placed a recall. Go to their recalls
  history and confirm that you see a "Patron has no recalls" message but
  no "Show old recalls" checkbox.

Sponsored-by: Athens County Public Libraries
Comment 2 David Nind 2025-11-08 03:41:21 UTC
The patch still applies, however:

1. I have no idea on the SQL required to make a recall overdue so that the "Mark as overdue" menu item is shown.

2. For sorting of date columns on the "Recalls queue" page, it seems to work OK before the patch--so I'm not sure how to reproduce that it is not working. (Example SQL to update the created date: update recalls set created_date = "2025-11-07 00:00:00" where recall_id=1