Bug 17234 made an effort to remove the ALTER IGNORE from updatedatabase.pl so that it could work with Mysql 5.7. But new entries have since been added to it: Bug 12395 - 18.06.00.005 Bug 19524 - 18.06.00.006 Bug 13560 - 18.06.00.008 These were oversights and must be removed.
Created attachment 77313 [details] [review] Bug 21129: New ALTER IGNORE TABLE entries need correction in updatedatabase.pl Bug 17234 made an effort to remove the ALTER IGNORE from updatedatabase.pl so that it could work with Mysql 5.7. But new entries have since been added to it: Bug 12395 - 18.06.00.005 Bug 19524 - 18.06.00.006 Bug 13560 - 18.06.00.008 This is corrected using column_exists to validate the existence of the new columns before ALTER
Comment on attachment 77313 [details] [review] Bug 21129: New ALTER IGNORE TABLE entries need correction in updatedatabase.pl Review of attachment 77313 [details] [review]: ----------------------------------------------------------------- ::: installer/data/mysql/updatedatabase.pl @@ +16129,4 @@ > > $DBversion = '18.06.00.005'; > if( CheckVersion( $DBversion ) ) { > + unless ( column_exists('aqorders', 'created') ) { It should be 'created_by' I think...
Created attachment 77330 [details] [review] Bug 21129: New ALTER IGNORE TABLE entries need correction in updatedatabase.pl Bug 17234 made an effort to remove the ALTER IGNORE from updatedatabase.pl so that it could work with Mysql 5.7. But new entries have since been added to it: Bug 12395 - 18.06.00.005 Bug 19524 - 18.06.00.006 Bug 13560 - 18.06.00.008 This is corrected using column_exists to validate the existence of the new columns before ALTER Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 77331 [details] [review] Bug 21129: Fix typo created vs created_by
Created attachment 77360 [details] [review] Bug 21129: New ALTER IGNORE TABLE entries need correction in updatedatabase.pl Bug 17234 made an effort to remove the ALTER IGNORE from updatedatabase.pl so that it could work with Mysql 5.7. But new entries have since been added to it: Bug 12395 - 18.06.00.005 Bug 19524 - 18.06.00.006 Bug 13560 - 18.06.00.008 This is corrected using column_exists to validate the existence of the new columns before ALTER Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 77361 [details] [review] Bug 21129: Fix typo created vs created_by Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Pushed to master. Thanks all!
Affected DB updates only exist in master branch, no need to apply to 18.05.x series.