From 589632e9d5d35c25c932aafa164feb92c26f5e3d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Buclin?= Date: Wed, 17 Aug 2011 13:05:31 +0200 Subject: Bug 661476: sanitycheck.pl should check if all products have components defined. Also, creating a new product from the web UI asks you to create a component too. r=glob a=LpSolit --- .../default/admin/components/edit-common.html.tmpl | 18 ++++++----- .../en/default/admin/products/create.html.tmpl | 17 ++++++++-- .../default/admin/products/edit-common.html.tmpl | 36 +++++++++++----------- .../default/admin/sanitycheck/messages.html.tmpl | 7 +++++ template/en/default/global/messages.html.tmpl | 4 +-- template/en/default/global/user-error.html.tmpl | 10 ++++++ 6 files changed, 61 insertions(+), 31 deletions(-) (limited to 'template') diff --git a/template/en/default/admin/components/edit-common.html.tmpl b/template/en/default/admin/components/edit-common.html.tmpl index 069b56cfd..6f65095af 100644 --- a/template/en/default/admin/components/edit-common.html.tmpl +++ b/template/en/default/admin/components/edit-common.html.tmpl @@ -22,16 +22,20 @@ # comp: object; Bugzilla::Component object. #%] +[%# When called from the "New Product" page, the component description field + # must have a name different from the product description field. %] +[% DEFAULT desc_name = "description" %] + - Component: + Component: - Component Description: + Component Description: [% INCLUDE global/textarea.html.tmpl - name = 'description' + name = desc_name minrows = 4 cols = 64 wrap = 'virtual' @@ -40,7 +44,7 @@ - + [% INCLUDE global/userselect.html.tmpl name => "initialowner" @@ -52,7 +56,7 @@ [% IF Param('useqacontact') %] - + [% INCLUDE global/userselect.html.tmpl name => "initialqacontact" @@ -65,9 +69,7 @@ [% END %] - - - + [% INCLUDE global/userselect.html.tmpl name => "initialcc" diff --git a/template/en/default/admin/products/create.html.tmpl b/template/en/default/admin/products/create.html.tmpl index 3af81fb23..2b60645ab 100644 --- a/template/en/default/admin/products/create.html.tmpl +++ b/template/en/default/admin/products/create.html.tmpl @@ -25,7 +25,8 @@ [% PROCESS global/header.html.tmpl title = title style_urls = ['skins/standard/admin.css'] - javascript_urls = ['js/util.js'] + javascript_urls = ['js/util.js', 'js/field.js'] + yui = [ 'autocomplete' ] %] [% DEFAULT @@ -42,7 +43,7 @@ Version: - @@ -52,6 +53,18 @@ + + +   + + + + This product must have at least one component. + You will be able to create additional components later: + + + + [% PROCESS "admin/components/edit-common.html.tmpl" desc_name = "comp_desc" %] diff --git a/template/en/default/admin/products/edit-common.html.tmpl b/template/en/default/admin/products/edit-common.html.tmpl index 4812707cd..eac33ea9a 100644 --- a/template/en/default/admin/products/edit-common.html.tmpl +++ b/template/en/default/admin/products/edit-common.html.tmpl @@ -25,7 +25,7 @@ [% IF Param('useclassification') %] - Classification: + Classification: [% classification.name FILTER html %] [% END %] @@ -43,6 +43,23 @@ + + Open for [% terms.bug %] entry: + + + + + + + + + + + [% IF Param('usetargetmilestone') -%] Default milestone: @@ -63,21 +80,4 @@ [% END %] - - Open for [% terms.bug %] entry: - - - - - - - - - - - [% Hook.process('rows') %] diff --git a/template/en/default/admin/sanitycheck/messages.html.tmpl b/template/en/default/admin/sanitycheck/messages.html.tmpl index 88264d820..494a8cdf0 100644 --- a/template/en/default/admin/sanitycheck/messages.html.tmpl +++ b/template/en/default/admin/sanitycheck/messages.html.tmpl @@ -229,6 +229,13 @@ [% ELSIF san_tag == "profile_login_start" %] Checking profile logins. + [% ELSIF san_tag == "product_alert" %] + Product + [%- name FILTER html %] has no components or no versions. + + [% ELSIF san_tag == "product_check_start" %] + Checking products with no components or versions. + [% ELSIF san_tag == "profile_login_alert" %] Bad profile email address, id=[% id FILTER html %], <[% email FILTER html %]>. diff --git a/template/en/default/global/messages.html.tmpl b/template/en/default/global/messages.html.tmpl index 01eb32651..a57449d7c 100644 --- a/template/en/default/global/messages.html.tmpl +++ b/template/en/default/global/messages.html.tmpl @@ -796,9 +796,7 @@ [% ELSIF message_tag == "product_created" %] [% title = "Product Created" %] - The product [% product.name FILTER html %] has been created. You will need to - - add at least one component before anyone can enter [% terms.bugs %] against this product. + The product [% product.name FILTER html %] has been created. [% ELSIF message_tag == "product_deleted" %] [% title = "Product Deleted" %] diff --git a/template/en/default/global/user-error.html.tmpl b/template/en/default/global/user-error.html.tmpl index 52c6156b6..7dac2ee55 100644 --- a/template/en/default/global/user-error.html.tmpl +++ b/template/en/default/global/user-error.html.tmpl @@ -373,6 +373,11 @@ You must reassign those [% terms.bugs %] to another component before you can delete this one. + [% ELSIF error == "component_is_last" %] + [% title = BLOCK %]Last Component in this Product[% END %] + '[% comp.name FILTER html %]' is the last component of the + '[% comp.product.name FILTER html %]' product. You cannot delete it. + [% ELSIF error == "component_name_too_long" %] [% title = "Component Name Is Too Long" %] The name of a component is limited to [% constants.MAX_COMPONENT_SIZE FILTER html %] @@ -1630,6 +1635,11 @@ version! You must reassign those [% terms.bugs %] to another version before you can delete this one. + [% ELSIF error == "version_is_last" %] + [% title = BLOCK %]Last Version in this Product[% END %] + '[% version.name FILTER html %]' is the last version of the + '[% version.product.name FILTER html %]' product. You cannot delete it. + [% ELSIF error == "users_deletion_disabled" %] [% title = "Deletion not activated" %] [% admindocslinks = {'useradmin.html' => 'User administration'} %] -- cgit v1.2.1