From bc067cad65d8d8492293a5c176c6e7c8d28b3ef7 Mon Sep 17 00:00:00 2001 From: Martin Persson Date: Wed, 27 May 2015 07:36:33 +0200 Subject: [PATCH] Bug 14246: Add borrowernumber, altered SQL As suggested by kind reviewers, an AFTER statement was added to the atomic upgrade script to ensure that the final field order is consistent is both fresh and upgraded databases. Also: * UPDATE action changed to CASCADE to improve robustness. Test plan: * Apply first patch in this set. * Apply this patch. * Perform a database upgrade. * Use mysql client to inspect field order. It should be identical to the kohastructure.sql order. Sponsored-by: Halland County Library --- installer/data/mysql/atomicupdate/bug_14246-add_news_author.sql | 4 ++-- installer/data/mysql/kohastructure.sql | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/installer/data/mysql/atomicupdate/bug_14246-add_news_author.sql b/installer/data/mysql/atomicupdate/bug_14246-add_news_author.sql index 1319d37..0a9362a 100644 --- a/installer/data/mysql/atomicupdate/bug_14246-add_news_author.sql +++ b/installer/data/mysql/atomicupdate/bug_14246-add_news_author.sql @@ -1,2 +1,2 @@ -ALTER TABLE `opac_news` ADD `borrowernumber` INT(11) default NULL; -ALTER TABLE `opac_news` ADD CONSTRAINT `borrowernumber_fk` FOREIGN KEY (`borrowernumber`) REFERENCES `borrowers`(`borrowernumber`) ON DELETE SET NULL ON UPDATE SET NULL; +ALTER TABLE `opac_news` ADD `borrowernumber` int(11) AFTER `number` default NULL; +ALTER TABLE `opac_news` ADD CONSTRAINT `borrowernumber_fk` FOREIGN KEY (`borrowernumber`) REFERENCES `borrowers`(`borrowernumber`) ON DELETE SET NULL ON UPDATE CASCADE; diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql index 5eb7eed..ef07ae5 100644 --- a/installer/data/mysql/kohastructure.sql +++ b/installer/data/mysql/kohastructure.sql @@ -1696,7 +1696,7 @@ CREATE TABLE `opac_news` ( -- data from the news tool `number` int(11) default NULL, -- the order in which this article appears in that specific location `borrowernumber` int(11) default NULL, -- The user who created the news article PRIMARY KEY (`idnew`), - CONSTRAINT `borrowernumber_fk` FOREIGN KEY (`borrowernumber`) REFERENCES `borrowers` (`borrowernumber`) ON DELETE SET NULL ON UPDATE SET NULL, + CONSTRAINT `borrowernumber_fk` FOREIGN KEY (`borrowernumber`) REFERENCES `borrowers` (`borrowernumber`) ON DELETE SET NULL ON UPDATE CASCADE, CONSTRAINT opac_news_branchcode_ibfk FOREIGN KEY (branchcode) REFERENCES branches (branchcode) ON DELETE CASCADE ON UPDATE CASCADE ) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci; -- 2.1.4