Bug 9158 - Correcting whitespace around display of series 440 marc fields
Summary: Correcting whitespace around display of series 440 marc fields
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor (vote)
Assignee: Chris Hall
QA Contact: Paul Poulain
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-27 23:48 UTC by Chris Hall
Modified: 2013-05-23 06:23 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
correcting whitespace in display of series (440) marc field (2.42 KB, patch)
2012-11-27 23:50 UTC, Chris Hall
Details | Diff | Splinter Review
bug 9158 correcting whitespace in series (440) display (2.54 KB, patch)
2012-12-07 13:41 UTC, Kyle M Hall
Details | Diff | Splinter Review
bug 9158 correcting whitespace in series (440) display (2.54 KB, patch)
2012-12-07 13:42 UTC, Kyle M Hall
Details | Diff | Splinter Review
bug 9158 correcting whitespace in series (440) display (2.64 KB, patch)
2012-12-21 13:32 UTC, Paul Poulain
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Hall 2012-11-27 23:48:26 UTC
The display of the  series marc field (440) on the opac and intranet details pages are
"first ; second ; third ."

but should be
"first ; second ; third. "

Notice the space before the trailing fullstop.

I also added a trailing space after the final fullstop as the concise aarc2 guide mentions using ". ", but I may have misunderstood.

I attached to OPAC as this affects only the display of the subfield, although both on OPAC an intranet.

I will attach the patch shortly (have to get bug id first..)
Comment 1 Chris Hall 2012-11-27 23:50:58 UTC Comment hidden (obsolete)
Comment 2 Kyle M Hall 2012-12-07 13:41:38 UTC Comment hidden (obsolete)
Comment 3 Kyle M Hall 2012-12-07 13:42:21 UTC Comment hidden (obsolete)
Comment 4 Paul Poulain 2012-12-21 13:32:02 UTC
Created attachment 14224 [details] [review]
bug 9158 correcting whitespace in series (440) display

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Patch changes 'first; second; third.' to 'first ; second ; third.'

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>

http://bugs.koha-community.org/show_bug.cgi?id=9157
Comment 5 Paul Poulain 2012-12-21 13:32:42 UTC
QA comment : trivial patch, only add a few whitespaces in XSLT

passed QA
Comment 6 Jared Camins-Esakov 2012-12-22 21:38:28 UTC
This patch has been pushed to master.
Comment 7 Chris Cormack 2012-12-26 19:52:20 UTC
Pushed to 3.10.x will be in 3.10.2
Comment 8 Chris Cormack 2012-12-26 19:59:29 UTC
Pushed to 3.8.x will be in 3.8.9