Deal with opt-in boundaries defensively
authorDan Scott <dscott@laurentian.ca>
Fri, 17 Aug 2012 20:07:19 +0000 (16:07 -0400)
committerBen Shum <bshum@biblio.org>
Tue, 29 Jan 2013 06:06:21 +0000 (01:06 -0500)
commit677167d56647dd7351b41aeee4851384d05c734e
tree503680afbb5bbfd405ff7c215e65ea29d3615707
parent50483a39db90045150343cc6ee98365a947c9fc3
Deal with opt-in boundaries defensively

If a site had not set an 'org.patron_opt_default' OU setting, then it
seemed that a DEFAULT value was getting dumped into the "create opt-in"
INSERT statement for the org_unit argument, and that (as there is a
non-NULL constraint on the column and no default value for the column)
resulted in the patron not getting opted in.

One way for sites to deal with this is to set an opt-in boundary at the
consortial level, along the lines of:

INSERT INTO actor.org_unit_setting (org_unit, name, value)
  VALUES (1, 'org.patron_opt_default', 2);

Alternatively, in the absense of any such setting, opt-in should
continue to work as it had before the new feature was added; this change
keeps the old behaviour active in that case.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: James Fournie <jfournie@sitka.bclibraries.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>
Open-ILS/src/perlmods/lib/OpenILS/Application/Actor.pm