[open-ils-commits] [GIT] Evergreen ILS branch rel_2_3 updated. d22b66ccd0b64f862f302db5534f550d05528683

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_2_3 has been updated
       via  d22b66ccd0b64f862f302db5534f550d05528683 (commit)
       via  b6621e6febdb6739017ec38ef9c93ada95ace332 (commit)
      from  bac86e4f87f1b36a7f46ed062eda19a6f6717494 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit d22b66ccd0b64f862f302db5534f550d05528683
Author: Ben Shum <bshum at biblio.org>
Date:   Tue Dec 18 00:12:38 2012 -0500

    Stamping upgrade script for hold request email notify default false
    
    Signed-off-by: Ben Shum <bshum at biblio.org>

diff --git a/Open-ILS/src/sql/Pg/002.schema.config.sql b/Open-ILS/src/sql/Pg/002.schema.config.sql
index 7b5b2c3..d069cc0 100644
--- a/Open-ILS/src/sql/Pg/002.schema.config.sql
+++ b/Open-ILS/src/sql/Pg/002.schema.config.sql
@@ -87,7 +87,7 @@ CREATE TRIGGER no_overlapping_deps
     BEFORE INSERT OR UPDATE ON config.db_patch_dependencies
     FOR EACH ROW EXECUTE PROCEDURE evergreen.array_overlap_check ('deprecates');
 
-INSERT INTO config.upgrade_log (version, applied_to) VALUES ('0744', :eg_version); -- Dyrcona/dbwells
+INSERT INTO config.upgrade_log (version, applied_to) VALUES ('0746', :eg_version); -- mrpeters-isl/bshum
 
 CREATE TABLE config.bib_source (
 	id		SERIAL	PRIMARY KEY,
diff --git a/Open-ILS/src/sql/Pg/upgrade/XXXX.action.hold_request.email_notify_default_false.sql b/Open-ILS/src/sql/Pg/upgrade/0746.action.hold_request.email_notify_default_false.sql
similarity index 60%
rename from Open-ILS/src/sql/Pg/upgrade/XXXX.action.hold_request.email_notify_default_false.sql
rename to Open-ILS/src/sql/Pg/upgrade/0746.action.hold_request.email_notify_default_false.sql
index f899cec..166f24f 100644
--- a/Open-ILS/src/sql/Pg/upgrade/XXXX.action.hold_request.email_notify_default_false.sql
+++ b/Open-ILS/src/sql/Pg/upgrade/0746.action.hold_request.email_notify_default_false.sql
@@ -1,6 +1,6 @@
 BEGIN;
 
-SELECT evergreen.upgrade_deps_block_check('XXXX', :eg_version);
+SELECT evergreen.upgrade_deps_block_check('0746', :eg_version);
 
 ALTER TABLE action.hold_request ALTER COLUMN email_notify SET DEFAULT 'false';
 

commit b6621e6febdb6739017ec38ef9c93ada95ace332
Author: Michael Peters <mrpeters at library.in.gov>
Date:   Thu Sep 13 16:28:04 2012 -0400

    LP#1047475 Set action.hold_request.email_notify to default to false
    
    LP#1047475 points out that email notify prefs are ignored in TPAC.
    Even if a patron deselects email notify during hold request time, it
    will always be set to true due to the column default.
    
    This change defaults the column in the database to false (like most
    other similar columns are) and resolves the bug.
    
    Credit to Steve Callender for the idea.
    
    Signed-off-by: Michael Peters <mrpeters at library.in.gov>
    Signed-off-by: Ben Shum <bshum at biblio.org>

diff --git a/Open-ILS/src/sql/Pg/090.schema.action.sql b/Open-ILS/src/sql/Pg/090.schema.action.sql
index 1af6e89..833d7bc 100644
--- a/Open-ILS/src/sql/Pg/090.schema.action.sql
+++ b/Open-ILS/src/sql/Pg/090.schema.action.sql
@@ -393,7 +393,7 @@ CREATE TABLE action.hold_request (
 	hold_type		TEXT				NOT NULL, -- CHECK (hold_type IN ('M','T','V','C')),  -- XXX constraint too constraining...
 	holdable_formats	TEXT,
 	phone_notify		TEXT,
-	email_notify		BOOL				NOT NULL DEFAULT TRUE,
+	email_notify		BOOL				NOT NULL DEFAULT FALSE,
 	sms_notify		TEXT,
 	sms_carrier		INT REFERENCES config.sms_carrier (id),
 	frozen			BOOL				NOT NULL DEFAULT FALSE,
diff --git a/Open-ILS/src/sql/Pg/upgrade/XXXX.action.hold_request.email_notify_default_false.sql b/Open-ILS/src/sql/Pg/upgrade/XXXX.action.hold_request.email_notify_default_false.sql
new file mode 100644
index 0000000..f899cec
--- /dev/null
+++ b/Open-ILS/src/sql/Pg/upgrade/XXXX.action.hold_request.email_notify_default_false.sql
@@ -0,0 +1,7 @@
+BEGIN;
+
+SELECT evergreen.upgrade_deps_block_check('XXXX', :eg_version);
+
+ALTER TABLE action.hold_request ALTER COLUMN email_notify SET DEFAULT 'false';
+
+COMMIT;

-----------------------------------------------------------------------

Summary of changes:
 Open-ILS/src/sql/Pg/002.schema.config.sql          |    2 +-
 Open-ILS/src/sql/Pg/090.schema.action.sql          |    2 +-
 ...ion.hold_request.email_notify_default_false.sql |    7 +++++++
 3 files changed, 9 insertions(+), 2 deletions(-)
 create mode 100644 Open-ILS/src/sql/Pg/upgrade/0746.action.hold_request.email_notify_default_false.sql


hooks/post-receive
-- 
Evergreen ILS