From 89df27fc144d1c1a7f6beed0e0bd0038f38ec14e Mon Sep 17 00:00:00 2001 From: Krishnajinka Date: Thu, 26 Jul 2018 13:06:15 +0900 Subject: Fix nested stmts issue of Sonar Policy rest util MSModeUtils have Sonar flagged issues about nesting more than 3 control stmts. Fix these. Also a minor change from prev commit about BrmsPush.java. Rework based on review Issue-ID: POLICY-1009 Change-Id: Ibd3cebcdebf9cf26233e810033793a5315b93272 Signed-off-by: Krishnajinka --- BRMSGateway/src/main/java/org/onap/policy/brms/api/BrmsPush.java | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'BRMSGateway') diff --git a/BRMSGateway/src/main/java/org/onap/policy/brms/api/BrmsPush.java b/BRMSGateway/src/main/java/org/onap/policy/brms/api/BrmsPush.java index b8706bb9b..507e17fea 100644 --- a/BRMSGateway/src/main/java/org/onap/policy/brms/api/BrmsPush.java +++ b/BRMSGateway/src/main/java/org/onap/policy/brms/api/BrmsPush.java @@ -787,9 +787,7 @@ public class BrmsPush { LOGGER.error("Error while starting Transaction " + e); } if (!modifiedGroups.isEmpty()) { - Boolean flag; - flag = buildAndGenerateJarFile(); - if (flag) { + if (buildAndGenerateJarFile()) { sendNotification(controllers); } } -- cgit 1.2.3-korg