opendev/gerrit
Code Issues Proposed changes

Merge "For external group fail with 405 if group endpoint expects internal group"

Browse Source
This commit is contained in:
Shawn Pearce
2013-03-13 04:31:22 +00:00
committed by Gerrit Code Review
parent 5f9af5be22 7fe85c7392
commit e5edaa8056
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gerrit-server/src/main/java/com/google/gerrit/server/group/GetDescription.java
View File

@@ -15,16 +15,16 @@
package com.google.gerrit.server.group;
import com.google.common.base.Strings;
import com.google.gerrit.extensions.restapi.ResourceNotFoundException;
import com.google.gerrit.extensions.restapi.MethodNotAllowedException;
import com.google.gerrit.extensions.restapi.RestReadView;
import com.google.gerrit.reviewdb.client.AccountGroup;
class GetDescription implements RestReadView<GroupResource> {
@Override
public String apply(GroupResource resource) throws ResourceNotFoundException {
public String apply(GroupResource resource) throws MethodNotAllowedException {
AccountGroup group = resource.toAccountGroup();
if (group == null) {
throw new ResourceNotFoundException();
throw new MethodNotAllowedException();
}
return Strings.nullToEmpty(group.getDescription());
}