thing.delete() tries to refresh

[Robert Collins]

 affects launchpad
 done

m = launchpad.load('foo/+milestone/bar'
m.delete() # boom - 404 errror

Doesn't make sense to refresh after doing a delete - or at most, should
catch 404 itself and raise *if 404 isn't returned*

-Rob

[Markus Korn (thekorn)]

For me the issue seem to be that milestone.delete() is exported as a POST method, and not DELETE

[Stephane Wirtel (OpenERP) (stephane-openerp)]

Hi all,

Please, Could you provide a patch and update the python-launchpadlib package.

Thank you so much

[Leonard Richardson (leonardr)]

Markus is right. If you publish a method as a write operation, launchpadlib will invoke it with POST and refresh the object afterwards, even if the method happens to be called 'delete'. This method needs to be published as a destructor operation. (@export_destructor_operation instead of @export_write_operation) Then it will be invoked with DELETE and launchpadlib will know better than to refresh it.

This is entirely a problem with Launchpad. Once it's fixed, existing installations of launchpadlib will start working automatically.

[Gary Poster (gary)]

Curtis, if you are willing to triage this as high, that would be great. However, if you triage this as low, I'll push the task up to high in Foundations and try to make the change this cycle.

[Robert Collins (lifeless)]

On Wed, 2010-04-07 at 12:09 +0000, Stephane (Open ERP) wrote:
> Hi all,
>
> Please, Could you provide a patch and update the python-launchpadlib
> package.

the package doesn't need to change - see the discussion.

-Rob

[Brad Crittenden (bac)]

The @export_destructor_operation approach does not work at this time. Using it causes the method to be uncallable. Leaving the method exported as a 'write' is the best work-around, even though it does throw the subsequent NotFound error.

It is the best that can be done until bug 534363 is fixed.

[Brad Crittenden (bac)]

Actually, the answer given to question 102714 (linked) shows the work-around for calling the destructor. The delete method for milestones can be converted to a destructor but the call sites will have to use the work-around for now.

[Ursula Junque (ursinha)]

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

[Curtis Hovey (sinzui)]

Fixed released in launchpad-project 10.04.