From 9726fb767a8016e6205fbaf24492049b67fb74e3 Mon Sep 17 00:00:00 2001
From: "mkanat%bugzilla.org" <>
Date: Fri, 7 Aug 2009 21:45:31 +0000
Subject: The original patch for bug 508186 actually broke utf-8
 conversion--the tables were converted, but the columns were not. This fixes
 it.

---
 Bugzilla/DB/Mysql.pm | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

(limited to 'Bugzilla/DB')

diff --git a/Bugzilla/DB/Mysql.pm b/Bugzilla/DB/Mysql.pm
index 54659f5b0..380448ba3 100644
--- a/Bugzilla/DB/Mysql.pm
+++ b/Bugzilla/DB/Mysql.pm
@@ -748,6 +748,8 @@ EOT
                     # right after the type, which will always come first.
                     my ($binary, $utf8) = ($sql_def, $sql_def);
                     my $type = $self->_bz_schema->convert_type($col_info->{TYPE});
+                    $binary =~ s/(\Q$type\E)/$1 CHARACTER SET binary/;
+                    $utf8   =~ s/(\Q$type\E)/$1 CHARACTER SET utf8/;
                     push(@binary_sql, "MODIFY COLUMN $name $binary");
                     push(@utf8_sql, "MODIFY COLUMN $name $utf8");
                 }
@@ -767,7 +769,8 @@ EOT
 
                 print "Converting the $table table to UTF-8...\n";
                 my $bin = "ALTER TABLE $table " . join(', ', @binary_sql);
-                my $utf = "ALTER TABLE $table " . join(', ', @utf8_sql);
+                my $utf = "ALTER TABLE $table " . join(', ', @utf8_sql,
+                          'DEFAULT CHARACTER SET utf8');
                 $self->do($bin);
                 $self->do($utf);
 
@@ -776,8 +779,9 @@ EOT
                     $self->bz_add_index($table, $index, $indexes{$index});
                 }
             }
-
-            $self->do("ALTER TABLE $table DEFAULT CHARACTER SET utf8");
+            else {
+                $self->do("ALTER TABLE $table DEFAULT CHARACTER SET utf8");
+            }
 
         } # foreach my $table (@tables)
 
-- 
cgit v1.2.3-24-g4f1b