Attachment 101963 Details for Bug 18177 – Bug 18177: DB changes

[patch] Bug 18177: DB changes

Bug-18177-DB-changes.patch (text/plain), 2.47 KB, created by Marcel de Rooy on 2020-03-27 08:26:15 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2020-03-27 08:26:15 UTC

Size: 2.47 KB

patch

obsolete

>From be6a5684607c38dfd6b65b5e6659671e1d54bf33 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Tue, 17 Mar 2020 09:50:18 +0100
>Subject: [PATCH] Bug 18177: DB changes
>Content-Type: text/plain; charset=utf-8
>
>Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> installer/data/mysql/atomicupdate/bug_18177.perl | 15 +++++++++++++++
> installer/data/mysql/kohastructure.sql           |  7 +------
> 2 files changed, 16 insertions(+), 6 deletions(-)
> create mode 100644 installer/data/mysql/atomicupdate/bug_18177.perl
>
>diff --git a/installer/data/mysql/atomicupdate/bug_18177.perl b/installer/data/mysql/atomicupdate/bug_18177.perl
>new file mode 100644
>index 0000000000..0426fda963
>--- /dev/null
>+++ b/installer/data/mysql/atomicupdate/bug_18177.perl
>@@ -0,0 +1,15 @@
>+$DBversion = 'XXX'; # will be replaced by the RM
>+if( CheckVersion( $DBversion ) ) {
>+
>+    for my $column ( qw(othersupplier booksellerfax booksellerurl bookselleremail currency) ) {
>+        if( column_exists( 'aqbooksellers', $column ) ) {
>+            $dbh->do(qq|
>+                ALTER TABLE aqbooksellers
>+                DROP COLUMN $column
>+            |);
>+        }
>+    }
>+
>+    SetVersion( $DBversion );
>+    print "Upgrade to $DBversion done (Bug 18177 - Remove some unused columns from aqbooksellers)\n";
>+}
>diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql
>index a7126a56ec..40dd796b9a 100644
>--- a/installer/data/mysql/kohastructure.sql
>+++ b/installer/data/mysql/kohastructure.sql
>@@ -2783,13 +2783,8 @@ CREATE TABLE `aqbooksellers` ( -- information about the vendors listed in acquis
>   `address3` LONGTEXT, -- third line of vendor physical address
>   `address4` LONGTEXT, -- fourth line of vendor physical address
>   `phone` varchar(30) default NULL, -- vendor phone number
>-  `accountnumber` LONGTEXT, -- unused in Koha
>-  `othersupplier` LONGTEXT,  -- unused in Koha
>-  `currency` varchar(10) NOT NULL default '', -- unused in Koha
>-  `booksellerfax` LONGTEXT, -- vendor fax number
>+  `accountnumber` LONGTEXT, -- vendor account number
>   `notes` LONGTEXT, -- order notes
>-  `bookselleremail` LONGTEXT, -- vendor email
>-  `booksellerurl` LONGTEXT, -- unused in Koha
>   `postal` LONGTEXT, -- vendor postal address (all lines)
>   `url` varchar(255) default NULL, -- vendor web address
>   `active` tinyint(4) default NULL, -- is this vendor active (1 for yes, 0 for no)
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 18177: 100861 | 100862 | 100863 | 100864 | 100865 | 100866 | 100867 | 100868 | 100869 | 100870 | 100871 | 100872 | 100873 | 100874 | 101612 | 101687 | 101688 | 101689 | 101690 | 101691 | 101692 | 101693 | 101694 | 101958 | 101959 | 101960 | 101961 | 101962 | 101963 | 101964 | 101965 | 102011