Bug 9993: On editing basket group delivery place resets to logged in branch
commited5e9732d9d078483918f594c9fdca7da737faec
authorOwen Leonard <oleonard@myacpl.org>
Wed, 20 Nov 2019 17:00:52 +0000 (20 17:00 +0000)
committerLucas Gass <lucas@bywatersolutions.com>
Fri, 14 Feb 2020 21:29:38 +0000 (14 21:29 +0000)
tree4214d00d3aea94c36876c8d6ad346fbf1f660010
parent667f7204678a9dafd01a960cdd294e130e31d267
Bug 9993: On editing basket group delivery place resets to logged in branch

This patch modifies basketgroups.pl so that existing data about a basket
group's billing and delivery place are correctly preselected in the edit
form. These fields shouldn't reset to the logged-in user's home branch
just because they have an empty value.

To test, apply the patch and go to Acquisitions -> Vendor -> Basket
groups.

When adding or editing a basket group the pre-selected value for
"Billing place" and "Delivery place" should be correct:

   - When creating a new basket group: The logged-in user's library
     should be pre-selected.

   - When editing a basket group which has a library defined for either
     the billing or delivery places, the correct library should be
     pre-selected.

   - When editing a basket group which has no library defined for either
     the billing or delivery place there should be no library
     pre-selected (the "--" option should be pre-selected).

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Joy Nelson <joy@bywatersolutions.com>
(cherry picked from commit 69afaa42391d358fac90a0fab8e00f1d10b75dc1)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
acqui/basketgroup.pl