[open-ils-commits] [GIT] Evergreen ILS branch master updated. 9e60f478f574f546bd40a019f75cb4119130ae8f

Evergreen Git git at git.evergreen-ils.org
Tue Aug 29 11:30:25 EDT 2017


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, master has been updated
       via  9e60f478f574f546bd40a019f75cb4119130ae8f (commit)
       via  75c7e2c0081583c4bb32e5a483baa9c30601b1c2 (commit)
       via  1bb10d44996351c362ac837dc8f9a0a641903e05 (commit)
      from  8c7bdaa944117eb29cce47d5d23a842db1586596 (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 9e60f478f574f546bd40a019f75cb4119130ae8f
Author: Kathy Lussier <klussier at masslnc.org>
Date:   Tue Aug 29 11:27:30 2017 -0400

    LP#1048822: Release notes entry for fuller simplified pull list title
    
    Signed-off-by: Kathy Lussier <klussier at masslnc.org>

diff --git a/docs/RELEASE_NOTES_NEXT/Circulation/simplified_pull_list_fuller_title.adoc b/docs/RELEASE_NOTES_NEXT/Circulation/simplified_pull_list_fuller_title.adoc
new file mode 100644
index 0000000..954bc69
--- /dev/null
+++ b/docs/RELEASE_NOTES_NEXT/Circulation/simplified_pull_list_fuller_title.adoc
@@ -0,0 +1,6 @@
+Fuller title in XUL client Simplified Pull List
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+The Simplified Pull List in the XUL client will now display subfields n and
+p in the title field. The addition will make it easier for staff to distinguish
+between different parts or seasons in a series.
+

commit 75c7e2c0081583c4bb32e5a483baa9c30601b1c2
Author: Kathy Lussier <klussier at masslnc.org>
Date:   Tue Aug 29 11:10:12 2017 -0400

    LP#1048822: Stamping upgrade script for simplified pull list fuller title
    
    Signed-off-by: Kathy Lussier <klussier at masslnc.org>

diff --git a/Open-ILS/src/sql/Pg/002.schema.config.sql b/Open-ILS/src/sql/Pg/002.schema.config.sql
index 9ae05d8..e73c016 100644
--- a/Open-ILS/src/sql/Pg/002.schema.config.sql
+++ b/Open-ILS/src/sql/Pg/002.schema.config.sql
@@ -90,7 +90,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 ('1058', :eg_version); -- mccanna/csharp/gmcharlt
+INSERT INTO config.upgrade_log (version, applied_to) VALUES ('1059', :eg_version); -- Stompro/DPearl/kmlussier
 
 CREATE TABLE config.bib_source (
 	id		SERIAL	PRIMARY KEY,
diff --git a/Open-ILS/src/sql/Pg/upgrade/XXXX.lp1048822_fuller_title_super_simple.sql b/Open-ILS/src/sql/Pg/upgrade/1059.lp1048822_fuller_title_super_simple.sql
similarity index 94%
rename from Open-ILS/src/sql/Pg/upgrade/XXXX.lp1048822_fuller_title_super_simple.sql
rename to Open-ILS/src/sql/Pg/upgrade/1059.lp1048822_fuller_title_super_simple.sql
index cffad4e..0c69753 100644
--- a/Open-ILS/src/sql/Pg/upgrade/XXXX.lp1048822_fuller_title_super_simple.sql
+++ b/Open-ILS/src/sql/Pg/upgrade/1059.lp1048822_fuller_title_super_simple.sql
@@ -1,5 +1,8 @@
 BEGIN;
 
+-- check whether patch can be applied
+SELECT evergreen.upgrade_deps_block_check('1059', :eg_version); --Stompro/DPearl/kmlussier
+
 CREATE OR REPLACE VIEW reporter.old_super_simple_record AS
 SELECT  r.id,
     r.fingerprint,

commit 1bb10d44996351c362ac837dc8f9a0a641903e05
Author: Josh Stompro <stomproj at larl.org>
Date:   Fri Jul 1 13:54:54 2016 -0500

    LP#1048822 - Fuller Title for reporter.super_simple_record
    
    The simplified pull list uses reporter.super_simple_record for
    title information.  reporter.super_simple_record currently only
    pulls the first 245a value for the title.  For titles that
    use 245(n|p) to fully identify the title this leaves out information
    that staff need to find the item.
    
    This patch adds all of the 245(n|p) to the 245a in the title.  I've found
    this covers most cases where staff need a fuller title.  The order of
    the 245(n|p) also matters, so steps are taken to keep the marc ordering
    of those fields.
    
    Testing:
    
    Before Patch - find a title that uses a 245(n|p) field to fully identify the
    title.  Place a hold on the title.  Use the simplified pull list to view the
    location that has that item on the shelf.  Observe that only the 245a is in
    the title field.
    
    After Patch - Look at the simplified pull list again and see that the title now
    includes more information.
    
    Signed-off-by: Josh Stompro <stomproj at larl.org>
    Signed-off-by: Dan Pearl <dpearl at cwmars.org>
    Signed-off-by: Kathy Lussier <klussier at masslnc.org>

diff --git a/Open-ILS/src/sql/Pg/reporter-schema.sql b/Open-ILS/src/sql/Pg/reporter-schema.sql
index 985ca6f..28d9d44 100644
--- a/Open-ILS/src/sql/Pg/reporter-schema.sql
+++ b/Open-ILS/src/sql/Pg/reporter-schema.sql
@@ -154,7 +154,7 @@ SELECT  r.id,
     r.quality,
     r.tcn_source,
     r.tcn_value,
-    FIRST(title.value) AS title,
+    CONCAT_WS(' ', FIRST(title.value),FIRST(title_np.val)) AS title,
     FIRST(author.value) AS author,
     STRING_AGG(DISTINCT publisher.value, ', ') AS publisher,
     STRING_AGG(DISTINCT SUBSTRING(pubdate.value FROM $$\d+$$), ', ') AS pubdate,
@@ -168,6 +168,17 @@ SELECT  r.id,
     END AS issn
   FROM  biblio.record_entry r
     LEFT JOIN metabib.full_rec title ON (r.id = title.record AND title.tag = '245' AND title.subfield = 'a')
+    LEFT JOIN ( -- Grab 245 N and P subfields in the order that they appear.
+      SELECT b.record, string_agg(val, ' ') AS val FROM (
+             SELECT title_np.record, title_np.value AS val
+              FROM metabib.full_rec title_np
+              WHERE
+              title_np.tag = '245'
+                        AND title_np.subfield IN ('p','n')
+                        ORDER BY title_np.id
+                ) b
+                GROUP BY 1
+         ) title_np ON (title_np.record=r.id)
     LEFT JOIN metabib.full_rec author ON (r.id = author.record AND author.tag IN ('100','110','111') AND author.subfield = 'a')
     LEFT JOIN metabib.full_rec publisher ON (r.id = publisher.record AND (publisher.tag = '260' OR (publisher.tag = '264' AND publisher.ind2 = '1')) AND publisher.subfield = 'b')
     LEFT JOIN metabib.full_rec pubdate ON (r.id = pubdate.record AND (pubdate.tag = '260' OR (pubdate.tag = '264' AND pubdate.ind2 = '1')) AND pubdate.subfield = 'c')
diff --git a/Open-ILS/src/sql/Pg/upgrade/XXXX.lp1048822_fuller_title_super_simple.sql b/Open-ILS/src/sql/Pg/upgrade/XXXX.lp1048822_fuller_title_super_simple.sql
new file mode 100644
index 0000000..cffad4e
--- /dev/null
+++ b/Open-ILS/src/sql/Pg/upgrade/XXXX.lp1048822_fuller_title_super_simple.sql
@@ -0,0 +1,48 @@
+BEGIN;
+
+CREATE OR REPLACE VIEW reporter.old_super_simple_record AS
+SELECT  r.id,
+    r.fingerprint,
+    r.quality,
+    r.tcn_source,
+    r.tcn_value,
+    CONCAT_WS(' ', FIRST(title.value),FIRST(title_np.val)) AS title,
+    FIRST(author.value) AS author,
+    STRING_AGG(DISTINCT publisher.value, ', ') AS publisher,
+    STRING_AGG(DISTINCT SUBSTRING(pubdate.value FROM $$\d+$$), ', ') AS pubdate,
+    CASE WHEN ARRAY_AGG( DISTINCT REPLACE(SUBSTRING(isbn.value FROM $$^\S+$$), '-', '') ) = '{NULL}'
+        THEN NULL
+        ELSE ARRAY_AGG( DISTINCT REPLACE(SUBSTRING(isbn.value FROM $$^\S+$$), '-', '') )
+    END AS isbn,
+    CASE WHEN ARRAY_AGG( DISTINCT REGEXP_REPLACE(issn.value, E'^\\S*(\\d{4})[-\\s](\\d{3,4}x?)', E'\\1 \\2') ) = '{NULL}'
+        THEN NULL
+        ELSE ARRAY_AGG( DISTINCT REGEXP_REPLACE(issn.value, E'^\\S*(\\d{4})[-\\s](\\d{3,4}x?)', E'\\1 \\2') )
+    END AS issn
+  FROM  biblio.record_entry r
+    LEFT JOIN metabib.full_rec title ON (r.id = title.record AND title.tag = '245' AND title.subfield = 'a')
+    LEFT JOIN ( -- Grab 245 N and P subfields in the order that they appear.
+      SELECT b.record, string_agg(val, ' ') AS val FROM (
+	     SELECT title_np.record, title_np.value AS val
+	      FROM metabib.full_rec title_np
+	      WHERE
+	      title_np.tag = '245'
+			AND title_np.subfield IN ('p','n')			
+			ORDER BY title_np.id
+		) b
+		GROUP BY 1
+	 ) title_np ON (title_np.record=r.id) 
+    LEFT JOIN metabib.full_rec author ON (r.id = author.record AND author.tag IN ('100','110','111') AND author.subfield = 'a')
+    LEFT JOIN metabib.full_rec publisher ON (r.id = publisher.record AND (publisher.tag = '260' OR (publisher.tag = '264' AND publisher.ind2 = '1')) AND publisher.subfield = 'b')
+    LEFT JOIN metabib.full_rec pubdate ON (r.id = pubdate.record AND (pubdate.tag = '260' OR (pubdate.tag = '264' AND pubdate.ind2 = '1')) AND pubdate.subfield = 'c')
+    LEFT JOIN metabib.full_rec isbn ON (r.id = isbn.record AND isbn.tag IN ('024', '020') AND isbn.subfield IN ('a','z'))
+    LEFT JOIN metabib.full_rec issn ON (r.id = issn.record AND issn.tag = '022' AND issn.subfield = 'a')
+  GROUP BY 1,2,3,4,5;
+
+  
+  -- Remove trigger on biblio.record_entry
+  SELECT reporter.disable_materialized_simple_record_trigger();
+  
+  -- Rebuild reporter.materialized_simple_record
+  SELECT reporter.enable_materialized_simple_record_trigger();
+  
+  COMMIT;

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

Summary of changes:
 Open-ILS/src/sql/Pg/002.schema.config.sql          |    2 +-
 Open-ILS/src/sql/Pg/reporter-schema.sql            |   13 +++++-
 .../1059.lp1048822_fuller_title_super_simple.sql   |   51 ++++++++++++++++++++
 .../simplified_pull_list_fuller_title.adoc         |    6 ++
 4 files changed, 70 insertions(+), 2 deletions(-)
 create mode 100644 Open-ILS/src/sql/Pg/upgrade/1059.lp1048822_fuller_title_super_simple.sql
 create mode 100644 docs/RELEASE_NOTES_NEXT/Circulation/simplified_pull_list_fuller_title.adoc


hooks/post-receive
-- 
Evergreen ILS


More information about the open-ils-commits mailing list