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

--- Comment #40 from David Cook <[email protected]> ---
(In reply to David Cook from comment #29)
> Curious to get other QAers/RM input about Koha::Util::Misc.

I'm still curious to hear what others think.

For me, I still think creating a Koha::Util::Misc class and using Sereal are
suboptimal choices. 

The functionality added by this patch could be done more simply using
$biblio->metadata and Digest::MD5 on its own. 

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.

I'll discuss all the above with the QA team and see what they say.

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