Bug 277504 : quips migrated from quip file have an (invalid) userid of 0

Patch by Frederic Buclin <LpSolit@gmail.com>  r=mkanat  a=myk

diff --git a/checksetup.pl b/checksetup.pl
index eb34ec6681ea21ac13cb663b6111bc09e79259b9..c0759d562dfab95ecb107ea9d31a1ba200d59f62 100755 (executable)
--- a/checksetup.pl
+++ b/checksetup.pl
@@ -1957,10 +1957,11 @@ $table{bug_group_map} =
      index(group_id)';
 
 # 2002-07-19, davef@tetsubo.com, bug 67950:
+# 2005-02-20, LpSolit@gmail.com, bug 277504
 # Store quips in the db.
 $table{quips} =
     'quipid mediumint not null auto_increment primary key,
-     userid mediumint not null default 0, 
+     userid mediumint null, 
      quip text not null,
      approved tinyint(1) not null default 1';
 
@@ -4093,6 +4094,16 @@ AddField('flags', 'is_active', 'tinyint not null default 1');
 
     
 
+# 2005-02-20 - LpSolit@gmail.com - Bug 277504
+# When migrating quips from the '$datadir/comments' file to the DB,
+# the user ID should be NULL instead of 0 (which is an invalid user ID).
+if (!GetFieldDef('quips', 'userid')->[2]) {
+    ChangeFieldType('quips', 'userid', 'mediumint null');
+    print "Changing owner to NULL for quips where the owner is unknown...\n";
+    $dbh->do('UPDATE quips SET userid = NULL WHERE userid = 0');
+}
+
+
 # If you had to change the --TABLE-- definition in any way, then add your
 # differential change code *** A B O V E *** this comment.
 #