Launchpad itself

IArchive.checkUpload() requires unnecessary parameters when asking about PPA permissions

Bug #821379 reported by Jonathan Lange
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
High
Unassigned

Bug Description

To use IArchive.checkUpload() to see if someone can upload to a PPA, you have to make a call like:

  >>> ppa.checkUpload(person=me, distroseries=ubuntu.current_series, sourcepackagename='hello', component='main', pocket='Release')

PPAs only have a main component and only have a release pocket. Requiring users to specify this to check permissions is redundant.

Tags: api ppa
Revision history for this message
Aaron Bentley (abentley) wrote :

PPAs happen to have only a main component and release pocket, but they are archives, and so they follow the same API as all archives. Are you suggesting a separate checkUploadPPA function that would only work on PPAs?

Changed in launchpad:
status: New → Incomplete
Revision history for this message
Jonathan Lange (jml) wrote :

I'm suggesting having those arguments – component and pocket – default to main and Release for PPAs. See https://bugs.launchpad.net/launchpad/+bug/821366/comments/1

Julian Edwards (julian-edwards)
Changed in launchpad:
status: Incomplete → Triaged
importance: Undecided → High
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.