More sonar cleanup and line consolidation
[policy/engine.git] / POLICY-SDK-APP / src / main / java / org / onap / policy / controller / CreateBRMSParamController.java
index b49f1fc..fa515b2 100644 (file)
@@ -2,15 +2,16 @@
  * ============LICENSE_START=======================================================
  * ONAP Policy Engine
  * ================================================================================
- * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved.
+ * Copyright (C) 2017-2019 AT&T Intellectual Property. All rights reserved.
  * Modified Copyright (C) 2018 Samsung Electronics Co., Ltd.
+ * Modifications Copyright (C) 2019 Bell Canada
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 
 package org.onap.policy.controller;
 
-import java.io.PrintWriter;
+import com.fasterxml.jackson.databind.DeserializationFeature;
+import com.fasterxml.jackson.databind.JsonNode;
+import com.fasterxml.jackson.databind.ObjectMapper;
+
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Collection;
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Map.Entry;
+import java.util.Objects;
 import java.util.Set;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
+import java.util.stream.Collectors;
 
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 import javax.xml.bind.JAXBElement;
 
+import oasis.names.tc.xacml._3_0.core.schema.wd_17.AdviceExpressionType;
+import oasis.names.tc.xacml._3_0.core.schema.wd_17.AdviceExpressionsType;
+import oasis.names.tc.xacml._3_0.core.schema.wd_17.AnyOfType;
+import oasis.names.tc.xacml._3_0.core.schema.wd_17.AttributeAssignmentExpressionType;
+import oasis.names.tc.xacml._3_0.core.schema.wd_17.AttributeDesignatorType;
+import oasis.names.tc.xacml._3_0.core.schema.wd_17.AttributeValueType;
+import oasis.names.tc.xacml._3_0.core.schema.wd_17.MatchType;
+import oasis.names.tc.xacml._3_0.core.schema.wd_17.PolicyType;
+import oasis.names.tc.xacml._3_0.core.schema.wd_17.RuleType;
+import oasis.names.tc.xacml._3_0.core.schema.wd_17.TargetType;
+
+import org.apache.commons.collections.CollectionUtils;
 import org.json.JSONObject;
 import org.onap.policy.common.logging.flexlogger.FlexLogger;
 import org.onap.policy.common.logging.flexlogger.Logger;
@@ -50,26 +69,12 @@ import org.onap.portalsdk.core.controller.RestrictedBaseController;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Controller;
 import org.springframework.web.bind.annotation.RequestMapping;
-
-import com.fasterxml.jackson.databind.DeserializationFeature;
-import com.fasterxml.jackson.databind.JsonNode;
-import com.fasterxml.jackson.databind.ObjectMapper;
-
-import oasis.names.tc.xacml._3_0.core.schema.wd_17.AdviceExpressionType;
-import oasis.names.tc.xacml._3_0.core.schema.wd_17.AdviceExpressionsType;
-import oasis.names.tc.xacml._3_0.core.schema.wd_17.AllOfType;
-import oasis.names.tc.xacml._3_0.core.schema.wd_17.AnyOfType;
-import oasis.names.tc.xacml._3_0.core.schema.wd_17.AttributeAssignmentExpressionType;
-import oasis.names.tc.xacml._3_0.core.schema.wd_17.AttributeDesignatorType;
-import oasis.names.tc.xacml._3_0.core.schema.wd_17.AttributeValueType;
-import oasis.names.tc.xacml._3_0.core.schema.wd_17.MatchType;
-import oasis.names.tc.xacml._3_0.core.schema.wd_17.PolicyType;
-import oasis.names.tc.xacml._3_0.core.schema.wd_17.RuleType;
-import oasis.names.tc.xacml._3_0.core.schema.wd_17.TargetType;
+import org.springframework.web.bind.annotation.RequestMethod;
 
 @Controller
 @RequestMapping("/")
 public class CreateBRMSParamController extends RestrictedBaseController {
+
     private static final Logger policyLogger = FlexLogger.getLogger(CreateBRMSParamController.class);
 
     private static CommonClassDao commonClassDao;
@@ -83,13 +88,14 @@ public class CreateBRMSParamController extends RestrictedBaseController {
     }
 
     @Autowired
-    private CreateBRMSParamController(CommonClassDao commonClassDao){
+    private CreateBRMSParamController(CommonClassDao commonClassDao) {
         CreateBRMSParamController.commonClassDao = commonClassDao;
     }
 
-    public CreateBRMSParamController(){
+    public CreateBRMSParamController() {
         // Empty constructor
     }
+
     protected PolicyRestAdapter policyAdapter = null;
 
     private HashMap<String, String> dynamicLayoutMap;
@@ -97,10 +103,15 @@ public class CreateBRMSParamController extends RestrictedBaseController {
     private static String brmsTemplateVlaue = "<$%BRMSParamTemplate=";
     private static String string = "String";
 
-
-    @RequestMapping(value={"/policyController/getBRMSTemplateData.htm"}, method={org.springframework.web.bind.annotation.RequestMethod.POST})
-    public void getBRMSParamPolicyRuleData(HttpServletRequest request, HttpServletResponse response){
-        try{
+    /**
+     * getBRMSParamPolicyRuleData.
+     *
+     * @param request HttpServletRequest
+     * @param response HttpServletResponse
+     */
+    @RequestMapping(value = {"/policyController/getBRMSTemplateData.htm"}, method = {RequestMethod.POST})
+    public void getBRMSParamPolicyRuleData(HttpServletRequest request, HttpServletResponse response) {
+        try {
             dynamicLayoutMap = new HashMap<>();
             ObjectMapper mapper = new ObjectMapper();
             mapper.configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);
@@ -111,26 +122,23 @@ public class CreateBRMSParamController extends RestrictedBaseController {
             response.setContentType(PolicyController.getContenttype());
             request.setCharacterEncoding(PolicyController.getCharacterencoding());
 
-            PrintWriter out = response.getWriter();
-            String responseString = mapper.writeValueAsString(dynamicLayoutMap);
-            JSONObject j = new JSONObject("{policyData: " + responseString + "}");
-            out.write(j.toString());
-        }catch(Exception e){
-            policyLogger.error("Exception Occured while getting BRMS Rule data" , e);
+            response.getWriter().write(new JSONObject("{policyData: " + mapper.writeValueAsString(dynamicLayoutMap)
+                + "}").toString());
+        } catch (Exception e) {
+            policyLogger.error("Exception Occured while getting BRMS Rule data", e);
         }
     }
 
-    protected String findRule(String ruleTemplate) {
+    private String findRule(String ruleTemplate) {
         List<Object> datas = commonClassDao.getDataById(BRMSParamTemplate.class, "ruleName", ruleTemplate);
-        if(datas != null && !datas.isEmpty()){
-            BRMSParamTemplate  bRMSParamTemplate = (BRMSParamTemplate) datas.get(0);
-            return bRMSParamTemplate.getRule();
+        if (CollectionUtils.isNotEmpty(datas)) {
+            return ((BRMSParamTemplate) datas.get(0)).getRule();
         }
         return null;
     }
 
-    protected void generateUI(String rule) {
-        if (rule==null){
+    private void generateUI(String rule) {
+        if (rule == null) {
             return;
         }
         try {
@@ -142,11 +150,44 @@ public class CreateBRMSParamController extends RestrictedBaseController {
     }
 
     private void processRule(String rule) {
-        StringBuilder params = new StringBuilder("");
-        Boolean flag = false;
-        Boolean comment = false;
+        StringBuilder params = getParamsBuilderFromRule(rule);
+        params = new StringBuilder(
+                params.toString().replace("declare Params", "").replace("end", "").replaceAll("\\s+", ""));
+        String[] components = params.toString().split(":");
+        String caption = "";
+        for (int i = 0; i < components.length; i++) {
+            String type;
+            if (i == 0) {
+                caption = components[i];
+            }
+            if ("".equals(caption)) {
+                break;
+            }
+            String nextComponent;
+            try {
+                nextComponent = components[i + 1];
+            } catch (Exception e) {
+                policyLogger.info("Error when procesing rule: " + e);
+                nextComponent = components[i];
+            }
+            if (nextComponent.startsWith(string)) {
+                type = "String";
+                createField(caption, type);
+                caption = nextComponent.replace(string, "");
+            } else if (nextComponent.startsWith("int")) {
+                type = "int";
+                createField(caption, type);
+                caption = nextComponent.replace("int", "");
+            }
+        }
+    }
+
+    private StringBuilder getParamsBuilderFromRule(final String rule) {
+        StringBuilder params = new StringBuilder();
+        boolean flag = false;
+        boolean comment = false;
         String[] lines = rule.split("\n");
-        for(String line : lines){
+        for (String line : lines) {
             if (line.isEmpty() || line.startsWith("//")) {
                 continue;
             }
@@ -184,34 +225,7 @@ public class CreateBRMSParamController extends RestrictedBaseController {
                 break;
             }
         }
-        params = new StringBuilder(params.toString().replace("declare Params", "").replace("end", "").replaceAll("\\s+", ""));
-        String[] components = params.toString().split(":");
-        String caption = "";
-        for (int i = 0; i < components.length; i++) {
-            String type;
-            if (i == 0) {
-                caption = components[i];
-            }
-            if("".equals(caption)){
-                break;
-            }
-            String nextComponent = "";
-            try {
-                nextComponent = components[i + 1];
-            } catch (Exception e) {
-                policyLogger.info("Just for Logging"+e);
-                nextComponent = components[i];
-            }
-            if (nextComponent.startsWith(string)) {
-                type = "String";
-                createField(caption, type);
-                caption = nextComponent.replace(string, "");
-            } else if (nextComponent.startsWith("int")) {
-                type = "int";
-                createField(caption, type);
-                caption = nextComponent.replace("int", "");
-            }
-        }
+        return params;
     }
 
     private String splitMultiLineStartComment(String line) {
@@ -231,7 +245,8 @@ public class CreateBRMSParamController extends RestrictedBaseController {
     }
 
     /*
-     * When the User Click Edit or View Policy the following method will get invoked for setting the data to PolicyRestAdapter.
+     * When the User Click Edit or View Policy the following method will get invoked for setting the data to
+     * PolicyRestAdapter.
      * Which is used to bind the data in GUI
      */
     public void prePopulateBRMSParamPolicyData(PolicyRestAdapter policyAdapter, PolicyEntity entity) {
@@ -241,28 +256,37 @@ public class CreateBRMSParamController extends RestrictedBaseController {
             policyAdapter.setOldPolicyFileName(policyAdapter.getPolicyName());
             // policy name value is the policy name without any prefix and
             // Extensions.
-            String policyNameValue = policyAdapter.getPolicyName().substring(policyAdapter.getPolicyName().indexOf("BRMS_Param_") + 11);
+            String policyNameValue =
+                    policyAdapter.getPolicyName().substring(policyAdapter.getPolicyName().indexOf("BRMS_Param_") + 11);
             if (policyLogger.isDebugEnabled()) {
-                policyLogger.debug("Prepopulating form data for BRMS RAW Policy selected:" + policyAdapter.getPolicyName());
+                policyLogger
+                        .debug("Prepopulating form data for BRMS RAW Policy selected:" + policyAdapter.getPolicyName());
             }
             policyAdapter.setPolicyName(policyNameValue);
-            String description = "";
-            try{
+            String description;
+            try {
                 description = policy.getDescription().substring(0, policy.getDescription().indexOf("@CreatedBy:"));
-            }catch(Exception e){
-                policyLogger.info("Just for Logging"+e);
+            } catch (Exception e) {
+                policyLogger.info("Error getting description: " + e);
                 description = policy.getDescription();
             }
             policyAdapter.setPolicyDescription(description);
             setDataAdapterFromAdviceExpressions(policy, policyAdapter);
-            paramUIGenerate(policyAdapter, entity);
+
+            // Generate Param UI
+            try {
+                paramUiGenerate(policyAdapter, entity);
+            } catch (Exception e) {
+                policyLogger.error(XACMLErrorConstants.ERROR_DATA_ISSUE + e.getMessage() + e);
+            }
+
             // Get the target data under policy.
             policyAdapter.setDynamicLayoutMap(dynamicLayoutMap);
-            if(policyAdapter.getDynamicLayoutMap().size() > 0){
-                LinkedHashMap<String,String> drlRule = new LinkedHashMap<>();
-                for(Object keyValue: policyAdapter.getDynamicLayoutMap().keySet()){
-                    drlRule.put(keyValue.toString(), policyAdapter.getDynamicLayoutMap().get(keyValue));
-                }
+            if (policyAdapter.getDynamicLayoutMap().size() > 0) {
+                LinkedHashMap<String, String> drlRule = policyAdapter.getDynamicLayoutMap().keySet().stream()
+                        .collect(Collectors.toMap(String::toString,
+                            keyValue -> policyAdapter.getDynamicLayoutMap().get(keyValue), (a, b) -> b,
+                                LinkedHashMap::new));
                 policyAdapter.setRuleData(drlRule);
             }
             TargetType target = policy.getTarget();
@@ -272,28 +296,31 @@ public class CreateBRMSParamController extends RestrictedBaseController {
         }
     }
 
-    private void setDataAdapterFromAdviceExpressions(PolicyType policy, PolicyRestAdapter policyAdapter){
+    private void setDataAdapterFromAdviceExpressions(PolicyType policy, PolicyRestAdapter policyAdapter) {
         ArrayList<Object> attributeList = new ArrayList<>();
         // Set Attributes.
-        AdviceExpressionsType expressionTypes = ((RuleType)policy.getCombinerParametersOrRuleCombinerParametersOrVariableDefinition().get(0)).getAdviceExpressions();
-        for( AdviceExpressionType adviceExpression: expressionTypes.getAdviceExpression()){
-            for(AttributeAssignmentExpressionType attributeAssignment: adviceExpression.getAttributeAssignmentExpression()){
-                if(attributeAssignment.getAttributeId().startsWith("key:")){
+        AdviceExpressionsType expressionTypes =
+                ((RuleType) policy.getCombinerParametersOrRuleCombinerParametersOrVariableDefinition().get(0))
+                        .getAdviceExpressions();
+        for (AdviceExpressionType adviceExpression : expressionTypes.getAdviceExpression()) {
+            for (AttributeAssignmentExpressionType attributeAssignment : adviceExpression
+                    .getAttributeAssignmentExpression()) {
+                if (attributeAssignment.getAttributeId().startsWith("key:")) {
                     Map<String, String> attribute = new HashMap<>();
                     String key = attributeAssignment.getAttributeId().replace("key:", "");
                     attribute.put("key", key);
                     @SuppressWarnings("unchecked")
-                    JAXBElement<AttributeValueType> attributevalue = (JAXBElement<AttributeValueType>) attributeAssignment.getExpression();
-                    String value = (String) attributevalue.getValue().getContent().get(0);
+                    JAXBElement<AttributeValueType> attributeValue =
+                            (JAXBElement<AttributeValueType>) attributeAssignment.getExpression();
+                    String value = (String) attributeValue.getValue().getContent().get(0);
                     attribute.put("value", value);
                     attributeList.add(attribute);
-                }else if(attributeAssignment.getAttributeId().startsWith("dependencies:")){
-                    ArrayList<String> dependencies = new ArrayList<>(Arrays.asList(attributeAssignment.getAttributeId().replace("dependencies:", "").split(",")));
-                    if(dependencies.contains("")){
-                        dependencies.remove("");
-                    }
+                } else if (attributeAssignment.getAttributeId().startsWith("dependencies:")) {
+                    ArrayList<String> dependencies = new ArrayList<>(Arrays
+                            .asList(attributeAssignment.getAttributeId().replace("dependencies:", "").split(",")));
+                    dependencies.remove("");
                     policyAdapter.setBrmsDependency(dependencies);
-                }else if(attributeAssignment.getAttributeId().startsWith("controller:")){
+                } else if (attributeAssignment.getAttributeId().startsWith("controller:")) {
                     policyAdapter.setBrmsController(attributeAssignment.getAttributeId().replace("controller:", ""));
                 }
             }
@@ -301,38 +328,20 @@ public class CreateBRMSParamController extends RestrictedBaseController {
         }
     }
 
-    private void setDataToAdapterFromTarget(TargetType target, PolicyRestAdapter policyAdapter){
+    private void setDataToAdapterFromTarget(TargetType target, PolicyRestAdapter policyAdapter) {
         // Under target we have AnyOFType
-        List<AnyOfType> anyOfList = target.getAnyOf();
-
-        if (anyOfList == null) {
+        if (target.getAnyOf() == null) {
             return;
         }
-
-        Iterator<AnyOfType> iterAnyOf = anyOfList.iterator();
-        while (iterAnyOf.hasNext()) {
-            AnyOfType anyOf = iterAnyOf.next();
-            // Under AnyOFType we have AllOFType
-            List<AllOfType> allOfList = anyOf.getAllOf();
-            if (allOfList == null) {
-                continue;
-            }
-            Iterator<AllOfType> iterAllOf = allOfList.iterator();
-            while (iterAllOf.hasNext()) {
-                AllOfType allOf = iterAllOf.next();
-                // Under AllOFType we have Match
-                List<MatchType> matchList = allOf.getMatch();
-                if (matchList != null) {
-                    setDataToAdapterFromMatchList(matchList, policyAdapter);
-                }
-            }
-        }
+        target.getAnyOf().stream().map(AnyOfType::getAllOf).filter(Objects::nonNull).flatMap(Collection::stream)
+                .forEach(allOf -> setDataToAdapterFromMatchList(allOf.getMatch(), policyAdapter));
     }
 
-    private void setDataToAdapterFromMatchList(List<MatchType> matchList, PolicyRestAdapter policyAdapter){
-        Iterator<MatchType> iterMatch = matchList.iterator();
-        while (iterMatch.hasNext()) {
-            MatchType match = iterMatch.next();
+    private void setDataToAdapterFromMatchList(List<MatchType> matchList, PolicyRestAdapter policyAdapter) {
+        if (matchList == null) {
+            return;
+        }
+        for (final MatchType match : matchList) {
             //
             // Under the match we have attribute value and
             // attributeDesignator. So,finally down to the actual attribute.
@@ -341,17 +350,16 @@ public class CreateBRMSParamController extends RestrictedBaseController {
             String value = (String) attributeValue.getContent().get(0);
             AttributeDesignatorType designator = match.getAttributeDesignator();
             String attributeId = designator.getAttributeId();
-
-            if ("RiskType".equals(attributeId)){
+            if ("RiskType".equals(attributeId)) {
                 policyAdapter.setRiskType(value);
             }
-            if ("RiskLevel".equals(attributeId)){
+            if ("RiskLevel".equals(attributeId)) {
                 policyAdapter.setRiskLevel(value);
             }
-            if ("guard".equals(attributeId)){
+            if ("guard".equals(attributeId)) {
                 policyAdapter.setGuard(value);
             }
-            if ("TTLDate".equals(attributeId) && !value.contains("NA")){
+            if ("TTLDate".equals(attributeId) && !value.contains("NA")) {
                 PolicyController controller = new PolicyController();
                 String newDate = controller.convertDate(value);
                 policyAdapter.setTtlDate(newDate);
@@ -360,109 +368,97 @@ public class CreateBRMSParamController extends RestrictedBaseController {
     }
 
     // This method generates the UI from rule configuration
-    public void paramUIGenerate(PolicyRestAdapter policyAdapter, PolicyEntity entity) {
+    private void paramUiGenerate(PolicyRestAdapter policyAdapter, PolicyEntity entity) {
         String data = entity.getConfigurationData().getConfigBody();
-        if(data == null){
+        if (data == null) {
             return;
         }
-
-        try {
-            StringBuilder params = new StringBuilder("");
-            Boolean flag = false;
-            Boolean comment = false;
-            for (String line : data.split("\n")) {
-                if (line.isEmpty() || line.startsWith("//")) {
-                    continue;
-                }
-                if(line.contains(brmsTemplateVlaue)){
-                    String value = line.substring(line.indexOf("<$%"),line.indexOf("%$>"));
-                    value = value.replace(brmsTemplateVlaue, "");
-                    policyAdapter.setRuleName(value);
-                }
-                if(line.contains("<%$Values=")) {
-                    String value = line.substring(line.indexOf("<%$"), line.indexOf("$%>"));
-                    value = value.replaceAll("<%\\$Values=", "");
-                    for( String keyValue : value.split(":\\|:")) {
-                        String[] pair = keyValue.split(":-:");
-                        if (pair != null && pair.length > 0) {
-                            dynamicLayoutMap.put(pair[0], (pair.length > 1) ? pair[1] : "");
-                        }
-                    }
-                    return;
-                }
-                if (line.startsWith("/*")) {
-                    comment = true;
-                    continue;
-                }
-                if ((line.contains("//"))&&(!(line.contains("http://") || line.contains("https://")))){
-                    line = splitSingleLineComment(line);
-                }
-                if (line.contains("/*")) {
-                    comment = true;
-                    if (line.contains("*/")) {
-                        comment = false;
-                        line = processMultiLineFullComment(line);
-                    } else {
-                        line = splitMultiLineStartComment(line);
-                    }
-                }
+        StringBuilder params = new StringBuilder();
+        boolean flag = false;
+        boolean comment = false;
+        for (String line : data.split("\n")) {
+            if (line.isEmpty() || line.startsWith("//")) {
+                continue;
+            }
+            if (line.contains(brmsTemplateVlaue)) {
+                String value = line.substring(line.indexOf("<$%"), line.indexOf("%$>"));
+                value = value.replace(brmsTemplateVlaue, "");
+                policyAdapter.setRuleName(value);
+            }
+            if (line.contains("<%$Values=")) {
+                String value = line.substring(line.indexOf("<%$"), line.indexOf("$%>"));
+                value = value.replaceAll("<%\\$Values=", "");
+                Arrays.stream(value.split(":\\|:")).map(keyValue -> keyValue.split(":-:"))
+                        .filter(pair -> pair.length > 0)
+                        .forEach(pair -> dynamicLayoutMap.put(pair[0], (pair.length > 1) ? pair[1] : ""));
+                return;
+            }
+            if (line.startsWith("/*")) {
+                comment = true;
+                continue;
+            }
+            if ((line.contains("//")) && (!(line.contains("http://") || line.contains("https://")))) {
+                line = splitSingleLineComment(line);
+            }
+            if (line.contains("/*")) {
+                comment = true;
                 if (line.contains("*/")) {
                     comment = false;
-                    line = processEndComment(line);
-                }
-                if (comment) {
-                    continue;
-                }
-                if (flag) {
-                    params.append(line);
-                }
-                if (line.contains("rule") && line.contains(".Params\"")) {
-                    params.append(line);
-                    flag = true;
-                }
-                if (line.contains("end") && flag) {
-                    break;
+                    line = processMultiLineFullComment(line);
+                } else {
+                    line = splitMultiLineStartComment(line);
                 }
             }
-            params = new StringBuilder(params.substring(params.indexOf(".Params\"")+ 11));
-            params = new StringBuilder(params.toString().replaceAll("\\s+", "").replace("salience1000whenthenParamsparams=newParams();","")
-                    .replace("insert(params);end", "")
-                    .replace("params.set", ""));
-            String[] components = params.toString().split("\\);");
-            if(components!= null && components.length > 0){
-                for (int i = 0; i < components.length; i++) {
-                    String value;
-                    components[i] = components[i]+")";
-                    String caption = components[i].substring(0,
-                            components[i].indexOf('('));
-                    caption = caption.substring(0, 1).toLowerCase() + caption.substring(1);
-                    if (components[i].contains("(\"")) {
-                        value = components[i]
-                                .substring(components[i].indexOf("(\""),
-                                        components[i].indexOf("\")"))
-                                .replace("(\"", "").replace("\")", "");
-                    } else {
-                        value = components[i]
-                                .substring(components[i].indexOf('('),
-                                        components[i].indexOf(')'))
-                                .replace("(", "").replace(")", "");
-                    }
-                    dynamicLayoutMap.put(caption, value);
+            if (line.contains("*/")) {
+                comment = false;
+                line = processEndComment(line);
+            }
+            if (comment) {
+                continue;
+            }
+            if (flag) {
+                params.append(line);
+            }
+            if (line.contains("rule") && line.contains(".Params\"")) {
+                params.append(line);
+                flag = true;
+            }
+            if (line.contains("end") && flag) {
+                break;
+            }
+        }
+        params = new StringBuilder(params.substring(params.indexOf(".Params\"") + 11));
+        params = new StringBuilder(
+                params.toString().replaceAll("\\s+", "").replace("salience1000whenthenParamsparams=newParams();", "")
+                        .replace("insert(params);end", "").replace("params.set", ""));
+        updateCaptionToDynamicLayoutMap(params);
+    }
 
+    private void updateCaptionToDynamicLayoutMap(final StringBuilder params) {
+        String[] components = params.toString().split("\\);");
+        if (components.length > 0) {
+            for (int i = 0; i < components.length; i++) {
+                String value;
+                components[i] = components[i] + ")";
+                String caption = components[i].substring(0, components[i].indexOf('('));
+                caption = caption.substring(0, 1).toLowerCase() + caption.substring(1);
+                if (components[i].contains("(\"")) {
+                    value = components[i].substring(components[i].indexOf("(\""), components[i].indexOf("\")"))
+                            .replace("(\"", "").replace("\")", "");
+                } else {
+                    value = components[i].substring(components[i].indexOf('('), components[i].indexOf(')'))
+                            .replace("(", "").replace(")", "");
                 }
+                dynamicLayoutMap.put(caption, value);
             }
-        } catch (Exception e) {
-            policyLogger.error(XACMLErrorConstants.ERROR_DATA_ISSUE + e.getMessage() + e);
         }
-
-
     }
 
     private String processEndComment(String line) {
         try {
             line = splitMultiLineEndComment(line);
         } catch (Exception e) {
-            policyLogger.info("Just for Logging"+e);
+            policyLogger.info("Just for Logging" + e);
             line = "";
         }
         return line;
@@ -470,41 +466,48 @@ public class CreateBRMSParamController extends RestrictedBaseController {
 
     private String processMultiLineFullComment(String line) {
         try {
-            line = splitMultiLineStartComment(line)
-                    + splitMultiLineEndComment(line);
+            line = splitMultiLineStartComment(line) + splitMultiLineEndComment(line);
         } catch (Exception e) {
-            policyLogger.info("Just for Logging"+e);
+            policyLogger.info("Just for Logging" + e);
             line = splitMultiLineStartComment(line);
         }
         return line;
     }
 
-    // set View Rule
+    /**
+     * setViewRule.
+     *
+     * @param request HttpServletRequest
+     * @param response HttpServletResponse
+     */
     @SuppressWarnings("unchecked")
-    @RequestMapping(value={"/policyController/ViewBRMSParamPolicyRule.htm"}, method={org.springframework.web.bind.annotation.RequestMethod.POST})
-    public void setViewRule(HttpServletRequest request, HttpServletResponse response){
+    @RequestMapping(value = {"/policyController/ViewBRMSParamPolicyRule.htm"}, method = {RequestMethod.POST})
+    public void setViewRule(HttpServletRequest request, HttpServletResponse response) {
         try {
             ObjectMapper mapper = new ObjectMapper();
             mapper.configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);
             JsonNode root = mapper.readTree(request.getReader());
-            PolicyRestAdapter policyData = mapper.readValue(root.get(PolicyController.getPolicydata()).get("policy").toString(), PolicyRestAdapter.class);
-            policyData.setDomainDir(root.get(PolicyController.getPolicydata()).get("model").get("name").toString().replace("\"", ""));
-            if(root.get(PolicyController.getPolicydata()).get("model").get("type").toString().replace("\"", "").equals(PolicyController.getFile())){
+            PolicyRestAdapter policyData = mapper.readValue(
+                    root.get(PolicyController.getPolicydata()).get("policy").toString(), PolicyRestAdapter.class);
+            policyData.setDomainDir(
+                    root.get(PolicyController.getPolicydata()).get("model").get("name").toString().replace("\"", ""));
+            if (root.get(PolicyController.getPolicydata()).get("model").get("type").toString().replace("\"", "")
+                    .equals(PolicyController.getFile())) {
                 policyData.setEditPolicy(true);
             }
 
-            String body;
-            body = findRule((String) policyData.getRuleName()) + "\n";
-            StringBuilder generatedMetadata = new StringBuilder();
-            generatedMetadata.append("/* Autogenerated Code Please Don't change/remove this comment section. This is for the UI purpose. \n\t " +
-                    brmsTemplateVlaue + policyData.getRuleName() + "%$> \n */ \n");
+            String body = findRule(policyData.getRuleName()) + "\n";
+            StringBuilder generatedMetadata = new StringBuilder().append(
+                    "/* Autogenerated Code Please Don't change/remove this comment section. "
+                            + "This is for the UI purpose. \n\t ")
+                    .append(brmsTemplateVlaue).append(policyData.getRuleName()).append("%$> \n */ \n");
 
-            if(policyData.getDynamicLayoutMap().size() > 0){
+            if (policyData.getDynamicLayoutMap().size() > 0) {
                 generatedMetadata.append("/* <%$Values=");
                 for (Entry<?, ?> entry : policyData.getRuleData().entrySet()) {
                     String uiKey = (String) entry.getKey();
-                    if(!"templateName".equals(uiKey)) {
-                        generatedMetadata.append(uiKey+":-:"+entry.getValue()+":|:");
+                    if (!"templateName".equals(uiKey)) {
+                        generatedMetadata.append(uiKey).append(":-:").append(entry.getValue()).append(":|:");
                     }
                 }
                 generatedMetadata.append("$%> \n*/ \n");
@@ -512,35 +515,31 @@ public class CreateBRMSParamController extends RestrictedBaseController {
             policyLogger.info("Metadata generated with :" + generatedMetadata.toString());
             body = generatedMetadata.toString() + body;
             // Expand the body.
-            Map<String,String> copyMap=new HashMap<>();
-            copyMap.putAll((Map<? extends String, ? extends String>) policyData.getRuleData());
-            copyMap.put("policyName", policyData.getDomainDir().replace("\\", ".") +".Config_BRMS_Param_" + policyData.getPolicyName());
+            Map<String, String> copyMap =
+                    new HashMap<>((Map<? extends String, ? extends String>) policyData.getRuleData());
+            copyMap.put("policyName",
+                    policyData.getDomainDir().replace("\\", ".") + ".Config_BRMS_Param_" + policyData.getPolicyName());
             copyMap.put("policyScope", policyData.getDomainDir().replace("\\", "."));
             copyMap.put("policyVersion", "1");
-            //Finding all the keys in the Map data-structure.
-            Set<String> keySet= copyMap.keySet();
+            // Finding all the keys in the Map data-structure.
+            Set<String> keySet = copyMap.keySet();
             Iterator<String> iterator = keySet.iterator();
-            Pattern p;
-            Matcher m;
-            while(iterator.hasNext()) {
-                //Converting the first character of the key into a lower case.
-                String input= iterator.next();
-                String output  = Character.toLowerCase(input.charAt(0)) +
-                        (input.length() > 1 ? input.substring(1) : "");
-                //Searching for a pattern in the String using the key.
-                p=Pattern.compile("\\$\\{"+output+"\\}");
-                m=p.matcher(body);
-                //Replacing the value with the inputs provided by the user in the editor.
-                body=m.replaceAll(copyMap.get(input));
+            while (iterator.hasNext()) {
+                // Converting the first character of the key into a lower case.
+                String input = iterator.next();
+                String output = Character.toLowerCase(input.charAt(0)) + (input.length() > 1 ? input.substring(1) : "");
+                // Searching for a pattern in the String using the key.
+                Pattern pattern = Pattern.compile("\\$\\{" + output + "\\}");
+                Matcher matcher = pattern.matcher(body);
+                // Replacing the value with the inputs provided by the user in the editor.
+                body = matcher.replaceAll(copyMap.get(input));
             }
             response.setCharacterEncoding("UTF-8");
             response.setContentType("application / json");
             request.setCharacterEncoding("UTF-8");
 
-            PrintWriter out = response.getWriter();
-            String responseString = mapper.writeValueAsString(body);
-            JSONObject j = new JSONObject("{policyData: " + responseString + "}");
-            out.write(j.toString());
+            response.getWriter().write(new JSONObject("{policyData: " + mapper.writeValueAsString(body)
+                + "}").toString());
         } catch (Exception e) {
             policyLogger.error(XACMLErrorConstants.ERROR_PROCESS_FLOW + e);
         }