Bug 30632

Summary: Fix report author display in list of saved reports
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: minor    
Priority: P5 - low CC: david, kyle, m.de.rooy, martin.renvoize
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This fixes the display of report authors in the list of saved reports to remove the extra space before the comma (Lastname, Firstname was displaying as Lastname , Firstname).
Version(s) released in:
22.05.00
Bug Depends on: 30491    
Bug Blocks: 30651    
Attachments: Bug 30632: Fix report author display in list of saved reports
Bug 30632: Fix report author display in list of saved reports
Bug 30632: Fix report author display in list of saved reports

Description Katrin Fischer 2022-04-27 16:28:21 UTC
The author of a report shows as:

Fischer , Katrin (51) 

The additional space before comma should be removed.

I assume it should be switched over to using one of the standardized ways to show patron names, see bug 30617
Comment 1 Owen Leonard 2022-04-28 15:22:12 UTC
Created attachment 134281 [details] [review]
Bug 30632: Fix report author display in list of saved reports

This patch removes whitespace in the template around the markup of
report author first name and surname. This prevents unwanted spaces in
the output.

To test, apply the patch and go to Reports -> Use saved.

The display of report author should be like this:

Leonard, Owen (123456)

Instead of Leonard , Owen (20375)
Comment 2 Katrin Fischer 2022-04-28 15:28:04 UTC
With all the patches for preferred name/middle name and such I was wondering if we could use patron-title.inc here. Is there a reason not to?
Comment 3 Owen Leonard 2022-04-28 15:36:03 UTC
I made this quick template fix because fixing it to use the patron-title include would require more complex updates to the way reports are queried. I certainly do not object if someone wants to take that route instead.
Comment 4 David Nind 2022-04-28 20:50:35 UTC
Created attachment 134303 [details] [review]
Bug 30632: Fix report author display in list of saved reports

This patch removes whitespace in the template around the markup of
report author first name and surname. This prevents unwanted spaces in
the output.

To test, apply the patch and go to Reports -> Use saved.

The display of report author should be like this:

Leonard, Owen (123456)

Instead of Leonard , Owen (20375)

Signed-off-by: David Nind <david@davidnind.com>
Comment 5 Marcel de Rooy 2022-04-29 09:51:15 UTC
Created attachment 134355 [details] [review]
Bug 30632: Fix report author display in list of saved reports

This patch removes whitespace in the template around the markup of
report author first name and surname. This prevents unwanted spaces in
the output.

To test, apply the patch and go to Reports -> Use saved.

The display of report author should be like this:

Leonard, Owen (123456)

Instead of Leonard , Owen (20375)

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 6 Katrin Fischer 2022-04-29 11:02:57 UTC
I am ok with the simple fix. I am filing a new one to "switch" to the include.
Comment 7 Fridolin Somers 2022-05-03 21:28:08 UTC
Pushed to master for 22.05, thanks to everybody involved 🦄