[Bug 11697] New: Thesaurus plugin sets indicators with a space
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11697 Bug ID: 11697 Summary: Thesaurus plugin sets indicators with a space Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Cataloging Assignee: gmcharlt@gmail.com Reporter: fridolyn.somers@biblibre.com QA Contact: testopia@bugs.koha-community.org CC: m.de.rooy@rijksmuseum.nl When using thesaurus plugin to link a biblio field with an authority, the indicators of the field of authority are set into indicators of biblio field. The bug is that when an indicator is not defined, it contains a space ' '. This space is then set into biblio field. The user must then erase this space before setting the indicator. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11697 Fridolin SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11697 --- Comment #1 from Fridolin SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 25071 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25071&action=edit Bug 11697 - Thesaurus plugin sets indicators with a space When using thesaurus plugin to link a biblio field with an authority, the indicators of the field of authority are set into indicators of biblio field. The bug is that when an indicator is not defined, it contains a space ' '. This space is then set into biblio field. The user must then erase this space before setting the indicator. This patch sets indicator as empty string when containing a space. Test plan : - Create a new biblio record - Select a field managed with a thesaurus. For example 700 - Click on thesaurus plugin icon - Search and choose an authority record with no indicator set on heading field => Without patch, the indicators of the biblio field (ie 700) are filled with a space character => With patch, the indicators of the biblio field (ie 700) are empty - Click on plugin icon - Search and choose an authority record with indicators set on heading field => The indicators are set in biblio field -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11697 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|gmcharlt@gmail.com |fridolyn.somers@biblibre.co | |m -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11697 Pascale Nalon <pascale.nalon@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |pascale.nalon@gmail.com --- Comment #2 from Pascale Nalon <pascale.nalon@gmail.com> --- It's applied well ! Both for 7xx and for 6xx fields, when the indicators are empty. But it's seems to fail when indicators is not empty. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11697 --- Comment #3 from Pascale Nalon <pascale.nalon@gmail.com> --- When indicators are like #1 or #0, they are not set in the biblio field -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11697 Pascale Nalon <pascale.nalon@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11697 Fridolin SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11697 Fridolin SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25071|0 |1 is obsolete| | --- Comment #4 from Fridolin SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 26441 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=26441&action=edit Indeed, there whas an error in my patch, please retest. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11697 M. Tompsett <mtompset@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |RESOLVED CC| |mtompset@hotmail.com Resolution|--- |DUPLICATE --- Comment #5 from M. Tompsett <mtompset@hotmail.com> --- I think the solution for bug 11955 is nicer. But I still haven't figured out how to test. The indicator fields always go blank after clicking tag editor and choosing an authority record, even if I've modified the authority record to have values in the indicators. *** This bug has been marked as a duplicate of bug 11955 *** -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org