Launchpad itself

Broken links at /people/$person/+packages

Bug #50399 reported by Diogo Matsubara
10
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Julian Edwards

Bug Description

The listing presented at the +packages page has broken links.
For instance, looking at https://launchpad.net/people/fabbione/+packages, the xvidtune package, points to 1:0.99.2-0ubuntu1 (https://launchpad.net/distros/ubuntu/dapper/+source/xvidtune/1:0.99.2-0ubuntu1) which is a 404.

OOPS-167C96

Tags: lp-soyuz oops ui
Diogo Matsubara (matsubara)
Changed in soyuz:
importance: Untriaged → Medium
status: Unconfirmed → Confirmed
Revision history for this message
Christian Reis (kiko) wrote :

I guess the package epoch is breaking canonical_url (or do we not use canonical_url there?)

Changed in soyuz:
assignee: nobody → kiko
importance: Medium → High
Revision history for this message
Christian Reis (kiko) wrote :

No, that's not it. Could it be that these packages have already been removed, perchance?

Revision history for this message
Christian Reis (kiko) wrote :

Found out what it is. The problem is that the distribution release +source page isn't traversing to REMOVED packages. The distribution +source page does, though. I'm not sure if linking to the distribution source page is the best thing to do, or if we should display removed packages in the distrorelease page.

Changed in soyuz:
status: Confirmed → In Progress
Revision history for this message
Celso Providelo (cprov) wrote :

DistributionSourcePackage.getVersion is able to traverse REMOVED version, ISourcePackage is intended to have a different behaviour (see bug #60440)

Then REMOVED version can be accessed, if required, via the "Publishing History" section in DistributionSourcePackage page, which linked from this page.

A good thing to do would be:

 * source_name: link to DistributionSourcePackage (https://launchpad.net/distros/ubuntu/+source/xvidtune)
 * distrorelease_name: link to SourcePackage (https://launchpad.net/distros/ubuntu/dapper/+source/xvidtune)
 * source_version: link to DistributionSourcePackageRelease (https://launchpad.net/distros/ubuntu/+source/xvidtune/1:0.99.2-0ubuntu1)

Does it make sense ?

Revision history for this message
Julian Edwards (julian-edwards) wrote :

Celso's proposed change makes sense, I'll do a fix for it. (although the /distros/ part of the URLs is not needed, I presume that was correct before my time?)

Revision history for this message
Diogo Matsubara (matsubara) wrote :

You're right Julian, with the Pillars in place, we don't use it anymore. Unless you're accessing launchpad.net/distros :-)

Celso Providelo (cprov)
Changed in soyuz:
assignee: kiko → julian-edwards
Revision history for this message
Julian Edwards (julian-edwards) wrote :

Fixed in RF 4340.

Changed in soyuz:
status: In Progress → Fix Committed
Julian Edwards (julian-edwards)
Changed in soyuz:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

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