[Bug 7727] New: NORMARC XSLT OPAC detail view shows double tabs
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7727 Priority: P5 - low Change sponsored?: --- Bug ID: 7727 Assignee: magnus@enger.priv.no Summary: NORMARC XSLT OPAC detail view shows double tabs QA Contact: koha.sekjal@gmail.com Severity: normal Classification: Unclassified OS: All Reporter: magnus@enger.priv.no Hardware: All Status: NEW Version: master Component: OPAC Product: Koha Created attachment 8232 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8232&action=edit Screenshot showing double tabs When marcflavour = NORMARC OPACXSLTDetailsDisplay = use XSLT the detail view has a double set of tabs for different views of the MARC record - see attached screenshot. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7727 --- Comment #1 from Magnus Enger <magnus@enger.priv.no> --- Created attachment 8604 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8604&action=edit Bug 7727 - NORMARC XSLT OPAC detail view shows double tabs With marcflavour = NORMARC and OPACXSLTDetailsDisplay = Using XSLT, the detail view displays a double set of tabs for Normal/MARC/ISBD views, one above the title and one below. To test: marcflavour = NORMARC OPACXSLTDetailsDisplay = Using XSLT Since the contents of the recaords do not matter here, the patch can be tested on MARC records of any flavour. - Check that double tabs are shown - Apply the patch - Check that only the set of tabs at the top is shown -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7727 Magnus Enger <magnus@enger.priv.no> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7727 Marijana Glavica <mglavica@ffzg.hr> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #8604|0 |1 is obsolete| | --- Comment #2 from Marijana Glavica <mglavica@ffzg.hr> --- Created attachment 8620 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8620&action=edit [SIGNED-OFF] Bug 7727 - NORMARC XSLT OPAC detail view shows double tabs With marcflavour = NORMARC and OPACXSLTDetailsDisplay = Using XSLT, the detail view displays a double set of tabs for Normal/MARC/ISBD views, one above the title and one below. To test: marcflavour = NORMARC OPACXSLTDetailsDisplay = Using XSLT Since the contents of the recaords do not matter here, the patch can be tested on MARC records of any flavour. - Check that double tabs are shown - Apply the patch - Check that only the set of tabs at the top is shown Signed-off-by: Marijana Glavica <mglavica@ffzg.hr> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7727 Marijana Glavica <mglavica@ffzg.hr> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |mglavica@ffzg.hr --- Comment #3 from Marijana Glavica <mglavica@ffzg.hr> --- Notice about multilanguage issue: non-english NORMARCslim2OPACDetail.xsl is located in english directory! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7727 --- Comment #4 from Magnus Enger <magnus@enger.priv.no> --- (In reply to comment #3)
Notice about multilanguage issue: non-english NORMARCslim2OPACDetail.xsl is located in english directory!
Thanks for signing off! Actually there is just the English directory to choose from: http://git.koha-community.org/gitweb/?p=koha.git;a=tree;f=koha-tmpl/opac-tmp... The other directories get created, and the XSLT-files get copied over, when you install more languages. And translating the NORMARC stuff into English and then back again to Norwegian makes no sense, I'm afraid... -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7727 --- Comment #5 from Marijana Glavica <mglavica@ffzg.hr> --- (In reply to comment #4)
Actually there is just the English directory to choose from: http://git.koha-community.org/gitweb/?p=koha.git;a=tree;f=koha-tmpl/opac- tmpl/prog
The other directories get created, and the XSLT-files get copied over, when you install more languages.
Yes, and that is why all templates should originally be written in English :)
And translating the NORMARC stuff into English and then back again to Norwegian makes no sense, I'm afraid...
unless you want to use NOMARC in multi-lingual environment :) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7727 --- Comment #6 from Magnus Enger <magnus@enger.priv.no> --- (In reply to comment #5)
unless you want to use NOMARC in multi-lingual environment :)
Hopefully, NORMARC will be dead long before that happens... ;-) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7727 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |m.de.rooy@rijksmuseum.nl QA Contact|koha.sekjal@gmail.com |m.de.rooy@rijksmuseum.nl --- Comment #7 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- QA Comment: Approving this change. Probably you would like to compare this xslt with the MARC21 counterpart in order to check if you missed more changes. For the record just mentioning that leaving Norwegian words in this English file is IMO not preferred. (Understand the pragmatic approach, but at the end we could have all languages in the English startup.) This patch does not add such words, but removes them. Great ;) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7727 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |paul.poulain@biblibre.com --- Comment #8 from Paul Poulain <paul.poulain@biblibre.com> --- pushed, not tested myself, but looking at the patch, should not be wrong. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7727 Magnus Enger <magnus@enger.priv.no> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |RESOLVED Resolution|--- |FIXED --- Comment #9 from Magnus Enger <magnus@enger.priv.no> --- Fixed in current master. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7727 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|master |rel_3_6 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7727 --- Comment #10 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- This bug will be included in the Koha 3.6.5 release. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org