1139872 - The URL in bugmail microdata has the '#' URI encoded, causing the links to break

Status: RESOLVED FIXED

(Bugzilla :: Email Notifications, defect)

Description

[Ed Morley [:emorley]]

Broken out from bug 1102364:

(In reply to Ed Morley [:edmorley] from comment #28)
> (In reply to David Lawrence [:dkl] from comment #15)
> > ::: template/en/default/email/bugmail.html.tmpl
> > @@ +64,5 @@
> > >        [% END %]
> > >        </ul>
> > > +      <div itemscope itemtype="http://schema.org/EmailMessage">
> > > +        <div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
> > > +          <link itemprop="url" href="[% urlbase FILTER html %]show_bug.cgi?id=[% "$bug.id$comment_anchor" FILTER html %]"/>
> > 
> > [% urlbase FILTER none %] and [% "$bug.id$comment_anchor" FILTER uri %]
> > instead.
> 
> Switching the latter to FILTER uri seems to have broken the URL, since the
> output (at least from what I receive; I can't test locally to see what is
> sent), is:
> https://bugzilla.mozilla.org/show_bug.cgi?id=1139664%23c2
> 
> Which results in "'1139664#c2' is not a valid bug number nor an alias to a
> bug."
> 
> Is there another filter type I can use instead that's more appropriate? Or
> can I just use |filter none| given the anchor is only generated from a
> .count? Or should I split the params up more and encode the bug number and
> comment number separately, and leave the hash untouched?

Comment 1

[:glob ✱]

(In reply to Ed Morley [:edmorley] from comment #0)
> > Is there another filter type I can use instead that's more appropriate? Or
> > can I just use |filter none| given the anchor is only generated from a
> > .count?

it's safe to use |FILTER none| here.

adding a template comment indicating that filtering isn't required and is in fact detrimental would help future developers touching this file.

Comment 2

[Ed Morley [:emorley]]

Attachment: Don't filter the bugmail microdata URL params

Comment 3

[:glob ✱]

Comment on attachment 8574388 [details] [diff] [review]
Don't filter the bugmail microdata URL params

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

r=glob

Comment 4

[Frédéric Buclin]

Comment on attachment 8574388 [details] [diff] [review]
Don't filter the bugmail microdata URL params

>+          [%# Filtering of the URL param is not required & would break the URL when the comment anchor is set %]
>+          <link itemprop="url" href="[% urlbase FILTER html %]show_bug.cgi?id=[% "$bug.id$comment_anchor" FILTER none %]"/>
>           <meta itemprop="name" content="View [% terms.bug %]"/>
>         </div>
>         <meta itemprop="description" content="[% terms.Bugzilla %] [%+ terms.bug %] update notification"/>

Could you please remove all these useless /> on checkin? HTML5 is not XHTML. This is the single HTML template in the whole codebase where this is in use.

Comment 5

[:glob ✱]

To ssh://[e-mail]/bugzilla/bugzilla.git
   e3fc6a0..592e6fd  master -> master