[Bug 9740] New: using XSLT modifies record authorized values
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 Bug ID: 9740 Summary: using XSLT modifies record authorized values Classification: Unclassified Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Templates Assignee: oleonard@myacpl.org Reporter: fridolyn.somers@biblibre.com When using XSLT display, in detail.pl, the record (MARC::Record) is get from database. This record is given to XSLTParse4Display() to transform into HTML using XSLT. This method uses transformMARCXML4XSLT() to replace in record authorized values code by there description. The bug is that this change is made on original record and remains in the rest of the script detail.pl. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Assignee|oleonard@myacpl.org |fridolyn.somers@biblibre.co | |m --- Comment #1 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 15803 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15803&action=edit Proposed patch See commit message -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- 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=9740 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |gmcharlt@gmail.com --- Comment #2 from Galen Charlton <gmcharlt@gmail.com> --- I agree with your analysis. However, I do have a question: have you seen an user-visible consequences? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |In Discussion -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |Needs Signoff --- Comment #3 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- (In reply to comment #2)
I agree with your analysis. However, I do have a question: have you seen an user-visible consequences?
A library has an authorized value defined on a 300$a (UNIMARC). That is how I saw the bug and composed the test plan. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |paul.poulain@biblibre.com --- Comment #4 from Paul Poulain <paul.poulain@biblibre.com> --- Patch tested with a sandbox, by Sonia Bouis <sonia.bouis@univ-lyon3.fr> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #15803|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 --- Comment #5 from Paul Poulain <paul.poulain@biblibre.com> --- Created attachment 16393 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=16393&action=edit Bug 9740: using XSLT modifies record authorized values When using XSLT display, in detail.pl, the record (MARC::Record) is get from database. This record is given to XSLTParse4Display() to transform into HTML using XSLT. This method uses transformMARCXML4XSLT() to replace in record authorized values code by there description. The bug is that this change is made on original record and remains in the rest of the script detail.pl. This patch adds a clonning operation into transformMARCXML4XSLT. Test plan : - Activate XSLT display in intranet details XSLTDetailsDisplay syspref - Create a new authorized value categorie : ie "AUTHVAL" - Add a new authorized value and description : ie "VAL" and "Description for VAL" - Edit default framework - Edit a note subfield : 3xx for UNIMARC, 5xx for others) : ie 326$a - Set authorized value "AUTHVAL" for this subfield - Edit a biblio with default framework - Choose authorized value for speficied subfield : ie "Description for VAL" for 326$a - Go to detail page and click on "Descriptions" tab => Whitout this patch you see authorized value description : ie "Description for VAL" => Whith this patch you see authorized value code : ie "VAL" "Descriptions" tab is set in detail.pl with C4::GetMarcNotes($record,$marcflavour) after the call to XSLTParse4Display($record), so test plan confirms that XSLTParse4Display() modifies or not $record. Signed-off-by: Sonia Bouis <sonia.bouis@univ-lyon3.fr> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |katrin.fischer@bsz-bw.de --- Comment #6 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Fridolyn, I am not sure I understand the problem here. I cataloged a record using authorised values for 500 (footnote) and 653 (subject). The code was "QA". In OPAC and staff all views show my definded description "Quality Assurance". When I export the record, it says "QA" in both fields. When I edit the record, both pull downs are preselected correctly. This is on master without your patch. Can you please explain? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 --- Comment #7 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Hie Katrin.
In OPAC and staff all views show my defined description "Quality Assurance". With your test, you should see that the code is displayed when you turn XSLT off.
Do you see the description in MARC detail page ? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 --- Comment #8 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Fridolyn, maybe I don't understand what the wanted/unwanted behaviour is? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 --- Comment #9 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- (In reply to comment #8)
Hi Fridolyn, maybe I don't understand what the wanted/unwanted behaviour is?
In my opinion, the unwanted behavior is that the "Descriptions" tab content depends on the use of XSLT : - without XSLT, the code is displayed - with XSLT, the description is displayed This is clearly a unwanted side-effect : changes on record appears in XSLT.pm but effect appears in details.pl. The wanted behavior is that the code is always displayed. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 --- Comment #10 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I think I disagree. We have authorized values with a special description for the OPAC - so that is what display. We also use authorised values for local subjectes (653) where the library does not want to create real authorities but have some control over the terms. So in that case we always want to display the long description and not the short code. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|master |3.14 Severity|normal |enhancement --- Comment #11 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- I think that the current behavior is a feature not a bug. There would have to be some sort of switch to make it possible to selectively disable the display of the label. If the issue is that you want the labels to match the codes of a particular set of authorized values, I'd suggest just setting the labels to match the codes (or, even better, train the librarians to use meaningful labels). Bumping back to 3.14 since we are in feature freeze. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 --- Comment #12 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- (In reply to comment #10)
I think I disagree. We have authorized values with a special description for the OPAC - so that is what display. We also use authorised values for local subjectes (653) where the library does not want to create real authorities but have some control over the terms. So in that case we always want to display the long description and not the short code.
The patch does not remove the conversion from code to description inside the block computed by XSLT, it removes this conversion for code outside of XSLT.pm -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 --- Comment #13 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Actually the side effect is only visible in fields shown in "Description" tab, that is what I describe in test plan. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 --- Comment #14 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- (In reply to comment #11)
I think that the current behavior is a feature not a bug. This can't be a feature, changes on var $record performed in XSLT.pm had effects in details.pl.
-- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 --- Comment #15 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- (In reply to comment #14)
(In reply to comment #11)
I think that the current behavior is a feature not a bug. This can't be a feature, changes on var $record performed in XSLT.pm had effects in details.pl.
No question that the code is poor, but surely it can be fixed without breaking functionality? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 --- Comment #16 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- (In reply to comment #15)
No question that the code is poor, but surely it can be fixed without breaking functionality? Just in case : I do not speak about the authorized value conversion in the block displayed by the XSLT.
I think no one add noticed that side effect and used it has a functionality. If you wan, I create a follow-up that replaces in "Description" tab the authorized values by there description. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |7706 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 --- Comment #17 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- I found a side effect of this bug : Bug 7706 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 mathieu saby <mathieu.saby@univ-rennes2.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mathieu.saby@univ-rennes2.f | |r -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9740 Fridolin SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |In Discussion -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org