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

--- Comment #9 from David Cook <[email protected]> ---
(In reply to Jonathan Druart from comment #7)
> Created attachment 98721 [details] [review]
> Bug 24586: Add tests
> 
> Not sure about this approach. Maybe we are hiding a problem coming from
> somewhere else. Maybe better to handle the error from the callers.

In fact, I forgot to mention it, but my patch will produce duplicate
authorities. 

In the short term, I was ok with making authority duplicates rather than
failing a bib import, but that would be a regression in Koha I think.

Better to fail rather than return nothing. I just haven't put the time into
creating a complete solution as I have competing priorities :/.

-- 
You are receiving this mail because:
You are the assignee for the bug.
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