Fix for bug 179876: Labels the "Requestee" field to reduce confusion about its purpose.

r=bbaetz
a=myk

webtools/bugzilla/attachment.cgi

@@ -563,6 +563,7 @@ sub edit
                                                     'attach_id' => $::FORM{'id'} });
   }
   $vars->{'flag_types'} = $flag_types;
+  $vars->{'any_flags_requesteeble'} = grep($_->{'is_requesteeble'}, @$flag_types);
   
   # Define the variables and functions that will be passed to the UI template.
   $vars->{'attachid'} = $::FORM{'id'}; 

webtools/bugzilla/bug_form.pl

@@ -217,6 +217,7 @@ sub show_bug {
                                   'target_type' => 'bug' });
     }
     $vars->{'flag_types'} = $flag_types;
+    $vars->{'any_flags_requesteeble'} = grep($_->{'is_requesteeble'}, @$flag_types);
 
     # The number of types of flags that can be set on attachments to this bug
     # and the number of flags on those attachments.  One of these counts must be

webtools/bugzilla/css/edit_bug.css

@@ -1,3 +1,4 @@
 
 .bz_private { color: darkred ; background : #f3eeee ; }
 
+table#flags th, table#flags td { vertical-align: baseline; text-align: left; }

webtools/bugzilla/template/en/default/attachment/edit.html.tmpl

@@ -33,7 +33,7 @@
     table.attachment_info td { text-align: left; vertical-align: top; }
     #noview { text-align: left; vertical-align: center; }
     
-    table#flags th, table#flags td { font-size: small; vertical-align: baseline; }
+    table#flags th, table#flags td { font-size: small; vertical-align: baseline; text-align: left; }
   "
 %]
 
@@ -173,7 +173,6 @@
           [% END %]
 
         [% IF flag_types.size > 0 %]
-          <b>Flags:</b><br>
           [% PROCESS "flag/list.html.tmpl" bug_id=bugid attach_id=attachid %]<br>
         [% END %]
         

webtools/bugzilla/template/en/default/bug/edit.html.tmpl

@@ -246,7 +246,6 @@ function updateRemainingTime() {
     </td>
     <td rowspan="4" colspan="2" valign="top"> 
         [% IF flag_types.size > 0 %]
-          <b>Flags:</b><br>
           [% PROCESS "flag/list.html.tmpl" %]
         [% END %]
     </td>

webtools/bugzilla/template/en/default/flag/list.html.tmpl

@@ -68,6 +68,16 @@
   #%]
 
 <table id="flags">
+  <tr>
+    <th colspan="3">
+      Flags:
+    </th>
+    [% IF any_flags_requesteeble %]
+      <th>
+        Requestee:
+      </th>
+    [% END %]
+  </tr>
 
   [%# Step 1: Display every flag type (except inactive types with no flags). %]
   [% FOREACH type = flag_types %]
@@ -96,19 +106,21 @@
             [% END %]
           </select>
         </td>
-        <td>
+        [% IF any_flags_requesteeble %]
-          [% IF type.is_active && type.is_requesteeble %]
+          <td>
-            <span style="white-space: nowrap;">
+            [% IF type.is_active && type.is_requesteeble %]
-              (<input type="text" size="8" maxlength="255"
+              <span style="white-space: nowrap;">
-                      id="requestee-[% flag.id %]" 
+                (<input type="text" size="8" maxlength="255"
-                      name="requestee-[% flag.id %]"
+                        id="requestee-[% flag.id %]" 
-                      [% IF flag.status == "?" && flag.requestee %]
+                        name="requestee-[% flag.id %]"
-                        value="[% flag.requestee.email FILTER html %]"
+                        [% IF flag.status == "?" && flag.requestee %]
-                      [% END %]
+                          value="[% flag.requestee.email FILTER html %]"
-               >)
+                        [% END %]
-            </span>
+                 >)
-          [% END %]
+              </span>
-        </td>
+            [% END %]
+          </td>
+        [% END %]
       </tr>
     [% END %]
     
@@ -116,7 +128,7 @@
     [% IF (!type.flags || type.flags.size == 0) && type.is_active %]
       <tr>
         <td>&nbsp;</td>
-        <td>[% type.name %]</td>
+        <td>[% type.name FILTER html %]</td>
         <td>
           <select id="flag_type-[% type.id %]" name="flag_type-[% type.id %]" 
                   onchange="toggleRequesteeField(this);">
@@ -128,15 +140,17 @@
             [% END %]
           </select>
         </td>
-        <td>
+        [% IF any_flags_requesteeble %]
-          [% IF type.is_requesteeble %]
+          <td>
-            <span style="white-space: nowrap;">
+            [% IF type.is_requesteeble %]
-              (<input type="text" size="8" maxlength="255"
+              <span style="white-space: nowrap;">
-                      id="requestee_type-[% type.id %]" 
+                (<input type="text" size="8" maxlength="255"
-                      name="requestee_type-[% type.id %]">)
+                        id="requestee_type-[% type.id %]" 
-            </span>
+                        name="requestee_type-[% type.id %]">)
-          [% END %]
+              </span>
-        </td>
+            [% END %]
+          </td>
+        [% END %]
       </tr>
     [% END %]
   [% END %]
@@ -149,7 +163,7 @@
         [% separator_displayed = 1 %]
     [% END %]
     <tr>
-      <td colspan="2">addl. [% type.name %]</td>
+      <td colspan="2">addl. [% type.name FILTER html %]</td>
       <td>
         <select id="flag_type-[% type.id %]" name="flag_type-[% type.id %]" 
                 onchange="toggleRequesteeField(this);">
@@ -161,15 +175,17 @@
           [% END %]
         </select>
       </td>
-      <td>
+      [% IF any_flags_requesteeble %]
-        [% IF type.is_requesteeble %]
+        <td>
-            <span style="white-space: nowrap;">
+          [% IF type.is_requesteeble %]
-              (<input type="text" size="8" maxlength="255"
+              <span style="white-space: nowrap;">
-                      id="requestee_type-[% type.id %]" 
+                (<input type="text" size="8" maxlength="255"
-                      name="requestee_type-[% type.id %]">)
+                        id="requestee_type-[% type.id %]" 
-            </span>
+                        name="requestee_type-[% type.id %]">)
-        [% END %]
+              </span>
-      </td>
+          [% END %]
+        </td>
+      [% END %]
     </tr>
   [% END %]