summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormyk%mozilla.org <>2001-09-28 08:49:07 +0200
committermyk%mozilla.org <>2001-09-28 08:49:07 +0200
commitc364f865c1d3d6cdf42aed51f5d6c1b3417a580e (patch)
treee6a36557b94da247e50b629ef20594b61f02f985
parent8d24bc783cc435d39fa3851992086321626d219f (diff)
downloadbugzilla-c364f865c1d3d6cdf42aed51f5d6c1b3417a580e.tar.gz
bugzilla-c364f865c1d3d6cdf42aed51f5d6c1b3417a580e.tar.xz
Fix for bug #53612: The milestone URL appears correctly when deleting a product or a component.
Patch by Jake <jake@acutex.net>. r=gerv@mozilla.org,myk@mozilla.org
-rwxr-xr-xeditcomponents.cgi6
-rwxr-xr-xeditproducts.cgi4
2 files changed, 6 insertions, 4 deletions
diff --git a/editcomponents.cgi b/editcomponents.cgi
index 30f9dd4ea..f48612d75 100755
--- a/editcomponents.cgi
+++ b/editcomponents.cgi
@@ -487,9 +487,9 @@ if ($action eq 'del') {
my $initialowner = $initialownerid ? DBID_to_name ($initialownerid) : "<FONT COLOR=\"red\">missing</FONT>";
my $initialqacontact = $initialqacontactid ? DBID_to_name ($initialqacontactid) : "<FONT COLOR=\"red\">missing</FONT>";
-
+ my $milestonelink = $milestoneurl ? "<A HREF=\"$milestoneurl\">$milestoneurl</A>"
+ : "<FONT COLOR=\"red\">missing</FONT>";
$pdesc ||= "<FONT COLOR=\"red\">missing</FONT>";
- $milestoneurl ||= "<FONT COLOR=\"red\">missing</FONT>";
$disallownew = $disallownew ? 'closed' : 'open';
$cdesc ||= "<FONT COLOR=\"red\">missing</FONT>";
@@ -530,7 +530,7 @@ if ($action eq 'del') {
if (Param('usetargetmilestone')) {
print "</TR><TR>\n";
print " <TD VALIGN=\"top\">Milestone URL:</TD>\n";
- print " <TD VALIGN=\"top\"><A HREF=\"$milestoneurl\">$milestoneurl</A></TD>\n";
+ print " <TD VALIGN=\"top\">$milestonelink</TD>\n";
}
print "</TR><TR>\n";
diff --git a/editproducts.cgi b/editproducts.cgi
index 73fdbf487..f7d479f1d 100755
--- a/editproducts.cgi
+++ b/editproducts.cgi
@@ -418,6 +418,8 @@ if ($action eq 'del') {
FROM products
WHERE product=" . SqlQuote($product));
my ($description, $milestoneurl, $disallownew) = FetchSQLData();
+ my $milestonelink = $milestoneurl ? "<a href=\"$milestoneurl\">$milestoneurl</a>"
+ : "<font color=\"red\">missing</font>";
$description ||= "<FONT COLOR=\"red\">description missing</FONT>";
$disallownew = $disallownew ? 'closed' : 'open';
@@ -437,7 +439,7 @@ if ($action eq 'del') {
if (Param('usetargetmilestone')) {
print "</TR><TR>\n";
print " <TD VALIGN=\"top\">Milestone URL:</TD>\n";
- print " <TD VALIGN=\"top\"><A HREF=\"$milestoneurl\">$milestoneurl</A></TD>\n";
+ print " <TD VALIGN=\"top\">$milestonelink</TD>\n";
}
# Added -JMR, 2/16/00