openstack/python-keystoneclient
Code Issues Proposed changes

Fix no content return type doc

Browse Source 
Replaces the occurrences of "204 No Content" with
the correct return and rtype.

Change-Id: Id782cda4d226c96c31d375fd2432e9d7e28e2f77
This commit is contained in:
Eduardo Magalhães
2016-08-17 15:32:14 +01:00
committed by Rodrigo Duarte
parent 63d039032c
commit 2bb07e746d
4 changed files with 16 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
keystoneclient/v3/domains.py
View File

@@ -116,7 +116,8 @@ class DomainManager(base.CrudManager):
:param domain: the domain to be deleted on the server.
:type domain: str or :class:`keystoneclient.v3.domains.Domain`
:returns: 204 No Content.
:returns: Response object with 204 status.
:rtype: :class:`requests.models.Response`
"""
return super(DomainManager, self).delete(

3
keystoneclient/v3/groups.py
View File

@@ -137,7 +137,8 @@ class GroupManager(base.CrudManager):
:param group: the group to be deleted on the server.
:type group: str or :class:`keystoneclient.v3.groups.Group`
:returns: 204 No Content.
:returns: Response object with 204 status.
:rtype: :class:`requests.models.Response`
"""
return super(GroupManager, self).delete(

3
keystoneclient/v3/projects.py
View File

@@ -214,7 +214,8 @@ class ProjectManager(base.CrudManager):
:param project: the project to be deleted on the server.
:type project: str or :class:`keystoneclient.v3.projects.Project`
:returns: 204 No Content.
:returns: Response object with 204 status.
:rtype: :class:`requests.models.Response`
"""
return super(ProjectManager, self).delete(

15
keystoneclient/v3/users.py
View File

@@ -210,7 +210,8 @@ class UserManager(base.CrudManager):
:param str old_password: the user's old password
:param str new_password: the user's new password
:returns: 204 No Content.
:returns: Response object with 204 status.
:rtype: :class:`requests.models.Response`
"""
if not (old_password and new_password):
@@ -236,7 +237,8 @@ class UserManager(base.CrudManager):
:param group: the group to put the user in.
:type group: str or :class:`keystoneclient.v3.groups.Group`
:returns: 204 No Content.
:returns: Response object with 204 status.
:rtype: :class:`requests.models.Response`
"""
self._require_user_and_group(user, group)
@@ -254,7 +256,8 @@ class UserManager(base.CrudManager):
:param group: the group to check the user in.
:type group: str or :class:`keystoneclient.v3.groups.Group`
:returns: 204 No Content.
:returns: Response object with 204 status.
:rtype: :class:`requests.models.Response`
"""
self._require_user_and_group(user, group)
@@ -272,7 +275,8 @@ class UserManager(base.CrudManager):
:param group: the group to remove the user from.
:type group: str or :class:`keystoneclient.v3.groups.Group`
:returns: 204 No Content.
:returns: Response object with 204 status.
:rtype: :class:`requests.models.Response`
"""
self._require_user_and_group(user, group)
@@ -288,7 +292,8 @@ class UserManager(base.CrudManager):
:param user: the user to be deleted on the server.
:type user: str or :class:`keystoneclient.v3.users.User`
:returns: 204 No Content.
:returns: Response object with 204 status.
:rtype: :class:`requests.models.Response`
"""
return super(UserManager, self).delete(