openstack/python-neutronclient
Code Issues Proposed changes

Merge "Revert parent_id and obj_id parameter order"

Browse Source
This commit is contained in:
Jenkins
2015-10-14 22:10:02 +00:00
committed by Gerrit Code Review
parent d908e97984 1ced38c1a7
commit e4ced578b2
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
neutronclient/v2_0/client.py
View File

@@ -1746,7 +1746,7 @@ class Client(ClientBase):
def _fx(obj, **_params): def _fx(obj, **_params):
return self.show_ext(path, obj, **_params) return self.show_ext(path, obj, **_params)
def _parent_fx(parent_id, obj, **_params): def _parent_fx(obj, parent_id, **_params):
return self.show_ext(path % parent_id, obj, **_params) return self.show_ext(path % parent_id, obj, **_params)
fn = _fx if not parent_resource else _parent_fx fn = _fx if not parent_resource else _parent_fx
setattr(self, "show_%s" % resource_singular, fn) setattr(self, "show_%s" % resource_singular, fn)
@@ -1773,7 +1773,7 @@ class Client(ClientBase):
def _fx(obj): def _fx(obj):
return self.delete_ext(path, obj) return self.delete_ext(path, obj)
def _parent_fx(parent_id, obj): def _parent_fx(obj, parent_id):
return self.delete_ext(path % parent_id, obj) return self.delete_ext(path % parent_id, obj)
fn = _fx if not parent_resource else _parent_fx fn = _fx if not parent_resource else _parent_fx
setattr(self, "delete_%s" % resource_singular, fn) setattr(self, "delete_%s" % resource_singular, fn)
@@ -1782,7 +1782,7 @@ class Client(ClientBase):
def _fx(obj, body=None): def _fx(obj, body=None):
return self.update_ext(path, obj, body) return self.update_ext(path, obj, body)
def _parent_fx(parent_id, obj, body=None): def _parent_fx(obj, parent_id, body=None):
return self.update_ext(path % parent_id, obj, body) return self.update_ext(path % parent_id, obj, body)
fn = _fx if not parent_resource else _parent_fx fn = _fx if not parent_resource else _parent_fx
setattr(self, "update_%s" % resource_singular, fn) setattr(self, "update_%s" % resource_singular, fn)