Currently the Holds to Pull report shows names as Surname Firstname It would look nicer and be less confusing if it said Surname, Firstname
That should read "in the overdues report" available from circulation.
Still true, would be an easy one :)
Created attachment 34021 [details] [review] Bug 10051: Separate first name and surname with comma in overdues report The name of the patron who has overdues is displayed as "surname first name" instead of "surname, first name". To test: - Go to circulation > overdues - Verify that first name and surname are not separated by comma - Apply patch - Verify the display is improved with a comma - Delete the first name from one of the patrons - Verify display is still ok
Patch tested with a sandbox, by David Roberts <david.roberts@ptfs-europe.com>
Created attachment 34042 [details] [review] Bug 10051: Separate first name and surname with comma in overdues report The name of the patron who has overdues is displayed as "surname first name" instead of "surname, first name". To test: - Go to circulation > overdues - Verify that first name and surname are not separated by comma - Apply patch - Verify the display is improved with a comma - Delete the first name from one of the patrons - Verify display is still ok Signed-off-by: David Roberts <david.roberts@ptfs-europe.com>
Created attachment 34043 [details] [review] [SIGNED OFF] Bug 10051: Separate first name and surname with comma in overdues report The name of the patron who has overdues is displayed as "surname first name" instead of "surname, first name". To test: - Go to circulation > overdues - Verify that first name and surname are not separated by comma - Apply patch - Verify the display is improved with a comma - Delete the first name from one of the patrons - Verify display is still ok Signed-off-by: David Roberts <david.roberts@ptfs-europe.com> Signed-off-by: Christopher Brannon <cbrannon@debian.localdomain> Current status: Signed Off
Created attachment 34179 [details] [review] Bug 10051: Separate first name and surname with comma in overdues report The name of the patron who has overdues is displayed as "surname first name" instead of "surname, first name". To test: - Go to circulation > overdues - Verify that first name and surname are not separated by comma - Apply patch - Verify the display is improved with a comma - Delete the first name from one of the patrons - Verify display is still ok Signed-off-by: David Roberts <david.roberts@ptfs-europe.com> Signed-off-by: Christopher Brannon <cbrannon@debian.localdomain> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Patch pushed to master. Thanks Katrina!
Pushed to 3.18.x will be in 3.18.2