Merge lp:~bac/launchpad/bug-524778-2 into lp:launchpad

[Brad Crittenden]

Fix the export of Milestone delete to use @export_destructor_operation.

= Summary =

The milestone 'destroySelf' method was exported as a write operation
which causes an unnecessary NotFound exception to be raised after the
successful deletion. It should've been exported as a destructor.

== Proposed fix ==

Change to use @export_destructor_operation

== Pre-implementation notes ==

Talks with Curtis, Leondard, and Francis.

== Implementation details ==

As above.

== Tests ==

There are currently no launchpadlib automated tests. The method can be
tested interactively using the following:

In [1]: from launchpadlib.launchpad import Launchpad
In [2]: lp = Launchpad.login_with('interactive', service_root='dev')
In [3]: ff = lp.projects['firefox']
In [4]: ff.getMilestone(name='deleteme') # None found.
In [5]: series = ff.series[0]
In [6]: nm = series.newMilestone(name='deleteme')
In [7]: ff.getMilestone(name='deleteme')
Out[7]: <milestone at
https://api.launchpad.dev/1.0/firefox/+milestone/deleteme>
In [8]: nm._root._browser.delete(nm.self_link)
In [9]: ff.getMilestone(name='deleteme') # None found.

== Demo and Q/A ==

As above against staging or edge.

= Launchpad lint =

Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.

Linting changed files:
  lib/lp/registry/interfaces/milestone.py

[Eleanor Berger (intellectronica)]

review: Approve

=== modified file 'lib/lp/registry/interfaces/milestone.py'
--- lib/lp/registry/interfaces/milestone.py	2010-02-27 10:19:18 +0000
+++ lib/lp/registry/interfaces/milestone.py	2010-04-27 10:40:56 +0000
@@ -32,10 +32,10 @@
 
 from lazr.restful.fields import CollectionField, Reference
 from lazr.restful.declarations import (
-    call_with, export_as_webservice_entry, export_factory_operation, exported,
-    export_operation_as, export_read_operation, export_write_operation,
-    operation_parameters, operation_returns_entry, rename_parameters_as,
-    REQUEST_USER)
+    call_with, export_as_webservice_entry, export_destructor_operation,
+    export_factory_operation, exported, export_operation_as,
+    export_read_operation, operation_parameters, operation_returns_entry,
+    rename_parameters_as, REQUEST_USER)
 
 
 class MilestoneNameField(ContentNameField):
@@ -173,7 +173,7 @@
         XXX sinzui 2010-01-27 bug=341687: blueprints not yet implemented.
         """
 
-    @export_write_operation()
+    @export_destructor_operation()
     @export_operation_as('delete')
     def destroySelf():
         """Delete this milestone.