Attachment 161221 Details for Bug 35687 – Bug 35687: Upgrade to 23.06.00.013 may fail

[patch] Bug 35687: Upgrade to 23.06.00.013 may fail

Bug-35687-Upgrade-to-230600013-may-fail.patch (text/plain), 1.96 KB, created by Victor Grousset/tuxayo on 2024-01-19 19:57:00 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Victor Grousset/tuxayo

Created: 2024-01-19 19:57:00 UTC

Size: 1.96 KB

patch

obsolete

>From d24f3adf21df04a50bca1d6976d5ca86731353a9 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Wed, 3 Jan 2024 14:07:49 +0000
>Subject: [PATCH] Bug 35687: Upgrade to 23.06.00.013 may fail
>
>There are reports that this update triggers the error Cannot change column 'itemnumber': used in a foreign key constraint 'tmp_holdsqueue_ibfk_1'
>
>Test Plan:
>1) Upgrade to post 23.06.00.013
>2) Note the failure
>3) Apply this patch set
>4) Run updatedatabase.pl
>5) Update should succeed!
>
>Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> .../data/mysql/atomicupdate/bug_35687.pl      | 24 +++++++++++++++++++
> 1 file changed, 24 insertions(+)
> create mode 100755 installer/data/mysql/atomicupdate/bug_35687.pl
>
>diff --git a/installer/data/mysql/atomicupdate/bug_35687.pl b/installer/data/mysql/atomicupdate/bug_35687.pl
>new file mode 100755
>index 0000000000..68380c0bb3
>--- /dev/null
>+++ b/installer/data/mysql/atomicupdate/bug_35687.pl
>@@ -0,0 +1,24 @@
>+use Modern::Perl;
>+
>+return {
>+    bug_number  => "35687",
>+    description => "Upgrade to 23.06.00.013 may fail, drop fk and recreate after adding the pk to tmp_holdsqueue",
>+    up          => sub {
>+        my ($args) = @_;
>+        my ( $dbh, $out ) = @$args{qw(dbh out)};
>+
>+        unless ( primary_key_exists( 'tmp_holdsqueue', 'itemnumber' ) ) {
>+            $dbh->do(q{ALTER TABLE tmp_holdsqueue DROP CONSTRAINT `tmp_holdsqueue_ibfk_1`});
>+            $dbh->do(q{ALTER TABLE tmp_holdsqueue ADD PRIMARY KEY (itemnumber)});
>+            $dbh->do(
>+                q{
>+                    ALTER TABLE tmp_holdsqueue ADD CONSTRAINT `tmp_holdsqueue_ibfk_1`
>+                    FOREIGN KEY (`itemnumber`) REFERENCES `items` (`itemnumber`)
>+                    ON DELETE CASCADE ON UPDATE CASCADE
>+                }
>+            );
>+        }
>+
>+        say $out "Set primary key for table 'tmp_holdsqueue' to 'itemnumber'";
>+    },
>+};
>-- 
>2.43.0

Actions: View | Diff | Splinter Review

Attachments on bug 35687: 160461 | 160462 | 161141 | 161142 | 161179 | 161180 | 161221