diff options
Diffstat (limited to 'main/src/main/java/org/onap/policy/pap/main/rest/depundep/PdpGroupDeleteProvider.java')
-rw-r--r-- | main/src/main/java/org/onap/policy/pap/main/rest/depundep/PdpGroupDeleteProvider.java | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/main/src/main/java/org/onap/policy/pap/main/rest/depundep/PdpGroupDeleteProvider.java b/main/src/main/java/org/onap/policy/pap/main/rest/depundep/PdpGroupDeleteProvider.java index d5acc624..6457efae 100644 --- a/main/src/main/java/org/onap/policy/pap/main/rest/depundep/PdpGroupDeleteProvider.java +++ b/main/src/main/java/org/onap/policy/pap/main/rest/depundep/PdpGroupDeleteProvider.java @@ -58,7 +58,10 @@ public class PdpGroupDeleteProvider extends ProviderBase<PdpGroupDeleteResponse> */ public Pair<Response.Status, PdpGroupDeleteResponse> deleteGroup(String groupName, String version) { - return Pair.of(Response.Status.OK, new PdpGroupDeleteResponse()); + PdpGroupDeleteResponse resp = new PdpGroupDeleteResponse(); + resp.setErrorDetails("not implemented yet"); + + return Pair.of(Response.Status.INTERNAL_SERVER_ERROR, resp); } /** @@ -107,14 +110,8 @@ public class PdpGroupDeleteProvider extends ProviderBase<PdpGroupDeleteResponse> return (group, subgroup) -> { - if (!subgroup.getPolicies().contains(desiredIdent)) { - // doesn't have the policy - return false; - } - // remove the policy from the subgroup - subgroup.getPolicies().remove(desiredIdent); - return true; + return subgroup.getPolicies().remove(desiredIdent); }; } } |