Blocked accounts can check out precats

[Terran McCanna]

Starting new ticket based on comments from bug 954263 - first noticed in 3.6 and confirmed again in 3.13:

Pre-cat checkouts ignore blocks on patron accounts that a normal circulation would respect (when a patron has too many fines, for example)

[Dan Briem (dbriem)]

After submitting a precat dialog, there was no indication if there were blocking events and they were automatically overridden.

However, permissions seemed to be respected. For example, after submitting the precat dialog for a patron that exceeded the fine limit as a staff member without the PATRON_EXCEEDS_FINES.override permission, I got the perm block dialog.

Here's a branch for testing: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbriem/lp2098898_precats_auto_override_failure_events.

This removes the automatic override designation for precats. This way, if there are blocking events after the precat dialog is submitted, they are returned so the user is notified and can choose to override them or not.

If I misunderstood the bug and there are permission issues I didn't notice, I will remove the pullrequest tag.

[Susan Morrison (smorrison425)]

Testing for Feedback Fest, confirmed that various blocks on a patron's account now prevent a precat checkout and require an override. Tested when account exceeds fines, exceeds checkouts, exceeds overdues, has a STAFF_CHR block, and is expired.

I have tested this code and consent to signing off on it with my name, Susan Morrison, and my email address, <email address hidden>.

[Terran McCanna (tmccanna)]

Thank you Dan and Susan!

Fix merged back to rel_3_13.