The overdue report (circ/overdue.pl) fails to display the patron name if the surname and/or firstname are null. This is because the concat() function in MySQL (properly) returns NULL if any of its arguments are NULL. Instead of using concat(), the individual name components should be returned, allowing the template to format the patron name however desired.
Created attachment 6260 [details] proposed bugfix
Created attachment 6261 [details] proposed bugfix
Hmm something odd with the patch attached here
Created attachment 6296 [details] [review] proposed fix
The content of attachment 6261 [details] has been deleted by Galen Charlton <gmcharlt@gmail.com> who provided the following reason: wrong attachment The token used to delete this attachment was generated at 2011-11-11 20:16:03 UTC.
Corrected version attached.
Created attachment 6300 [details] [review] bug 7198: show patron name in overdue list even if name column is null If the surname or firstname of a patron is NULL, the MySQL concat() function (properly) returns NULL, leading to the patron name not displaying in the overdue report. Signed-off-by: Galen Charlton <gmc@esilibrary.com> Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
QA: Looks good. Updating patch status.
Patch pushed, please test
Included in 3.6 prior to 3.6.5.