OOPS - KeyError: 'link' - in person-index.pt

Bug #485237 reported by Gavin Panella
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
High
Brad Crittenden

Bug Description

When ~spm visits https://launchpad.net/~csr21 or
https://edge.launchpad.net/~csr21 he gets OOPS-1419L89. This is
because the person-index.pt template refers to an unbound name at line
169:

  <a
      tal:condition="context/required:launchpad.Edit"
      tal:attributes="href overview_menu/editemailaddresses/url"
    ><img tal:attributes="alt link/text" src="/@@/edit" /></a>
                              ^^^^

Related branches

Revision history for this message
Gavin Panella (allenap) wrote :

BTW, I think this may be because ~spm has elevated privileges compared to the rest of us. For instance, the pages work fine for me.

affects: launchpad → launchpad-registry
Changed in launchpad-registry:
status: New → Triaged
Revision history for this message
Curtis Hovey (sinzui) wrote :

That is a crafted link. I bet the page does not fail if it used the proper tales
  <a tal:replace="overview_menu/editemailaddresses/fmt:icon" />

Changed in launchpad-registry:
importance: Undecided → High
milestone: none → 3.1.11
Brad Crittenden (bac)
Changed in launchpad-registry:
status: Triaged → In Progress
assignee: nobody → Brad Crittenden (bac)
Revision history for this message
Brad Crittenden (bac) wrote :

r 9925

Changed in launchpad-registry:
status: In Progress → Fix Committed
Revision history for this message
Steve McInerney (spm) wrote :

bac,
  works on edge.
As expected, still broken on prod.
awesome! ta. will process that merge request again.

Brad Crittenden (bac)
Changed in launchpad-registry:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers