Re-fixing bug 30824 the correct way (which also fixes bug 46753)

24870261 · dave%intrec.com · 52aae0f4 · 24870261
Commit 24870261 authored Aug 13, 2000 by dave%intrec.com
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 3 deletions

process_bug.cgi process_bug.cgi +2 -3

No files found.
--- a/process_bug.cgi
+++ b/process_bug.cgi
@@ -898,12 +898,11 @@ The changes made were:
        }
        if ($old ne $new) {
            if ($col eq 'assigned_to' || $col eq 'qa_contact') {
-                $old = DBID_to_name($old) if $old != 0;
-                $new = DBID_to_name($new) if $new != 0;
+                $old = ($old) ? DBID_to_name($old) : "";
+                $new = ($new) ? DBID_to_name($new) : "";
                $origCcString .= ",$old"; # make sure to send mail to people
                                          # if they are going to no longer get
                                          # updates about this bug.
-                $old = "" if $old == 0;
            }
            if ($col eq 'product') {
                RemoveVotes($id, 0,