@@ -, +, @@ --- .../data/mysql/atomicupdate/bug_24010.pl | 30 +++++++++++++++++++ installer/data/mysql/kohastructure.sql | 4 +-- 2 files changed, 32 insertions(+), 2 deletions(-) create mode 100644 installer/data/mysql/atomicupdate/bug_24010.pl --- a/installer/data/mysql/atomicupdate/bug_24010.pl +++ a/installer/data/mysql/atomicupdate/bug_24010.pl @@ -0,0 +1,30 @@ +use Modern::Perl; + +return { + bug_number => "24010", + description => "Make subscription.staffdisplaycount and opacdisplaycount integer columns", + up => sub { + my ($args) = @_; + my ($dbh, $out) = @$args{qw(dbh out)}; + + $dbh->do(q{ + UPDATE subscription + SET staffdisplaycount = NULL + WHERE staffdisplaycount = "" + }); + $dbh->do(q{ + ALTER TABLE subscription + MODIFY COLUMN staffdisplaycount INT(11) NULL DEFAULT NULL + }); + + $dbh->do(q{ + UPDATE subscription + SET opacdisplaycount = NULL + WHERE opacdisplaycount = "" + }); + $dbh->do(q{ + ALTER TABLE subscription + MODIFY COLUMN opacdisplaycount INT(11) NULL DEFAULT NULL + }); + }, +}; --- a/installer/data/mysql/kohastructure.sql +++ a/installer/data/mysql/kohastructure.sql @@ -4878,8 +4878,8 @@ CREATE TABLE `subscription` ( `branchcode` varchar(10) COLLATE utf8mb4_unicode_ci NOT NULL DEFAULT '' COMMENT 'default branches (items.homebranch)', `lastbranch` varchar(10) COLLATE utf8mb4_unicode_ci DEFAULT NULL, `serialsadditems` tinyint(1) NOT NULL DEFAULT 0 COMMENT 'does receiving this serial create an item record', - `staffdisplaycount` varchar(10) COLLATE utf8mb4_unicode_ci DEFAULT NULL COMMENT 'how many issues to show to the staff', - `opacdisplaycount` varchar(10) COLLATE utf8mb4_unicode_ci DEFAULT NULL COMMENT 'how many issues to show to the public', + `staffdisplaycount` INT(11) NULL DEFAULT NULL, -- how many issues to show to the staff + `opacdisplaycount` INT(11) NULL DEFAULT NULL, -- how many issues to show to the public `graceperiod` int(11) NOT NULL DEFAULT 0 COMMENT 'grace period in days', `enddate` date DEFAULT NULL COMMENT 'subscription end date', `closed` tinyint(1) NOT NULL DEFAULT 0 COMMENT 'yes / no if the subscription is closed', --