API: In distro.getSeries, it's not clear the name in name_or_version should be the launchpad name
Bug #344282 reported by
Ursula Junque
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
Curtis Hovey | ||
Bug Description
As seen in OOPS-1171EC107, a user tried, using the API, to getSeries of Ubuntu distro trying to use the display name, "Hardy", and it oopsed.
>>> distro = lp.distribution
>>> distro.
NotFoundError: u'Hardy'
It's not clear in the docstring of getSeries method that you should use the launchpad name, "hardy".
Also, it'd be good if the exception raised was NotFound (OOPS-1171EB48) instead of NotFoundError, to keep consistency.
| description: | updated |
| Changed in launchpad-registry: | |
| importance: | Undecided → Low |
| status: | New → Triaged |
| Changed in launchpad-registry: | |
| assignee: | nobody → Curtis Hovey (sinzui) |
| milestone: | none → 2.2.6 |
Revision history for this message
| Curtis Hovey (sinzui) wrote | #2 |
| Changed in launchpad-registry: | |
| status: | Triaged → Fix Committed |
Revision history for this message
| Curtis Hovey (sinzui) wrote Bug 344282 Fix released | #3 |
| Changed in launchpad-registry: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
We just need to update the docstring to state that name_or_version is `IDistroSeries.