Fix all bugs reported by Sonar in policy/engine

[policy/engine.git] / ONAP-XACML / src / main / java / org / onap / policy / xacml / std / pap / StdEngine.java

diff --git a/ONAP-XACML/src/main/java/org/onap/policy/xacml/std/pap/StdEngine.java b/ONAP-XACML/src/main/java/org/onap/policy/xacml/std/pap/StdEngine.java
index 273383a..d3dc7ae 100644 (file)
--- a/ONAP-XACML/src/main/java/org/onap/policy/xacml/std/pap/StdEngine.java
+++ b/ONAP-XACML/src/main/java/org/onap/policy/xacml/std/pap/StdEngine.java
@@ -180,18 +180,8 @@ public class StdEngine extends StdPDPItemSetChangeNotifier implements PAPPolicyE
                // We don't have the default group, create it
                //
                String defaultId = properties.getProperty(PROP_PAP_GROUPS_DEFAULT, PROP_PAP_GROUPS_DEFAULT_NAME);
-               if(defaultId == null){
-                       defaultId = PROP_PAP_GROUPS_DEFAULT_NAME;
-               }
-               if("".equals(defaultId)){
-                       defaultId = PROP_PAP_GROUPS_DEFAULT_NAME;
-               }
-               //we're going to check one more time in case the PROP_PAP_GROUPS_DEFAULT_NAME doesn't exist
-               if(defaultId == null){
-                       defaultId = "default";
-               }
                if("".equals(defaultId)){
-                       defaultId = "default";
+                  defaultId = PROP_PAP_GROUPS_DEFAULT_NAME;
                }
                logger.warn("Default group does NOT exist, creating " + defaultId);
                Path defaultPath = Paths.get(this.repository.toString(), defaultId);
@@ -349,7 +339,7 @@ public class StdEngine extends StdPDPItemSetChangeNotifier implements PAPPolicyE
                                        props.store(os, "");
                                }
                        } catch (IOException e) {
-                               PolicyLogger.error(MessageCodes.EXCEPTION_ERROR, e, "StdEngine", "Failed to create " + policyProperties);
+                               PolicyLogger.error(MessageCodes.EXCEPTION_ERROR, e, "StdEngine", "Failed to create policyProperties");
                                throw new PAPException("Failed to create " + id);
                        }
                }
@@ -368,7 +358,7 @@ public class StdEngine extends StdPDPItemSetChangeNotifier implements PAPPolicyE
                                        props.store(os, "");
                                }
                        } catch (IOException e) {
-                               PolicyLogger.error(MessageCodes.ERROR_DATA_ISSUE, e, "StdEngine", "Failed to create " + pipProperties);
+                               PolicyLogger.error(MessageCodes.ERROR_DATA_ISSUE, e, "StdEngine", "Failed to create pipProperties");
                                throw new PAPException("Failed to create " + id);
                        }