On Wed, Mar 13, 2013 at 8:26 PM, Ben Shum <email address hidden> wrote:
> 2) The upgrade SQL contains an entry to insert a default patron penalty for org unit 1 (usually CONS) and set a max lost penalty of 10. This might be bad for sites performing the upgrade and suddenly finding themselves with an unexpected new default block entry.
Agreed -- IMO, this must be changed before the patch can be merged.
Note that it's OK for the new default patron penalty to be present in
the seed data, it just shouldn't automatically part of the upgrade.
Hi,
On Wed, Mar 13, 2013 at 8:26 PM, Ben Shum <email address hidden> wrote:
> 2) The upgrade SQL contains an entry to insert a default patron penalty for org unit 1 (usually CONS) and set a max lost penalty of 10. This might be bad for sites performing the upgrade and suddenly finding themselves with an unexpected new default block entry.
Agreed -- IMO, this must be changed before the patch can be merged.
Note that it's OK for the new default patron penalty to be present in
the seed data, it just shouldn't automatically part of the upgrade.