Web Client: Opt-in Action Triggers, set as Registration Default will prevent Patron Registration screen from loading

[Michele Morgan]

Evergreen 2.12

When any opt-in action trigger is set to be a registration default, (i.e. the opt-in trigger is checked by default at registration time) the web client new registration page will not load.

To reproduce, make the stock opt-in action trigger "Email Checkout Receipt" opted in by default when a new patron is registered:

-Go to Administration -> Server Administration -> User Setting Types
-Double click to edit the setting 'circ.send_email_checkout_receipts'
-Enter 'True' (without the quotes) in the Registration Default field and Save

Login to the web client and go to Circulation -> Register Patron. The progress bar displays, but the page never loads.

In the xul client, setting Registration Default for the user setting type to True puts a check in the trigger's checkbox on the patron registration page, opting in the patron by default unless it's unchecked prior to saving.

[Cesar V (cesardv)]

A patch for this is here:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/cesardv/lp1695029_webstaff_patron_reg_screen

[Josh Stompro (u-launchpad-stompro-org)]

Signoff branch - user/stompro/lp1695029_webstaff_patron_reg_screen_signoff
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/stompro/lp1695029_webstaff_patron_reg_screen_signoff

I had no problem loading the patron registration screen after applying this patch and following the test plan. Tested on a 7/18/2017 version of master.

Josh

[Rogan Hamby (rogan-hamby)]

Tested and worked as expected, sign off here: user/rogan/lp1695029_patronreg_signoff

[Bill Erickson (berick)]

Follow-up branch pushed:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1695029-patron-reg-opt-in-defaults

1. I merged Rogan's and Josh's sign-offs to one commit. (Rogan's branch doesn't have them, presumably a thinko).

2. The feature described in this bug is not really a supported feature. It just happens to work.
 Putting /any/ value as the default ("true", "false", "What's the frequency Kenneth?") results in the checkbox being checked in the XUL editor. I get the gist, though, so I pushed a second commit to my branch that inspects the default value of boolean user settings for a true-looking thing and treats it accordingly. It also avoids attempts to Boolean-ize non-boolean (e.g. 'string') settings.

[Josh Stompro (u-launchpad-stompro-org)]

Bill, thanks for seeing and fixing that problem. I just tested your patch and can confirm that any value starting with 't' now causes the checkbox to be set, and any other value causes it to be unset. The form still loads like it should also.

Signoff Branch: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/stompro/lp1695029-patron-reg-opt-in-defaults-signoff2

Josh

[Jason Stephenson (jstephenson)]

I tested this branch on a rel_2_12_4 VM where the initial problem did not show, i.e. patron registration worked. After installing this branch, patron registration still worked.

On our training server with rel_2_12_3, otherwise practically the same code as the above, patron registration does not load and the progress bar spins. That behavior continues after loading this branch on training, so we have likely stumbled on another bug. I will do a little more investigation to see if I can determine the cause.

[Jason Stephenson (jstephenson)]

Turns out, I've stumbled across bug 1702756.

[Jason Etheridge (phasefx)]

Looks good to me, pushed to master. Thanks!

One note when testing, and probably worthy of a different bug, when I went to set the Registration Default, the dialog changed the Datatype to String instead of keeping it Boolean. It looks like the drop-down is actually sticky within any given page load of the User Settings Type page.

[Jason Etheridge (phasefx)]

also, I seem to have broken the milestone/series target thingy :)

[Jason Etheridge (phasefx)]

pushed to rel_2_12