https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41768
--- Comment #28 from David Cook <[email protected]> --- I find the code in Koha/REST/V1/Biblios.pm looks weird. Like it shouldn't default to koha.user in the private endpoint, but technically since patron_id is required for it, it will always reset it, so technically it should be fine... but it looks potentially confusing and it could impact long-term maintainability. Not a blocker I guess but I don't like it. Why add "use C4::Context" to Koha/REST/V1/Holds.pm and Koha/REST/V1/Items.pm? Otherwise... I think it's looking all right. I'll give it a test. -- 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/
