Bug 320436: New self-created accounts dont get regexp group memberships - Patch by...

diff --git a/Bugzilla/User.pm b/Bugzilla/User.pm
index fe46dc2921634c4954283eea4598d5528c187444..542e6e8d78488885e3290def9f9acc0a20a5a79e 100644 (file)
--- a/Bugzilla/User.pm
+++ b/Bugzilla/User.pm
@@ -1349,6 +1349,10 @@ sub insert_new_user {
                  "(user_id, relationship, event) " . 
                  "VALUES ($userid, " . REL_ANY . ", $event)");
     }
+
+    my $user = new Bugzilla::User($userid);
+    $user->derive_regexp_groups();
+
     
     # Return the password to the calling code so it can be included
     # in an email sent to the user.

diff --git a/createaccount.cgi b/createaccount.cgi
index 8bb7829efc2749e249a8cbd482c3fb8c533f45ac..01b746c89cffca4cd29eba67df8943a029d92256 100755 (executable)
--- a/createaccount.cgi
+++ b/createaccount.cgi
@@ -69,7 +69,9 @@ if (defined($login)) {
 
     $vars->{'login'} = $login;
 
-    $dbh->bz_lock_tables('profiles WRITE', 'email_setting WRITE', 'tokens READ');
+    $dbh->bz_lock_tables('profiles WRITE', 'groups READ',
+                         'user_group_map WRITE', 'email_setting WRITE',
+                         'tokens READ');
 
     if (!is_available_username($login)) {
         # Account already exists

diff --git a/editusers.cgi b/editusers.cgi
index bfc7eee144fb2bb9fce70b08fd2d1aaa66ecab34..290b8dbe4afcea08182c79af9bff9839ad965009 100755 (executable)
--- a/editusers.cgi
+++ b/editusers.cgi
@@ -180,6 +180,8 @@ if ($action eq 'search') {
     # Lock tables during the check+creation session.
     $dbh->bz_lock_tables('profiles WRITE',
                          'profiles_activity WRITE',
+                         'groups READ',
+                         'user_group_map WRITE',
                          'email_setting WRITE',
                          'namedqueries READ',
                          'whine_queries READ',
@@ -203,8 +205,6 @@ if ($action eq 'search') {
     insert_new_user($login, $realname, $password, $disabledtext);
     my $new_user_id = $dbh->bz_last_key('profiles', 'userid');
     $dbh->bz_unlock_tables();
-    my $newprofile = new Bugzilla::User($new_user_id);
-    $newprofile->derive_regexp_groups();
     userDataToVars($new_user_id);
 
     $vars->{'message'} = 'account_created';