Web Client: library setting Record In-House Use: # of uses threshold for Are You Sure? dialog. is not honoured

[[e-mail]]

Tested on 2.12.X master about 3 weeks ago.

[Galen Charlton (gmc)]

Library setting in question is ui.circ.in_house_use.entry_cap; Open-ILS/web/js/ui/default/staff/circ/in_house_use/app.js does reference it but may not be working.

[Bill Erickson (berick)]

Hm, yeah, the input's numeric plus/minus controls stop at max, but don't prevent you from typing a value higher than the max.

[Cesar V (cesardv)]

Bill, I tested it and it seems that while one can manually type in a "number of uses" value greater than the max set in the permission ("Record In-House Use: Maximum # of uses allowed per entry"), it doesn't actually let you continue with the submit. I get a html5 form validation error tooltip, saying my number was higher than the max. See screenshot.

Regarding the "Are you sure?" dialogue I am actually not even seeing it pop up at all... but that could be that one cannot have that permission and the "max # of uses allowed per entry" permission at the same time? I'm not sure... I'll test further though

[Cesar V (cesardv)]

I don't actually see any code that would pop up any sort of dialogue, to warn/confirm that # of uses for the in-house use entry is higher than what settings allow... I do see a "countWarn" variable that was declared to take in the setting's value but was never used. I'll add some code to actually

[Cesar V (cesardv)]

.. do this. (finishing my thought form yesterday (: )

I've a fix for this here: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/cesardv/lp1716962_inhouse_use_num_confirm_modal

Basically, ui.circ.in_house_use.entry_cap setting is taken care of by the form's validation, but ui.circ.in_house_use.entry_warn was being ignored. I added code to conditionally display a confirmation modal if this setting was triggered.

[Mike Rylander (mrylander)]

3.0.1 will be better for having this commit. Thanks, Cesar!