https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40955
Katrin Fischer <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|unspecified |Main Assignee|[email protected] |[email protected] |ity.org | Status|Needs Signoff |Failed QA --- Comment #2 from Katrin Fischer <[email protected]> --- Hi Marion, some first notes: 1) QA tests pass 2) Code review a) Terminology "Item reserved by another patron" We need to use "hold" instead of "reserve" See: https://wiki.koha-community.org/wiki/Terminology#R Suggestion: "There is a hold for another patron on the item" Do we need the information, that the hold is for another patron or could it just be: "There is a hold on the item" ? b) Unit tests I wonder if there should be a unit test. We do have tests for the other message in this section in t/db_dependent/SIP/Transaction.t. -- 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/
