[open-ils-commits] [GIT] Evergreen ILS branch rel_2_4 updated. 17a520e09079ebb66fceb86870614696c3a6ca35

Evergreen Git git at git.evergreen-ils.org
Wed Jul 17 13:34:10 EDT 2013


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_4 has been updated
       via  17a520e09079ebb66fceb86870614696c3a6ca35 (commit)
       via  4d689b86480fd3dd48c1e64d4a666bd4616f7aa9 (commit)
       via  1750611d7ed92e196d0ae63e36107385c97e397f (commit)
       via  a2e194d74237e5d5df4d945537c62d948918f8ba (commit)
      from  3d8ba21c666db3f69491c0c7c375940f9b84b0ac (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 17a520e09079ebb66fceb86870614696c3a6ca35
Author: Mike Rylander <mrylander at gmail.com>
Date:   Wed Jul 17 13:29:14 2013 -0400

    Stamping upgrade script for copy stats SP fix
    
    Signed-off-by: Mike Rylander <mrylander at gmail.com>

diff --git a/Open-ILS/src/sql/Pg/002.schema.config.sql b/Open-ILS/src/sql/Pg/002.schema.config.sql
index 984bd00..67b3c46 100644
--- a/Open-ILS/src/sql/Pg/002.schema.config.sql
+++ b/Open-ILS/src/sql/Pg/002.schema.config.sql
@@ -91,7 +91,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 ('0810', :eg_version); -- senator/miker
+INSERT INTO config.upgrade_log (version, applied_to) VALUES ('0811', :eg_version); -- fparks/miker
 
 CREATE TABLE config.bib_source (
 	id		SERIAL	PRIMARY KEY,
diff --git a/Open-ILS/src/sql/Pg/upgrade/XXXX.function.copy_related_hold_stats.sql b/Open-ILS/src/sql/Pg/upgrade/0811.function.copy_related_hold_stats.sql
similarity index 96%
rename from Open-ILS/src/sql/Pg/upgrade/XXXX.function.copy_related_hold_stats.sql
rename to Open-ILS/src/sql/Pg/upgrade/0811.function.copy_related_hold_stats.sql
index a59b517..81b7b93 100644
--- a/Open-ILS/src/sql/Pg/upgrade/XXXX.function.copy_related_hold_stats.sql
+++ b/Open-ILS/src/sql/Pg/upgrade/0811.function.copy_related_hold_stats.sql
@@ -1,7 +1,7 @@
 BEGIN;
 
 -- check whether patch can be applied
-SELECT evergreen.upgrade_deps_block_check('XXXX', :eg_version);
+SELECT evergreen.upgrade_deps_block_check('0811', :eg_version);
 
 DROP FUNCTION action.copy_related_hold_stats(integer);
 

commit 4d689b86480fd3dd48c1e64d4a666bd4616f7aa9
Author: Fredrick Parks <fparks at catalystitservices.com>
Date:   Wed Jul 10 11:12:17 2013 -0700

    LP 1103706 Hold ratios in circ policies cause errors when trying to renew items
    
    Changed the function action.copy_related_hold_stats to accept a bigint as the perameter instead of an integer.
    Copy_related_hold_stats is only called by the function action.item_user_circ_test which trys to pass a bigint.
    
    Signed-off-by: Fredrick Parks <fparks at catalystitservices.com>
    Signed-off-by: Mike Rylander <mrylander at gmail.com>

diff --git a/Open-ILS/src/sql/Pg/100.circ_matrix.sql b/Open-ILS/src/sql/Pg/100.circ_matrix.sql
index 3dbb328..1e4761a 100644
--- a/Open-ILS/src/sql/Pg/100.circ_matrix.sql
+++ b/Open-ILS/src/sql/Pg/100.circ_matrix.sql
@@ -365,7 +365,7 @@ CREATE TYPE action.hold_stats AS (
     available_copy_ratio    FLOAT
 );
 
-CREATE OR REPLACE FUNCTION action.copy_related_hold_stats (copy_id INT) RETURNS action.hold_stats AS $func$
+CREATE OR REPLACE FUNCTION action.copy_related_hold_stats (copy_id BIGINT) RETURNS action.hold_stats AS $func$
 DECLARE
     output          action.hold_stats%ROWTYPE;
     hold_count      INT := 0;
diff --git a/Open-ILS/src/sql/Pg/upgrade/XXXX.function.copy_related_hold_stats.sql b/Open-ILS/src/sql/Pg/upgrade/XXXX.function.copy_related_hold_stats.sql
new file mode 100644
index 0000000..a59b517
--- /dev/null
+++ b/Open-ILS/src/sql/Pg/upgrade/XXXX.function.copy_related_hold_stats.sql
@@ -0,0 +1,60 @@
+BEGIN;
+
+-- check whether patch can be applied
+SELECT evergreen.upgrade_deps_block_check('XXXX', :eg_version);
+
+DROP FUNCTION action.copy_related_hold_stats(integer);
+
+CREATE OR REPLACE FUNCTION action.copy_related_hold_stats(copy_id bigint)
+  RETURNS action.hold_stats AS
+$BODY$
+DECLARE
+    output          action.hold_stats%ROWTYPE;
+    hold_count      INT := 0;
+    copy_count      INT := 0;
+    available_count INT := 0;
+    hold_map_data   RECORD;
+BEGIN
+
+    output.hold_count := 0;
+    output.copy_count := 0;
+    output.available_count := 0;
+
+    SELECT  COUNT( DISTINCT m.hold ) INTO hold_count
+      FROM  action.hold_copy_map m
+            JOIN action.hold_request h ON (m.hold = h.id)
+      WHERE m.target_copy = copy_id
+            AND NOT h.frozen;
+
+    output.hold_count := hold_count;
+
+    IF output.hold_count > 0 THEN
+        FOR hold_map_data IN
+            SELECT  DISTINCT m.target_copy,
+                    acp.status
+              FROM  action.hold_copy_map m
+                    JOIN asset.copy acp ON (m.target_copy = acp.id)
+                    JOIN action.hold_request h ON (m.hold = h.id)
+              WHERE m.hold IN ( SELECT DISTINCT hold FROM action.hold_copy_map WHERE target_copy = copy_id ) AND NOT h.frozen
+        LOOP
+            output.copy_count := output.copy_count + 1;
+            IF hold_map_data.status IN (0,7,12) THEN
+                output.available_count := output.available_count + 1;
+            END IF;
+        END LOOP;
+        output.total_copy_ratio = output.copy_count::FLOAT / output.hold_count::FLOAT;
+        output.available_copy_ratio = output.available_count::FLOAT / output.hold_count::FLOAT;
+
+    END IF;
+
+    RETURN output;
+
+END;
+$BODY$
+  LANGUAGE plpgsql VOLATILE
+  COST 100;
+ALTER FUNCTION action.copy_related_hold_stats(bigint)
+  OWNER TO evergreen;
+
+COMMIT;
+

commit 1750611d7ed92e196d0ae63e36107385c97e397f
Author: Mike Rylander <mrylander at gmail.com>
Date:   Wed Jul 17 13:25:29 2013 -0400

    Stamping upgrade script for authority labeling fix
    
    Signed-off-by: Mike Rylander <mrylander at gmail.com>

diff --git a/Open-ILS/src/sql/Pg/002.schema.config.sql b/Open-ILS/src/sql/Pg/002.schema.config.sql
index 85d523a..984bd00 100644
--- a/Open-ILS/src/sql/Pg/002.schema.config.sql
+++ b/Open-ILS/src/sql/Pg/002.schema.config.sql
@@ -91,7 +91,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 ('0804', :eg_version); -- paxed/remingtron/bshum/dbwells
+INSERT INTO config.upgrade_log (version, applied_to) VALUES ('0810', :eg_version); -- senator/miker
 
 CREATE TABLE config.bib_source (
 	id		SERIAL	PRIMARY KEY,
diff --git a/Open-ILS/src/sql/Pg/upgrade/XXXX.data.authority_4xx_fields.sql b/Open-ILS/src/sql/Pg/upgrade/0810.data.authority_4xx_fields.sql
similarity index 76%
rename from Open-ILS/src/sql/Pg/upgrade/XXXX.data.authority_4xx_fields.sql
rename to Open-ILS/src/sql/Pg/upgrade/0810.data.authority_4xx_fields.sql
index c6383f1..8398747 100644
--- a/Open-ILS/src/sql/Pg/upgrade/XXXX.data.authority_4xx_fields.sql
+++ b/Open-ILS/src/sql/Pg/upgrade/0810.data.authority_4xx_fields.sql
@@ -1,7 +1,7 @@
 BEGIN;
 
 -- check whether patch can be applied
-SELECT evergreen.upgrade_deps_block_check('XXXX', :eg_version);
+SELECT evergreen.upgrade_deps_block_check('0810', :eg_version);
 
 UPDATE authority.control_set_authority_field
     SET name = REGEXP_REPLACE(name, '^See Also', 'See From')

commit a2e194d74237e5d5df4d945537c62d948918f8ba
Author: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
Date:   Thu Jul 11 15:31:37 2013 -0400

    Authorities: 4XX fields have wrong names
    
    In Evergreen we have authority fields defined in the database that
    relate to control sets.  These are used somewhere today (I'm fuzzy on
    that at the moment; some places once had and may still have hardcoded
    labels) but they also will be used by the bib and auth browser
    that's not yet merged to master (see LP #1177810).
    
    The 4XX tags among the set I'm talking about (in
    authority.control_set_authority_fields) are mislabled, saying See Also
    where they ought to say See From, e.g.
    http://www.loc.gov/marc/authority/ad400.html
    
    Signed-off-by: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
    Signed-off-by: Mike Rylander <mrylander at gmail.com>

diff --git a/Open-ILS/src/sql/Pg/950.data.seed-values.sql b/Open-ILS/src/sql/Pg/950.data.seed-values.sql
index 43ca400..4dee9d9 100644
--- a/Open-ILS/src/sql/Pg/950.data.seed-values.sql
+++ b/Open-ILS/src/sql/Pg/950.data.seed-values.sql
@@ -10203,7 +10203,7 @@ INSERT INTO authority.control_set_authority_field (id, control_set, main_entry,
     (4, 1, NULL, '130', 'adfgklmnoprstvxyz', oils_i18n_gettext('4','Heading -- Uniform Title','acsaf','name'), '2'),
     (24, 1, 4, '530', 'adfgiklmnoprstvwxyz4', oils_i18n_gettext('24','See Also From Tracing -- Uniform Title','acsaf','name'), '2'),
     (44, 1, 4, '730', 'adfghklmnoprstvwxyz25', oils_i18n_gettext('44','Established Heading Linking Entry -- Uniform Title','acsaf','name'), '2'),
-    (64, 1, 4, '430', 'adfgiklmnoprstvwxyz4', oils_i18n_gettext('64','See Also Tracing -- Uniform Title','acsaf','name'), '2');
+    (64, 1, 4, '430', 'adfgiklmnoprstvwxyz4', oils_i18n_gettext('64','See From Tracing -- Uniform Title','acsaf','name'), '2');
 
 INSERT INTO authority.control_set_authority_field (id, control_set, main_entry, tag, sf_list, name) VALUES
 
@@ -10247,17 +10247,17 @@ INSERT INTO authority.control_set_authority_field (id, control_set, main_entry,
     (52, 1, 12, '748', 'avwxyz25', oils_i18n_gettext('52','Established Heading Linking Entry -- Chronological Term','acsaf','name')),
 
 -- See From tracings
-    (61, 1, 1, '400', 'abcdefiklmnopqrstvwxyz4', oils_i18n_gettext('61','See Also Tracing -- Personal Name','acsaf','name')),
-    (62, 1, 2, '410', 'abcdefgiklmnoprstvwxyz4', oils_i18n_gettext('62','See Also Tracing -- Corporate Name','acsaf','name')),
-    (63, 1, 3, '411', 'acdefgiklnpqstvwxyz4', oils_i18n_gettext('63','See Also Tracing -- Meeting Name','acsaf','name')),
-    (65, 1, 5, '450', 'abivwxyz4', oils_i18n_gettext('65','See Also Tracing -- Topical Term','acsaf','name')),
-    (66, 1, 6, '451', 'aivwxyz4', oils_i18n_gettext('66','See Also Tracing -- Geographic Name','acsaf','name')),
-    (67, 1, 7, '455', 'aivwxyz4', oils_i18n_gettext('67','See Also Tracing -- Genre/Form Term','acsaf','name')),
-    (68, 1, 8, '480', 'ivwxyz4', oils_i18n_gettext('68','See Also Tracing -- General Subdivision','acsaf','name')),
-    (69, 1, 9, '481', 'ivwxyz4', oils_i18n_gettext('69','See Also Tracing -- Geographic Subdivision','acsaf','name')),
-    (70, 1, 10, '482', 'ivwxyz4', oils_i18n_gettext('70','See Also Tracing -- Chronological Subdivision','acsaf','name')),
-    (71, 1, 11, '485', 'ivwxyz4', oils_i18n_gettext('71','See Also Tracing -- Form Subdivision','acsaf','name')),
-    (72, 1, 12, '448', 'aivwxyz4', oils_i18n_gettext('72','See Also Tracing -- Chronological Term','acsaf','name'));
+    (61, 1, 1, '400', 'abcdefiklmnopqrstvwxyz4', oils_i18n_gettext('61','See From Tracing -- Personal Name','acsaf','name')),
+    (62, 1, 2, '410', 'abcdefgiklmnoprstvwxyz4', oils_i18n_gettext('62','See From Tracing -- Corporate Name','acsaf','name')),
+    (63, 1, 3, '411', 'acdefgiklnpqstvwxyz4', oils_i18n_gettext('63','See From Tracing -- Meeting Name','acsaf','name')),
+    (65, 1, 5, '450', 'abivwxyz4', oils_i18n_gettext('65','See From Tracing -- Topical Term','acsaf','name')),
+    (66, 1, 6, '451', 'aivwxyz4', oils_i18n_gettext('66','See From Tracing -- Geographic Name','acsaf','name')),
+    (67, 1, 7, '455', 'aivwxyz4', oils_i18n_gettext('67','See From Tracing -- Genre/Form Term','acsaf','name')),
+    (68, 1, 8, '480', 'ivwxyz4', oils_i18n_gettext('68','See From Tracing -- General Subdivision','acsaf','name')),
+    (69, 1, 9, '481', 'ivwxyz4', oils_i18n_gettext('69','See From Tracing -- Geographic Subdivision','acsaf','name')),
+    (70, 1, 10, '482', 'ivwxyz4', oils_i18n_gettext('70','See From Tracing -- Chronological Subdivision','acsaf','name')),
+    (71, 1, 11, '485', 'ivwxyz4', oils_i18n_gettext('71','See From Tracing -- Form Subdivision','acsaf','name')),
+    (72, 1, 12, '448', 'aivwxyz4', oils_i18n_gettext('72','See From Tracing -- Chronological Term','acsaf','name'));
 
 INSERT INTO authority.browse_axis (code,name,description,sorter) VALUES
     ('title','Title','Title axis','titlesort'),
diff --git a/Open-ILS/src/sql/Pg/upgrade/XXXX.data.authority_4xx_fields.sql b/Open-ILS/src/sql/Pg/upgrade/XXXX.data.authority_4xx_fields.sql
new file mode 100644
index 0000000..c6383f1
--- /dev/null
+++ b/Open-ILS/src/sql/Pg/upgrade/XXXX.data.authority_4xx_fields.sql
@@ -0,0 +1,10 @@
+BEGIN;
+
+-- check whether patch can be applied
+SELECT evergreen.upgrade_deps_block_check('XXXX', :eg_version);
+
+UPDATE authority.control_set_authority_field
+    SET name = REGEXP_REPLACE(name, '^See Also', 'See From')
+    WHERE tag LIKE '4__' AND control_set = 1;
+
+COMMIT;

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

Summary of changes:
 Open-ILS/src/sql/Pg/002.schema.config.sql          |    2 +-
 Open-ILS/src/sql/Pg/100.circ_matrix.sql            |    2 +-
 Open-ILS/src/sql/Pg/950.data.seed-values.sql       |   24 ++++++++++----------
 .../Pg/upgrade/0810.data.authority_4xx_fields.sql  |   10 ++++++++
 ...l => 0811.function.copy_related_hold_stats.sql} |   21 +++++++++--------
 5 files changed, 35 insertions(+), 24 deletions(-)
 create mode 100644 Open-ILS/src/sql/Pg/upgrade/0810.data.authority_4xx_fields.sql
 copy Open-ILS/src/sql/Pg/upgrade/{0067.schema.hold-copy-ratios.sql => 0811.function.copy_related_hold_stats.sql} (78%)


hooks/post-receive
-- 
Evergreen ILS


More information about the open-ils-commits mailing list