---- Reported by toni.rosa@gmail.com 2010-04-07 16:39:50 ---- A change in this line (1472) would suffice... Current Line: push @subfields_loop, {value => $value, link_loop => \@this_link_loop, separator => $separator} unless ($authors_subfield->[0] eq '9' ); New proposed line: push @subfields_loop, {tag=> $field->tag(),code => $subfieldcode, value => $value, link_loop => \@this_link_loop, separator => $separator} unless ($authors_subfield->[0] eq '9' ); This change would allow doing things like this in the templates: <TMPL_IF EXPR="tag == 700" && code eq 'a' > <strong>Author:</strong> <!-- TMPL_ELSE --> <TMPL_IF EXPR="tag == 710" && code eq 'a' > <strong>Corpotation Author:</strong> <!-- /TMPL_IF --> <!-- /TMPL_IF --> ---- Additional Comments From paul.poulain@biblibre.com 2010-04-07 16:54:30 ---- pls note that for UNIMARC ppl, such a behaviour is correct too. 70x is "physical author" and 71x is "corporation author" ( note that I don't know what is a "corpotation author" :D ) --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:26 UTC --- This bug was previously known as _bug_ 4376 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4376 Actual time not defined. Setting to 0.0 The original reporter of this bug does not have an account here. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. Previous reporter was toni.rosa@gmail.com. Setting qa contact to the default for this product. This bug either had no qa contact or an invalid one.
Created attachment 4808 [details] [review] proposed patch the subfieldcode suggested already in master. Just adding the tag
The change seems like a good idea, but are there instances where we need to make use of this? XSLT gives us an alternate tool for it in most cases. Paul's comment suggests that to make this work we have to create new sections in the template for each MARC flavor.
Owen: I agree that with XSLT this feature will be less and less used, and will probably be deprecated soon. In the meantime, a library that want to have local templates can have a good use of this feature. So I propose to add it for now, and when XSLT is used everywhere we will completly remove the GetMarcAuthor anyway (and GetMarcSubject, and many other things in Biblio.pm)
Updating Version : This ENH will be for Koha 3.8
Bug versionned for master. entries will be made against rel_3_8 once the patch has been applied (see thread about that on koha-devel yesterday)
Created attachment 7097 [details] [review] BZ4376 Minor change in GetMarcAuthor A minor change in the “GetMarcAuthors” function of C4/Biblio.pm allow differentiate the type of authors in the templates This change allow doing things like this in the templates: <TMPL_IF EXPR="tag == 700" && code eq 'a' > <strong>Author:</strong> <!-- TMPL_ELSE --> <TMPL_IF EXPR="tag == 710" && code eq 'a' > <strong>Corpotation Author:</strong> <!-- /TMPL_IF --> <!-- /TMPL_IF --> (html template syntax, but also applicable to template toolkit) Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com> Works as claimed and doesn't break existing functionality.
QA comment: trivial change
Created attachment 7214 [details] [review] BZ4376 Minor change in GetMarcAuthor A minor change in the GetMarcAuthors function of C4/Biblio.pm allow differentiate the type of authors in the templates This change allow doing things like this in the templates: <TMPL_IF EXPR="tag == 700" && code eq 'a' > <strong>Author:</strong> <!-- TMPL_ELSE --> <TMPL_IF EXPR="tag == 710" && code eq 'a' > <strong>Corpotation Author:</strong> <!-- /TMPL_IF --> <!-- /TMPL_IF --> (html template syntax, but also applicable to template toolkit) Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com> Works as claimed and doesn't break existing functionality.
Comment on attachment 7097 [details] [review] BZ4376 Minor change in GetMarcAuthor obsoleted by my patch, there is an encoding problem with this one
There have been no further reports of problems so I am marking this bug resolved.