KMAIN 1448

	Added a check to prevent a merged copy being associated with a deleted
	call number in asset.merge_record_assets.

	new file:   KCLS/sql/kmain-1448/KMAIN_1448_Bug_from_KMAIN_268.sql
	new file:   KCLS/sql/kmain-1448/KMAIN_1448_Bug_from_KMAIN_268_rollback.sql

Signed-off-by: Michael Glass <mglass@catalystitservices.com>

diff --git a/KCLS/sql/kmain-1448/KMAIN_1448_Bug_from_KMAIN_268.sql b/KCLS/sql/kmain-1448/KMAIN_1448_Bug_from_KMAIN_268.sql
new file mode 100644 (file)
index 0000000..c4f7230
--- /dev/null
+++ b/KCLS/sql/kmain-1448/KMAIN_1448_Bug_from_KMAIN_268.sql
@@ -0,0 +1,163 @@
+-- Function: asset.merge_record_assets(bigint, bigint)\r
+\r
+-- DROP FUNCTION asset.merge_record_assets(bigint, bigint);\r
+\r
+CREATE OR REPLACE FUNCTION asset.merge_record_assets(\r
+    target_record bigint,\r
+    source_record bigint)\r
+  RETURNS integer AS\r
+$BODY$\r
+DECLARE\r
+    moved_objects INT := 0;\r
+    source_cn     asset.call_number%ROWTYPE;\r
+    target_cn     asset.call_number%ROWTYPE;\r
+    metarec       metabib.metarecord%ROWTYPE;\r
+    hold          action.hold_request%ROWTYPE;\r
+    ser_rec       serial.record_entry%ROWTYPE;\r
+    uri_count     INT := 0;\r
+    counter       INT := 0;\r
+    uri_datafield TEXT;\r
+    uri_text      TEXT := '';\r
+BEGIN\r
+\r
+    -- move any 856 entries on records that have at least one MARC-mapped URI entry\r
+    SELECT  INTO uri_count COUNT(*)\r
+      FROM  asset.uri_call_number_map m\r
+            JOIN asset.call_number cn ON (m.call_number = cn.id)\r
+      WHERE cn.record = source_record;\r
+\r
+    IF uri_count > 0 THEN\r
+\r
+        SELECT  COUNT(*) INTO counter\r
+          FROM  oils_xpath_table(\r
+                    'id',\r
+                    'marc',\r
+                    'biblio.record_entry',\r
+                    '//*[@tag="856"]',\r
+                    'id=' || source_record\r
+                ) as t(i int,c text);\r
+\r
+        FOR i IN 1 .. counter LOOP\r
+            SELECT  '<datafield xmlns="http://www.loc.gov/MARC21/slim"' ||\r
+                        ' tag="856"' || \r
+                        ' ind1="' || FIRST(ind1) || '"'  || \r
+                        ' ind2="' || FIRST(ind2) || '">' || \r
+                        array_to_string(\r
+                            array_accum(\r
+                                '<subfield code="' || subfield || '">' ||\r
+                                regexp_replace(\r
+                                    regexp_replace(\r
+                                        regexp_replace(data,'&','&amp;','g'),\r
+                                        '>', '&gt;', 'g'\r
+                                    ),\r
+                                    '<', '&lt;', 'g'\r
+                                ) || '</subfield>'\r
+                            ), ''\r
+                        ) || '</datafield>' INTO uri_datafield\r
+              FROM  oils_xpath_table(\r
+                        'id',\r
+                        'marc',\r
+                        'biblio.record_entry',\r
+                        '//*[@tag="856"][position()=' || i || ']/@ind1|' || \r
+                        '//*[@tag="856"][position()=' || i || ']/@ind2|' || \r
+                        '//*[@tag="856"][position()=' || i || ']/*/@code|' ||\r
+                        '//*[@tag="856"][position()=' || i || ']/*[@code]',\r
+                        'id=' || source_record\r
+                    ) as t(id int,ind1 text, ind2 text,subfield text,data text);\r
+\r
+            uri_text := uri_text || uri_datafield;\r
+        END LOOP;\r
+\r
+        IF uri_text <> '' THEN\r
+            UPDATE  biblio.record_entry\r
+              SET   marc = regexp_replace(marc,'(</[^>]*record>)', uri_text || E'\\1')\r
+              WHERE id = target_record;\r
+        END IF;\r
+\r
+    END IF;\r
+\r
+    -- Find and move metarecords to the target record\r
+    SELECT  INTO metarec *\r
+      FROM  metabib.metarecord\r
+      WHERE master_record = source_record;\r
+\r
+    IF FOUND THEN\r
+        UPDATE  metabib.metarecord\r
+          SET   master_record = target_record,\r
+            mods = NULL\r
+          WHERE id = metarec.id;\r
+\r
+        moved_objects := moved_objects + 1;\r
+    END IF;\r
+\r
+    -- Find call numbers attached to the source ...\r
+    FOR source_cn IN SELECT * FROM asset.call_number WHERE record = source_record LOOP\r
+\r
+        SELECT  INTO target_cn *\r
+          FROM  asset.call_number\r
+          WHERE label = source_cn.label\r
+            AND owning_lib = source_cn.owning_lib\r
+            AND record = target_record\r
+            AND NOT deleted;\r
+\r
+        -- ... and if there's a conflicting one on the target ...\r
+        IF FOUND THEN\r
+\r
+            -- ... move the copies to that, and ...\r
+            UPDATE  asset.copy\r
+              SET   call_number = target_cn.id\r
+              WHERE call_number = source_cn.id;\r
+\r
+            -- ... move V holds to the move-target call number\r
+            FOR hold IN SELECT * FROM action.hold_request WHERE target = source_cn.id AND hold_type = 'V' LOOP\r
+\r
+                UPDATE  action.hold_request\r
+                  SET   target = target_cn.id\r
+                  WHERE id = hold.id;\r
+\r
+                moved_objects := moved_objects + 1;\r
+            END LOOP;\r
+\r
+        -- ... if not ...\r
+        ELSE\r
+            -- ... just move the call number to the target record\r
+            UPDATE  asset.call_number\r
+              SET   record = target_record\r
+              WHERE id = source_cn.id;\r
+        END IF;\r
+\r
+        moved_objects := moved_objects + 1;\r
+    END LOOP;\r
+\r
+    -- Find T holds targeting the source record ...\r
+    FOR hold IN SELECT * FROM action.hold_request WHERE target = source_record AND hold_type = 'T' LOOP\r
+\r
+        -- ... and move them to the target record\r
+        UPDATE  action.hold_request\r
+          SET   target = target_record\r
+          WHERE id = hold.id;\r
+\r
+        moved_objects := moved_objects + 1;\r
+    END LOOP;\r
+\r
+    -- Find serial records targeting the source record ...\r
+    FOR ser_rec IN SELECT * FROM serial.record_entry WHERE record = source_record LOOP\r
+        -- ... and move them to the target record\r
+        UPDATE  serial.record_entry\r
+          SET   record = target_record\r
+          WHERE id = ser_rec.id;\r
+\r
+        moved_objects := moved_objects + 1;\r
+    END LOOP;\r
+\r
+    -- Finally, "delete" the source record\r
+    DELETE FROM biblio.record_entry WHERE id = source_record;\r
+\r
+    -- That's all, folks!\r
+    RETURN moved_objects;\r
+END;\r
+$BODY$\r
+  LANGUAGE plpgsql VOLATILE\r
+  COST 100;\r
+ALTER FUNCTION asset.merge_record_assets(bigint, bigint)\r
+  OWNER TO evergreen;\r

diff --git a/KCLS/sql/kmain-1448/KMAIN_1448_Bug_from_KMAIN_268_rollback.sql b/KCLS/sql/kmain-1448/KMAIN_1448_Bug_from_KMAIN_268_rollback.sql
new file mode 100644 (file)
index 0000000..377cc2c
--- /dev/null
+++ b/KCLS/sql/kmain-1448/KMAIN_1448_Bug_from_KMAIN_268_rollback.sql
@@ -0,0 +1,162 @@
+-- Function: asset.merge_record_assets(bigint, bigint)\r
+\r
+-- DROP FUNCTION asset.merge_record_assets(bigint, bigint);\r
+\r
+CREATE OR REPLACE FUNCTION asset.merge_record_assets(\r
+    target_record bigint,\r
+    source_record bigint)\r
+  RETURNS integer AS\r
+$BODY$\r
+DECLARE\r
+    moved_objects INT := 0;\r
+    source_cn     asset.call_number%ROWTYPE;\r
+    target_cn     asset.call_number%ROWTYPE;\r
+    metarec       metabib.metarecord%ROWTYPE;\r
+    hold          action.hold_request%ROWTYPE;\r
+    ser_rec       serial.record_entry%ROWTYPE;\r
+    uri_count     INT := 0;\r
+    counter       INT := 0;\r
+    uri_datafield TEXT;\r
+    uri_text      TEXT := '';\r
+BEGIN\r
+\r
+    -- move any 856 entries on records that have at least one MARC-mapped URI entry\r
+    SELECT  INTO uri_count COUNT(*)\r
+      FROM  asset.uri_call_number_map m\r
+            JOIN asset.call_number cn ON (m.call_number = cn.id)\r
+      WHERE cn.record = source_record;\r
+\r
+    IF uri_count > 0 THEN\r
+\r
+        SELECT  COUNT(*) INTO counter\r
+          FROM  oils_xpath_table(\r
+                    'id',\r
+                    'marc',\r
+                    'biblio.record_entry',\r
+                    '//*[@tag="856"]',\r
+                    'id=' || source_record\r
+                ) as t(i int,c text);\r
+\r
+        FOR i IN 1 .. counter LOOP\r
+            SELECT  '<datafield xmlns="http://www.loc.gov/MARC21/slim"' ||\r
+                        ' tag="856"' || \r
+                        ' ind1="' || FIRST(ind1) || '"'  || \r
+                        ' ind2="' || FIRST(ind2) || '">' || \r
+                        array_to_string(\r
+                            array_accum(\r
+                                '<subfield code="' || subfield || '">' ||\r
+                                regexp_replace(\r
+                                    regexp_replace(\r
+                                        regexp_replace(data,'&','&amp;','g'),\r
+                                        '>', '&gt;', 'g'\r
+                                    ),\r
+                                    '<', '&lt;', 'g'\r
+                                ) || '</subfield>'\r
+                            ), ''\r
+                        ) || '</datafield>' INTO uri_datafield\r
+              FROM  oils_xpath_table(\r
+                        'id',\r
+                        'marc',\r
+                        'biblio.record_entry',\r
+                        '//*[@tag="856"][position()=' || i || ']/@ind1|' || \r
+                        '//*[@tag="856"][position()=' || i || ']/@ind2|' || \r
+                        '//*[@tag="856"][position()=' || i || ']/*/@code|' ||\r
+                        '//*[@tag="856"][position()=' || i || ']/*[@code]',\r
+                        'id=' || source_record\r
+                    ) as t(id int,ind1 text, ind2 text,subfield text,data text);\r
+\r
+            uri_text := uri_text || uri_datafield;\r
+        END LOOP;\r
+\r
+        IF uri_text <> '' THEN\r
+            UPDATE  biblio.record_entry\r
+              SET   marc = regexp_replace(marc,'(</[^>]*record>)', uri_text || E'\\1')\r
+              WHERE id = target_record;\r
+        END IF;\r
+\r
+    END IF;\r
+\r
+    -- Find and move metarecords to the target record\r
+    SELECT  INTO metarec *\r
+      FROM  metabib.metarecord\r
+      WHERE master_record = source_record;\r
+\r
+    IF FOUND THEN\r
+        UPDATE  metabib.metarecord\r
+          SET   master_record = target_record,\r
+            mods = NULL\r
+          WHERE id = metarec.id;\r
+\r
+        moved_objects := moved_objects + 1;\r
+    END IF;\r
+\r
+    -- Find call numbers attached to the source ...\r
+    FOR source_cn IN SELECT * FROM asset.call_number WHERE record = source_record LOOP\r
+\r
+        SELECT  INTO target_cn *\r
+          FROM  asset.call_number\r
+          WHERE label = source_cn.label\r
+            AND owning_lib = source_cn.owning_lib\r
+            AND record = target_record;\r
+\r
+        -- ... and if there's a conflicting one on the target ...\r
+        IF FOUND THEN\r
+\r
+            -- ... move the copies to that, and ...\r
+            UPDATE  asset.copy\r
+              SET   call_number = target_cn.id\r
+              WHERE call_number = source_cn.id;\r
+\r
+            -- ... move V holds to the move-target call number\r
+            FOR hold IN SELECT * FROM action.hold_request WHERE target = source_cn.id AND hold_type = 'V' LOOP\r
+\r
+                UPDATE  action.hold_request\r
+                  SET   target = target_cn.id\r
+                  WHERE id = hold.id;\r
+\r
+                moved_objects := moved_objects + 1;\r
+            END LOOP;\r
+\r
+        -- ... if not ...\r
+        ELSE\r
+            -- ... just move the call number to the target record\r
+            UPDATE  asset.call_number\r
+              SET   record = target_record\r
+              WHERE id = source_cn.id;\r
+        END IF;\r
+\r
+        moved_objects := moved_objects + 1;\r
+    END LOOP;\r
+\r
+    -- Find T holds targeting the source record ...\r
+    FOR hold IN SELECT * FROM action.hold_request WHERE target = source_record AND hold_type = 'T' LOOP\r
+\r
+        -- ... and move them to the target record\r
+        UPDATE  action.hold_request\r
+          SET   target = target_record\r
+          WHERE id = hold.id;\r
+\r
+        moved_objects := moved_objects + 1;\r
+    END LOOP;\r
+\r
+    -- Find serial records targeting the source record ...\r
+    FOR ser_rec IN SELECT * FROM serial.record_entry WHERE record = source_record LOOP\r
+        -- ... and move them to the target record\r
+        UPDATE  serial.record_entry\r
+          SET   record = target_record\r
+          WHERE id = ser_rec.id;\r
+\r
+        moved_objects := moved_objects + 1;\r
+    END LOOP;\r
+\r
+    -- Finally, "delete" the source record\r
+    DELETE FROM biblio.record_entry WHERE id = source_record;\r
+\r
+    -- That's all, folks!\r
+    RETURN moved_objects;\r
+END;\r
+$BODY$\r
+  LANGUAGE plpgsql VOLATILE\r
+  COST 100;\r
+ALTER FUNCTION asset.merge_record_assets(bigint, bigint)\r
+  OWNER TO evergreen;\r