LP#1812900: fix retention of saved defaults in holdings editor
authorGalen Charlton <gmc@equinoxinitiative.org>
Fri, 19 Apr 2019 20:29:21 +0000 (16:29 -0400)
committerJane Sandberg <sandbej@linnbenton.edu>
Wed, 19 Jun 2019 21:59:14 +0000 (14:59 -0700)
commit90d48f2d5fd193f6d0fc733eda9e9ae3014535ac
tree7e8b73cef385bc0d26039c2dfeaa18ed4218989f
parentdbdf120763c97fce16751bae9a1dbd9e0673fc8d
LP#1812900: fix retention of saved defaults in holdings editor

Because the $watch on the statistical category default OU filter
in the holdings editor can fire before the form is fully initialized,
saved defaults (i.e., the cat.copy.defaults workstation setting)
could get overwritten with the hard-coded default value for that, er,
set of defaults.

This patch ensures that that $watch saves the defaults only when the
value of the default stat cat OU filter has changed.

To test
-------
[1] Use the holdings editor on an item. Change to the defaults
    tab and check or uncheck checkboxes such as "Use checkdigit"
    and "Print Item Labels on Save and Exit"
[2] Refresh the page or open the holdings editor on a different item.
    Note that the settings from step #1 are not retained.
[3] Apply the patch.
[4] Repeat steps #1 and #2. This time, the settings should be retained.
[5] Set the Default Filter Library in the same tab and refresh. Verify
    that the OU you chose is retained.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: John Amundson <jsamundson@cwmars.org>
Signed-off-by: Janet Schrader <jschrader@cwmars.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>
Open-ILS/web/js/ui/default/staff/cat/volcopy/app.js