From: bbaetz%acm.org <>
Date: Sun, 16 Mar 2003 14:19:34 +0000 (+0000)
Subject: Bug 197180 - long component name not flagged as error
X-Git-Tag: bugzilla-2.16.3~17
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cb16021208623e65e6f68185f85523e8bd790585;p=thirdparty%2Fbugzilla.git

Bug 197180 - long component name not flagged as error
Because of a mismatch between the size of bugs.component and
components.program, this caused silent failures when creating/moving bugs
in that component.
r/a=justdave
---

diff --git a/editcomponents.cgi b/editcomponents.cgi
index 4f76593df6..1dc89925f0 100755
--- a/editcomponents.cgi
+++ b/editcomponents.cgi
@@ -123,7 +123,7 @@ sub EmitFormElements ($$$$$)
                                              $initialqacontactid ? DBID_to_name ($initialqacontactid) : '');
 
     print "  <TH ALIGN=\"right\">Component:</TH>\n";
-    print "  <TD><INPUT SIZE=64 MAXLENGTH=255 NAME=\"component\" VALUE=\"" .
+    print "  <TD><INPUT SIZE=64 MAXLENGTH=50 NAME=\"component\" VALUE=\"" .
         value_quote($component) . "\">\n";
     print "      <INPUT TYPE=HIDDEN NAME=\"product\" VALUE=\"" .
         value_quote($product) . "\"></TD>\n";
@@ -385,6 +385,11 @@ if ($action eq 'new') {
         PutTrailer($localtrailer);
         exit;
     }
+    if (length($component) > 50) {
+        print "Sorry, the name of a component is limited to 50 characters.";
+        PutTrailer($localtrailer);
+        exit;
+    }
 
     my $description = trim($::FORM{description} || '');
 
@@ -709,6 +714,12 @@ if ($action eq 'update') {
 
     CheckComponent($product,$componentold);
 
+    if (length($component) > 50) {
+        print "Sorry, the name of a component is limited to 50 characters.";
+        PutTrailer($localtrailer);
+        exit;
+    }
+
     # Note that the order of this tests is important. If you change
     # them, be sure to test for WHERE='$component' or WHERE='$componentold'