From b6004bbf18e311b9d9856d1c674092406454c6b0 Mon Sep 17 00:00:00 2001
From: "lpsolit%gmail.com" <>
Date: Wed, 28 Sep 2005 04:47:32 +0000
Subject: Bug 308318: Can create a product without description but cannot
 remove an existing description - Patch by Frédéric Buclin <LpSolit@gmail.com>
 r=wicked a=justdave
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 editproducts.cgi                                | 6 ++++++
 template/en/default/global/user-error.html.tmpl | 6 +++++-
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/editproducts.cgi b/editproducts.cgi
index 54f1de190..82ce7433d 100755
--- a/editproducts.cgi
+++ b/editproducts.cgi
@@ -303,6 +303,12 @@ if ($action eq 'new') {
     }
 
     my $description  = trim($cgi->param('description')  || '');
+
+    if ($description eq '') {
+        ThrowUserError('product_must_have_description',
+                       {'product' => $product});
+    }
+
     my $milestoneurl = trim($cgi->param('milestoneurl') || '');
     my $disallownew = 0;
     $disallownew = 1 if $cgi->param('disallownew');
diff --git a/template/en/default/global/user-error.html.tmpl b/template/en/default/global/user-error.html.tmpl
index 8288a309d..f23220d29 100644
--- a/template/en/default/global/user-error.html.tmpl
+++ b/template/en/default/global/user-error.html.tmpl
@@ -1015,8 +1015,12 @@
     You must reassign those [% terms.bugs %] to another product before you
     can delete this one.
 
+  [% ELSIF error == "product_must_have_description" %]
+    [% title = "Product needs Description" %]
+    You must enter a description for product '[% product FILTER html %]'.
+
   [% ELSIF error == "product_must_have_version" %]
-    [% title = "Product needs version" %]
+    [% title = "Product needs Version" %]
     You must enter a version for product '[% product FILTER html %]'.
 
   [% ELSIF error == "product_not_specified" %]
-- 
cgit v1.2.3-24-g4f1b