Commit da1e0f0e authored by mkanat%bugzilla.org's avatar mkanat%bugzilla.org

Bug 516712: Allow checksetup.pl to upgrade installations from before 3.4 where…

Bug 516712: Allow checksetup.pl to upgrade installations from before 3.4 where profiles.extern_id contains multiple empty string values. Patch by Max Kanat-Alexander <mkanat@bugzilla.org> r=dkl, a=mkanat
parent be1c9700
......@@ -559,8 +559,7 @@ sub update_table_definitions {
_add_visiblity_value_to_value_tables();
# 2009-03-02 arbingersys@gmail.com - Bug 423613
$dbh->bz_add_index('profiles', 'profiles_extern_id_idx',
{TYPE => 'UNIQUE', FIELDS => [qw(extern_id)]});
_add_extern_id_index();
# 2009-03-31 LpSolit@gmail.com - Bug 478972
$dbh->bz_alter_column('group_control_map', 'entry',
......@@ -3180,6 +3179,17 @@ sub _add_visiblity_value_to_value_tables {
}
}
sub _add_extern_id_index {
my $dbh = Bugzilla->dbh;
if (!$dbh->bz_index_info('profiles', 'profiles_extern_id_idx')) {
# Some Bugzillas have a multiple empty strings in extern_id,
# which need to be converted to NULLs before we add the index.
$dbh->do("UPDATE profiles SET extern_id = NULL WHERE extern_id = ''");
$dbh->bz_add_index('profiles', 'profiles_extern_id_idx',
{TYPE => 'UNIQUE', FIELDS => [qw(extern_id)]});
}
}
sub _convert_disallownew_to_isactive {
my $dbh = Bugzilla->dbh;
if ($dbh->bz_column_info('products', 'disallownew')){
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment