1272548 - bug link in header broken if format param specified

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: User Interface, defect)

Description

[:glob ✱]

str:

1. https://bugzilla.mozilla.org/show_bug.cgi?id=1150541&format=modal
2. click on the "bug 1150541" link in the top left corner

expected:

page refreshed, with the format retained

actual:

https://bugzilla.mozilla.org/show_bug.cgi?id=1150541%26amp%3Bformat%3Dmodal
'1150541&format=modal' is not a valid bug number nor an alias to a bug.

Comment 1

[Dylan Hardison [:dylan] (he/him)]

Attachment: 1272548_1.patch

Comment 2

[David Lawrence [:dkl]]

Comment on attachment 8753970 [details] [diff] [review]
1272548_1.patch

Review of attachment 8753970 [details] [diff] [review]:
-----------------------------------------------------------------

r=dkl

::: extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
@@ +170,5 @@
> +        [% IF cgi.param("format") %]
> +          [% format_uri = cgi.param("format") FILTER uri %]
> +          [% this_bug_href = this_bug_href _ "&format=" _ format_uri %]
> +        [% END %]
> +        <a id="this-bug" href="[% this_bug_href FILTER html %]">

Your way is cleaner in that it leaves off &format= if not defined. We could also just do:

diff --git a/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl b/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
index 25f785a..959dcf9 100644
--- a/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
+++ b/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
@@ -166,8 +166,8 @@
         view_only = 1
     %]
       <div id="field-value-bug_id">
-        <a id="this-bug" href="show_bug.cgi?id=[% bug.id FILTER none %]
-                [%~ '&amp;format=' _ cgi.param("format") FILTER uri IF cgi.param("format") %]"
+        <a id="this-bug" href="show_bug.cgi?id=[% bug.id FILTER none %]&amp;format=
+                [%~ cgi.param("format") FILTER uri IF cgi.param("format") %]"
         >
           [%~ terms.Bug _ " " _ bug.id FILTER none ~%]
         </a>

Leave it up to you.

Comment 3

[Dylan Hardison [:dylan] (he/him)]

To ssh://[e-mail]/webtools/bmo/bugzilla.git
   3eb0bba..01ea5ee  master -> master