| Summary: | Display patron name in 'Holds to pull' report using standard template | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Fridolin Somers <fridolin.somers> |
| Component: | Templates | Assignee: | Fridolin Somers <fridolin.somers> |
| Status: | Pushed to stable --- | QA Contact: | Lucas Gass (lukeg) <lucas> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | e.betemps, jesse, lucas |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | Trivial patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: |
25.05.00,24.11.07
|
|
| Circulation function: | |||
| Attachments: |
Bug 39626: Use patron-title.inc in pendingreserves.tt
Bug 39626: Use patron-title.inc in pendingreserves.tt Bug 39626: Use patron-title.inc in pendingreserves.tt Bug 39626: Use patron-title.inc in pendingreserves.tt Bug 39626: Use patron-title.inc in pendingreserves.tt |
||
|
Description
Fridolin Somers
2025-04-14 12:36:45 UTC
Created attachment 180920 [details] [review] Bug 39626: Use patron-title.inc in pendingreserves.tt In "Holds to pull" page, patron column should use patron-title.inc like in other pages (like waiting holds). This allows to use surname first to allow better sorting/finding. Test plan : 1) Place an hold on some record 2) Go to Circulation > Holds to pull 3) Check you see in column "First patron" : surname then firstname Created attachment 180921 [details] [review] Bug 39626: Use patron-title.inc in pendingreserves.tt In "Holds to pull" page, patron column should use patron-title.inc like in other pages (like waiting holds). This allows to use surname first to allow better sorting/finding. Test plan : 1) Place an hold on some record 2) Go to Circulation > Holds to pull 3) Check you see in column "First patron" : surname then firstname Also allows to display middle name, other name ... Created attachment 180928 [details] [review] Bug 39626: Use patron-title.inc in pendingreserves.tt In "Holds to pull" page, patron column should use patron-title.inc like in other pages (like waiting holds). This allows to use surname first to allow better sorting/finding. Test plan : 1) Place an hold on some record 2) Go to Circulation > Holds to pull 3) Check you see in column "First patron" : surname then firstname Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Test/SO note: it also adds cardnumber at the end. Created attachment 180929 [details] [review] Bug 39626: Use patron-title.inc in pendingreserves.tt In "Holds to pull" page, patron column should use patron-title.inc like in other pages (like waiting holds). This allows to use surname first to allow better sorting/finding. Test plan : 1) Place an hold on some record 2) Go to Circulation > Holds to pull 3) Check you see in column "First patron" : surname then firstname Signed-off-by: Emmanuel Bétemps <e.betemps@gmail.com> Hi, middle and other names are effectively displayed but there's no space between them Acevedo, Henry middle(other) (23529000035676) Manu. (In reply to Emmanuel Bétemps from comment #6) > Hi, > > middle and other names are effectively displayed but there's no space > between them > > Acevedo, Henry > middle(other) > (23529000035676) > > > Manu. This was fixed by Bug 39308. Created attachment 181184 [details] [review] Bug 39626: Use patron-title.inc in pendingreserves.tt In "Holds to pull" page, patron column should use patron-title.inc like in other pages (like waiting holds). This allows to use surname first to allow better sorting/finding. Test plan : 1) Place an hold on some record 2) Go to Circulation > Holds to pull 3) Check you see in column "First patron" : surname then firstname Signed-off-by: Emmanuel Bétemps <e.betemps@gmail.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> I love these small fixes! Pushed for 25.05! Well done everyone, thank you! Pushed to 24.11 for 24.11.07 Merge conflicts with 24.05.x Please resolve if needed for this version. |