Bug 31364: Check length of the multiple syspref array instead whether it is null
authorJoonas Kylmälä <joonas.kylmala@iki.fi>
Sun, 4 Sep 2022 11:37:46 +0000 (14:37 +0300)
committerTomas Cohen Arazi <tomascohen@theke.io>
Wed, 7 Sep 2022 19:07:00 +0000 (12:07 -0700)
commitce1efe7de0d71362f930a8fc5415fda797531a0b
tree7747efbeff508b42bc0042d432892ecfa0ba5431
parent7a90be2ff221df5e7ab575e89063f6e8dcc99c29
Bug 31364: Check length of the multiple syspref array instead whether it is null

The multiple systempreferences are always defined as an empty array at
the minimum, so in order to determine whether we need to remove all
the selections from the systempreference we should check whether the
array is empty and not whether the array doesn't exist.

To test:
 1) Set at least 1 value for OPACHoldsIfAvailableAtPickupExceptions
 2) Remove all the values for OPACHoldsIfAvailableAtPickupExceptions
 and notice you get the message "Nothing to save"
 3) Apply patch and repeat, removing all the values should work now.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/js/pages/preferences.js