Bug 312195: Add an index to all parameters to editparams - Patch by Frédéric…

editparams.cgi

@@ -65,7 +65,7 @@ foreach my $panel (@parampanels) {
 
 $vars->{panels} = \@panels;
 
-if ($action eq "save") {
+if ($action eq 'save' && $current_module) {
     my @changes = ();
     my @module_param_list = "Bugzilla::Config::${current_module}"->get_param_list();
 

skins/standard/params.css

@@ -35,6 +35,11 @@ td.selected_section {
   background-color: white;
 }
 
+td.index {
+  color: black;
+  background-color: #edf;
+}
+
 dt {
   font-weight: bold;
 }

template/en/default/admin/params/common.html.tmpl

@@ -24,7 +24,7 @@
 
 <dl>
   [% FOREACH param = panel.param_list %]
-    <dt>[% param.name FILTER html %]</dt>
+    <dt><a name="[% param.name FILTER html %]">[% param.name FILTER html %]</a></dt>
     <dd>[% panel.param_descs.${param.name} FILTER none %]
       <p>
       [% IF param.type == "t" %]

template/en/default/admin/params/editparams.html.tmpl

@@ -27,7 +27,7 @@
 
 [% PROCESS global/variables.none.tmpl %]
 
-[% curpanel = 0 %]
+[% curpanel = -1 %]
 [% panels = panels.sort('sortkey') %]
 
 [% FOREACH panel = panels %]
@@ -41,9 +41,20 @@
 
 [% current_panel = panels.$curpanel %]
 
-[%# We cannot call header.html.tmpl earlier as we have to know which panel is active first %]
+[%# We cannot call header.html.tmpl earlier as we have to know
+    which panel is active first, in order to get its title %]
+
+[% title = BLOCK %]
+  [% IF curpanel == -1 %]
+    [% terms.Bugzilla FILTER html %] Parameters: Index
+  [% ELSE %]
+    [% terms.Bugzilla FILTER html %] Configuration:
+    [%+ current_panel.title FILTER html %]
+  [% END %]
+[% END %]
+
 [% PROCESS global/header.html.tmpl
-   title = "$terms.Bugzilla Configuration: $current_panel.title"
+   title = title
    message = message
    style_urls = ['skins/standard/params.css']
 %]
@@ -53,6 +64,11 @@
     <td>
       [%# NAVIGATION BAR %]
       <table id="menu">
+        <tr>
+          <td class="index">
+            <a href="editparams.cgi?section=index" title="Show all parameters">Index</a>
+          </td>
+        </tr>
         [% FOREACH panel = panels %]
           <tr>
             [% IF panel.current %]
@@ -70,6 +86,9 @@
       </table>
     </td>
     <td>
+      [% IF curpanel == -1 %]
+        [% PROCESS admin/params/index.html.tmpl panels = panels %]
+      [% ELSE %]
         <p>
           This lets you edit the basic operating parameters of [% terms.Bugzilla %].
           Be careful!<br>
@@ -83,6 +102,7 @@
           <input type="reset" value="Reset form">
           <input type="submit" name="action" value="Save Changes">
         </form>
+      [% END %]
     </td>
   </tr>
 </table>

template/en/default/admin/params/index.html.tmpl

+[%# 1.0@bugzilla.org %]
+[%# The contents of this file are subject to the Mozilla Public
+  # License Version 1.1 (the "License"); you may not use this file
+  # except in compliance with the License. You may obtain a copy of
+  # the License at http://www.mozilla.org/MPL/
+  #
+  # Software distributed under the License is distributed on an "AS
+  # IS" basis, WITHOUT WARRANTY OF ANY KIND, either express or
+  # implied. See the License for the specific language governing
+  # rights and limitations under the License.
+  #
+  # The Original Code is the Bugzilla Bug Tracking System.
+  #
+  # The Initial Developer of the Original Code is Netscape Communications
+  # Corporation. Portions created by Netscape are
+  # Copyright (C) 1998 Netscape Communications Corporation. All
+  # Rights Reserved.
+  #
+  # Contributor(s): Frédéric Buclin <LpSolit@gmail.com>
+  #%]
+
+<p>
+  All parameters are displayed below, per section.
+  If you cannot find one from here, then the parameter does not exist.
+</p>
+
+<div align="center">
+  <table>
+    <tr>
+      <th>Parameter</th>
+      <th>Section</th>
+    </tr>
+    [% FOREACH panel = panels %]
+      [% FOREACH param = panel.param_list.sort('name') %]
+        <tr>
+          <td>[% param.name FILTER html %]</td>
+          <td>
+            <a href="editparams.cgi?section=
+               [%- panel.name FILTER url_quote %]#[% param.name FILTER url_quote %]">
+               [% panel.title FILTER html %]</a>
+          </td>
+        </tr>
+      [% END %]
+      <tr>
+        <td>&nbsp;</td><td>&nbsp;</td>
+      </tr>
+    [% END %]
+  </table>
+</div>