http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9282 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #10 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Looks quite good to me, but I am wondering about this now: } elsif ($tag eq '180') { $subfields_to_report = 'vxyz'; [etc. etc. etc.] if ($subfields_to_report) { push @authorized, { heading => $field->as_string($subfields_to_report), hemain => $field->subfield( substr($subfields_to_report, 0, 1) ), If you use the first char here, you will take $v into hemain. But I think that you should take $x for the 18X fields. I would like to see a response from Jared here, as I assume that he was responsible for quite some changes in this module. Could you please attract his attention? Thanks. Changing status to reflect need for clarification. -- You are receiving this mail because: You are watching all bug changes.