[Koha-bugs] [Bug 9886] ParseLetter removes too often punctuation characters from fields

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9886

--- Comment #3 from M. de Rooy <m.de.rooy at rijksmuseum.nl> ---
Created attachment 16959
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=16959&action=edit
Bug 9886: ParseLetter removes too often punctuation characters from fields

This patch makes ParseLetter somewhat more restrictive in removing
punctuation characters from the end of a table field.
Based on the assumption that we want to remove punctuation from fields in
biblio and biblioitems (like ISBD).
ParseLetter should not remove e.g. a parenthesis in itemcallnumber, but still
removes e.g. a colon (:) at the end of a title.
Removed an unneeded global and lookahead from the regex.

Test plan:
1) Add a colon (:) to the end of a title.
2) Add a colon to the end of item copynumber.
3) Place a hold on that item. Check it in. Confirm hold.
4) Check the email or print notice generated. There should be no colon at the
end of the title, but the colon in the copynumber should still be there.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.