Bug 31086: (QA follow-up) Remove DEFAULT from column
There's no need to have a default here, and in fact it doesn't make sense. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
parent
bf2a9b5279
commit
d731943c24
|
|
@ -8,7 +8,7 @@ return {
|
|||
my ($dbh, $out) = @$args{qw(dbh out)};
|
||||
$dbh->do(q{
|
||||
ALTER TABLE reserves
|
||||
MODIFY COLUMN `branchcode` varchar(10) COLLATE utf8mb4_unicode_ci NOT NULL DEFAULT 0 COMMENT 'foreign key from the branches table defining which branch the patron wishes to pick this hold up at'
|
||||
MODIFY COLUMN `branchcode` varchar(10) COLLATE utf8mb4_unicode_ci NOT NULL COMMENT 'foreign key from the branches table defining which branch the patron wishes to pick this hold up at'
|
||||
});
|
||||
# Print useful stuff here
|
||||
say $out "Removed NULL option from branchcode for reserves";
|
||||
|
|
|
|||
|
|
@ -4456,7 +4456,7 @@ CREATE TABLE `reserves` (
|
|||
`borrowernumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from the borrowers table defining which patron this hold is for',
|
||||
`reservedate` date DEFAULT NULL COMMENT 'the date the hold was placed',
|
||||
`biblionumber` int(11) NOT NULL DEFAULT 0 COMMENT 'foreign key from the biblio table defining which bib record this hold is on',
|
||||
`branchcode` varchar(10) COLLATE utf8mb4_unicode_ci NOT NULL DEFAULT 0 COMMENT 'foreign key from the branches table defining which branch the patron wishes to pick this hold up at',
|
||||
`branchcode` varchar(10) COLLATE utf8mb4_unicode_ci NOT NULL COMMENT 'foreign key from the branches table defining which branch the patron wishes to pick this hold up at',
|
||||
`desk_id` int(11) DEFAULT NULL COMMENT 'foreign key from the desks table defining which desk the patron should pick this hold up at',
|
||||
`notificationdate` date DEFAULT NULL COMMENT 'currently unused',
|
||||
`reminderdate` date DEFAULT NULL COMMENT 'currently unused',
|
||||
|
|
|
|||
Loading…
Reference in New Issue