https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40272

--- Comment #15 from Marcel de Rooy <[email protected]> ---
(In reply to Marcel de Rooy from comment #10)
> Hi Lucas,
> 
> Good catch. The new biblio case should surely have been included in the
> original test plan..
> 
> I did a bit more than adding the strictly needed "if $biblio"-check in the
> addbiblio script on the follow-up.
> I also added a check too in the extractor code on having a MARC::Record
> object. Actually this should imo be in Extractor::MARC->metadata. But that
> should enlarge the scope of this report too much for now. Will open a new
> report for that.
> And added both cases in unit tests: metadata is just a string, metadata is
> an empty MARC::Record object.

Sorry for the noise here.
But I finally decided not to include more changes but do a quick fix here.
And add more changes on the new report. Adjusting the check_fixed_length too.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to