From 1c9bef4b8c7ed2a7a2208844599b34f237197501 Mon Sep 17 00:00:00 2001 From: Agustin Moyano Date: Wed, 15 Apr 2020 13:52:21 -0300 Subject: [PATCH] Bug 25086: (follow-up) Set changed_fields column as nullable This patch reverts previous patch (left there because it was signed off), and sets changed_fields column as nullable. Attend only this test plan and not previous patch. To test: 1. Do not apply any patch 2. Set PatronSelfRegistrationVerifyByEmail preference to require 3. Fill self registration form in opac and submit CHECK => OPAC dies with a nasty message, that says Field 'changed_fields' doesn't have a default value 4. Apply this patch and updatedatabase 5. Repeat step 3 SUCCESS => Registry process continues as expected 6. Sign off Sponsored-by: Orex Digital --- installer/data/mysql/atomicupdate/bug_25086.perl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/installer/data/mysql/atomicupdate/bug_25086.perl b/installer/data/mysql/atomicupdate/bug_25086.perl index f1869a2680..1ecdced1d7 100644 --- a/installer/data/mysql/atomicupdate/bug_25086.perl +++ b/installer/data/mysql/atomicupdate/bug_25086.perl @@ -2,7 +2,7 @@ $DBversion = 'XXX'; # will be replaced by the RM if( CheckVersion( $DBversion ) ) { # you can use $dbh here like: - $dbh->do( "ALTER TABLE borrower_modifications ALTER COLUMN changed_fields MEDIUMTEXT DEFAULT NULL" ); + $dbh->do( "ALTER TABLE borrower_modifications MODIFY changed_fields MEDIUMTEXT DEFAULT NULL" ); # Always end with this (adjust the bug info) NewVersion( $DBversion, 25086, "Set changed_fields column of borrower_modifications as nullable"); -- 2.25.0