Bug 395924: Deleting a user account may delete other user's whine - Patch by…

editusers.cgi

@@ -551,9 +551,6 @@ if ($action eq 'search') {
     }
 
     # 2) Whines
-    my $sth_whineidFromSchedules = $dbh->prepare(
-           qq{SELECT eventid FROM whine_schedules
-              WHERE mailto = ? AND mailto_type = ?});
     my $sth_whineidFromEvents = $dbh->prepare(
            'SELECT id FROM whine_events WHERE owner_userid = ?');
     my $sth_deleteWhineEvent = $dbh->prepare(
@@ -563,12 +560,8 @@ if ($action eq 'search') {
     my $sth_deleteWhineSchedule = $dbh->prepare(
            'DELETE FROM whine_schedules WHERE eventid = ?');
 
-    $sth_whineidFromSchedules->execute($otherUserID, MAILTO_USER);
-    while ($id = $sth_whineidFromSchedules->fetchrow_array()) {
-        $sth_deleteWhineQuery->execute($id);
-        $sth_deleteWhineSchedule->execute($id);
-        $sth_deleteWhineEvent->execute($id);
-    }
+    $dbh->do('DELETE FROM whine_schedules WHERE mailto = ? AND mailto_type = ?',
+             undef, ($otherUserID, MAILTO_USER));
 
     $sth_whineidFromEvents->execute($otherUserID);
     while ($id = $sth_whineidFromEvents->fetchrow_array()) {

template/en/default/admin/users/confirm-delete.html.tmpl

@@ -372,32 +372,32 @@
             will cease along with the deletion of the user account.
           </li>
         [% END %]
-        [% IF whine_events || whine_schedules %]
-          <li>
-            [% otheruser.login FILTER html %]
         [% IF whine_events %]
-              has scheduled
+          <li>
+            [% otheruser.login FILTER html %] has scheduled
             [% IF whine_events == 1 %]
               a whine
             [% ELSE %]
               [%+ whine_events %] whines
+            [% END %].
+            [% IF whine_events == 1 %]
+              This whine
+            [% ELSE %]
+              These whines
             [% END %]
+            will be deleted along with the user account.
+          </li>
         [% END %]
         [% IF whine_schedules %]
-              [%+ 'and' IF whine_events %]
-              is on the receiving end of
+          <li>
+            [% otheruser.login FILTER html %] is on the receiving end of
             [% IF whine_schedules == 1 %]
               a whine
             [% ELSE %]
               [%+ whine_schedules %] whines
-              [% END %]
             [% END %].
-            [% IF whine_events + whine_schedules == 1 %]
-              This whine
-            [% ELSE %]
-              These whines
-            [% END %]
-            will be deleted along with the user account.
+            The corresponding schedules will be deleted along with the user account,
+            but the whines themselves will be left unaltered.
           </li>
         [% END %]
       </ul>