Launchpad itself

Oops showing unpublished dsp summary

Bug #612408 reported by Curtis Hovey
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
j.c.sackett
Launchpad itself 10.09 "Hungry Hippo"

Bug Description

OOPS-1674EA685 shows that dsp+index (and presumably sp+index) can oops at context/summary/fmt:text-to-html because the package has no publication history. Only Ubuntu is guaranteeds to have spph. Debian will often have it, though it is often out of date. The dsp and sp templates that access summary need to handle situations these non-soyuz-based distros.

  LocationError: (<lp.registry.model.distributionsourcepackage.DistributionSourcePackage object at 0x2aaab1b98990>, 'summary')

Something simple like this may work.
<p tal:define="summary context/summary|nothing"
   tal:condition="summary"
   tal:content="summary/fmt:text-to-html" />

See original description

Related branches

lp:~jcsackett/launchpad/bugfix-612408
Curtis Hovey (community): Approve (code)
Diff: 66 lines (+25/-3)
3 files modified
lib/lp/registry/model/sourcepackage.py (+2/-0)
lib/lp/registry/tests/test_sourcepackage.py (+21/-0)
lib/lp/soyuz/tests/test_publishing.py (+2/-3)
lp:~jcsackett/launchpad/distributionsourcepackage-summary-fix (Merged)
Curtis Hovey (sinzui)
Changed in launchpad-registry:
assignee: nobody → j.c.sackett (jcsackett)
Curtis Hovey (sinzui)
Changed in launchpad-registry:
status: Triaged → In Progress
Ursula Junque (ursinha)
tags: added: oops
removed: oop
description: updated
j.c.sackett (jcsackett)
Changed in launchpad-registry:
status: In Progress → Fix Committed
Revision history for this message
Ursula Junque (ursinha) wrote : Bug fixed by a commit

Fixed in stable r11310 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/11310>.

tags: added: qa-needstesting
Revision history for this message
Curtis Hovey (sinzui) wrote : Bug 612408 Fix released

Fixed released in launchpad-project 10.08.

Curtis Hovey (sinzui)
Changed in launchpad-registry:
status: Fix Committed → Fix Released
Curtis Hovey (sinzui)
Changed in launchpad-registry:
status: Fix Released → In Progress
milestone: 10.08 → 10.09
Curtis Hovey (sinzui)
tags: added: qa-bad
removed: qa-needstesting
j.c.sackett (jcsackett)
Changed in launchpad-registry:
status: In Progress → Fix Committed
Revision history for this message
Launchpad QA Bot (lpqabot) wrote : Bug fixed by a commit

Fixed in stable r11360 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/11360>.

tags: added: qa-needstesting
removed: qa-bad
j.c.sackett (jcsackett)
tags: added: qa-bad
removed: qa-needstesting
Changed in launchpad-registry:
status: Fix Committed → In Progress
j.c.sackett (jcsackett)
tags: added: qa-needstesting
removed: qa-bad
j.c.sackett (jcsackett)
Changed in launchpad-registry:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
tags: added: qa-ok
removed: qa-needstesting
Curtis Hovey (sinzui)
Changed in launchpad-registry:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.