opendev/gerrit
Code Issues Proposed changes

GetAuditLog: Fix NPE when setting the name of a subgroup

Browse Source 
When a subgroup is not available any more, we couldn't load the
group from GroupBackend. Thus in this case, we should not try
to set the name of the subgroup.

Change-Id: I2be5e620fba847cb3fd8d0e364926141e4c33300
This commit is contained in:
Changcheng Xiao
2018-02-27 09:53:37 +01:00
parent ba6851811a
commit 219e3c5f67
3 changed files with 63 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
Documentation/rest-api-groups.txt
View File

@@ -1518,7 +1518,9 @@ The group member that is added/removed. If `type` is `ADD_USER` or
`REMOVE_USER` the member is returned as detailed
link:rest-api-accounts.html#account-info[AccountInfo] entity, if `type`
is `ADD_GROUP` or `REMOVE_GROUP` the member is returned as
link:#group-info[GroupInfo] entity.
link:#group-info[GroupInfo] entity. Note that the `name` in
link:#group-info[GroupInfo] will not be set if the member group is not
available.
|`type` |
The event type, can be: `ADD_USER`, `REMOVE_USER`, `ADD_GROUP` or
`REMOVE_GROUP`.