From: lpsolit%gmail.com <> Date: Fri, 5 Aug 2005 01:29:33 +0000 (+0000) Subject: Bug 302723: Deleting a user account when this user is the assignee or QA contact... X-Git-Tag: bugzilla-2.20rc2~5 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=dfb7f52c9bcb4e8d01af6e00fbf420416ca06b8e;p=thirdparty%2Fbugzilla.git Bug 302723: Deleting a user account when this user is the assignee or QA contact of a bug should reassign this bug to the default ones - Patch by Frédéric Buclin r=wurblzap a=justdave --- diff --git a/editusers.cgi b/editusers.cgi index 18005fd940..5ccd1d06f2 100755 --- a/editusers.cgi +++ b/editusers.cgi @@ -404,14 +404,14 @@ if ($action eq 'search') { $vars->{'editcomponents'} = UserInGroup('editcomponents'); # Find other cross references. - $vars->{'bugs'} = $dbh->selectrow_array( + $vars->{'assignee_or_qa'} = $dbh->selectrow_array( qq{SELECT COUNT(*) FROM bugs - WHERE assigned_to = ? OR - qa_contact = ? OR - reporter = ? - }, - undef, ($otherUserID, $otherUserID, $otherUserID)); + WHERE assigned_to = ? OR qa_contact = ?}, + undef, ($otherUserID, $otherUserID)); + $vars->{'reporter'} = $dbh->selectrow_array( + 'SELECT COUNT(*) FROM bugs WHERE reporter = ?', + undef, $otherUserID); $vars->{'cc'} = $dbh->selectrow_array( 'SELECT COUNT(*) FROM cc WHERE who = ?', undef, $otherUserID); @@ -487,7 +487,8 @@ if ($action eq 'search') { 'series_data WRITE', 'whine_schedules WRITE', 'whine_queries WRITE', - 'whine_events WRITE'); + 'whine_events WRITE', + 'bugs WRITE'); Param('allowuserdeletion') || ThrowUserError('users_deletion_disabled'); @@ -565,6 +566,37 @@ if ($action eq 'search') { $sth_deleteWhineEvent->execute($id); } + # 3) Bugs + # 3.1) fall back to the default assignee + my $buglist = $dbh->selectall_arrayref( + 'SELECT bug_id, initialowner + FROM bugs + INNER JOIN components ON components.id = bugs.component_id + WHERE assigned_to = ?', undef, $otherUserID); + + my $sth_updateAssignee = $dbh->prepare( + 'UPDATE bugs SET assigned_to = ? WHERE bug_id = ?'); + + foreach my $bug (@$buglist) { + my ($bug_id, $default_assignee) = @$bug; + $sth_updateAssignee->execute($default_assignee, $bug_id); + } + + # 3.2) fall back to the default QA contact + $buglist = $dbh->selectall_arrayref( + 'SELECT bug_id, initialqacontact + FROM bugs + INNER JOIN components ON components.id = bugs.component_id + WHERE qa_contact = ?', undef, $otherUserID); + + my $sth_updateQAcontact = $dbh->prepare( + 'UPDATE bugs SET qa_contact = ? WHERE bug_id = ?'); + + foreach my $bug (@$buglist) { + my ($bug_id, $default_qa_contact) = @$bug; + $sth_updateQAcontact->execute($default_qa_contact, $bug_id); + } + # Finally, remove the user account itself. $dbh->do('DELETE FROM profiles WHERE userid = ?', undef, $otherUserID); diff --git a/template/en/default/admin/users/confirm-delete.html.tmpl b/template/en/default/admin/users/confirm-delete.html.tmpl index 205f96b151..e8bfe8609d 100644 --- a/template/en/default/admin/users/confirm-delete.html.tmpl +++ b/template/en/default/admin/users/confirm-delete.html.tmpl @@ -21,7 +21,9 @@ # editusers: is viewing user member of editusers? # editcomponents: is viewing user member of editcomponents? # otheruser: Bugzilla::User object of the viewed user. - # bugs: number of bugs the viewed user has a role in + # reporter: number of bugs reported by the user + # assignee_or_qa: number of bugs the user is either the assignee + # or the QA contact # bug_activity: number of bugs the viewed user has activity # entries on # cc number of bugs the viewed user is cc list member @@ -129,24 +131,22 @@

Confirmation

- [% IF bugs || bug_activity || cc || flags.requestee || flags.setter || - longdescs || namedqueries || profiles_activity || series || votes || - watch.watched || watch.watcher || whine_events || whine_schedules %] + [% IF reporter || assignee_or_qa || bug_activity || cc || flags.requestee || + flags.setter || longdescs || namedqueries || profiles_activity || series || + votes || watch.watched || watch.watcher || whine_events || whine_schedules %]