Re-fix for bug 28458: AddFDef always replaces the fielddefs every time you run…

Re-fix for bug 28458: AddFDef always replaces the fielddefs every time you run checksetup.pl, so the change to them during doeditparams was nullified if you updated. Other recent changes have nullified the reason for changing it in editparams anyway, so just backing that part out. Patch by Jake Steenhagen <jake@acutex.net> r= justdave@syndicomm.com

Re-fix for bug 28458: AddFDef always replaces the fielddefs every time you run…
b6622d0c · justdave%syndicomm.com · 5f4abd29 · b6622d0c · b6622d0c
Commit b6622d0c authored Jun 21, 2001 by justdave%syndicomm.com
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 7 deletions

checksetup.pl checksetup.pl +1 -1

doeditparams.cgi doeditparams.cgi +0 -6

No files found.
--- a/checksetup.pl
+++ b/checksetup.pl
@@ -1097,7 +1097,7 @@ AddFDef("component", "Component", 1);
 AddFDef("assigned_to", "AssignedTo", 1);
 AddFDef("reporter", "ReportedBy", 1);
 AddFDef("votes", "Votes", 0);
-AddFDef("qa_contact", "QAContact", 0);
+AddFDef("qa_contact", "QAContact", 1);
 AddFDef("cc", "CC", 1);
 AddFDef("dependson", "BugsThisDependsOn", 0);
 AddFDef("blocked", "OtherBugsDependingOnThis", 0);

--- a/doeditparams.cgi
+++ b/doeditparams.cgi
@@ -68,12 +68,6 @@ foreach my $i (@::param_list) {
 #      print "Old: '" . url_quote(Param($i)) . "'<BR>\n";
 #      print "New: '" . url_quote($::FORM{$i}) . "'<BR>\n";
        $::param{$i} = $::FORM{$i};
-        # If the useqacontact Param is changing, update the mailheader
-        if ($i eq 'useqacontact') {
-            print "&nbsp;&nbsp;&nbsp;- Updating mailhead information<br>\n";
-            SendSQL("UPDATE fielddefs SET mailhead = " . SqlQuote($::param{$i}) .
-                    "WHERE name = 'qa_contact'");
-        }
    }
 }