From: jouni%heikniemi.net <>
Date: Tue, 20 Jul 2004 12:06:18 +0000 (+0000)
Subject: Bug 86051: Enable changing product name case.
X-Git-Tag: bugzilla-2.19.1~149
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0ed8d17388e304fe1d32f78d71a88597929236e5;p=thirdparty%2Fbugzilla.git

Bug 86051: Enable changing product name case.
Patch by GavinS <bugzilla@chimpychompy.org>
r=jouni, a=justdave
---

diff --git a/editproducts.cgi b/editproducts.cgi
index c539465af0..17bcfd7038 100755
--- a/editproducts.cgi
+++ b/editproducts.cgi
@@ -306,13 +306,27 @@ if ($action eq 'new') {
         PutTrailer($localtrailer);
         exit;
     }
-    if (TestProduct($product)) {
+
+    my $existing_product = TestProduct($product);
+
+    # Check for exact case sensitive match:
+    if ($existing_product eq $product) {
         print "The product '$product' already exists. Please press\n";
         print "<b>Back</b> and try again.\n";
         PutTrailer($localtrailer);
         exit;
     }
 
+    # Next check for a case-insensitive match:
+    if (lc($existing_product) eq lc($product)) {
+        print "The new product '$product' differs from existing product ";
+        print "'$existing_product' only in case. Please press\n";
+        print "<b>Back</b> and try again.\n";
+        PutTrailer($localtrailer);
+        exit;
+    }
+
+
     my $version = trim($::FORM{version} || '');
 
     if ($version eq '') {
@@ -1190,7 +1204,9 @@ if ($action eq 'update') {
             PutTrailer($localtrailer);
             exit;
         }
-        if (TestProduct($product)) {
+
+        if (lc($product) ne lc($productold) &&
+            TestProduct($product)) {
             print "Sorry, product name '$product' is already in use.";
             SendSQL("UNLOCK TABLES");
             PutTrailer($localtrailer);