|
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. Not backporting to 22.11.x as it wasn't backported to 24.05.x |