Created attachment 9851 [details] [review]
Bug 8167 - Add syspref allow a choice of blocking, non-blocking, or do nothing when issuing lost items.
Signed-off-by: Liz Rea <email@example.com>
Works as advertised.
Code looks good to me.
I've pushed the patch because it works as advertised BUT I feel that:
+ nothing : "do nothing"
can be confusing. Does it mean "do the check-in without saying anything" or "refuse to do the check-in".
(Reading the code it's "do the check-in saying nothing")
There have been no further reports of problems so I am marking this bug resolved.