Commit 14b373c1 authored by mkanat%bugzilla.org's avatar mkanat%bugzilla.org

Bug 482584: Add a parameter to hide the "See Also" field

Patch by Max Kanat-Alexander <mkanat@bugzilla.org> r=LpSolit, r=dkl, a=LpSolit
parent 27d919b3
...@@ -84,6 +84,12 @@ sub get_param_list { ...@@ -84,6 +84,12 @@ sub get_param_list {
}, },
{ {
name => 'use_see_also',
type => 'b',
default => 1
},
{
name => 'defaultpriority', name => 'defaultpriority',
type => 's', type => 's',
choices => \@legal_priorities, choices => \@legal_priorities,
......
...@@ -42,6 +42,12 @@ ...@@ -42,6 +42,12 @@
usebugaliases => "Do you wish to use $terms.bug aliases, which allow you to assign " _ usebugaliases => "Do you wish to use $terms.bug aliases, which allow you to assign " _
"$terms.bugs an easy-to-remember name by which you can refer to them?", "$terms.bugs an easy-to-remember name by which you can refer to them?",
use_see_also =>
"Do you wish to use the See Also field? It allows you refer to"
_ " $terms.bugs in other installations. Even if you disable this field,"
_ " $terms.bug relationships (URLs) already set on $terms.bugs will"
_ " still appear and can be removed.",
defaultpriority => "This is the priority that newly entered $terms.bugs are set to.", defaultpriority => "This is the priority that newly entered $terms.bugs are set to.",
defaultseverity => "This is the severity that newly entered $terms.bugs are set to.", defaultseverity => "This is the severity that newly entered $terms.bugs are set to.",
...@@ -57,4 +63,4 @@ ...@@ -57,4 +63,4 @@
"You can leave this empty: " _ "You can leave this empty: " _
"$terms.Bugzilla will then use the operating system that the browser " _ "$terms.Bugzilla will then use the operating system that the browser " _
"reports to be running on as the default." } "reports to be running on as the default." }
%] %]
\ No newline at end of file
...@@ -908,13 +908,15 @@ ...@@ -908,13 +908,15 @@
[%# Block for See Also #%] [%# Block for See Also #%]
[%############################################################################%] [%############################################################################%]
[% BLOCK section_see_also %] [% BLOCK section_see_also %]
<tr> [% IF Param('use_see_also') || bug.see_also.size %]
[% INCLUDE bug/field.html.tmpl <tr>
field = bug_fields.see_also [% INCLUDE bug/field.html.tmpl
value = bug.see_also field = bug_fields.see_also
editable = bug.check_can_change_field('see_also', 0, 1) value = bug.see_also
%] editable = bug.check_can_change_field('see_also', 0, 1)
</tr> %]
</tr>
[% END %]
[% END %] [% END %]
[%############################################################################%] [%############################################################################%]
......
...@@ -174,18 +174,22 @@ ...@@ -174,18 +174,22 @@
[% FOREACH url = value %] [% FOREACH url = value %]
<li> <li>
<a href="[% url FILTER html %]">[% url FILTER html %]</a> <a href="[% url FILTER html %]">[% url FILTER html %]</a>
<label><input type="checkbox" value="[% url FILTER html %]" [% IF editable %]
name="remove_[% field.name FILTER html %]"> <label><input type="checkbox" value="[% url FILTER html %]"
Remove</label> name="remove_[% field.name FILTER html %]">
Remove</label>
[% END %]
</li> </li>
[% END %] [% END %]
[% '</ul>' IF value.size %] [% '</ul>' IF value.size %]
<label for="[% field.name FILTER html %]"> [% IF editable && Param('use_see_also') %]
<strong>Add [% terms.Bug %] URLs:</strong> <label for="[% field.name FILTER html %]">
</label><br> <strong>Add [% terms.Bug %] URLs:</strong>
<input type="text" id="[% field.name FILTER html %]" </label><br>
class="text_input" name="[% field.name FILTER html %]" size="40"> <input type="text" id="[% field.name FILTER html %]" size="40"
class="text_input" name="[% field.name FILTER html %]">
[% END %]
[% END %] [% END %]
[% ELSIF field.type == constants.FIELD_TYPE_TEXTAREA %] [% ELSIF field.type == constants.FIELD_TYPE_TEXTAREA %]
<div class="uneditable_textarea">[% value FILTER wrap_comment(60) <div class="uneditable_textarea">[% value FILTER wrap_comment(60)
......
...@@ -315,16 +315,18 @@ When searching for [% terms.bugs %] that have been resolved or ...@@ -315,16 +315,18 @@ When searching for [% terms.bugs %] that have been resolved or
verified, remember to set the status field appropriately. verified, remember to set the status field appropriately.
</p> </p>
<h2><a name="see_also"></a>See Also</h2> [% IF Param("use_see_also") %]
<h2><a name="see_also"></a>See Also</h2>
<p>This allows you to refer to [% terms.bugs %] in other installations.
You can enter a URL to a [%+ terms.bug %] in the "Add [% terms.Bug %] URLs" <p>This allows you to refer to [% terms.bugs %] in other installations.
field to note that that [% terms.bug %] is related to this one. You can You can enter a URL to a [%+ terms.bug %] in the "Add [% terms.Bug %] URLs"
enter multiple URLs at once by separating them with a comma.</p> field to note that that [% terms.bug %] is related to this one. You can
enter multiple URLs at once by separating them with a comma.</p>
<p>You should normally use this field to refer to [% terms.bugs %] in
<em>other</em> installations. For [% terms.bugs %] in this <p>You should normally use this field to refer to [% terms.bugs %] in
installation, it is better to use the "Depends On" and "Blocks" <em>other</em> installations. For [% terms.bugs %] in this
fields.</p> installation, it is better to use the "Depends On" and "Blocks"
fields.</p>
[% END %]
[% INCLUDE global/footer.html.tmpl %] [% INCLUDE global/footer.html.tmpl %]
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment