summaryrefslogtreecommitdiffstats
path: root/template/en/default
diff options
context:
space:
mode:
authorlpsolit%gmail.com <>2005-10-20 07:21:04 +0200
committerlpsolit%gmail.com <>2005-10-20 07:21:04 +0200
commitc9e867f7b1b01aff688df7fcb7c37f48cf8a39ca (patch)
treeab8d1d386a101cafbc4960a0eabf50d4dce82cdf /template/en/default
parentb2966165174d850b13be0e25eba6e9053142ed00 (diff)
downloadbugzilla-c9e867f7b1b01aff688df7fcb7c37f48cf8a39ca.tar.gz
bugzilla-c9e867f7b1b01aff688df7fcb7c37f48cf8a39ca.tar.xz
Bug 298688: Forbid the deletion of values used by default for the priority, severity, platform and OS and update data/params when editing them - Patch by Frédéric Buclin <LpSolit@gmail.com> r=GavinS a=justdave
Diffstat (limited to 'template/en/default')
-rw-r--r--template/en/default/admin/fieldvalues/confirm-delete.html.tmpl80
-rw-r--r--template/en/default/admin/fieldvalues/list.html.tmpl22
-rw-r--r--template/en/default/admin/fieldvalues/updated.html.tmpl6
-rw-r--r--template/en/default/global/user-error.html.tmpl9
4 files changed, 77 insertions, 40 deletions
diff --git a/template/en/default/admin/fieldvalues/confirm-delete.html.tmpl b/template/en/default/admin/fieldvalues/confirm-delete.html.tmpl
index d1a016d8d..31deb547f 100644
--- a/template/en/default/admin/fieldvalues/confirm-delete.html.tmpl
+++ b/template/en/default/admin/fieldvalues/confirm-delete.html.tmpl
@@ -20,6 +20,8 @@
# value_count: number; The number of values left for this field, including
# this value.
# field: string; The name of the field.
+ # param_name: string; The name of the parameter (defaultxxx) associated
+ # with the field.
#%]
[% title = BLOCK %]Delete Value '[% value FILTER html %]' from the
@@ -27,6 +29,7 @@
[% PROCESS global/header.html.tmpl
title = title
+ style_urls = ['skins/standard/admin.css']
%]
<table border="1" cellpadding="4" cellspacing="0">
@@ -57,49 +60,58 @@
</table>
<h2>Confirmation</h2>
-
-[% IF bug_count %]
- Sorry, there
+[% IF (Param(param_name) == value) || bug_count || (value_count == 1) %]
- [% IF bug_count > 1 %]
- are [% bug_count FILTER html %] [%+ terms.bugs %]
- [% ELSE %]
- is [% bug_count FILTER html %] [%+ terms.bug %]
- [% END %]
+ <p>Sorry, but the '[% value FILTER html %]' value cannot be deleted
+ from the '[% field FILTER html %]' field for the following reason(s):</p>
- that still have this field value. You must change the field value on
- <a title="List of [% terms.bugs %] where '[% field FILTER html %]' is '[% value FILTER html %]'"
- href="buglist.cgi?[% field FILTER url_quote %]=[%- value FILTER url_quote %]">
- [% IF bug_count > 1 %]
- those [% terms.bugs %]
- [% ELSE %]
- that [% terms.bug %]
+ <ul class="warningmessages">
+ [% IF Param(param_name) == value %]
+ <li>'[% value FILTER html %]' is the default value for
+ the '[% field FILTER html %]' field.
+ [% IF user.groups.tweakparams %]
+ You first have to <a href="editparams.cgi?section=bugfields#
+ [%- param_name FILTER url_quote %]">change the default value</a> for
+ this field before you can delete this value.
+ [% END %]
[% END %]
- </a>
- to another value before you can delete this value.
-
-[% END %]
-[% IF value_count == 1 %]
-
- <p>Sorry, but '[% value FILTER html %]' is the last value for
- '[%- field FILTER html %]', and so it can not be deleted.</p>
+ [% IF bug_count %]
+ <li>There
+ [% IF bug_count > 1 %]
+ are [% bug_count FILTER html %] [%+ terms.bugs %]
+ [% ELSE %]
+ is 1 [% terms.bug %]
+ [% END %]
+ with this field value. You must change the field value on
+ <a title="List of [% terms.bugs %] where '[% field FILTER html %]' is '[% value FILTER html %]'"
+ href="buglist.cgi?[% field FILTER url_quote %]=[% value FILTER url_quote %]">
+ [% IF bug_count > 1 %]
+ those [% terms.bugs %]
+ [% ELSE %]
+ that [% terms.bug %]
+ [% END %]
+ </a>
+ to another value before you can delete this value.
+ [% END %]
- [% ELSE %]
+ [% IF value_count == 1 %]
+ <li>'[% value FILTER html %]' is the last value for
+ '[%- field FILTER html %]', and so it can not be deleted.
+ [% END %]
+ </ul>
- [% IF bug_count == 0 %]
+[% ELSE %]
- <p>Do you really want to delete this value?<p>
-
- <form method="post" action="editvalues.cgi">
- <input type="submit" value="Yes, delete">
- <input type="hidden" name="action" value="delete">
- <input type="hidden" name="field" value="[% field FILTER html %]">
- <input type="hidden" name="value" value="[% value FILTER html %]">
- </form>
+ <p>Do you really want to delete this value?</p>
- [% END %]
+ <form method="post" action="editvalues.cgi">
+ <input type="submit" value="Yes, delete">
+ <input type="hidden" name="action" value="delete">
+ <input type="hidden" name="field" value="[% field FILTER html %]">
+ <input type="hidden" name="value" value="[% value FILTER html %]">
+ </form>
[% END %]
diff --git a/template/en/default/admin/fieldvalues/list.html.tmpl b/template/en/default/admin/fieldvalues/list.html.tmpl
index b661653de..d25adf2b1 100644
--- a/template/en/default/admin/fieldvalues/list.html.tmpl
+++ b/template/en/default/admin/fieldvalues/list.html.tmpl
@@ -50,18 +50,28 @@
name => "sortkey"
heading => "Sortkey"
},
- ]
+ {
+ name => "action"
+ heading => "Action"
+ content => "Delete"
+ contentlink => delete_contentlink
+ } ]
%]
-[% columns.push({
- heading => "Action"
- content => "Delete"
- contentlink => delete_contentlink
- }) %]
+[% overrides.action = [ {
+ match_value => "$default"
+ match_field => 'name'
+ override_content => 1
+ content => "(Default value)"
+ override_contentlink => 1
+ contentlink => undef
+ } ]
+%]
[% PROCESS admin/table.html.tmpl
columns = columns
data = values
+ overrides = overrides
%]
<p>
diff --git a/template/en/default/admin/fieldvalues/updated.html.tmpl b/template/en/default/admin/fieldvalues/updated.html.tmpl
index fc9fcafbc..c7f676e67 100644
--- a/template/en/default/admin/fieldvalues/updated.html.tmpl
+++ b/template/en/default/admin/fieldvalues/updated.html.tmpl
@@ -24,6 +24,8 @@
# value & updated_value: the name of the field value
# sortkey & updated_sortkey: the field value sortkey
# field: string; the field that the value belongs to
+ # default_value_updated: boolean; whether the default value for
+ # this field has been updated
#%]
[% title = BLOCK %]Updating Value '[% value FILTER html %]' of the
@@ -34,6 +36,10 @@
[% IF updated_value %]
<p>Updated field value to: '[% value FILTER html %]'.</p>
+ [% IF default_value_updated %]
+ <p>Note that this value is the default for this field.
+ All references to the default value will now point to this new value.</p>
+ [% END %]
[% END %]
[% IF updated_sortkey %]
diff --git a/template/en/default/global/user-error.html.tmpl b/template/en/default/global/user-error.html.tmpl
index 09f3d6e8a..334f1c54e 100644
--- a/template/en/default/global/user-error.html.tmpl
+++ b/template/en/default/global/user-error.html.tmpl
@@ -384,6 +384,15 @@
The value '[% value FILTER html %]' does not exist for
the '[% field FILTER html %]' field.
+ [% ELSIF error == "fieldvalue_is_default" %]
+ [% title = "Specified Field Value Is Default" %]
+ '[% value FILTER html %]' is the default value for
+ the '[% field FILTER html %]' field and cannot be deleted.
+ [% IF user.groups.tweakparams %]
+ You have to <a href="editparams.cgi?section=bugfields#
+ [%- param_name FILTER url_quote %]">change</a> the default value first.
+ [% END %]
+
[% ELSIF error == "fieldvalue_name_too_long" %]
[% title = "Field Value Is Too Long" %]
The value of a field is limited to 60 characters.