metacpan / metacpan-api

A free, open API for everything you want to know about CPAN
http://www.metacpan.org/
Other
288 stars 196 forks source link

Ancient distribution has apparent index priority over current #416

Open kentfredric opened 9 years ago

kentfredric commented 9 years ago

Dropping this here because something is wrong at the data level which is obviously leaking into the UI. There may be secondary bugs here that are being exposed in the front end as a result of the primary data level bug.

https://metacpan.org/release/Test-Builder-Tester

sn141

You will note how the coloration of this release indicates it is both

sn142

And clicking on any of the modules in this "latest release" takes you somewhere else entirely:

sn143

sn144

Exploring deep in "Browse source" gets you here:

https://metacpan.org/source/MARKF/Test-Builder-Tester-1.01/lib/Test/Builder/Tester.pm

sn145

Clicking the "module documentation link" sends you to .... an entirely different release, in an entirely different distribution:

sn146

O_o

mickeyn commented 7 years ago

the result you see is for the exact match on the distribution name - that's prioritized by design. may be changed in v1 with the suggester work (#470)

ranguard commented 7 years ago

Is this our pod disambiguation ?

ranguard commented 7 years ago

@haarg to cleanup ticket issue description after our discussion

haarg commented 7 years ago

For some reason, the modules in the Test::Builder::Tester are listed as latest. MARKF does hace comaint on the packages, but they aren't in 02packages as they have been superseded by those in Test-Simple.

If that was fixed, the URLs on the dist page would be correct, and they should also drop out of the search result.

tsibley commented 7 years ago

This appears to be MetaCPAN::Script::Latest (invoked either by Script::Release or on its own from cron) catching that packages can move between distributions and setting "latest" on the new dist, but incorrectly not downgrading the file status in the old distribution.