Merge lp://staging/~cjwatson/lazr.restful/mutator-replacement into lp://staging/lazr.restful
Proposed by
Colin Watson
| Status: | Merged |
|---|---|
| Merged at revision: | 226 |
| Proposed branch: | lp://staging/~cjwatson/lazr.restful/mutator-replacement |
| Merge into: | lp://staging/lazr.restful |
| Diff against target: |
156 lines (+40/-29) 3 files modified
src/lazr/restful/NEWS.txt (+9/-0) src/lazr/restful/docs/webservice-declarations.txt (+14/-21) src/lazr/restful/metazcml.py (+17/-8) |
| To merge this branch: | bzr merge lp://staging/~cjwatson/lazr.restful/mutator-replacement |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Ioana Lasc (community) | Approve | ||
| LAZR Developers | Pending | ||
|
Review via email:
|
|||
Commit message
Remove limitation on immediately reinstating a mutator as a named operation.
Description of the change
This previously only worked if the registration code happened to encounter the replacing named operation before it encountered the replaced mutator, which ultimately depended on dict iteration order. I considered making it enforce the limitation consistently, but it was just as easy to remove the limitation.
To post a comment you must log in.
Revision history for this message
| Ioana Lasc (ilasc) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
