Part 2 of bug 182083: add Help for more admin pages - Patch by me, r=Colin

diff --git a/docs/xml/administration.xml b/docs/xml/administration.xml
index 50ffe43f09f8581a9b9df519df3b19d2f1e09d32..cdb48cdbd7e652bb25b5f2069d7aee50f695b395 100644 (file)
--- a/docs/xml/administration.xml
+++ b/docs/xml/administration.xml
@@ -1036,9 +1036,9 @@
      <title>Administering Flags</title>
 
      <para>
-       If you have the <quote>editcomponents</quote> permission, you will
-       have <quote>Edit: ... | Flags | ...</quote> in your page footer.
-       Clicking on that link will bring you to the <quote>Administer 
+       If you have the <quote>editcomponents</quote> permission, you can
+       edit Flag Types from the main administration page. Clicking the
+       <quote>Flags</quote> link will bring you to the <quote>Administer
        Flag Types</quote> page. Here, you can select whether you want 
        to create (or edit) a Bug flag, or an Attachment flag.
      </para>
@@ -1047,6 +1047,15 @@
        just go over it once.
      </para>
 
+     <section id="flags-edit">
+       <title>Editing a Flag</title>
+       <para>
+         To edit a flag's properties, just click on the <quote>Edit</quote>
+         link next to the flag's description. That will take you to the same
+         form as described below (<xref linkend="flags-create"/>).
+       </para>
+     </section>
+
      <section id="flags-create">
        <title>Creating a Flag</title>
        
@@ -1257,15 +1266,6 @@
         </warning>
       </section>
 
-      <section id="flags-edit">
-        <title>Editing a Flag</title>
-        <para>
-          To edit a flag's properties, just click on the <quote>Edit</quote>
-          link next to the flag's description. That will take you to the same
-          form described in the <quote>Creating a Flag</quote> section.
-        </para>
-      </section>
-
     </section> <!-- flags-admin -->
 
     <!-- XXX We should add a "Uses of Flags" section, here, with examples. -->

diff --git a/editclassifications.cgi b/editclassifications.cgi
index 03d3af81cb22d9525fc79c503eb86fcc24f3e365..44a627054acd697fbce1ccdf8513e156dd7422e8 100755 (executable)
--- a/editclassifications.cgi
+++ b/editclassifications.cgi
@@ -39,6 +39,9 @@ sub LoadTemplate {
     my $action = shift;
     my $cgi = Bugzilla->cgi;
     my $template = Bugzilla->template;
+    # There is currently only one section about classifications,
+    # so all pages point to it. Let's define it here.
+    $vars->{'doc_section'} = 'classifications.html';
 
     $action =~ /(\w+)/;
     $action = $1;

diff --git a/editcomponents.cgi b/editcomponents.cgi
index 46b319edd3e8d57f4ac3991b0270d5c5e727edda..2dabaf898bf090440d31ec65e09cacf8d25ead2f 100755 (executable)
--- a/editcomponents.cgi
+++ b/editcomponents.cgi
@@ -37,6 +37,9 @@ use Bugzilla::Token;
 my $cgi = Bugzilla->cgi;
 my $template = Bugzilla->template;
 my $vars = {};
+# There is only one section about components in the documentation,
+# so all actions point to the same page.
+$vars->{'doc_section'} = 'components.html';
 
 #
 # Preliminary checks:

diff --git a/editmilestones.cgi b/editmilestones.cgi
index 0be0f6c7f1eff1e376ca92e81eac5c7986c00445..2682b6fcabcdebeccf8f04d80f19311c4a84eead 100755 (executable)
--- a/editmilestones.cgi
+++ b/editmilestones.cgi
@@ -29,6 +29,9 @@ my $cgi = Bugzilla->cgi;
 my $dbh = Bugzilla->dbh;
 my $template = Bugzilla->template;
 my $vars = {};
+# There is only one section about milestones in the documentation,
+# so all actions point to the same page.
+$vars->{'doc_section'} = 'milestones.html';
 
 #
 # Preliminary checks:

diff --git a/editproducts.cgi b/editproducts.cgi
index 7c6c52f7d3cda65673d3177d19985c56c005bd69..82f0bf99f9fd36e60a34c97f619120eba03fd791 100755 (executable)
--- a/editproducts.cgi
+++ b/editproducts.cgi
@@ -61,6 +61,9 @@ my $dbh = Bugzilla->dbh;
 my $cgi = Bugzilla->cgi;
 my $template = Bugzilla->template;
 my $vars = {};
+# Remove this as soon as the documentation about products has been
+# improved and each action has its own section.
+$vars->{'doc_section'} = 'products.html';
 
 print $cgi->header();
 

diff --git a/editversions.cgi b/editversions.cgi
index ac7b2c7a03d22bb2c06fb79ae94c2bb27c8c29ac..06a58ea03af77e80e3ce6a6158ac0e99e5a6f934 100755 (executable)
--- a/editversions.cgi
+++ b/editversions.cgi
@@ -42,6 +42,9 @@ my $cgi = Bugzilla->cgi;
 my $dbh = Bugzilla->dbh;
 my $template = Bugzilla->template;
 my $vars = {};
+# There is only one section about versions in the documentation,
+# so all actions point to the same page.
+$vars->{'doc_section'} = 'versions.html';
 
 #
 # Preliminary checks:

diff --git a/template/en/default/admin/admin.html.tmpl b/template/en/default/admin/admin.html.tmpl
index 0a1aed28997d70c22e3e396b6409cfd4e712d5e6..dc8859537f14490055ea3835d745efbca3f89f40 100644 (file)
--- a/template/en/default/admin/admin.html.tmpl
+++ b/template/en/default/admin/admin.html.tmpl
@@ -22,6 +22,7 @@
 
 [% PROCESS global/header.html.tmpl title = title
                                    style_urls = ['skins/standard/admin.css']
+                                   doc_section = "administration.html"
 %]
 
 <div>

diff --git a/template/en/default/admin/flag-type/confirm-delete.html.tmpl b/template/en/default/admin/flag-type/confirm-delete.html.tmpl
index 131dc73d951585f57b6570ff3b95c413273ed173..cc6a064a92832c360ea1056e1bb5ad95f975b271 100644 (file)
--- a/template/en/default/admin/flag-type/confirm-delete.html.tmpl
+++ b/template/en/default/admin/flag-type/confirm-delete.html.tmpl
@@ -22,7 +22,10 @@
 
 [% title = BLOCK %]Confirm Deletion of Flag Type '[% flag_type.name FILTER html %]'[% END %]
 
-[% PROCESS global/header.html.tmpl title = title %]
+[% PROCESS global/header.html.tmpl
+  title = title
+  doc_section = "flags-overview.html#flags-delete"
+%]
 
 <p>
    There are [% flag_type.flag_count %] flags of type [% flag_type.name FILTER html %].

diff --git a/template/en/default/admin/flag-type/edit.html.tmpl b/template/en/default/admin/flag-type/edit.html.tmpl
index 609e3db9dd214293af303aaffe0bbda1e3f22737..ebebf50821a0ed73ed6e5783876ad1ece3e170cc 100644 (file)
--- a/template/en/default/admin/flag-type/edit.html.tmpl
+++ b/template/en/default/admin/flag-type/edit.html.tmpl
@@ -33,10 +33,12 @@
   [% typeLabelLowerSingular = BLOCK %]attachment[% END %]
 [% END %]
 
+[% doc_section = "flags-overview.html#flags-create" %]
 [% IF last_action == "copy" %]
   [% title = BLOCK %]Create Flag Type Based on [% type.name FILTER html %][% END %]
 [% ELSIF last_action == "edit" %]
   [% title = BLOCK %]Edit Flag Type [% type.name FILTER html %][% END %]
+  [% doc_section = "flags-overview.html#flags-edit" %]
 [% END %]
 
 [% PROCESS global/header.html.tmpl
@@ -47,6 +49,7 @@
   "
   onload="var f = document.forms[0]; selectProduct(f.product, f.component, null, null, '__Any__');"
   javascript_urls=["js/productform.js"]
+  doc_section = doc_section
 %]
 
 <form method="post" action="editflagtypes.cgi">

diff --git a/template/en/default/admin/flag-type/list.html.tmpl b/template/en/default/admin/flag-type/list.html.tmpl
index 6f9fe4d0d149af514fae4dcb1d8f2b82e702d749..d4bba945a289a8f8f4866db0c31584e89d76bea1 100644 (file)
--- a/template/en/default/admin/flag-type/list.html.tmpl
+++ b/template/en/default/admin/flag-type/list.html.tmpl
@@ -32,6 +32,7 @@
   "
   onload="var f = document.forms[0]; selectProduct(f.product, f.component, null, null, '__All__');"
   javascript_urls=["js/productform.js"]
+  doc_section = "flags-overview.html#flag-types"
 %]
 
 <p>

diff --git a/template/en/default/admin/params/editparams.html.tmpl b/template/en/default/admin/params/editparams.html.tmpl
index ac646f97bf15712cb3dc5b5cce0c2b7bbc5cca21..a35ec0f4ae7ae10c90eff2f4d8a60385cabe1cdb 100644 (file)
--- a/template/en/default/admin/params/editparams.html.tmpl
+++ b/template/en/default/admin/params/editparams.html.tmpl
@@ -57,6 +57,7 @@
    message = message
    style_urls = ['skins/standard/params.css']
    javascript_urls = ['js/params.js']
+   doc_section = "parameters.html"
 %]
 
 <table border="0" width="100%">

diff --git a/template/en/default/admin/sudo.html.tmpl b/template/en/default/admin/sudo.html.tmpl
index 3bdbd9fab3eab13d487e6fdacb165052413d2e9c..4dc70f95ac9697fc598c87d1b80003fea3e6d4d9 100644 (file)
--- a/template/en/default/admin/sudo.html.tmpl
+++ b/template/en/default/admin/sudo.html.tmpl
@@ -23,6 +23,7 @@
 [% PROCESS global/header.html.tmpl
    title = "Begin sudo session"
    style_urls = ['skins/standard/admin.css']
+   doc_section = "useradmin.html#impersonatingusers"
  %]
  
 [% DEFAULT target_login = "" %]

diff --git a/template/en/default/admin/users/confirm-delete.html.tmpl b/template/en/default/admin/users/confirm-delete.html.tmpl
index d6607c397e4ca1ea1dd076797dcad82dbd1d5644..52c98f0b1bea1143deccdd12f76320743f81ab31 100644 (file)
--- a/template/en/default/admin/users/confirm-delete.html.tmpl
+++ b/template/en/default/admin/users/confirm-delete.html.tmpl
@@ -48,6 +48,7 @@
   title = title
   style_urls = ['skins/standard/admin.css',
                 'skins/standard/editusers.css']
+  doc_section = "useradmin.html#user-account-deletion"
 %]
 
 [% PROCESS admin/users/listselectvars.html.tmpl

diff --git a/template/en/default/admin/users/create.html.tmpl b/template/en/default/admin/users/create.html.tmpl
index dd674b723834af1460fba3bdb0339ca7719807db..6fd5b67e79ee374c69365fae76f03d719e83c025 100644 (file)
--- a/template/en/default/admin/users/create.html.tmpl
+++ b/template/en/default/admin/users/create.html.tmpl
@@ -23,6 +23,7 @@
   title = "Add user"
   style_urls = ['skins/standard/editusers.css']
   onload = "document.forms['f'].login.focus()"
+  doc_section = "useradmin.html#createnewusers"
 %]
 
 [% PROCESS admin/users/listselectvars.html.tmpl

diff --git a/template/en/default/admin/users/edit.html.tmpl b/template/en/default/admin/users/edit.html.tmpl
index 37c0a74b2ed3422be50d70061c6a744cda4c5a0d..3efa4b8bf1b2de51616d77e27bf6196628e312e3 100644 (file)
--- a/template/en/default/admin/users/edit.html.tmpl
+++ b/template/en/default/admin/users/edit.html.tmpl
@@ -30,6 +30,7 @@
   title = title
   message = message
   style_urls = ['skins/standard/admin.css', 'skins/standard/editusers.css']
+  doc_section = "useradmin.html#modifyusers"
 %]
 
 [% PROCESS admin/users/listselectvars.html.tmpl

diff --git a/template/en/default/admin/users/list.html.tmpl b/template/en/default/admin/users/list.html.tmpl
index 4ccc1464d71d50b5ffc40932216f11e68246b945..4788e527d31addc8144bc9e97d3dd821c28ee4e7 100644 (file)
--- a/template/en/default/admin/users/list.html.tmpl
+++ b/template/en/default/admin/users/list.html.tmpl
@@ -24,6 +24,7 @@
 [% PROCESS global/header.html.tmpl
   title = "Select user"
   style_urls = ['skins/standard/editusers.css']
+  doc_section = "useradmin.html"
 %]
 
 [% PROCESS admin/users/listselectvars.html.tmpl

diff --git a/template/en/default/admin/users/search.html.tmpl b/template/en/default/admin/users/search.html.tmpl
index ff1419d9818a0a2f5314d23ccd043b2e19689f71..82e0afda71f3ea83cbcdbf957831d1ec901e0343 100644 (file)
--- a/template/en/default/admin/users/search.html.tmpl
+++ b/template/en/default/admin/users/search.html.tmpl
@@ -27,6 +27,7 @@
   title = "Search users"
   style_urls = ['skins/standard/editusers.css']
   onload = "document.forms['f'].matchstr.focus()"
+  doc_section = "useradmin.html#user-account-search"
 %]
 
 [% PROCESS admin/users/listselectvars.html.tmpl