https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39180
--- Comment #2 from Andrew Fuerste-Henry <[email protected]> --- (In reply to Nick Clemens (kidclamp) from comment #1) > Created attachment 181941 [details] [review] > Bug 39180: Catch exception when updating patron lastseen date > > This patch adds a catch during checkin and passes a message back to the > caller > > For checkouts and renewals we don't have a method for passing messages so we > simply catch the exception > and warn - this seems reasonable, I think we should add a warning on the > patron display that a required guarantor > is missing, but not break circulation. > > We will need to add unit tests The general approach here is good, but the popup at checkin should tell the user more specifically what has happened. Something like "This item has been checked-in, but the following warning still needs to be addressed: [Child patron needs a guarantor]." If we put some obvious alert in the patron record, we don't need to give a special message at checkout and renewal. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. _______________________________________________ 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/
