[Koha-bugs] [Bug 33036] REST API: Merge biblio records

[bugzilla-daemon at bugs.koha-community.org]

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

--- Comment #17 from Jonathan Druart <jonathan.druart+koha at gmail.com> ---
(In reply to Zeno Tajoli from comment #16)
> Hi Jonathan, I'm trying to develop this patch so I ask same explanation
> about your requests:
> 1)'first move the code to a module'.
> In this patch the new verb 'merge' is inside an existing module,
> 'Koha/REST/V1/Biblios.pm' and it is also a Mojo controller.
> So the code is in the correct place.

To make it reusable from another place than the REST API.
It needs to be placed into Koha::Biblio (see Koha::Patron::merge_with)

> 3)'then call that method from merge.pl and your Mojo controller'
> Yes, I need to change the merge.pl and use the REST API instead of the
> present code.

No, you need to call Koha::Biblio::merge_with from merge.pl and the REST API
controller.

-- 
You are receiving this mail because:
You are watching all bug changes.