Bug 305941: Remove profiles.refreshed_when and groups.last_changed - Patch by Frédéric Buclin <LpSolit@gmail.com> r=mkanat a=myk
git-svn-id: svn://10.0.0.236/trunk@206625 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
@@ -618,7 +618,6 @@ use constant ABSTRACT_SCHEMA => {
|
||||
DEFAULT => 'FALSE'},
|
||||
mybugslink => {TYPE => 'BOOLEAN', NOTNULL => 1,
|
||||
DEFAULT => 'TRUE'},
|
||||
refreshed_when => {TYPE => 'DATETIME', NOTNULL => 1},
|
||||
extern_id => {TYPE => 'varchar(64)'},
|
||||
],
|
||||
INDEXES => [
|
||||
@@ -738,7 +737,6 @@ use constant ABSTRACT_SCHEMA => {
|
||||
name => {TYPE => 'varchar(255)', NOTNULL => 1},
|
||||
description => {TYPE => 'TEXT', NOTNULL => 1},
|
||||
isbuggroup => {TYPE => 'BOOLEAN', NOTNULL => 1},
|
||||
last_changed => {TYPE => 'DATETIME', NOTNULL => 1},
|
||||
userregexp => {TYPE => 'TINYTEXT', NOTNULL => 1,
|
||||
DEFAULT => "''"},
|
||||
isactive => {TYPE => 'BOOLEAN', NOTNULL => 1,
|
||||
|
||||
@@ -37,7 +37,6 @@ use constant DB_COLUMNS => qw(
|
||||
groups.name
|
||||
groups.description
|
||||
groups.isbuggroup
|
||||
groups.last_changed
|
||||
groups.userregexp
|
||||
groups.isactive
|
||||
);
|
||||
@@ -99,7 +98,6 @@ sub id { return $_[0]->{'id'}; }
|
||||
sub name { return $_[0]->{'name'}; }
|
||||
sub description { return $_[0]->{'description'}; }
|
||||
sub is_bug_group { return $_[0]->{'isbuggroup'}; }
|
||||
sub last_changed { return $_[0]->{'last_changed'}; }
|
||||
sub user_regexp { return $_[0]->{'userregexp'}; }
|
||||
sub is_active { return $_[0]->{'isactive'}; }
|
||||
|
||||
@@ -157,7 +155,6 @@ Bugzilla::Group - Bugzilla group class.
|
||||
my $id = $group->id;
|
||||
my $name = $group->name;
|
||||
my $description = $group->description;
|
||||
my $last_changed = $group->last_changed;
|
||||
my $user_reg_exp = $group->user_reg_exp;
|
||||
my $is_active = $group->is_active;
|
||||
|
||||
|
||||
@@ -459,7 +459,11 @@ sub update_table_definitions {
|
||||
{TYPE => 'INT2', NOTNULL => 1, DEFAULT => 0});
|
||||
$dbh->bz_alter_column('products', 'votestoconfirm',
|
||||
{TYPE => 'INT2', NOTNULL => 1, DEFAULT => 0});
|
||||
|
||||
|
||||
# 2006-08-04 LpSolit@gmail.com - Bug 305941
|
||||
$dbh->bz_drop_column('profiles', 'refreshed_when');
|
||||
$dbh->bz_drop_column('groups', 'last_changed');
|
||||
|
||||
################################################################
|
||||
# New --TABLE-- changes should go *** A B O V E *** this point #
|
||||
################################################################
|
||||
@@ -1315,9 +1319,6 @@ sub _convert_groups_system_from_groupset {
|
||||
|
||||
# The groups system needs to be converted if groupset exists
|
||||
if ($dbh->bz_column_info("profiles", "groupset")) {
|
||||
$dbh->bz_add_column('groups', 'last_changed',
|
||||
{TYPE => 'DATETIME', NOTNULL => 1}, '0000-00-00 00:00:00');
|
||||
|
||||
# Some mysql versions will promote any unique key to primary key
|
||||
# so all unique keys are removed first and then added back in
|
||||
$dbh->bz_drop_index('groups', 'groups_bit_idx');
|
||||
@@ -1331,8 +1332,6 @@ sub _convert_groups_system_from_groupset {
|
||||
|
||||
$dbh->bz_add_index('groups', 'groups_name_idx',
|
||||
{TYPE => 'UNIQUE', FIELDS => [qw(name)]});
|
||||
$dbh->bz_add_column('profiles', 'refreshed_when',
|
||||
{TYPE => 'DATETIME', NOTNULL => 1}, '0000-00-00 00:00:00');
|
||||
|
||||
# Convert all existing groupset records to map entries before removing
|
||||
# groupset fields or removing "bit" from groups.
|
||||
@@ -1979,9 +1978,6 @@ sub _add_user_group_map_grant_type {
|
||||
$dbh->bz_add_index('user_group_map', 'user_group_map_user_id_idx',
|
||||
{TYPE => 'UNIQUE',
|
||||
FIELDS => [qw(user_id group_id grant_type isbless)]});
|
||||
|
||||
# Make sure groups get rederived
|
||||
$dbh->do("UPDATE groups SET last_changed = NOW() WHERE name = 'admin'");
|
||||
}
|
||||
}
|
||||
|
||||
@@ -2065,12 +2061,6 @@ sub _rename_votes_count_and_force_group_refresh {
|
||||
# Renaming the 'count' column in the votes table because Sybase doesn't
|
||||
# like it
|
||||
if ($dbh->bz_column_info('votes', 'count')) {
|
||||
# 2003-04-24 - myk@mozilla.org/bbaetz@acm.org, bug 201018
|
||||
# Force all cached groups to be updated at login, due to security bug
|
||||
# Do this here, inside the next schema change block, so that it doesn't
|
||||
# get invalidated on every checksetup run.
|
||||
$dbh->do("UPDATE profiles SET refreshed_when='1900-01-01 00:00:00'");
|
||||
|
||||
$dbh->bz_rename_column('votes', 'count', 'vote_count');
|
||||
}
|
||||
}
|
||||
|
||||
@@ -707,9 +707,6 @@ sub derive_regexp_groups {
|
||||
|
||||
my $sth;
|
||||
|
||||
# avoid races, we are only up to date as of the BEGINNING of this process
|
||||
my $time = $dbh->selectrow_array("SELECT NOW()");
|
||||
|
||||
# add derived records for any matching regexps
|
||||
|
||||
$sth = $dbh->prepare("SELECT id, userregexp, user_group_map.group_id
|
||||
@@ -735,9 +732,6 @@ sub derive_regexp_groups {
|
||||
$group_delete->execute($id, $group, GRANT_REGEXP) if $present;
|
||||
}
|
||||
}
|
||||
|
||||
$dbh->do(q{UPDATE profiles SET refreshed_when = ? WHERE userid = ?},
|
||||
undef, ($time, $id));
|
||||
}
|
||||
|
||||
sub product_responsibilities {
|
||||
@@ -1363,8 +1357,8 @@ sub insert_new_user {
|
||||
# Insert the new user record into the database.
|
||||
$dbh->do("INSERT INTO profiles
|
||||
(login_name, realname, cryptpassword, disabledtext,
|
||||
refreshed_when, disable_mail)
|
||||
VALUES (?, ?, ?, ?, '1901-01-01 00:00:00', ?)",
|
||||
disable_mail)
|
||||
VALUES (?, ?, ?, ?, ?)",
|
||||
undef,
|
||||
($username, $realname, $cryptpassword, $disabledtext,
|
||||
$disable_mail));
|
||||
|
||||
Reference in New Issue
Block a user