Bug 289455: products.classification_id has a different definition in an upgraded...

Patch By Albert Ting <altlst@sonic.net> r=mkanat, a=justdave

diff --git a/checksetup.pl b/checksetup.pl
index 8e076426da20605bb672fff989c07c5c97a81b23..f556682f3c572bf92d3772adc4f0c4397f9a0199 100755 (executable)
--- a/checksetup.pl
+++ b/checksetup.pl
@@ -3716,7 +3716,7 @@ if ($dbh->bz_get_field_def('bugs', 'short_desc')->[2]) { # if it allows nulls
 
 # 2003-10-24 - alt@sonic.net, bug 224208
 # Support classification level
-$dbh->bz_add_field('products', 'classification_id', 'smallint DEFAULT 1');
+$dbh->bz_add_field('products', 'classification_id', 'smallint NOT NULL DEFAULT 1');
 
 # 2004-08-29 - Tomas.Kopal@altap.cz, bug 257303
 # Change logincookies.lastused type from timestamp to datetime
@@ -3939,6 +3939,11 @@ add_setting ("comment_sort_order", {"oldest_to_newest" => 1,
              "oldest_to_newest" );
 
 
+# 2005-04-07 - alt@sonic.net, bug 289455
+# make classification_id field type be consistent with DB:Schema
+$dbh->bz_change_field_type('products', 'classification_id',
+                           'smallint NOT NULL DEFAULT 1');
+
 } # END LEGACY CHECKS
 
 # If you had to change the --TABLE-- definition in any way, then add your