Summary: | Complete cataloguing plugin unimarc_field_4XX | ||
---|---|---|---|
Product: | Koha | Reporter: | Julian Maurice <julian.maurice> |
Component: | Cataloging | Assignee: | Julian Maurice <julian.maurice> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | chris, jonathan.druart, liz, m.de.rooy, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 14766: Complete cataloguing plugin unimarc_field_4XX
Bug 14766: Complete cataloguing plugin unimarc_field_4XX Bug 14766: Complete cataloguing plugin unimarc_field_4XX Bug 14766: unimarc_field_4XX: escape ', ", \n, \r Bug 14766: Complete cataloguing plugin unimarc_field_4XX Bug 14766: unimarc_field_4XX: escape ', ", \n, \r |
Description
Julian Maurice
2015-09-01 12:54:39 UTC
Created attachment 42148 [details] [review] Bug 14766: Complete cataloguing plugin unimarc_field_4XX Add subfields l, n and o for better UNIMARC compliance Created attachment 42149 [details] [review] Bug 14766: Complete cataloguing plugin unimarc_field_4XX Add subfields l, n and o for better UNIMARC compliance Created attachment 42263 [details] [review] Bug 14766: Complete cataloguing plugin unimarc_field_4XX Add subfields l, n and o for better UNIMARC compliance Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Note: I just did a code audit here, as I don't know enough about UNIMARC to know if the 4XX fields should have these subfields. All other fields replace some chars (', ", \n, \r), I am sure you should do it for these 3 ones too. Created attachment 42374 [details] [review] Bug 14766: unimarc_field_4XX: escape ', ", \n, \r Created attachment 42376 [details] [review] Bug 14766: Complete cataloguing plugin unimarc_field_4XX Add subfields l, n and o for better UNIMARC compliance Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Note: I just did a code audit here, as I don't know enough about UNIMARC to know if the 4XX fields should have these subfields. Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Created attachment 42377 [details] [review] Bug 14766: unimarc_field_4XX: escape ', ", \n, \r Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Patches pushed to master. Thanks Paul! Pushed to 3.20.x will be in 3.20.4 Pushed to 3.18.x will be in 3.18.11 |