]> git.evergreen-ils.org Git - working/Evergreen.git/commit
LP#1821804 - Cloned patron address set to pending user/stompro/lp1821804_cloned_address_pending
authorJosh Stompro <stompro@stompro.org>
Wed, 23 Jun 2021 19:21:46 +0000 (14:21 -0500)
committerJosh Stompro <stompro@stompro.org>
Wed, 23 Jun 2021 19:21:46 +0000 (14:21 -0500)
commitebd5500b51f2a6f17ecb3595f00830ec17fffefc
treec82bcf2009fb7c68a7518b4a034b0ae78c883a9e
parent6022ddc5895d19dbd35923679a419233cd73551d
LP#1821804 - Cloned patron address set to pending

When cloning a patron record, the address gets set to pending and cannot
be fixed until the new patron record is saved.

The address creation steps for cloned records are a little different if
the library setting "Patron Registration: Cloned patrons get address copy" is
set, so test with that enabled and disabled.  In our system we have that setting
enabled.

The issue is that the values come in as a string like 't' or 'f' and need to be converted
to a boolean value.  That is done in the normal address loading code, but wasn't in
the cloning code.

I also added in a fix for the city limits flag not getting cloned which seems
like an related issue.  This seems like it was just cosmetic, the city limits
does get set when the record is saved, based on the parent records address.

Testing Plan:

Before change:

1. Enable "Cloned patrons get address copy" setting.
2. Open up a patron record that has a valid non pending address with city limits checked.
3. Save and Clone
4. Notice that the address in the new cloned record says pending.
5. Notice that the city limits checkbox isn't checked.

After change applied:

1. Enable "Cloned patrons get address copy" setting.
2. Open up a patron record that has a valid non pending address with city limits checked.
3. Save and Clone
4. Notice that the address in the new cloned record doesn't say pending.
5. Notice that the city limits checkbox is checked.

Signed-off-by: Josh Stompro <stompro@stompro.org>
Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js