Attachment 99317 Details for Bug 24131 – Bug 24131: (follow-up) Fix and rebase

[patch] Bug 24131: (follow-up) Fix and rebase

Bug-24131-follow-up-Fix-and-rebase.patch (text/plain), 4.74 KB, created by Martin Renvoize (ashimema) on 2020-02-20 14:33:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2020-02-20 14:33:21 UTC

Size: 4.74 KB

patch

obsolete

>From f419f5dd1b82ce286485725551040f99ff01094d Mon Sep 17 00:00:00 2001
>From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>Date: Thu, 20 Feb 2020 14:29:47 +0000
>Subject: [PATCH] Bug 24131: (follow-up) Fix and rebase
>
>Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>---
> installer/data/mysql/updatedatabase.pl | 34 +++++++++++---------------
> 1 file changed, 14 insertions(+), 20 deletions(-)
>
>diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl
>index 2742e6c8f1..826c1b831e 100755
>--- a/installer/data/mysql/updatedatabase.pl
>+++ b/installer/data/mysql/updatedatabase.pl
>@@ -20228,7 +20228,7 @@ if( CheckVersion( $DBversion ) ) {
> 
> $DBversion = '19.11.00.000';
> if( CheckVersion( $DBversion ) ) {
>-    NewVersion( $DBversion, undef '19.11.00 release' );
>+    NewVersion( $DBversion, undef, '19.11.00 release' );
> }
> 
> $DBversion = '19.12.00.000';
>@@ -20725,8 +20725,8 @@ if ( CheckVersion($DBversion) ) {
>     if ( !column_exists( 'marc_tag_structure', 'important') ){
>         $dbh->do("ALTER TABLE marc_tag_structure ADD COLUMN important TINYINT(4) NOT NULL DEFAULT 0  AFTER mandatory");
>     }
>-    SetVersion($DBversion);
>-    print "Upgrade to $DBversion done (Bug 8643 - Add important constraint to marc subfields)\n";
>+
>+    NewVersion( $DBversion, 8643, 'Add important constraint to marc subfields' );
> }
> 
> $DBversion = '19.12.00.021';
>@@ -20775,8 +20775,7 @@ if( CheckVersion( $DBversion ) ) {
>           ( 'Lost Item Found' )
>     });
> 
>-    SetVersion( $DBversion );
>-    print "Upgrade to $DBversion done (Bug 24592 - Update LOST_RETURN to LOST_FOUND)\n";
>+    NewVersion( $DBversion, 24592, 'Update LOST_RETURN to LOST_FOUND');
> }
> 
> $DBversion = '19.12.00.022';
>@@ -20784,16 +20783,14 @@ if( CheckVersion( $DBversion ) ) {
>     $dbh->do( "ALTER TABLE items MODIFY COLUMN uri MEDIUMTEXT" );
>     $dbh->do( "ALTER TABLE deleteditems MODIFY COLUMN uri MEDIUMTEXT" );
> 
>-    SetVersion( $DBversion );
>-    print "Upgrade to $DBversion done (Bug 20882 - items.uri to MEDIUMTEXT)\n";
>+    NewVersion( $DBversion, 20882, 'items.uri to MEDIUMTEXT');
> }
> 
> $DBversion = '19.12.00.023';
> if( CheckVersion( $DBversion ) ) {
>     $dbh->do( "ALTER TABLE quotes MODIFY timestamp datetime NULL" );
> 
>-    SetVersion( $DBversion );
>-    print "Upgrade to $DBversion done (Bug 24640 - Allow quotes.timestamp to be NULL)\n";
>+    NewVersion( $DBversion, 24640, 'Allow quotes.timestamp to be NULL');
> }
> 
> $DBversion = '19.12.00.024';
>@@ -20807,8 +20804,8 @@ if( CheckVersion( $DBversion ) ) {
>         explanation = "Choose the fines mode, 'off' (do not accrue fines) or 'production' (accrue overdue fines).  Requires accruefines cronjob or CalculateFinesOnReturn system preference."
>         WHERE variable = 'finesMode'
>     });
>-    SetVersion( $DBversion );
>-    print "Upgrade to $DBversion done (Bug 21633  - Remove finesMode 'test')\n";
>+
>+    NewVersion( $DBversion, 21633, 'Remove finesMode "test"');
> }
> 
> $DBversion = '19.12.00.025';
>@@ -20817,8 +20814,8 @@ if( CheckVersion( $DBversion ) ) {
>         INSERT IGNORE INTO `systempreferences` (variable,value,options,explanation,type)
>         VALUES ('DumpSearchQueryTemplate',0,'','Add the search query being passed to the search engine into the template for debugging','YesNo')
>     });
>-    SetVersion( $DBversion );
>-    print "Upgrade to $DBversion done (Bug 24103 - add DumpSearchQueryTemplate syspref)\n";
>+
>+    NewVersion( $DBversion, 24103, 'add DumpSearchQueryTemplate syspref');
> }
> 
> $DBversion = '19.12.00.026';
>@@ -20827,8 +20824,7 @@ if( CheckVersion( $DBversion ) ) {
>         $dbh->do( "ALTER TABLE z3950servers ADD COLUMN attributes VARCHAR(255) after add_xslt" );
>     }
> 
>-    SetVersion( $DBversion );
>-    print "Upgrade to $DBversion done (Bug 11297 - Add support for custom PQF attributes for Z39.50 server searches)\n";
>+    NewVersion( $DBversion, 11297, 'Add support for custom PQF attributes for Z39.50 server searches');
> }
> 
> $DBversion = '19.12.00.027';
>@@ -20870,9 +20866,7 @@ if( CheckVersion( $DBversion ) ) {
>         debit_type_code IS NOT NULL
>     });
> 
>-    # Always end with this (adjust the bug info)
>-    SetVersion( $DBversion );
>-    print "Upgrade to $DBversion done (Bug 24532 - Fix pathological cases of negative debits)\n";
>+    NewVersion( $DBversion, 24532, 'Fix pathological cases of negative debits');
> }
> 
> $DBversion = '19.12.00.028';
>@@ -20882,8 +20876,8 @@ if( CheckVersion( $DBversion ) ) {
>         VALUES
>         ('OpacBrowseSearch', '0',NULL, "Elasticsearch only: add a page allowing users to 'browse' all items in the collection",'YesNo')
>     });
>-    SetVersion( $DBversion );
>-    print "Upgrade to $DBversion done (Bug 14567: Add OpacBrowseSearch syspref)\n";
>+
>+    NewVersion( $DBversion, 14567, 'Add OpacBrowseSearch syspref');
> }
> 
> # SEE bug 13068
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 24131: 98536 | 98537 | 99315 | 99316 | 99317 | 99521 | 99522 | 99523 | 99524 | 100161 | 100162 | 100163 | 100164 | 100464 | 100465 | 100466 | 100467 | 100468 | 100530 | 100531 | 100532 | 100533 | 100534 | 100535 | 100536 | 100538 | 101155 | 101159 | 101160 | 101162 | 101164 | 101166 | 101168 | 101187 | 103001