diff options
author | Endi S. Dewata <edewata@redhat.com> | 2014-02-18 17:02:08 -0500 |
---|---|---|
committer | Endi S. Dewata <edewata@redhat.com> | 2014-02-19 12:51:08 -0500 |
commit | fda9b5c579d62498f846ab9c313ccbe1a85b0cdb (patch) | |
tree | 5bb5aaa4a45c7a5e421e02cf01d36e0191a9f54e /base | |
parent | d345bab896d2ac16dca164adb48031047affdf09 (diff) | |
download | pki-fda9b5c579d62498f846ab9c313ccbe1a85b0cdb.tar.gz pki-fda9b5c579d62498f846ab9c313ccbe1a85b0cdb.tar.xz pki-fda9b5c579d62498f846ab9c313ccbe1a85b0cdb.zip |
Refactored GroupMemberProcessor.
The GroupMemberProcessor.addGroupMember() has been modified such that it returns
the newly created group member data. This way the GroupService can use the common
method to create the response object.
Ticket #554
Diffstat (limited to 'base')
-rw-r--r-- | base/server/cms/src/com/netscape/cms/servlet/admin/GroupMemberProcessor.java | 8 | ||||
-rw-r--r-- | base/server/cms/src/com/netscape/cms/servlet/admin/GroupService.java | 4 |
2 files changed, 5 insertions, 7 deletions
diff --git a/base/server/cms/src/com/netscape/cms/servlet/admin/GroupMemberProcessor.java b/base/server/cms/src/com/netscape/cms/servlet/admin/GroupMemberProcessor.java index 429fbea17..ff56c2139 100644 --- a/base/server/cms/src/com/netscape/cms/servlet/admin/GroupMemberProcessor.java +++ b/base/server/cms/src/com/netscape/cms/servlet/admin/GroupMemberProcessor.java @@ -23,7 +23,6 @@ import java.util.Enumeration; import java.util.Locale; import java.util.Map; -import javax.ws.rs.core.Response; import javax.ws.rs.core.UriInfo; import org.jboss.resteasy.plugins.providers.atom.Link; @@ -184,7 +183,7 @@ public class GroupMemberProcessor extends Processor { } } - public Response addGroupMember(GroupMemberData groupMemberData) { + public GroupMemberData addGroupMember(GroupMemberData groupMemberData) { String groupID = groupMemberData.getGroupID(); try { if (groupID == null) { @@ -241,10 +240,7 @@ public class GroupMemberProcessor extends Processor { // read the data back groupMemberData = getGroupMember(groupID, memberID); - return Response - .created(groupMemberData.getLink().getHref()) - .entity(groupMemberData) - .build(); + return groupMemberData; } catch (PKIException e) { auditAddGroupMember(groupID, groupMemberData, ILogger.FAILURE); diff --git a/base/server/cms/src/com/netscape/cms/servlet/admin/GroupService.java b/base/server/cms/src/com/netscape/cms/servlet/admin/GroupService.java index 4fc4a7cfc..3af4bae43 100644 --- a/base/server/cms/src/com/netscape/cms/servlet/admin/GroupService.java +++ b/base/server/cms/src/com/netscape/cms/servlet/admin/GroupService.java @@ -390,7 +390,9 @@ public class GroupService extends PKIService implements GroupResource { try { GroupMemberProcessor processor = new GroupMemberProcessor(getLocale(headers)); processor.setUriInfo(uriInfo); - return processor.addGroupMember(groupMemberData); + groupMemberData = processor.addGroupMember(groupMemberData); + + return createCreatedResponse(groupMemberData, groupMemberData.getLink().getHref()); } catch (PKIException e) { throw e; |