Branch fix for bug 232659: Fix inconsistent attachment links (and clean

up diff viewer UI while we're at it). Includes minor change to list that
makes the URLs actually as consistent as on trunk. r=myk, a218=myk.

diff --git a/template/en/default/attachment/diff-file.html.tmpl b/template/en/default/attachment/diff-file.html.tmpl
index ffd525db5434027da1dd844f064c4c2ea037d3e7..75078b6792e198df7b0f97a5c6f60ccc567b4cbf 100644 (file)
--- a/template/en/default/attachment/diff-file.html.tmpl
+++ b/template/en/default/attachment/diff-file.html.tmpl
@@ -71,7 +71,7 @@ incremental_restore()
       </a>
     [% END %]
   [% END %] 
-  (<a name="[% file.filename FILTER html %]_sec[% section_num %]"><a href="#[% file.filename FILTER html %]_sec[% section_num %]">Link Here</a></a>)
+  (<a name="[% file.filename FILTER html %]_sec[% section_num %]" href="#[% file.filename FILTER html %]_sec[% section_num %]">Link Here</a>)
   </th></tr>
   [% FOREACH group = section.groups %]
     [% IF group.context %]

diff --git a/template/en/default/attachment/diff-header.html.tmpl b/template/en/default/attachment/diff-header.html.tmpl
index 3d96d0f7024a2b0fbcb47bceceff9c057949b4b5..e3ca1ac416ee8ed2aec4b492adc4c173ac91f4f9 100644 (file)
--- a/template/en/default/attachment/diff-header.html.tmpl
+++ b/template/en/default/attachment/diff-header.html.tmpl
@@ -25,60 +25,77 @@
 
 [% title = BLOCK %]
   [% IF attachid %]
-Attachment #[% attachid %] for [% terms.Bug %] #[% bugid %]
+Attachment #[% attachid %] for [% terms.bug %] #[% bugid %]
   [% ELSE %]
-Interdiff of #[% oldid %] and #[% newid %] for #[% terms.Bug %] #[% bugid %]
+Interdiff of #[% oldid %] and #[% newid %] for #[% terms.bug %] #[% bugid %]
   [% END %]
 [% END %]
 
 [% style = BLOCK %]
 .file_head {
-  font-size: x-large;
   font-weight: bold;
-  background-color: #d3d3d3;
+  font-size: 1em;
+  background-color: #c3c3c3;
   border: 1px solid black;
   width: 100%;
 }
+
+.file_head a {
+  text-decoration: none; 
+  font-family: monospace; 
+  font-size: 1.1em;
+}
+
 .file_collapse {
   display: none;
 }
+
 .section_head {
   width: 100%;
-  font-weight: bold;
-  background-color: #d3d3d3;
+  background-color: #f0f0f0;
   border: 1px solid black;
   text-align: left;
 }
+
 table.file_table {
   table-layout: fixed;
   width: 100%;
   empty-cells: show;
   border-spacing: 0px;
   border-collapse: collapse;
+  /* draw border below last open context section in listing */
+  border-bottom: 1px solid black;
 }
+
 tbody.file td {
   border-left: 1px dashed black;
   border-right: 1px dashed black;
   width: 50%;
 }
+
 tbody.file pre {
   display: inline;
   white-space: -moz-pre-wrap;
   font-size: 0.9em;
 }
+
 tbody.file pre:empty {
   display: block;
   height: 1em;
 }
+
 .changed {
   background-color: lightblue;
 }
+
 .added {
   background-color: lightgreen;
 }
+
 .removed {
   background-color: #FFCC99;
 }
+
 .warning {
   color: red
 }
@@ -186,22 +203,21 @@ tbody.file pre:empty {
 
 [% onload = 'restore_all(); document.checkboxform.restore_indicator.checked = true' %]
 
+[% BLOCK viewurl %]attachment.cgi?id=[% id %][% END %]
+[% BLOCK editurl %][% PROCESS viewurl %]&action=edit[% END %]
+[% BLOCK diffurl %][% PROCESS viewurl %]&action=diff[% END %]
+
 [% IF headers %]
   [% h1 = BLOCK %]
     [% IF attachid %]
-      [% description FILTER html %] (#[% attachid %])
+      Attachment #[% attachid %]: [% description FILTER html %]
     [% ELSE %]
-      [% USE url %]
-      [% old_url = url('attachment.cgi', action = 'diff', id = oldid) %]
-      [% new_url = url('attachment.cgi', action = 'diff', id = newid) %]
       Diff Between 
-      <a href="[% old_url %]">[% old_desc FILTER html %]</a>
-      (#[% oldid %])
+       #[% oldid %]: <a href="[% PROCESS diffurl id=oldid %]">[% old_desc FILTER html %]</a>
       and 
-      <a href="[% new_url %]">[% new_desc FILTER html %]</a>
-      (#[% newid %])
+       #[% newid %]: <a href="[% PROCESS diffurl id=newid %]">[% new_desc FILTER html %]</a>
     [% END %]
-    for <a href="show_bug.cgi?id=[% bugid %]">[% terms.Bug %] #[% bugid %]</a>
+    for <a href="show_bug.cgi?id=[% bugid %]">[% terms.bug %] #[% bugid %]</a>
   [% END %]
   [% h2 = BLOCK %]
     [% bugsummary FILTER html %]
@@ -226,18 +242,14 @@ tbody.file pre:empty {
 [% IF attachid %]
   [%# HEADER %]
   [% IF headers %]
-    [% USE url('attachment.cgi', id = attachid) %]
-    <a href="[% url() %]">View</a>
-    | <a href="[% url(action = 'edit') %]">Edit</a>
-    [% USE url('attachment.cgi', id = attachid, context = context,
-                                 collapsed = collapsed, headers = headers,
-                                 action = 'diff') %]
-    | <a href="[% url(format = 'raw') %]">Raw Unified</a>
+    <a href="[% PROCESS viewurl id=attachid %]">View</a>
+    | <a href="[% PROCESS editurl id=attachid %]">Edit</a>
+    | <a href="[% PROCESS diffurl id=attachid %]&amp;context=[% context FILTER html %]&amp;collapsed=[% collapsed FILTER html %]&amp;headers=[% headers FILTER html %]&amp;format=raw">Raw&nbsp;Unified</a>
   [% END %]
   [% IF other_patches %]
     [% IF headers %] |[%END%]
     Differences between
-    <form style="display: inline">
+    <form style="display: inline" action="">
       <select name="oldid">
       [% FOREACH patch = other_patches %]
         <option value="[% patch.id %]"
@@ -255,13 +267,8 @@ tbody.file pre:empty {
   <br>
 [% ELSE %]
   [% IF headers %]
-    [% USE url('attachment.cgi', newid = newid, oldid = oldid, action = 'interdiff') %]
-    <a href="[% url(format = 'raw') %]">Raw Unified</a>
-    [% IF attachid %]
-    <br>
-    [% ELSE %]
+    <a href="attachment.cgi?oldid=[% oldid %]&amp;newid=[% newid %]&amp;action=interdiff&amp;format=raw">Raw Unified</a>
     |
-    [% END %]
   [% END %]
 [% END %]
   
@@ -276,23 +283,24 @@ tbody.file pre:empty {
    onclick="return expand_all()">Expand All</a>
 
 [% IF do_context %]
+  [%# only happens for normal viewing, not interdiff %]
   | <span style='font-weight: bold'>Context:</span>
   [% IF context == "patch" %]
     (<strong>Patch</strong> / 
   [% ELSE %]
-    (<a href="[% url(context = '') %]">Patch</a> / 
+    (<a href="[% PROCESS diffurl id=attachid %]&amp;headers=[% headers FILTER html %]">Patch</a> / 
   [% END %]
   [% IF context == "file" %]
     <strong>File</strong> /
   [% ELSE %]
-    <a href="[% url(context = 'file') %]">File</a> / 
+    <a href="[% PROCESS diffurl id=attachid %]&amp;headers=[% headers FILTER html %]&amp;context=file">File</a> / 
   [% END %]
 
   [% IF context == "patch" || context == "file" %]
     [% context = 3 %]
   [% END %]
   [%# textbox for context %]
-  <form style="display: inline"><input type="hidden" name="action" value="diff"><input type="hidden" name="id" value="[% attachid %]"><input type="hidden" name="collapsed" value="[% collapsed FILTER html %]"><input type="hidden" name="headers" value="[% headers FILTER html %]"><input type="text" name="context" value="[% context FILTER html %]" size="3"></form>)
+  <form style="display: inline" action=""><input type="hidden" name="action" value="diff"><input type="hidden" name="id" value="[% attachid %]"><input type="hidden" name="collapsed" value="[% collapsed FILTER html %]"><input type="hidden" name="headers" value="[% headers FILTER html %]"><input type="text" name="context" value="[% context FILTER html %]" size="3"></form>)
 [% END %]
 
 [% IF warning %]
@@ -307,10 +315,12 @@ tbody.file pre:empty {
   [%+ terms.Bugzilla %] when comparing patches made against different revisions.
   [% END %]
 </h2>
+[% ELSE %]
+    <br><br>
 [% END %]
  
 [%# Restore Stuff %]
-<form name="checkboxform">
+<form name="checkboxform" action="">
 <input type="checkbox" name="restore_indicator" style="display: none">
 
 

diff --git a/template/en/default/attachment/edit.html.tmpl b/template/en/default/attachment/edit.html.tmpl
index a05945df0893b4136069a51faa3dad3de9237ac6..a4888f80d560e202bddb4a20c9a775d0812d26bd 100644 (file)
--- a/template/en/default/attachment/edit.html.tmpl
+++ b/template/en/default/attachment/edit.html.tmpl
@@ -247,9 +247,9 @@
       [% IF isviewable %]
         <td width="75%">
           <textarea id="editFrame" name="comment" style="height: 400px; width: 100%; display: none;" cols="80" wrap="soft"></textarea>
-          <iframe id="viewFrame" src="attachment.cgi?id=[% attachid %]&amp;action=view" style="height: 400px; width: 100%;">
+          <iframe id="viewFrame" src="attachment.cgi?id=[% attachid %]" style="height: 400px; width: 100%;">
             <b>You cannot view the attachment while editing it because your browser does not support IFRAMEs.
-            <a href="attachment.cgi?id=[% attachid %]&amp;action=view">View the attachment on a separate page</a>.</b>
+            <a href="attachment.cgi?id=[% attachid %]">View the attachment on a separate page</a>.</b>
           </iframe>
           <script type="application/x-javascript" language="JavaScript">
             <!--
@@ -276,8 +276,7 @@
             able to display.
           </b></p>
           <p><b>
-            <a href="attachment.cgi?id=[% attachid %]&amp;action=view">Download
-            the attachment</a>.
+            <a href="attachment.cgi?id=[% attachid %]">Download the attachment</a>.
           </b></p>
         </td>
       [% END %]

diff --git a/template/en/default/attachment/list.html.tmpl b/template/en/default/attachment/list.html.tmpl
index 8558a5959e96178762951bd42b6ec9c1b76f40be..d5741d6a2cd2a5be4c110999fa946b7b1f14080e 100644 (file)
--- a/template/en/default/attachment/list.html.tmpl
+++ b/template/en/default/attachment/list.html.tmpl
@@ -36,7 +36,7 @@
         [% IF !attachment.isprivate || canseeprivate %]
     <tr [% "class=\"bz_private\"" IF attachment.isprivate %]>
       <td valign="top">
-        <a href="attachment.cgi?id=[% attachment.attachid %]&amp;action=view">[% attachment.description FILTER html FILTER obsolete(attachment.isobsolete) %]</a>
+        <a href="attachment.cgi?id=[% attachment.attachid %]">[% attachment.description FILTER html FILTER obsolete(attachment.isobsolete) %]</a>
       </td>
 
       <td valign="top">

diff --git a/template/en/default/filterexceptions.pl b/template/en/default/filterexceptions.pl
index 3f9f9ab01731be6d2b33177fe76645ac4424f456..e94cc625777c232a7ee7d4a7fe5b1165025a3f03 100644 (file)
--- a/template/en/default/filterexceptions.pl
+++ b/template/en/default/filterexceptions.pl
@@ -460,9 +460,8 @@
 
 'attachment/diff-header.html.tmpl' => [
   'attachid',
+  'id',
   'bugid',
-  'old_url',
-  'new_url',
   'oldid',
   'newid',
   'style',
@@ -479,11 +478,6 @@
   'section_num'
 ],
 
-'admin/table.html.tmpl' => [
-  'link_uri',
-  'c.content'
-],
-
 'admin/products/groupcontrol/confirm-edit.html.tmpl' => [
   'group.count', 
 ],
@@ -498,6 +492,11 @@
   'const.CONTROLMAPMANDATORY', 
 ],
 
+'admin/keywords/list.html.tmpl' => [
+  'keyword.id',
+  'keyword.bug_count',
+],
+
 'admin/keywords/edit.html.tmpl' => [
   'keyword_id',
   'bug_count',