Summary: | Fix item search CSV export - obsolete "by" and display of publication year for MARC21 | ||
---|---|---|---|
Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
Component: | Templates | Assignee: | Katrin Fischer <katrin.fischer> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | bgkriegel, chris, hector.hecaxmmx, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13859 | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 14303: Fix item search CSV export - obsolete "by" and display of publication year for MARC21
Bug 14303: Fix item search CSV export - obsolete "by" and display of publication year for MARC21 Bug 14303: Fix item search CSV export - obsolete "by" and display of publication year for MARC21 |
Description
Katrin Fischer
2015-05-31 20:30:05 UTC
Created attachment 40006 [details] [review] Bug 14303: Fix item search CSV export - obsolete "by" and display of publication year for MARC21 The exported CSV file from the item search didn't display the publication year in MARC21 installations and the title always contained 'by' even if there was no author information to display. Also the by is not needed by MARC21 as the data should include punctuation. This basically copies the changes done to the JSON format on bug 13859 to the CSV include. To test: Switch from 'Screen' to 'CSV' - Check publication date always displays (MARC21 and UNIMARC) - Check that for MARC21 the 'by' has been removed from the title information - Check that for UNIMARC the 'by' only displays when there is also an author to display Created attachment 40029 [details] [review] Bug 14303: Fix item search CSV export - obsolete "by" and display of publication year for MARC21 The exported CSV file from the item search didn't display the publication year in MARC21 installations and the title always contained 'by' even if there was no author information to display. Also the by is not needed by MARC21 as the data should include punctuation. This basically copies the changes done to the JSON format on bug 13859 to the CSV include. To test: Switch from 'Screen' to 'CSV' - Check publication date always displays (MARC21 and UNIMARC) - Check that for MARC21 the 'by' has been removed from the title information - Check that for UNIMARC the 'by' only displays when there is also an author to display Tested for MARC21, works as expected. Signed-off-by: Marc Véron <veron@veron.ch> (In reply to Marc Véron from comment #2) ... > Signed-off-by: Marc Véron <veron@veron.ch> Marc, you forgot to set to 'Signed Off' (In reply to Bernardo Gonzalez Kriegel from comment #3) > (In reply to Marc Véron from comment #2) > ... > > Signed-off-by: Marc Véron <veron@veron.ch> > > Marc, you forgot to set to 'Signed Off' It's right, Marc, you forget to change the status of the bug to Signed-off, the bug still appear as Needs Signoff Regards Created attachment 40876 [details] [review] Bug 14303: Fix item search CSV export - obsolete "by" and display of publication year for MARC21 The exported CSV file from the item search didn't display the publication year in MARC21 installations and the title always contained 'by' even if there was no author information to display. Also the by is not needed by MARC21 as the data should include punctuation. This basically copies the changes done to the JSON format on bug 13859 to the CSV include. To test: Switch from 'Screen' to 'CSV' - Check publication date always displays (MARC21 and UNIMARC) - Check that for MARC21 the 'by' has been removed from the title information - Check that for UNIMARC the 'by' only displays when there is also an author to display Tested for MARC21, works as expected. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Patch pushed to master. Thanks Katrina! Pushed to 3.20.x will be in 3.20.2 |