diff options
| author |   Krishnajinka <kris.jinka@samsung.com> | 2018-07-26 13:06:15 +0900 |
|---|---|---|
| committer | Krishnajinka <kris.jinka@samsung.com> | 2018-07-26 19:59:54 +0900 |
| commit | 89df27fc144d1c1a7f6beed0e0bd0038f38ec14e (patch) | |
| tree | 07183a6c9e8c7c04d5ff379a58b0ef65a7287621 /BRMSGateway | |
| parent | 7957dd169d09ff5fcfdfc0384e8ed7d3534d3dc0 (diff) | |
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 <kris.jinka@samsung.com>
Diffstat (limited to 'BRMSGateway')
| -rw-r--r-- | BRMSGateway/src/main/java/org/onap/policy/brms/api/BrmsPush.java | 4 |
1 files changed, 1 insertions, 3 deletions
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); } } |