https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40955 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|unspecified |Main Assignee|koha-bugs@lists.koha-commun |marion.durand@biblibre.com |ity.org | Status|Needs Signoff |Failed QA --- Comment #2 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- 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.