Attachment 101722 Details for Bug 24412 – Bug 24412: kohastructure and atomic update

[patch] Bug 24412: kohastructure and atomic update

Bug-24412-kohastructure-and-atomic-update.patch (text/plain), 4.30 KB, created by Nicolas Legrand on 2020-03-25 15:21:53 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nicolas Legrand

Created: 2020-03-25 15:21:53 UTC

Size: 4.30 KB

patch

obsolete

>From 859f2a6b8649a8b45201b48d7d7f7738f730c19b Mon Sep 17 00:00:00 2001
>From: Nicolas Legrand <nicolas.legrand@bulac.fr>
>Date: Mon, 13 Jan 2020 17:55:46 +0100
>Subject: [PATCH] Bug 24412: kohastructure and atomic update
>
>---
> .../bug_24412_Attach_waiting_reserve_to_desk.perl        | 16 ++++++++++++++++
> installer/data/mysql/kohastructure.sql                   |  4 ++++
> 2 files changed, 20 insertions(+)
> create mode 100644 installer/data/mysql/atomicupdate/bug_24412_Attach_waiting_reserve_to_desk.perl
>
>diff --git a/installer/data/mysql/atomicupdate/bug_24412_Attach_waiting_reserve_to_desk.perl b/installer/data/mysql/atomicupdate/bug_24412_Attach_waiting_reserve_to_desk.perl
>new file mode 100644
>index 0000000..c743eab
>--- /dev/null
>+++ b/installer/data/mysql/atomicupdate/bug_24412_Attach_waiting_reserve_to_desk.perl
>@@ -0,0 +1,16 @@
>+$DBversion = 'XXX';  # will be replaced by the RM
>+if( CheckVersion( $DBversion ) ) {
>+    # you can use $dbh here like:
>+    $dbh->do(q{
>+ALTER TABLE reserves ADD COLUMN desk_id INT(11) DEFAULT NULL AFTER branchcode,
>+  ADD KEY desk_id (`desk_id`),
>+  ADD CONSTRAINT `reserves_ibfk_6` FOREIGN KEY (`desk_id`) REFERENCES `desks` (`desk_id`) ON DELETE CASCADE ON UPDATE CASCADE ;
>+             });
>+    $dbh->do(q{
>+ALTER TABLE old_reserves ADD COLUMN desk_id INT(11) DEFAULT NULL AFTER branchcode,
>+  ADD KEY `old_desk_id` (`desk_id`);
>+             });
>+
>+    SetVersion( $DBversion );
>+    print "Upgrade to $DBversion done (Bug XXXXX - Attach waiting reserve to desk)\n";
>+}
>diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql
>index a7126a5..e6addf2 100644
>--- a/installer/data/mysql/kohastructure.sql
>+++ b/installer/data/mysql/kohastructure.sql
>@@ -1771,6 +1771,7 @@ CREATE TABLE `reserves` ( -- information related to holds/reserves in Koha
>   `reservedate` date default NULL, -- the date the hold was placed
>   `biblionumber` int(11) NOT NULL default 0, -- foreign key from the biblio table defining which bib record this hold is on
>   `branchcode` varchar(10) default NULL, -- foreign key from the branches table defining which branch the patron wishes to pick this hold up at
>+  `desk_id` int(11) default NULL, -- foreign key from the desks table defining which desk the patron should pick this hold up at
>   `notificationdate` date default NULL, -- currently unused
>   `reminderdate` date default NULL, -- currently unused
>   `cancellationdate` date default NULL, -- the date this hold was cancelled
>@@ -1792,12 +1793,14 @@ CREATE TABLE `reserves` ( -- information related to holds/reserves in Koha
>   KEY `biblionumber` (`biblionumber`),
>   KEY `itemnumber` (`itemnumber`),
>   KEY `branchcode` (`branchcode`),
>+  KEY desk_id (`desk_id`),
>   KEY `itemtype` (`itemtype`),
>   CONSTRAINT `reserves_ibfk_1` FOREIGN KEY (`borrowernumber`) REFERENCES `borrowers` (`borrowernumber`) ON DELETE CASCADE ON UPDATE CASCADE,
>   CONSTRAINT `reserves_ibfk_2` FOREIGN KEY (`biblionumber`) REFERENCES `biblio` (`biblionumber`) ON DELETE CASCADE ON UPDATE CASCADE,
>   CONSTRAINT `reserves_ibfk_3` FOREIGN KEY (`itemnumber`) REFERENCES `items` (`itemnumber`) ON DELETE CASCADE ON UPDATE CASCADE,
>   CONSTRAINT `reserves_ibfk_4` FOREIGN KEY (`branchcode`) REFERENCES `branches` (`branchcode`) ON DELETE CASCADE ON UPDATE CASCADE,
>   CONSTRAINT `reserves_ibfk_5` FOREIGN KEY (`itemtype`) REFERENCES `itemtypes` (`itemtype`) ON DELETE CASCADE ON UPDATE CASCADE
>+  CONSTRAINT `reserves_ibfk_6` FOREIGN KEY (`desk_id`) REFERENCES `desks` (`desk_id`) ON DELETE CASCADE ON UPDATE CASCADE ;
> ) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci;
> 
> --
>@@ -1811,6 +1814,7 @@ CREATE TABLE `old_reserves` ( -- this table holds all holds/reserves that have b
>   `reservedate` date default NULL, -- the date the hold was places
>   `biblionumber` int(11) default NULL, -- foreign key from the biblio table defining which bib record this hold is on
>   `branchcode` varchar(10) default NULL, -- foreign key from the branches table defining which branch the patron wishes to pick this hold up at
>+  `desk_id` int(11) default NULL, -- foreign key from the desks table defining which desk the patron should pick this hold up at
>   `notificationdate` date default NULL, -- currently unused
>   `reminderdate` date default NULL, -- currently unused
>   `cancellationdate` date default NULL, -- the date this hold was cancelled
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 24412: 97328 | 97329 | 97330 | 101625 | 101626 | 101627 | 101628 | 101629 | 101630 | 101721 | 101722 | 101723 | 101825 | 101884 | 101885 | 101886 | 101898 | 101899 | 101900 | 101955 | 101956 | 101957 | 106586 | 106648 | 110067 | 110068 | 110069 | 110070 | 110450 | 110451 | 110452 | 110453 | 110454 | 110455 | 110582 | 110583 | 110584 | 110585 | 110586 | 112046 | 112047 | 112048 | 112049 | 112050 | 112092 | 112099 | 112165 | 112166 | 112167 | 112168 | 112169 | 112170 | 112171 | 112172 | 112201 | 112202 | 112203 | 112204 | 112205 | 112206 | 112207 | 112208 | 112209 | 112210 | 112211 | 112212 | 112213 | 112214 | 112215 | 112216 | 112217 | 112218 | 112219 | 112264 | 112265 | 112266 | 112267 | 112268 | 112269 | 112270 | 112271 | 112272 | 112292 | 112293 | 113108 | 113109 | 113116 | 113228 | 113229 | 113230 | 113231 | 113232 | 113233 | 113234 | 113235 | 113236