From 5eee338175a2f2bf78290ad4154639a2972077ee Mon Sep 17 00:00:00 2001 From: "terry%mozilla.org" <> Date: Sat, 15 Jan 2000 06:35:24 +0000 Subject: Patch by Ramon Felciano , with many tweaks by me. Added a footer to every page. Add some options to do things like display checkboxes instead of scrolling lists, and a new formatting for email diffs, and show list items capitalized instead of all upper case. --- process_bug.cgi | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) (limited to 'process_bug.cgi') diff --git a/process_bug.cgi b/process_bug.cgi index c1c7e63a4..4d053745d 100755 --- a/process_bug.cgi +++ b/process_bug.cgi @@ -97,6 +97,7 @@ if ($::FORM{'product'} ne $::dontchange) { print "\n"; print "\n"; print "Cancel all this and go to the query page.\n"; + PutFooter(); exit; } } @@ -196,6 +197,7 @@ sub CheckonComment( $ ) { "

" . "Please press Back and give some words " . "on the reason of the your change.\n" ); + PutFooter(); exit( 0 ); } else { $ret = 0; @@ -294,6 +296,7 @@ SWITCH: for ($::FORM{'knob'}) { trim($::FORM{'assigned_to'}) eq "") { print "You cannot reassign to a bug to noone. Unless you intentionally cleared out the \"Reassign bug to\" field, "; print Param("browserbugmessage"); + PutFooter(); exit 0; } } @@ -305,11 +308,13 @@ SWITCH: for ($::FORM{'knob'}) { if ($::FORM{'product'} eq $::dontchange) { print "You must specify a product to help determine the new\n"; print "owner of these bugs.\n"; + PutFooter(); exit 0 } if ($::FORM{'component'} eq $::dontchange) { print "You must specify a component whose owner should get\n"; print "assigned these bugs.\n"; + PutFooter(); exit 0 } ChangeStatus('NEW'); @@ -344,11 +349,13 @@ SWITCH: for ($::FORM{'knob'}) { if ($num !~ /^[0-9]*$/) { print "You must specify a bug number of which this bug is a\n"; print "duplicate. The bug has not been changed.\n"; + PutFooter(); exit; } if (defined($::FORM{'id'}) && $::FORM{'dup_id'} == $::FORM{'id'}) { print "Nice try, $::FORM{'who'}. But it doesn't really make sense to mark a\n"; print "bug as a duplicate of itself, does it?\n"; + PutFooter(); exit; } AppendComment($::FORM{'dup_id'}, $::FORM{'who'}, "*** Bug $::FORM{'id'} has been marked as a duplicate of this bug. ***"); @@ -365,6 +372,7 @@ SWITCH: for ($::FORM{'knob'}) { }; # default print "Unknown action $::FORM{'knob'}!\n"; + PutFooter(); exit; } @@ -372,6 +380,7 @@ SWITCH: for ($::FORM{'knob'}) { if ($#idlist < 0) { print "You apparently didn't choose any bugs to modify.\n"; print "

Click Back and try again.\n"; + PutFooter(); exit; } @@ -388,6 +397,7 @@ if ($::FORM{'keywords'}) { print "

The legal keyword names are "; print "listed here.\n"; print "

Please click the Back button and try again.\n"; + PutFooter(); exit; } if (!$keywordseen{$i}) { @@ -403,7 +413,8 @@ if ($::comma eq "" && 0 == @keywordlist && $keywordaction ne "makeexact") { if (!defined $::FORM{'comment'} || $::FORM{'comment'} =~ /^\s*$/) { print "Um, you apparently did not change anything on the selected\n"; print "bugs.

Click Back and try again.\n"; - exit + PutFooter(); + exit; } } @@ -501,7 +512,7 @@ The changes made were: print ", except for the changes to the description"; } print qq{.\n

  • Throw away my changes, and go revisit bug $id\n}; - navigation_header(); + PutFooter(); exit; } @@ -523,6 +534,7 @@ The changes made were: if ($comp ne $i) { print "

    $i is not a legal bug number

    \n"; print "

    Click Back and try again.\n"; + PutFooter(); exit; } if (!exists $seen{$i}) { @@ -541,6 +553,7 @@ The changes made were: print "The change you are making to dependencies\n"; print "has caused a circular dependency chain.\n"; print "

    Click Back and try again.\n"; + PutFooter(); exit; } if (!exists $seen{$t}) { @@ -717,4 +730,5 @@ if (defined $::next_bug) { do "bug_form.pl"; } else { navigation_header(); + PutFooter(); } -- cgit v1.2.3-24-g4f1b