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

--- Comment #41 from Janusz Kaczmarek <[email protected]> ---
(In reply to David Cook from comment #40)
> The functionality added by this patch could be done more simply using
> $biblio->metadata and Digest::MD5 on its own. 

Note that we do not have a Koha::Biblio object for the modified version coming
from the editor/mergeform.  We just have the MARC::Record object.

> I also notice that
> "koha-tmpl/intranet-tmpl/prog/en/includes/cateditor-ui.inc" uses a new SVC
> API called "svc/mid_air_collision".
> 
> While I don't think we have an explicit coding guideline forbidding new SVC
> API endpoints, I think there is an unwritten rule about this. I think all
> new API endpoints must be REST API endpoints.

The functionality of svc/mid_air_collision does not fit to REST API.  I'm not
quite happy happy with my svc/mid_air_collision idea, but:

Feci, quod potui, faciant meliora potentes

-- 
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