When you try to check out an item that is on hold for someone else the wording of the message is confusing: "Item is on hold for 2010-05-13 : Jordan Alford (23529000050113)" It should instead read: "Item is on hold for Jordan Alford (23529000050113) since 2010-05-13" or whatever the date is that's listed - but right now it looks like it's on hold for the date ...
Created attachment 2325 [details] Confirmation when item is checked out to another patron Now with additional information.
Created attachment 2326 [details] Confirmation when item is on hold for another patron Now with additional information.
Created attachment 2327 [details] Confirmation when item is already checked out to the patron Now with additional information
Created attachment 2328 [details] Confirmation when item is waiting for another patron Now with additional information
Created attachment 2329 [details] Confirmation when patron has too many checked out Improved phrasing
Created attachment 2330 [details] [review] Proposed fix This patch improves the phrasing of several messages by breaking the message variable into distinct parts for more natural-sounding warnings when: - Checking out an item on hold for another patron - Checking out an item which is waiting for another patron - Checking out an item which is checked out to another patron - Checking out an item which is checked out to this patron - Checking out to a patron who has too many checked out I would appreciate special attention to my changes to the TooMany function in Circulation.pm to make sure I handled it correctly.
Patch pushed, please test