When a patron has charges on their account in excess of the amount set in MaxOutstanding, they see a message on their account that says "Please note: Your account has outstanding fees & charges of X.XX. Holds are blocked because your fine balance is over the limit." If the syspref OPACHoldRequests (formerly known as RequestOnOPAC) is disabled, we should not show this alert to the patron even if they have fines over the MaxOutstanding amount.
Created attachment 133308 [details] [review] Bug 30508: Do not display OPAC message about block from holds when OPACHoldRequests is disabled This patch adds a check for the OPACHoldRequests to determine whether a message should be shown to patrons whose account is over the maxoutstanding limit. To test, apply the patch and set the "maxoutstanding" system preference to a low value. Set OPACHoldRequests to "Allow." - Log in to the OPAC as a patron who has outstanding fines or charges which exceed the maxoutstanding value you set. - You should see a message on the "Your summary" page, "Holds are blocked because your fine balance is over the limit." - Set OPACHoldRequests to "Don't allow" and refresh the "Your summary" page in the OPAC. The message should disappear.
Shouldn't we just hide the "Holds are blocked because your fine balance is over the limit" part of the message? Seems like it would still be useful to show the "Please note: Your account has outstanding fees & charges of X.XX" part of the message, regardless of OPACHoldRequests.
Created attachment 133721 [details] [review] Bug 30508: Do not display OPAC message about block from holds when OPACHoldRequests is disabled To test, apply the patch and set the "maxoutstanding" system preference to a low value. Set OPACHoldRequests to "Allow." - Log in to the OPAC as a patron who has outstanding fines or charges which exceed the maxoutstanding value you set. - You should see a message on the "Your summary" page, "Holds are blocked because your fine balance is over the limit." - Set OPACHoldRequests to "Don't allow" and refresh the "Your summary" page in the OPAC. The message should disappear. Works OK!
Created attachment 133722 [details] [review] Bug 30508: Do not display OPAC message about block from holds when OPACHoldRequests is disabled This patch adds a check for the OPACHoldRequests to determine whether a message should be shown to patrons whose account is over the maxoutstanding limit. To test, apply the patch and set the "maxoutstanding" system preference to a low value. Set OPACHoldRequests to "Allow." - Log in to the OPAC as a patron who has outstanding fines or charges which exceed the maxoutstanding value you set. - You should see a message on the "Your summary" page, "Holds are blocked because your fine balance is over the limit." - Set OPACHoldRequests to "Don't allow" and refresh the "Your summary" page in the OPAC. The message should disappear. Signed-off-by: Amit Gupta <amit.gupta@informaticsglobal.com>
Thanks for testing Amit :) . For some reason when attaching the patch it hasn't been marked as a patch. And don't forget to add your signoff line to the patch with: git format-patch -s origin/master Here has been attached a version of the patch with your signoff added (assuming you want to use the same email address as in Bugzilla) Wait, it's not the same email address as the one I tried to attach! So there has been something else weird with the patch attachment. Comment 3 doesn't show the signoff line but it was there and resurfaced when I attached the patch. (Also for some reason, my attempt to add a SO line with a custom name and email didn't work) Anyway, what is your procedure to attach a patch here?
Hi Victor, not sure about the confusion here, but Amit's sign-off line seems ok to me in his attached patch: Signed-off-by: Amit Gupta <amit.gupta@informaticsglobal.com> Maybe it was uploaded manually, that could explain the missing 'patch' flag.
Created attachment 133740 [details] [review] Bug 30508: Do not display OPAC message about block from holds when OPACHoldRequests is disabled This patch adds a check for the OPACHoldRequests to determine whether a message should be shown to patrons whose account is over the maxoutstanding limit. To test, apply the patch and set the "maxoutstanding" system preference to a low value. Set OPACHoldRequests to "Allow." - Log in to the OPAC as a patron who has outstanding fines or charges which exceed the maxoutstanding value you set. - You should see a message on the "Your summary" page, "Holds are blocked because your fine balance is over the limit." - Set OPACHoldRequests to "Don't allow" and refresh the "Your summary" page in the OPAC. The message should disappear. Signed-off-by: Amit Gupta <amit.gupta@informaticsglobal.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
(In reply to Lucas Gass from comment #2) > Shouldn't we just hide the "Holds are blocked because your fine balance is > over the limit" part of the message? > > Seems like it would still be useful to show the "Please note: Your account > has outstanding fees & charges of X.XX" part of the message, regardless of > OPACHoldRequests. Good point
I prefer to leave it to next cycle
Pushed to master for 22.11. Nice work everyone, thanks!