Bug 110711 - various query.cgi/tmpl problems. This patch resolves all known issues.

defparams.pl

@@ -468,7 +468,7 @@ You will get this message once a day until you've dealt with these bugs!
 DefParam("defaultquery",
           "This is the default query that initially comes up when you submit a bug.  It's in URL parameter format, which makes it hard to read.  Sorry!",
          "t",
-         "bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&order=%22Importance%22");
+         "bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&emailassigned_to1=1&emailassigned_to2=1&emailreporter2=1&emailcc2=1&emailqa_contact2=1&order=%22Importance%22");
 
 
 DefParam("letsubmitterchoosepriority",

template/default/query/query.atml

@@ -347,19 +347,17 @@ function selectProduct(f) {
       </select>
     </td>
     <td>
-      <input name="short_desc" size="30" value="[% default.short_desc.0 FILTER html %]" />
+      <input name="short_desc" size="40" value="[% default.short_desc.0 FILTER html %]" />
     </td>
     <td>
       <input type="submit" value="Search" />
     </td>
   </tr>  
-</table>
 
 [%# *** Product Component Version Target *** %]
 
-<table>
   <tr>
-  <td>
+    <td colspan="4">
       <table>
         <tr valign="bottom">
           <th align="left">Product:</th>
@@ -407,12 +405,9 @@ function selectProduct(f) {
   
 [%# *** Comment URL Whiteboard Keywords *** %]
 
-  <tr>
-  <td>
-    <table border="0">
   [% FOREACH field = [ 
-      { name => "long_desc", description => "A comment" },
-      { name => "bug_file_loc", description => "The URL" },
+    { name => "long_desc", description => "A&nbsp;comment" },
+    { name => "bug_file_loc", description => "The&nbsp;URL" },
     { name => "status_whiteboard", description => "Whiteboard" } ] %]
 
     [% UNLESS field.name == 'status_whiteboard' AND NOT Param('usestatuswhiteboard') %]
@@ -428,7 +423,9 @@ function selectProduct(f) {
         </select>
       </td>
       <td><input name="[% field.name %]" size="40" value="
-          [% default.${field.name}.0 FILTER html %]" /></td>
+        [% default.${field.name}.0 FILTER html %]" />
+      </td>
+      <td></td>
     </tr>  
     [% END %]
   [% END %]
@@ -454,10 +451,6 @@ function selectProduct(f) {
       </td>
     </tr>
   [% END %]
-
-    </table>
-  </td>  
-  </tr>
 </table>  
 
 <hr>
@@ -498,6 +491,7 @@ function selectProduct(f) {
           </strong>  
         </legend>
 
+
 <table>
   <tr>
   [% FOREACH n = [1, 2] %]
@@ -511,33 +505,63 @@ function selectProduct(f) {
     </td>
   </tr>
   <tr>
-    <td><input type="checkbox" name="emailassigned_to[% n %]" value="1"
-        [% " checked" IF default.emailassigned_to.$n %] />bug owner</td>
+    <td>
+      <input type="checkbox" name="emailassigned_to[% n %]" 
+             id="emailassigned_to[% n %]" value="1"
+             [% " checked" IF default.emailassigned_to.$n %] />
+      <label for="emailassigned_to[% n %]">
+        bug owner
+      </label>
+    </td>
   </tr>
   <tr>
-    <td><input type="checkbox" name="emailreporter[% n %]" value="1"
-        [% " checked" IF default.emailreporter.$n %] />reporter</td>
+    <td>
+      <input type="checkbox" name="emailreporter[% n %]" 
+             id="emailreporter[% n %]" value="1"
+             [% " checked" IF default.emailreporter.$n %] />
+      <label for="emailreporter[% n %]">
+        reporter
+      </label>
+    </td>
   </tr>
   [% IF Param('useqacontact') %]
   <tr>
-    <td><input type="checkbox" name="emailqa_contact[% n %]" value="1"
-        [% " checked" IF default.emailqa_contact.$n %] />QA Contact</td>
+    <td>
+      <input type="checkbox" name="emailqa_contact[% n %]" 
+             id="emailqa_contact[% n %]" value="1"
+             [% " checked" IF default.emailqa_contact.$n %] />
+      <label for="emailqa_contact[% n %]">
+        QA contact
+      </label>
+    </td>
   </tr>
   [% END %]
   <tr>
-    <td><input type="checkbox" name="emailcc[% n %]" value="1"
-        [% " checked" IF default.emailcc.$n %] />CC list member</td>
+    <td>
+      <input type="checkbox" name="emailcc[% n %]" 
+             id="emailcc[% n %]" value="1"
+             [% " checked" IF default.emailcc.$n %] />
+      <label for="emailcc[% n %]">
+        CC list member
+      </label>
+    </td>
   </tr>
   <tr>
-    <td><input type="checkbox" name="emaillongdesc[% n %]" value="1"
-        [% " checked" IF default.emaillongdesc.$n %] />commenter</td>
+    <td>
+        <input type="checkbox" name="emaillongdesc[% n %]" 
+               id="emaillongdesc[% n %]" value="1"
+               [% " checked" IF default.emaillongdesc.$n %] />
+      <label for="emaillongdesc[% n %]">
+        commenter
+      </label>
+    </td>
   </tr>
   <tr>
     <td>
       <select name="emailtype[% n %]">
       [% FOREACH qv = [ 
-        { name => "exact", description => "is" },
         { name => "substring", description => "contains" },
+        { name => "exact", description => "is" },
         { name => "regexp", description => "matches regexp" },
         { name => "notregexp", description => "doesn&#8217;t match regexp" } ] %]
         
@@ -587,6 +611,7 @@ function selectProduct(f) {
   </tr>
 </table>
 
+
       </fieldset>
     </td>
    
@@ -596,6 +621,7 @@ function selectProduct(f) {
       <fieldset>
         <legend><strong>Bug Changes</strong></legend>
 
+
 <dl>
   <dt>Only bugs changed in the last </dt>
   <dd><input name=changedin size=3 value="[% default.changedin.0 FILTER html %]" /> days</dd>
@@ -617,7 +643,7 @@ function selectProduct(f) {
   <dd>
     <input name="chfieldfrom" size="10" value="[% default.chfieldfrom.0 FILTER html %]" />
     and <input name="chfieldto" size="10" value="[% default.chfieldto.0 FILTER html %]" />
-    <br>(YYYY-MM-DD)
+    <br />(YYYY-MM-DD)
   </dd>
   <dt>to this value: (optional)</dt>
   <dd>
@@ -625,6 +651,7 @@ function selectProduct(f) {
   </dd>
 </dl>
 
+
        </fieldset>
      </td>
   </tr>
@@ -637,9 +664,9 @@ function selectProduct(f) {
     [% IF NOT userid %]
       <input type="hidden" name="cmdtype" value="doit" />
     [% ELSE %]
-      <br>
+      <br />
       <input type="radio" name="cmdtype" value="doit" checked /> Run this query
-      <br>
+      <br />
 
       [% IF namedqueries.size > 0 %]
         <p>
@@ -675,15 +702,16 @@ function selectProduct(f) {
 
       <input type="radio" name="cmdtype" value="asdefault" />
       Remember this as my default query
-      <br>
+      <br />
       <input type="radio" name="cmdtype" value="asnamed" />
       Remember this query, and name it:
       <input type="text" name="newqueryname">
-      <br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" name="tofooter" value="1" />
+      <br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" name="tofooter" value="1" />
           and put it in my page footer
-      <br>
+      <br />
     [% END %]
-      <p>
+      <br />
+      <div>
       Sort results by:
       <select name="order">
       [% FOREACH order = orders %]
@@ -692,14 +720,14 @@ function selectProduct(f) {
       [% END %]
       </select>
 
-      <input type="submit" value="Submit query" />
+      <input type="submit" value="Search" />
       [% IF userdefaultquery %]
          <p>
            <a href="query.cgi?nukedefaultquery=1">
              Set my default query back to the system default</a>
          </p>
       [% END %]
-      </p>
+      </div>
     </td>
   </tr>
 </table>