Fix sonar issues in policy-api
[policy/api.git] / main / src / main / java / org / onap / policy / api / main / rest / ApiRestController.java
index 0c46df4..30b004b 100644 (file)
@@ -48,17 +48,21 @@ import javax.ws.rs.Path;
 import javax.ws.rs.PathParam;\r
 import javax.ws.rs.Produces;\r
 import javax.ws.rs.core.Response;\r
-import javax.ws.rs.core.Response.ResponseBuilder;\r
 import org.apache.commons.lang3.tuple.Pair;\r
 import org.onap.policy.api.main.rest.provider.HealthCheckProvider;\r
 import org.onap.policy.api.main.rest.provider.PolicyProvider;\r
 import org.onap.policy.api.main.rest.provider.PolicyTypeProvider;\r
 import org.onap.policy.api.main.rest.provider.StatisticsProvider;\r
+import org.onap.policy.common.endpoints.event.comm.Topic.CommInfrastructure;\r
 import org.onap.policy.common.endpoints.report.HealthCheckReport;\r
+import org.onap.policy.common.endpoints.utils.NetLoggerUtil;\r
+import org.onap.policy.common.endpoints.utils.NetLoggerUtil.EventType;\r
 import org.onap.policy.models.base.PfModelException;\r
 import org.onap.policy.models.base.PfModelRuntimeException;\r
 import org.onap.policy.models.tosca.authorative.concepts.ToscaPolicy;\r
 import org.onap.policy.models.tosca.authorative.concepts.ToscaServiceTemplate;\r
+import org.slf4j.Logger;\r
+import org.slf4j.LoggerFactory;\r
 \r
 /**\r
  * Class to provide REST API services.\r
@@ -87,7 +91,9 @@ import org.onap.policy.models.tosca.authorative.concepts.ToscaServiceTemplate;
         }),\r
         schemes = { SwaggerDefinition.Scheme.HTTP, SwaggerDefinition.Scheme.HTTPS },\r
         securityDefinition = @SecurityDefinition(basicAuthDefinitions = { @BasicAuthDefinition(key = "basicAuth") }))\r
-public class ApiRestController {\r
+public class ApiRestController extends CommonRestController {\r
+\r
+    private static final Logger LOGGER = LoggerFactory.getLogger(ApiRestController.class);\r
 \r
     /**\r
      * Retrieves the healthcheck status of the API component.\r
@@ -133,8 +139,7 @@ public class ApiRestController {
             @HeaderParam("X-ONAP-RequestID") @ApiParam("RequestID for http transaction") UUID requestId) {\r
 \r
         updateApiStatisticsCounter(Target.OTHER, Result.SUCCESS, HttpMethod.GET);\r
-        return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-            .entity(new HealthCheckProvider().performHealthCheck()).build();\r
+        return makeOkResponse(requestId, new HealthCheckProvider().performHealthCheck());\r
     }\r
 \r
     /**\r
@@ -181,8 +186,8 @@ public class ApiRestController {
             @HeaderParam("X-ONAP-RequestID") @ApiParam("RequestID for http transaction") UUID requestId) {\r
 \r
         updateApiStatisticsCounter(Target.OTHER, Result.SUCCESS, HttpMethod.GET);\r
-        return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-            .entity(new StatisticsProvider().fetchCurrentStatistics()).build();\r
+\r
+        return makeOkResponse(requestId, new StatisticsProvider().fetchCurrentStatistics());\r
     }\r
 \r
     /**\r
@@ -231,13 +236,11 @@ public class ApiRestController {
         try (PolicyTypeProvider policyTypeProvider = new PolicyTypeProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyTypeProvider.fetchPolicyTypes(null, null);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.SUCCESS, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-                    .entity(serviceTemplate).build();\r
+            return makeOkResponse(requestId, serviceTemplate);\r
         } catch (PfModelException | PfModelRuntimeException pfme) {\r
+            LOGGER.error("GET /policytypes", pfme);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.FAILURE, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(\r
-                    Response.status(pfme.getErrorResponse().getResponseCode())), requestId)\r
-                    .entity(pfme.getErrorResponse()).build();\r
+            return makeErrorResponse(requestId, pfme);\r
         }\r
     }\r
 \r
@@ -291,13 +294,11 @@ public class ApiRestController {
         try (PolicyTypeProvider policyTypeProvider = new PolicyTypeProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyTypeProvider.fetchPolicyTypes(policyTypeId, null);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.SUCCESS, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-                    .entity(serviceTemplate).build();\r
+            return makeOkResponse(requestId, serviceTemplate);\r
         } catch (PfModelException | PfModelRuntimeException pfme) {\r
+            LOGGER.error("GET /policytypes/{}", policyTypeId, pfme);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.FAILURE, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(\r
-                    Response.status(pfme.getErrorResponse().getResponseCode())), requestId)\r
-                    .entity(pfme.getErrorResponse()).build();\r
+            return makeErrorResponse(requestId, pfme);\r
         }\r
     }\r
 \r
@@ -353,13 +354,69 @@ public class ApiRestController {
         try (PolicyTypeProvider policyTypeProvider = new PolicyTypeProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyTypeProvider.fetchPolicyTypes(policyTypeId, versionId);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.SUCCESS, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-                    .entity(serviceTemplate).build();\r
+            return makeOkResponse(requestId, serviceTemplate);\r
+        } catch (PfModelException | PfModelRuntimeException pfme) {\r
+            LOGGER.error("GET /policytypes/{}/versions/{}", policyTypeId, versionId, pfme);\r
+            updateApiStatisticsCounter(Target.POLICY_TYPE, Result.FAILURE, HttpMethod.GET);\r
+            return makeErrorResponse(requestId, pfme);\r
+        }\r
+    }\r
+\r
+    /**\r
+     * Retrieves latest version of a particular policy type.\r
+     *\r
+     * @param policyTypeId the ID of specified policy type\r
+     *\r
+     * @return the Response object containing the results of the API operation\r
+     */\r
+    @GET\r
+    @Path("/policytypes/{policyTypeId}/versions/latest")\r
+    @ApiOperation(value = "Retrieve latest version of a policy type",\r
+            notes = "Returns latest version for the specified policy type",\r
+            response = ToscaServiceTemplate.class,\r
+            responseHeaders = {\r
+                    @ResponseHeader(name = "X-MinorVersion",\r
+                                    description = "Used to request or communicate a MINOR version back from the client"\r
+                                                + " to the server, and from the server back to the client",\r
+                                    response = String.class),\r
+                    @ResponseHeader(name = "X-PatchVersion",\r
+                                    description = "Used only to communicate a PATCH version in a response for"\r
+                                                + " troubleshooting purposes only, and will not be provided by"\r
+                                                + " the client on request",\r
+                                    response = String.class),\r
+                    @ResponseHeader(name = "X-LatestVersion",\r
+                                    description = "Used only to communicate an API's latest version",\r
+                                    response = String.class),\r
+                    @ResponseHeader(name = "X-ONAP-RequestID",\r
+                                    description = "Used to track REST transactions for logging purpose",\r
+                                    response = UUID.class)\r
+            },\r
+            authorizations = @Authorization(value = "basicAuth"),\r
+            tags = { "PolicyType", },\r
+            extensions = {\r
+                    @Extension(name = "interface info", properties = {\r
+                            @ExtensionProperty(name = "api-version", value = "1.0.0"),\r
+                            @ExtensionProperty(name = "last-mod-release", value = "Dublin")\r
+                    })\r
+            })\r
+    @ApiResponses(value = {\r
+            @ApiResponse(code = 401, message = "Authentication Error"),\r
+            @ApiResponse(code = 403, message = "Authorization Error"),\r
+            @ApiResponse(code = 404, message = "Resource Not Found"),\r
+            @ApiResponse(code = 500, message = "Internal Server Error")\r
+        })\r
+    public Response getLatestVersionOfPolicyType(\r
+            @PathParam("policyTypeId") @ApiParam(value = "ID of policy type", required = true) String policyTypeId,\r
+            @HeaderParam("X-ONAP-RequestID") @ApiParam("RequestID for http transaction") UUID requestId) {\r
+\r
+        try (PolicyTypeProvider policyTypeProvider = new PolicyTypeProvider()) {\r
+            ToscaServiceTemplate serviceTemplate = policyTypeProvider.fetchLatestPolicyTypes(policyTypeId);\r
+            updateApiStatisticsCounter(Target.POLICY_TYPE, Result.SUCCESS, HttpMethod.GET);\r
+            return makeOkResponse(requestId, serviceTemplate);\r
         } catch (PfModelException | PfModelRuntimeException pfme) {\r
+            LOGGER.error("GET /policytypes/{}/versions/latest", policyTypeId, pfme);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.FAILURE, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(\r
-                    Response.status(pfme.getErrorResponse().getResponseCode())), requestId)\r
-                    .entity(pfme.getErrorResponse()).build();\r
+            return makeErrorResponse(requestId, pfme);\r
         }\r
     }\r
 \r
@@ -410,16 +467,18 @@ public class ApiRestController {
             @ApiParam(value = "Entity body of policy type", required = true) ToscaServiceTemplate body,\r
             @HeaderParam("X-ONAP-RequestID") @ApiParam("RequestID for http transaction") UUID requestId) {\r
 \r
+        if (NetLoggerUtil.getNetworkLogger().isInfoEnabled()) {\r
+            NetLoggerUtil.log(EventType.IN, CommInfrastructure.REST, "/policytypes", toJson(body));\r
+        }\r
+\r
         try (PolicyTypeProvider policyTypeProvider = new PolicyTypeProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyTypeProvider.createPolicyType(body);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.SUCCESS, HttpMethod.POST);\r
-            return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-                    .entity(serviceTemplate).build();\r
+            return makeOkResponse(requestId, serviceTemplate);\r
         } catch (PfModelException | PfModelRuntimeException pfme) {\r
+            LOGGER.error("POST /policytypes", pfme);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.FAILURE, HttpMethod.POST);\r
-            return addLoggingHeaders(addVersionControlHeaders(\r
-                    Response.status(pfme.getErrorResponse().getResponseCode())), requestId)\r
-                    .entity(pfme.getErrorResponse()).build();\r
+            return makeErrorResponse(requestId, pfme);\r
         }\r
     }\r
 \r
@@ -477,12 +536,10 @@ public class ApiRestController {
 \r
         try (PolicyTypeProvider policyTypeProvider = new PolicyTypeProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyTypeProvider.deletePolicyType(policyTypeId, versionId);\r
-            return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-                    .entity(serviceTemplate).build();\r
+            return makeOkResponse(requestId, serviceTemplate);\r
         } catch (PfModelException | PfModelRuntimeException pfme) {\r
-            return addLoggingHeaders(addVersionControlHeaders(\r
-                    Response.status(pfme.getErrorResponse().getResponseCode())), requestId)\r
-                    .entity(pfme.getErrorResponse()).build();\r
+            LOGGER.error("DELETE /policytypes/{}/versions/{}", policyTypeId, versionId, pfme);\r
+            return makeErrorResponse(requestId, pfme);\r
         }\r
     }\r
 \r
@@ -540,13 +597,11 @@ public class ApiRestController {
             ToscaServiceTemplate serviceTemplate =\r
                     policyProvider.fetchPolicies(policyTypeId, policyTypeVersion, null, null);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-                    .entity(serviceTemplate).build();\r
+            return makeOkResponse(requestId, serviceTemplate);\r
         } catch (PfModelException | PfModelRuntimeException pfme) {\r
+            LOGGER.error("GET /policytypes/{}/versions/{}/policies", policyTypeId, policyTypeVersion, pfme);\r
             updateApiStatisticsCounter(Target.POLICY, Result.FAILURE, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(\r
-                    Response.status(pfme.getErrorResponse().getResponseCode())), requestId)\r
-                    .entity(pfme.getErrorResponse()).build();\r
+            return makeErrorResponse(requestId, pfme);\r
         }\r
     }\r
 \r
@@ -606,13 +661,12 @@ public class ApiRestController {
             ToscaServiceTemplate serviceTemplate = policyProvider\r
                     .fetchPolicies(policyTypeId, policyTypeVersion, policyId, null);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-                    .entity(serviceTemplate).build();\r
+            return makeOkResponse(requestId, serviceTemplate);\r
         } catch (PfModelException | PfModelRuntimeException pfme) {\r
+            LOGGER.error("/policytypes/{}/versions/{}/policies/{}", policyTypeId, policyTypeVersion, policyId,\r
+                    pfme);\r
             updateApiStatisticsCounter(Target.POLICY, Result.FAILURE, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(\r
-                    Response.status(pfme.getErrorResponse().getResponseCode())), requestId)\r
-                    .entity(pfme.getErrorResponse()).build();\r
+            return makeErrorResponse(requestId, pfme);\r
         }\r
     }\r
 \r
@@ -674,13 +728,12 @@ public class ApiRestController {
             ToscaServiceTemplate serviceTemplate = policyProvider\r
                     .fetchPolicies(policyTypeId, policyTypeVersion, policyId, policyVersion);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-                    .entity(serviceTemplate).build();\r
+            return makeOkResponse(requestId, serviceTemplate);\r
         } catch (PfModelException | PfModelRuntimeException pfme) {\r
+            LOGGER.error("GET /policytypes/{}/versions/{}/policies/{}/versions/{}", policyTypeId,\r
+                    policyTypeVersion, policyId, policyVersion, pfme);\r
             updateApiStatisticsCounter(Target.POLICY, Result.FAILURE, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(\r
-                    Response.status(pfme.getErrorResponse().getResponseCode())), requestId)\r
-                    .entity(pfme.getErrorResponse()).build();\r
+            return makeErrorResponse(requestId, pfme);\r
         }\r
     }\r
 \r
@@ -694,7 +747,7 @@ public class ApiRestController {
      * @return the Response object containing the results of the API operation\r
      */\r
     @GET\r
-    @Path("/policytypes/{policyTypeId}/versions/{policyTypeVersion}/policies/{policyId}/latest")\r
+    @Path("/policytypes/{policyTypeId}/versions/{policyTypeVersion}/policies/{policyId}/versions/latest")\r
     @ApiOperation(value = "Retrieve the latest version of a particular policy",\r
             notes = "Returns the latest version of specified policy",\r
             response = ToscaServiceTemplate.class,\r
@@ -740,13 +793,12 @@ public class ApiRestController {
             ToscaServiceTemplate serviceTemplate =\r
                     policyProvider.fetchLatestPolicies(policyTypeId, policyTypeVersion, policyId);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-                    .entity(serviceTemplate).build();\r
+            return makeOkResponse(requestId, serviceTemplate);\r
         } catch (PfModelException | PfModelRuntimeException pfme) {\r
+            LOGGER.error("GET /policytypes/{}/versions/{}/policies/{}/versions/latest", policyTypeId,\r
+                    policyTypeVersion, policyId, pfme);\r
             updateApiStatisticsCounter(Target.POLICY, Result.FAILURE, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(\r
-                    Response.status(pfme.getErrorResponse().getResponseCode())), requestId)\r
-                    .entity(pfme.getErrorResponse()).build();\r
+            return makeErrorResponse(requestId, pfme);\r
         }\r
     }\r
 \r
@@ -760,7 +812,7 @@ public class ApiRestController {
      * @return the Response object containing the results of the API operation\r
      */\r
     @GET\r
-    @Path("/policytypes/{policyTypeId}/versions/{policyTypeVersion}/policies/{policyId}/deployed")\r
+    @Path("/policytypes/{policyTypeId}/versions/{policyTypeVersion}/policies/{policyId}/versions/deployed")\r
     @ApiOperation(value = "Retrieve deployed versions of a particular policy in pdp groups",\r
             notes = "Returns deployed versions of specified policy in pdp groups",\r
             response = ToscaPolicy.class, responseContainer = "List",\r
@@ -806,13 +858,12 @@ public class ApiRestController {
             Map<Pair<String, String>, List<ToscaPolicy>> deployedPolicies = policyProvider\r
                     .fetchDeployedPolicies(policyTypeId, policyTypeVersion, policyId);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-                    .entity(deployedPolicies).build();\r
+            return makeOkResponse(requestId, deployedPolicies);\r
         } catch (PfModelException | PfModelRuntimeException pfme) {\r
+            LOGGER.error("GET /policytypes/{}/versions/{}/policies/{}/versions/deployed", policyTypeId,\r
+                    policyTypeVersion, policyId, pfme);\r
             updateApiStatisticsCounter(Target.POLICY, Result.FAILURE, HttpMethod.GET);\r
-            return addLoggingHeaders(addVersionControlHeaders(\r
-                    Response.status(pfme.getErrorResponse().getResponseCode())), requestId)\r
-                    .entity(pfme.getErrorResponse()).build();\r
+            return makeErrorResponse(requestId, pfme);\r
         }\r
     }\r
 \r
@@ -869,17 +920,21 @@ public class ApiRestController {
             @HeaderParam("X-ONAP-RequestID") @ApiParam("RequestID for http transaction") UUID requestId,\r
             @ApiParam(value = "Entity body of policy", required = true) ToscaServiceTemplate body) {\r
 \r
+        if (NetLoggerUtil.getNetworkLogger().isInfoEnabled()) {\r
+            NetLoggerUtil.log(EventType.IN, CommInfrastructure.REST,\r
+                            "/policytypes/" + policyTypeId + "/versions/" + policyTypeVersion + "/policies",\r
+                            toJson(body));\r
+        }\r
+\r
         try (PolicyProvider policyProvider = new PolicyProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyProvider\r
                     .createPolicy(policyTypeId, policyTypeVersion, body);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.POST);\r
-            return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-                    .entity(serviceTemplate).build();\r
+            return makeOkResponse(requestId, serviceTemplate);\r
         } catch (PfModelException | PfModelRuntimeException pfme) {\r
+            LOGGER.error("POST /policytypes/{}/versions/{}/policies", policyTypeId, policyTypeVersion, pfme);\r
             updateApiStatisticsCounter(Target.POLICY, Result.FAILURE, HttpMethod.POST);\r
-            return addLoggingHeaders(addVersionControlHeaders(\r
-                    Response.status(pfme.getErrorResponse().getResponseCode())), requestId)\r
-                    .entity(pfme.getErrorResponse()).build();\r
+            return makeErrorResponse(requestId, pfme);\r
         }\r
     }\r
 \r
@@ -941,27 +996,14 @@ public class ApiRestController {
         try (PolicyProvider policyProvider = new PolicyProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyProvider\r
                     .deletePolicy(policyTypeId, policyTypeVersion, policyId, policyVersion);\r
-            return addLoggingHeaders(addVersionControlHeaders(Response.status(Response.Status.OK)), requestId)\r
-                    .entity(serviceTemplate).build();\r
+            return makeOkResponse(requestId, serviceTemplate);\r
         } catch (PfModelException | PfModelRuntimeException pfme) {\r
-            return addLoggingHeaders(addVersionControlHeaders(\r
-                    Response.status(pfme.getErrorResponse().getResponseCode())), requestId)\r
-                    .entity(pfme.getErrorResponse()).build();\r
+            LOGGER.error("DELETE /policytypes/{}/versions/{}/policies/{}/versions/{}", policyTypeId,\r
+                    policyTypeVersion, policyId, policyVersion, pfme);\r
+            return makeErrorResponse(requestId, pfme);\r
         }\r
     }\r
 \r
-    private ResponseBuilder addVersionControlHeaders(ResponseBuilder rb) {\r
-        return rb.header("X-MinorVersion", "0").header("X-PatchVersion", "0").header("X-LatestVersion", "1.0.0");\r
-    }\r
-\r
-    private ResponseBuilder addLoggingHeaders(ResponseBuilder rb, UUID requestId) {\r
-        if (requestId == null) {\r
-            // Generate a random uuid if client does not embed requestId in rest request\r
-            return rb.header("X-ONAP-RequestID", UUID.randomUUID());\r
-        }\r
-        return rb.header("X-ONAP-RequestID", requestId);\r
-    }\r
-\r
     private enum Target {\r
         POLICY, POLICY_TYPE, OTHER\r
     }\r
@@ -977,52 +1019,65 @@ public class ApiRestController {
     private void updateApiStatisticsCounter(Target target, Result result, HttpMethod http) {\r
 \r
         ApiStatisticsManager.updateTotalApiCallCount();\r
-        if (target == Target.POLICY) {\r
-            if (result == Result.SUCCESS) {\r
-                if (http == HttpMethod.GET) {\r
-                    ApiStatisticsManager.updateApiCallSuccessCount();\r
-                    ApiStatisticsManager.updateTotalPolicyGetCount();\r
-                    ApiStatisticsManager.updatePolicyGetSuccessCount();\r
-                } else if (http == HttpMethod.POST) {\r
-                    ApiStatisticsManager.updateApiCallSuccessCount();\r
-                    ApiStatisticsManager.updateTotalPolicyPostCount();\r
-                    ApiStatisticsManager.updatePolicyPostSuccessCount();\r
-                }\r
-            } else {\r
-                if (http == HttpMethod.GET) {\r
-                    ApiStatisticsManager.updateApiCallFailureCount();\r
-                    ApiStatisticsManager.updateTotalPolicyGetCount();\r
-                    ApiStatisticsManager.updatePolicyGetFailureCount();\r
-                } else {\r
-                    ApiStatisticsManager.updateApiCallFailureCount();\r
-                    ApiStatisticsManager.updateTotalPolicyPostCount();\r
-                    ApiStatisticsManager.updatePolicyPostFailureCount();\r
-                }\r
+\r
+        switch (target) {\r
+            case POLICY:\r
+                updatePolicyStats(result, http);\r
+                break;\r
+            case POLICY_TYPE:\r
+                updatePolicyTypeStats(result, http);\r
+                break;\r
+            default:\r
+                ApiStatisticsManager.updateApiCallSuccessCount();\r
+                break;\r
+        }\r
+    }\r
+\r
+    private void updatePolicyStats(Result result, HttpMethod http) {\r
+        if (result == Result.SUCCESS) {\r
+            if (http == HttpMethod.GET) {\r
+                ApiStatisticsManager.updateApiCallSuccessCount();\r
+                ApiStatisticsManager.updateTotalPolicyGetCount();\r
+                ApiStatisticsManager.updatePolicyGetSuccessCount();\r
+            } else if (http == HttpMethod.POST) {\r
+                ApiStatisticsManager.updateApiCallSuccessCount();\r
+                ApiStatisticsManager.updateTotalPolicyPostCount();\r
+                ApiStatisticsManager.updatePolicyPostSuccessCount();\r
             }\r
-        } else if (target == Target.POLICY_TYPE) {\r
-            if (result == Result.SUCCESS) {\r
-                if (http == HttpMethod.GET) {\r
-                    ApiStatisticsManager.updateApiCallSuccessCount();\r
-                    ApiStatisticsManager.updateTotalPolicyTypeGetCount();\r
-                    ApiStatisticsManager.updatePolicyTypeGetSuccessCount();\r
-                } else if (http == HttpMethod.POST) {\r
-                    ApiStatisticsManager.updateApiCallSuccessCount();\r
-                    ApiStatisticsManager.updatePolicyTypePostSuccessCount();\r
-                    ApiStatisticsManager.updatePolicyTypePostSuccessCount();\r
-                }\r
+        } else {\r
+            if (http == HttpMethod.GET) {\r
+                ApiStatisticsManager.updateApiCallFailureCount();\r
+                ApiStatisticsManager.updateTotalPolicyGetCount();\r
+                ApiStatisticsManager.updatePolicyGetFailureCount();\r
             } else {\r
-                if (http == HttpMethod.GET) {\r
-                    ApiStatisticsManager.updateApiCallFailureCount();\r
-                    ApiStatisticsManager.updateTotalPolicyTypeGetCount();\r
-                    ApiStatisticsManager.updatePolicyTypeGetFailureCount();\r
-                } else {\r
-                    ApiStatisticsManager.updateApiCallFailureCount();\r
-                    ApiStatisticsManager.updateTotalPolicyTypePostCount();\r
-                    ApiStatisticsManager.updatePolicyTypePostFailureCount();\r
-                }\r
+                ApiStatisticsManager.updateApiCallFailureCount();\r
+                ApiStatisticsManager.updateTotalPolicyPostCount();\r
+                ApiStatisticsManager.updatePolicyPostFailureCount();\r
+            }\r
+        }\r
+    }\r
+\r
+    private void updatePolicyTypeStats(Result result, HttpMethod http) {\r
+        if (result == Result.SUCCESS) {\r
+            if (http == HttpMethod.GET) {\r
+                ApiStatisticsManager.updateApiCallSuccessCount();\r
+                ApiStatisticsManager.updateTotalPolicyTypeGetCount();\r
+                ApiStatisticsManager.updatePolicyTypeGetSuccessCount();\r
+            } else if (http == HttpMethod.POST) {\r
+                ApiStatisticsManager.updateApiCallSuccessCount();\r
+                ApiStatisticsManager.updatePolicyTypePostSuccessCount();\r
+                ApiStatisticsManager.updatePolicyTypePostSuccessCount();\r
             }\r
         } else {\r
-            ApiStatisticsManager.updateApiCallSuccessCount();\r
+            if (http == HttpMethod.GET) {\r
+                ApiStatisticsManager.updateApiCallFailureCount();\r
+                ApiStatisticsManager.updateTotalPolicyTypeGetCount();\r
+                ApiStatisticsManager.updatePolicyTypeGetFailureCount();\r
+            } else {\r
+                ApiStatisticsManager.updateApiCallFailureCount();\r
+                ApiStatisticsManager.updateTotalPolicyTypePostCount();\r
+                ApiStatisticsManager.updatePolicyTypePostFailureCount();\r
+            }\r
         }\r
     }\r
 }
\ No newline at end of file