Fix major sonar issues in SDK APP

Fix major issues reported by sonar in POLICY SDK APP files such as
CreateBRMSParamController, PolicyManagerServlet. Also modify tabs to
space. Rework2 based on review comments

Issue-ID: POLICY-1016
Change-Id: I5407ade1ce9f597fa91d9ca18506f5ac2d8b1411
Signed-off-by: Krishnajinka <kris.jinka@samsung.com>

2 files changed, 714 insertions, 666 deletions

diff --git a/POLICY-SDK-APP/src/main/java/org/onap/policy/admin/PolicyManagerServlet.java b/POLICY-SDK-APP/src/main/java/org/onap/policy/admin/PolicyManagerServlet.java
index de83b045b..2492ab024 100644
--- a/POLICY-SDK-APP/src/main/java/org/onap/policy/admin/PolicyManagerServlet.java
+++ b/POLICY-SDK-APP/src/main/java/org/onap/policy/admin/PolicyManagerServlet.java
@@ -326,8 +326,6 @@ public class PolicyManagerServlet extends HttpServlet {
     }
 
     private JSONObject searchPolicyList(JSONObject params, HttpServletRequest request) {
-        Set<String> scopes;
-        List<String> roles;
         List<Object> policyData = new ArrayList<>();
         JSONArray policyList = null;
         if(params.has("policyList")){
@@ -336,55 +334,66 @@ public class PolicyManagerServlet extends HttpServlet {
         PolicyController controller = getPolicyControllerInstance();
         List<JSONObject> resultList = new ArrayList<>();
         try {
-            //Get the Login Id of the User from Request
-            String userId =  UserUtils.getUserSession(request).getOrgUserId();
-            List<Object> userRoles = controller.getRoles(userId);
-            Pair<Set<String>, List<String>> pair = org.onap.policy.utils.UserUtils.checkRoleAndScope(userRoles);
-            roles = pair.u;
-            scopes = pair.t;
-            if (roles.contains(ADMIN) || roles.contains(EDITOR) || roles.contains(GUEST) ) {
-                if(scopes.isEmpty()){
-                    return error("No Scopes has been Assigned to the User. Please, Contact Super-Admin");
-                }
-                Set<String> tempScopes = scopes;
-                for(String scope : tempScopes){
-                    addScope(scopes, scope);
-                }
+            if (!lookupPolicyData(request, policyData, policyList, controller, resultList))
+                return error("No Scopes has been Assigned to the User. Please, Contact Super-Admin");
+
+        }catch(Exception e){
+            LOGGER.error("Exception occured while reading policy Data from Policy Version table for Policy Search Data"+e);
+        }
+
+        return new JSONObject().put(RESULT, resultList);
+    }
+
+    private boolean lookupPolicyData(HttpServletRequest request, List<Object> policyData, JSONArray policyList, PolicyController controller, List<JSONObject> resultList) throws ServletException {
+        List<String> roles;
+        Set<String> scopes;//Get the Login Id of the User from Request
+        String userId =  UserUtils.getUserSession(request).getOrgUserId();
+        List<Object> userRoles = controller.getRoles(userId);
+        Pair<Set<String>, List<String>> pair = org.onap.policy.utils.UserUtils.checkRoleAndScope(userRoles);
+        roles = pair.u;
+        scopes = pair.t;
+        if (roles.contains(ADMIN) || roles.contains(EDITOR) || roles.contains(GUEST) ) {
+            if(scopes.isEmpty()){
+                return false;
             }
-            if(policyList!= null){
-                for(int i = 0; i < policyList.length(); i++){
-                    String policyName = policyList.get(i).toString().replace(".xml", "");
-                    String version = policyName.substring(policyName.lastIndexOf('.')+1);
-                    policyName = policyName.substring(0, policyName.lastIndexOf('.')).replace(".", File.separator);
-                    parsePolicyList(resultList, controller, policyName, version);
-                }
+            Set<String> tempScopes = scopes;
+            for(String scope : tempScopes){
+                addScope(scopes, scope);
+            }
+        }
+        if(policyList!= null){
+            for(int i = 0; i < policyList.length(); i++){
+                String policyName = policyList.get(i).toString().replace(".xml", "");
+                String version = policyName.substring(policyName.lastIndexOf('.')+1);
+                policyName = policyName.substring(0, policyName.lastIndexOf('.')).replace(".", File.separator);
+                parsePolicyList(resultList, controller, policyName, version);
+            }
+        }else{
+            if (roles.contains(SUPERADMIN) || roles.contains(SUPEREDITOR)   || roles.contains(SUPERGUEST) ){
+                policyData = controller.getData(PolicyVersion.class);
             }else{
-                if (roles.contains(SUPERADMIN) || roles.contains(SUPEREDITOR)   || roles.contains(SUPERGUEST) ){
-                    policyData = controller.getData(PolicyVersion.class);
-                }else{
-                    List<Object> filterdatas = controller.getData(PolicyVersion.class);
-                    for(Object filter : filterdatas){
-                        PolicyVersion filterdata = (PolicyVersion) filter;
-                        try{
-                            String scopeName = filterdata.getPolicyName().substring(0, filterdata.getPolicyName().lastIndexOf(File.separator));
-                            if(scopes.contains(scopeName)){
-                                policyData.add(filterdata);
-                            }
-                        }catch(Exception e){
-                            LOGGER.error("Exception occured while filtering policyversion data"+e);
-                        }
-                    }
+                List<Object> filterdatas = controller.getData(PolicyVersion.class);
+                for(Object filter : filterdatas) {
+                    addFilterData(policyData, scopes, (PolicyVersion) filter);
                 }
+            }
 
-                if(!policyData.isEmpty()){
-                    updateResultList(policyData, resultList);
-                }
+            if(!policyData.isEmpty()){
+                updateResultList(policyData, resultList);
             }
-        }catch(Exception e){
-            LOGGER.error("Exception occured while reading policy Data from Policy Version table for Policy Search Data"+e);
         }
+        return true;
+    }
 
-        return new JSONObject().put(RESULT, resultList);
+    private void addFilterData(List<Object> policyData, Set<String> scopes, PolicyVersion filter) {
+        try{
+            String scopeName = filter.getPolicyName().substring(0, filter.getPolicyName().lastIndexOf(File.separator));
+            if(scopes.contains(scopeName)){
+                policyData.add(filter);
+            }
+        }catch(Exception e){
+            LOGGER.error("Exception occured while filtering policyversion data"+e);
+        }
     }
 
     private void updateResultList(List<Object> policyData, List<JSONObject> resultList) {
@@ -570,82 +579,90 @@ public class PolicyManagerServlet extends HttpServlet {
 
     //Get the List of Policies and Scopes for Showing in Editor tab
     private JSONObject list(JSONObject params, HttpServletRequest request) throws ServletException {
-        Set<String> scopes = null;
-        List<String> roles = null;
         try {
-            PolicyController controller = getPolicyControllerInstance();
-            //Get the Login Id of the User from Request
-            String testUserID = getTestUserId();
-            String userId =  testUserID != null ? testUserID : UserUtils.getUserSession(request).getOrgUserId();
-            List<Object> userRoles = controller.getRoles(userId);
-            Pair<Set<String>, List<String>> pair = org.onap.policy.utils.UserUtils.checkRoleAndScope(userRoles);
-            roles = pair.u;
-            scopes = pair.t;
-
-            List<JSONObject> resultList = new ArrayList<>();
-            boolean onlyFolders = params.getBoolean("onlyFolders");
-            String path = params.getString("path");
-            if(path.contains("..xml")){
-                path = path.replaceAll("..xml", "").trim();
-            }
+            return processPolicyList(params, request);
+        } catch (Exception e) {
+            LOGGER.error("list", e);
+            return error(e.getMessage());
+        }
+    }
 
-            if (roles.contains(ADMIN) || roles.contains(EDITOR) || roles.contains(GUEST) ) {
-                if(scopes.isEmpty()){
-                    return error("No Scopes has been Assigned to the User. Please, Contact Super-Admin");
-                }else{
-                    if(!"/".equals(path)){
-                        String tempScope = path.substring(1, path.length());
-                        tempScope = tempScope.replace("/", File.separator);
-                        scopes.add(tempScope);
-                    }
-                }
-            }
+    private JSONObject processPolicyList(JSONObject params, HttpServletRequest request) throws ServletException {
+        List<String> roles;
+        Set<String> scopes;PolicyController controller = getPolicyControllerInstance();
+        //Get the Login Id of the User from Request
+        String testUserID = getTestUserId();
+        String userId =  testUserID != null ? testUserID : UserUtils.getUserSession(request).getOrgUserId();
+        List<Object> userRoles = controller.getRoles(userId);
+        Pair<Set<String>, List<String>> pair = org.onap.policy.utils.UserUtils.checkRoleAndScope(userRoles);
+        roles = pair.u;
+        scopes = pair.t;
 
-            if("/".equals(path)){
-                if(roles.contains(SUPERADMIN) || roles.contains(SUPEREDITOR) || roles.contains(SUPERGUEST)){
-                    List<Object> scopesList = queryPolicyEditorScopes(null);
-                    for(Object list : scopesList){
-                        PolicyEditorScopes scope = (PolicyEditorScopes) list;
-                        if(!(scope.getScopeName().contains(File.separator))){
-                            JSONObject el = new JSONObject();
-                            el.put(NAME, scope.getScopeName());
-                            el.put(DATE, scope.getModifiedDate());
-                            el.put(SIZE, "");
-                            el.put(TYPE, "dir");
-                            el.put(CREATED_BY, scope.getUserCreatedBy().getUserName());
-                            el.put(MODIFIED_BY, scope.getUserModifiedBy().getUserName());
-                            resultList.add(el);
-                        }
-                    }
-                }else if(roles.contains(ADMIN) || roles.contains(EDITOR) || roles.contains(GUEST)){
-                    for(Object scope : scopes){
-                        JSONObject el = new JSONObject();
-                        List<Object> scopesList = queryPolicyEditorScopes(scope.toString());
-                        if(!scopesList.isEmpty()){
-                            PolicyEditorScopes scopeById = (PolicyEditorScopes) scopesList.get(0);
-                            el.put(NAME, scopeById.getScopeName());
-                            el.put(DATE, scopeById.getModifiedDate());
-                            el.put(SIZE, "");
-                            el.put(TYPE, "dir");
-                            el.put(CREATED_BY, scopeById.getUserCreatedBy().getUserName());
-                            el.put(MODIFIED_BY, scopeById.getUserModifiedBy().getUserName());
-                            resultList.add(el);
-                        }
-                    }
-                }
+        List<JSONObject> resultList = new ArrayList<>();
+        boolean onlyFolders = params.getBoolean("onlyFolders");
+        String path = params.getString("path");
+        if(path.contains("..xml")){
+            path = path.replaceAll("..xml", "").trim();
+        }
+
+        if (roles.contains(ADMIN) || roles.contains(EDITOR) || roles.contains(GUEST) ) {
+            if(scopes.isEmpty()){
+                return error("No Scopes has been Assigned to the User. Please, Contact Super-Admin");
             }else{
-                try{
-                    String scopeName = path.substring(path.indexOf('/') +1);
-                    activePolicyList(scopeName, resultList, roles, scopes, onlyFolders);
-                } catch (Exception ex) {
-                    LOGGER.error("Error Occured While reading Policy Files List"+ex );
+                if(!"/".equals(path)){
+                    String tempScope = path.substring(1, path.length());
+                    tempScope = tempScope.replace("/", File.separator);
+                    scopes.add(tempScope);
                 }
             }
+        }
 
+        if (!"/".equals(path)) {
+            try{
+                String scopeName = path.substring(path.indexOf('/') +1);
+                activePolicyList(scopeName, resultList, roles, scopes, onlyFolders);
+            } catch (Exception ex) {
+                LOGGER.error("Error Occured While reading Policy Files List"+ex );
+            }
             return new JSONObject().put(RESULT, resultList);
-        } catch (Exception e) {
-            LOGGER.error("list", e);
-            return error(e.getMessage());
+        }
+
+        processRoles(scopes, roles, resultList);
+
+        return new JSONObject().put(RESULT, resultList);
+    }
+
+    private void processRoles(Set<String> scopes, List<String> roles, List<JSONObject> resultList) {
+        if(roles.contains(SUPERADMIN) || roles.contains(SUPEREDITOR) || roles.contains(SUPERGUEST)){
+            List<Object> scopesList = queryPolicyEditorScopes(null);
+            for(Object list : scopesList){
+                PolicyEditorScopes scope = (PolicyEditorScopes) list;
+                if(!(scope.getScopeName().contains(File.separator))){
+                    JSONObject el = new JSONObject();
+                    el.put(NAME, scope.getScopeName());
+                    el.put(DATE, scope.getModifiedDate());
+                    el.put(SIZE, "");
+                    el.put(TYPE, "dir");
+                    el.put(CREATED_BY, scope.getUserCreatedBy().getUserName());
+                    el.put(MODIFIED_BY, scope.getUserModifiedBy().getUserName());
+                    resultList.add(el);
+                }
+            }
+        }else if(roles.contains(ADMIN) || roles.contains(EDITOR) || roles.contains(GUEST)){
+            for(Object scope : scopes){
+                JSONObject el = new JSONObject();
+                List<Object> scopesList = queryPolicyEditorScopes(scope.toString());
+                if(!scopesList.isEmpty()){
+                    PolicyEditorScopes scopeById = (PolicyEditorScopes) scopesList.get(0);
+                    el.put(NAME, scopeById.getScopeName());
+                    el.put(DATE, scopeById.getModifiedDate());
+                    el.put(SIZE, "");
+                    el.put(TYPE, "dir");
+                    el.put(CREATED_BY, scopeById.getUserCreatedBy().getUserName());
+                    el.put(MODIFIED_BY, scopeById.getUserModifiedBy().getUserName());
+                    resultList.add(el);
+                }
+            }
         }
     }
 
@@ -693,31 +710,8 @@ public class PolicyManagerServlet extends HttpServlet {
             activePolicies = controller.getDataByQuery(query, params);
             scopesList = controller.getDataByQuery(scopeNamequery, params);
         }
-        for(Object list : scopesList){
-            PolicyEditorScopes scopeById = (PolicyEditorScopes) list;
-            String scope = scopeById.getScopeName();
-            if(scope.contains(File.separator)){
-                String checkScope = scope.substring(0, scope.lastIndexOf(File.separator));
-                if(scopeName.contains("\\\\")){
-                    scopeName = scopeName.replace("\\\\", File.separator);
-                }
-                if(scope.contains(File.separator)){
-                    scope = scope.substring(checkScope.length()+1);
-                    if(scope.contains(File.separator)){
-                        scope = scope.substring(0, scope.indexOf(File.separator));
-                    }
-                }
-                if(scopeName.equalsIgnoreCase(checkScope)){
-                    JSONObject el = new JSONObject();
-                    el.put(NAME, scope);
-                    el.put(DATE, scopeById.getModifiedDate());
-                    el.put(SIZE, "");
-                    el.put(TYPE, "dir");
-                    el.put(CREATED_BY, scopeById.getUserCreatedBy().getUserName());
-                    el.put(MODIFIED_BY, scopeById.getUserModifiedBy().getUserName());
-                    resultList.add(el);
-                }
-            }
+        for(Object list : scopesList) {
+            scopeName = checkScope(resultList, scopeName, (PolicyEditorScopes) list);
         }
         String scopeNameCheck;
         for (Object list : activePolicies) {
@@ -754,6 +748,33 @@ public class PolicyManagerServlet extends HttpServlet {
         }
     }
 
+    private String checkScope(List<JSONObject> resultList, String scopeName, PolicyEditorScopes scopeById) {
+        String scope = scopeById.getScopeName();
+        if(scope.contains(File.separator)){
+            String targetScope = scope.substring(0, scope.lastIndexOf(File.separator));
+            if(scopeName.contains("\\\\")){
+                scopeName = scopeName.replace("\\\\", File.separator);
+            }
+            if(scope.contains(File.separator)){
+                scope = scope.substring(targetScope.length()+1);
+                if(scope.contains(File.separator)){
+                    scope = scope.substring(0, scope.indexOf(File.separator));
+                }
+            }
+            if(scopeName.equalsIgnoreCase(targetScope)){
+                JSONObject el = new JSONObject();
+                el.put(NAME, scope);
+                el.put(DATE, scopeById.getModifiedDate());
+                el.put(SIZE, "");
+                el.put(TYPE, "dir");
+                el.put(CREATED_BY, scopeById.getUserCreatedBy().getUserName());
+                el.put(MODIFIED_BY, scopeById.getUserModifiedBy().getUserName());
+                resultList.add(el);
+            }
+        }
+        return scopeName;
+    }
+
     private String getUserName(String loginId){
         PolicyController controller = getPolicyControllerInstance();
         UserInfo userInfo = (UserInfo) controller.getEntityItem(UserInfo.class, "userLoginId", loginId);
@@ -766,88 +787,92 @@ public class PolicyManagerServlet extends HttpServlet {
     //Rename Policy
     private JSONObject rename(JSONObject params, HttpServletRequest request) throws ServletException {
         try {
-            boolean isActive = false;
-            List<String> policyActiveInPDP = new ArrayList<>();
-            Set<String>  scopeOfPolicyActiveInPDP = new HashSet<>();
-            String userId = UserUtils.getUserSession(request).getOrgUserId();
-            String oldPath = params.getString("path");
-            String newPath = params.getString("newPath");
-            oldPath = oldPath.substring(oldPath.indexOf('/')+1);
-            newPath = newPath.substring(newPath.indexOf('/')+1);
-            String checkValidation = null;
-            if(oldPath.endsWith(".xml")){
-                checkValidation = newPath.replace(".xml", "");
-                checkValidation = checkValidation.substring(checkValidation.indexOf('_') + 1, checkValidation.lastIndexOf("."));
-                checkValidation = checkValidation.substring(checkValidation.lastIndexOf("/")+1);
-                if(!PolicyUtils.policySpecialCharValidator(checkValidation).contains("success")){
-                    return error("Policy Rename Failed. The Name contains special characters.");
-                }
-                JSONObject result = policyRename(oldPath, newPath, userId);
+            return handlePolicyRename(params, request);
+        } catch (Exception e) {
+            LOGGER.error(XACMLErrorConstants.ERROR_DATA_ISSUE+"Exception Occured While Renaming Policy"+e);
+            return error(e.getMessage());
+        }
+    }
+
+    private JSONObject handlePolicyRename(JSONObject params, HttpServletRequest request) throws ServletException {
+        boolean isActive = false;
+        List<String> policyActiveInPDP = new ArrayList<>();
+        Set<String> scopeOfPolicyActiveInPDP = new HashSet<>();
+        String userId = UserUtils.getUserSession(request).getOrgUserId();
+        String oldPath = params.getString("path");
+        String newPath = params.getString("newPath");
+        oldPath = oldPath.substring(oldPath.indexOf('/')+1);
+        newPath = newPath.substring(newPath.indexOf('/')+1);
+        String checkValidation = null;
+        if(oldPath.endsWith(".xml")){
+            checkValidation = newPath.replace(".xml", "");
+            checkValidation = checkValidation.substring(checkValidation.indexOf('_') + 1, checkValidation.lastIndexOf("."));
+            checkValidation = checkValidation.substring(checkValidation.lastIndexOf("/")+1);
+            if(!PolicyUtils.policySpecialCharValidator(checkValidation).contains("success")){
+                return error("Policy Rename Failed. The Name contains special characters.");
+            }
+            JSONObject result = policyRename(oldPath, newPath, userId);
+            if(!(Boolean)(result.getJSONObject("result").get("success"))){
+                return result;
+            }
+        }else{
+            String scopeName = oldPath;
+            String newScopeName = newPath;
+            if(scopeName.contains("/")){
+                scopeName = scopeName.replace("/", File.separator);
+                newScopeName = newScopeName.replace("/", File.separator);
+            }
+            checkValidation = newScopeName.substring(newScopeName.lastIndexOf(File.separator)+1);
+            if(scopeName.contains("\\")){
+                scopeName = scopeName.replace("\\", "\\\\\\\\");
+                newScopeName = newScopeName.replace("\\", "\\\\\\\\");
+            }
+            if(!PolicyUtils.policySpecialCharValidator(checkValidation).contains("success")){
+                return error("Scope Rename Failed. The Name contains special characters.");
+            }
+            PolicyController controller = getPolicyControllerInstance();
+            String query = "from PolicyVersion where POLICY_NAME like :scopeName";
+            String scopeNamequery = "from PolicyEditorScopes where SCOPENAME like :scopeName";
+            SimpleBindings pvParams = new SimpleBindings();
+            pvParams.put("scopeName", scopeName + "%");
+            List<Object> activePolicies = controller.getDataByQuery(query, pvParams);
+            List<Object> scopesList = controller.getDataByQuery(scopeNamequery, pvParams);
+            for(Object object : activePolicies){
+                PolicyVersion activeVersion = (PolicyVersion) object;
+                String policyOldPath = activeVersion.getPolicyName().replace(File.separator, "/") + "." + activeVersion.getActiveVersion() + ".xml";
+                String policyNewPath = policyOldPath.replace(oldPath, newPath);
+                JSONObject result = policyRename(policyOldPath, policyNewPath, userId);
                 if(!(Boolean)(result.getJSONObject("result").get("success"))){
-                    return result;
-                }
-            }else{
-                String scopeName = oldPath;
-                String newScopeName = newPath;
-                if(scopeName.contains("/")){
-                    scopeName = scopeName.replace("/", File.separator);
-                    newScopeName = newScopeName.replace("/", File.separator);
-                }
-                checkValidation = newScopeName.substring(newScopeName.lastIndexOf(File.separator)+1);
-                if(scopeName.contains("\\")){
-                    scopeName = scopeName.replace("\\", "\\\\\\\\");
-                    newScopeName = newScopeName.replace("\\", "\\\\\\\\");
-                }
-                if(!PolicyUtils.policySpecialCharValidator(checkValidation).contains("success")){
-                    return error("Scope Rename Failed. The Name contains special characters.");
-                }
-                PolicyController controller = getPolicyControllerInstance();
-                String query = "from PolicyVersion where POLICY_NAME like :scopeName";
-                String scopeNamequery = "from PolicyEditorScopes where SCOPENAME like :scopeName";
-                SimpleBindings pvParams = new SimpleBindings();
-                pvParams.put("scopeName", scopeName + "%");
-                List<Object> activePolicies = controller.getDataByQuery(query, pvParams);
-                List<Object> scopesList = controller.getDataByQuery(scopeNamequery, pvParams);
-                for(Object object : activePolicies){
-                    PolicyVersion activeVersion = (PolicyVersion) object;
-                    String policyOldPath = activeVersion.getPolicyName().replace(File.separator, "/") + "." + activeVersion.getActiveVersion() + ".xml";
-                    String policyNewPath = policyOldPath.replace(oldPath, newPath);
-                    JSONObject result = policyRename(policyOldPath, policyNewPath, userId);
-                    if(!(Boolean)(result.getJSONObject("result").get("success"))){
-                        isActive = true;
-                        policyActiveInPDP.add(policyOldPath);
-                        String scope = policyOldPath.substring(0, policyOldPath.lastIndexOf('/'));
-                        scopeOfPolicyActiveInPDP.add(scope.replace("/", File.separator));
-                    }
-                }
-                boolean rename = false;
-                if(activePolicies.size() != policyActiveInPDP.size()){
-                    rename = true;
+                    isActive = true;
+                    policyActiveInPDP.add(policyOldPath);
+                    String scope = policyOldPath.substring(0, policyOldPath.lastIndexOf('/'));
+                    scopeOfPolicyActiveInPDP.add(scope.replace("/", File.separator));
                 }
+            }
+            boolean rename = false;
+            if(activePolicies.size() != policyActiveInPDP.size()){
+                rename = true;
+            }
 
-                UserInfo userInfo = new UserInfo();
-                userInfo.setUserLoginId(userId);
-                if(policyActiveInPDP.isEmpty()){
-                    renameScope(scopesList, scopeName, newScopeName, controller);
-                }else if(rename){
-                    renameScope(scopesList, scopeName, newScopeName, controller);
-                    for(String scope : scopeOfPolicyActiveInPDP){
-                        PolicyEditorScopes editorScopeEntity = new PolicyEditorScopes();
-                        editorScopeEntity.setScopeName(scope.replace("\\", "\\\\\\\\"));
-                        editorScopeEntity.setUserCreatedBy(userInfo);
-                        editorScopeEntity.setUserModifiedBy(userInfo);
-                        controller.saveData(editorScopeEntity);
-                    }
-                }
-                if(isActive){
-                    return error("The Following policies rename failed. Since they are active in PDP Groups" +policyActiveInPDP);
+            UserInfo userInfo = new UserInfo();
+            userInfo.setUserLoginId(userId);
+            if(policyActiveInPDP.isEmpty()){
+                renameScope(scopesList, scopeName, newScopeName, controller);
+            }else if(rename){
+                renameScope(scopesList, scopeName, newScopeName, controller);
+                for(String scope : scopeOfPolicyActiveInPDP){
+                    PolicyEditorScopes editorScopeEntity = new PolicyEditorScopes();
+                    editorScopeEntity.setScopeName(scope.replace("\\", "\\\\\\\\"));
+                    editorScopeEntity.setUserCreatedBy(userInfo);
+                    editorScopeEntity.setUserModifiedBy(userInfo);
+                    controller.saveData(editorScopeEntity);
                 }
             }
-            return success();
-        } catch (Exception e) {
-            LOGGER.error(XACMLErrorConstants.ERROR_DATA_ISSUE+"Exception Occured While Renaming Policy"+e);
-            return error(e.getMessage());
+            if(isActive){
+                return error("The Following policies rename failed. Since they are active in PDP Groups" +policyActiveInPDP);
+            }
         }
+        return success();
     }
 
     private void renameScope(List<Object> scopesList, String inScopeName, String newScopeName, PolicyController controller){
diff --git a/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateBRMSParamController.java b/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateBRMSParamController.java
index 40cceead5..b49f1fc35 100644
--- a/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateBRMSParamController.java
+++ b/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateBRMSParamController.java
@@ -3,6 +3,7 @@
  * ONAP Policy Engine
  * ================================================================================
  * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved.
+ * Modified Copyright (C) 2018 Samsung Electronics Co., Ltd.
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -69,457 +70,479 @@ import oasis.names.tc.xacml._3_0.core.schema.wd_17.TargetType;
 @Controller
 @RequestMapping("/")
 public class CreateBRMSParamController extends RestrictedBaseController {
-	private static final Logger policyLogger = FlexLogger.getLogger(CreateBRMSParamController.class);
-
-	private static CommonClassDao commonClassDao;
-
-	public static CommonClassDao getCommonClassDao() {
-		return commonClassDao;
-	}
-
-	public static void setCommonClassDao(CommonClassDao commonClassDao) {
-		CreateBRMSParamController.commonClassDao = commonClassDao;
-	}
-	
-	@Autowired
-	private CreateBRMSParamController(CommonClassDao commonClassDao){
-		CreateBRMSParamController.commonClassDao = commonClassDao;
-	}
-
-	public CreateBRMSParamController(){
-		// Empty constructor
-	}
-	protected PolicyRestAdapter policyAdapter = null; 
-
-	private HashMap<String, String> dynamicLayoutMap;
-	
-	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{
-			dynamicLayoutMap = new HashMap<>();
-			ObjectMapper mapper = new ObjectMapper();
-			mapper.configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);
-			JsonNode root = mapper.readTree(request.getReader());
-			String rule = findRule(root.get(PolicyController.getPolicydata()).toString().replaceAll("^\"|\"$", ""));
-			generateUI(rule);
-			response.setCharacterEncoding(PolicyController.getCharacterencoding());
-			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);
-		}
-	}
-
-	protected 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();
-		}
-		return null;
-	}
-
-	protected void generateUI(String rule) {
-		if(rule!=null){
-			try {
-				StringBuilder params = new StringBuilder("");
-				Boolean flag = false;
-				Boolean comment = false;
-				String[] lines = rule.split("\n");
-				for(String line : lines){
-					if (line.isEmpty() || line.startsWith("//")) {
-						continue;
-					}
-					if (line.startsWith("/*")) {
-						comment = true;
-						continue;
-					}
-					if (line.contains("//")) {
-						line = line.split("\\/\\/")[0];
-					}
-					if (line.contains("/*")) {
-						comment = true;
-						if (line.contains("*/")) {
-							try {
-								comment = false;
-								line = line.split("\\/\\*")[0]
-										+ line.split("\\*\\/")[1].replace("*/", "");
-							} catch (Exception e) {
-								policyLogger.info("Just for Logging"+e);
-								line = line.split("\\/\\*")[0];
-							}
-						} else {
-							line = line.split("\\/\\*")[0];
-						}
-					}
-					if (line.contains("*/")) {
-						comment = false;
-						try {
-							line = line.split("\\*\\/")[1].replace("*/", "");
-						} catch (Exception e) {
-							policyLogger.info("Just for Logging"+e);
-							line = "";
-						}
-					}
-					if (comment) {
-						continue;
-					}
-					if (flag) {
-						params.append(line);
-					}
-					if (line.contains("declare Params")) {
-						params.append(line);
-						flag = true;
-					}
-					if (line.contains("end") && flag) {
-						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", "");
-					}
-				}
-			} catch (Exception e) {
-				policyLogger.error(XACMLErrorConstants.ERROR_SYSTEM_ERROR + e);
-			}
-		}
-	}
-	
-	private void createField(String caption, String type) {
-		dynamicLayoutMap.put(caption, type);
-	}
-
-	/*
-	 * 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) {
-		dynamicLayoutMap = new HashMap<>();
-		if (policyAdapter.getPolicyData() instanceof PolicyType) {
-			PolicyType policy = (PolicyType) policyAdapter.getPolicyData();
-			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);
-			if (policyLogger.isDebugEnabled()) {
-				policyLogger.debug("Prepopulating form data for BRMS RAW Policy selected:" + policyAdapter.getPolicyName());
-			}
-			policyAdapter.setPolicyName(policyNameValue);
-			String description = "";
-			try{
-				description = policy.getDescription().substring(0, policy.getDescription().indexOf("@CreatedBy:"));
-			}catch(Exception e){
-				policyLogger.info("Just for Logging"+e);
-				description = policy.getDescription();
-			}
-			policyAdapter.setPolicyDescription(description);
-			setDataAdapterFromAdviceExpressions(policy, policyAdapter);
-			paramUIGenerate(policyAdapter, entity);
-			// 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));
-				}
-				policyAdapter.setRuleData(drlRule);
-			}	
-			TargetType target = policy.getTarget();
-			if (target != null) {
-				setDataToAdapterFromTarget(target, 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:")){
-					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);
-					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("");
-					}
-					policyAdapter.setBrmsDependency(dependencies);
-				}else if(attributeAssignment.getAttributeId().startsWith("controller:")){
-					policyAdapter.setBrmsController(attributeAssignment.getAttributeId().replace("controller:", ""));
-				}
-			}
-			policyAdapter.setAttributes(attributeList);
-		}
-	}
-	
-	private void setDataToAdapterFromTarget(TargetType target, PolicyRestAdapter policyAdapter){
-		// Under target we have AnyOFType
-		List<AnyOfType> anyOfList = target.getAnyOf();
-		if (anyOfList != null) {
-			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) {
-					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);
-						}
-					}
-				}
-			}
-		}
-	}
-	
-	private void setDataToAdapterFromMatchList(List<MatchType> matchList, PolicyRestAdapter policyAdapter){
-		Iterator<MatchType> iterMatch = matchList.iterator();
-		while (iterMatch.hasNext()) {
-			MatchType match = iterMatch.next();
-			//
-			// Under the match we have attribute value and
-			// attributeDesignator. So,finally down to the actual attribute.
-			//
-			AttributeValueType attributeValue = match.getAttributeValue();
-			String value = (String) attributeValue.getContent().get(0);
-			AttributeDesignatorType designator = match.getAttributeDesignator();
-			String attributeId = designator.getAttributeId();
-
-			if ("RiskType".equals(attributeId)){
-				policyAdapter.setRiskType(value);
-			}
-			if ("RiskLevel".equals(attributeId)){
-				policyAdapter.setRiskLevel(value);
-			}
-			if ("guard".equals(attributeId)){
-				policyAdapter.setGuard(value);
-			}
-			if ("TTLDate".equals(attributeId) && !value.contains("NA")){
-				PolicyController controller = new PolicyController();
-				String newDate = controller.convertDate(value);
-				policyAdapter.setTtlDate(newDate);
-			}
-		}
-	}
-
-	// This method generates the UI from rule configuration
-	public void paramUIGenerate(PolicyRestAdapter policyAdapter, PolicyEntity entity) {
-		String data = entity.getConfigurationData().getConfigBody();
-		if(data != null){
-			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 = line.split("\\/\\/")[0];
-					}
-					if (line.contains("/*")) {
-						comment = true;
-						if (line.contains("*/")) {
-							try {
-								comment = false;
-								line = line.split("\\/\\*")[0]
-										+ line.split("\\*\\/")[1].replace(
-												"*/", "");
-							} catch (Exception e) {
-								policyLogger.info("Just for Logging"+e);
-								line = line.split("\\/\\*")[0];
-							}
-						} else {
-							line = line.split("\\/\\*")[0];
-						}
-					}
-					if (line.contains("*/")) {
-						comment = false;
-						try {
-							line = line.split("\\*\\/")[1]
-									.replace("*/", "");
-						} catch (Exception e) {
-							policyLogger.info("Just for Logging"+e);
-							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", ""));
-				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);
-
-					}
-				}
-			} catch (Exception e) {
-				policyLogger.error(XACMLErrorConstants.ERROR_DATA_ISSUE + e.getMessage() + e);
-			} 
-		}
-		
-	}
-
-	// set View Rule
-	@SuppressWarnings("unchecked")
-	@RequestMapping(value={"/policyController/ViewBRMSParamPolicyRule.htm"}, method={org.springframework.web.bind.annotation.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())){
-				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");
-
-			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()+":|:");
-			        }
-			    }
-			    generatedMetadata.append("$%> \n*/ \n");			    
-			}
-			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());
-			copyMap.put("policyScope", policyData.getDomainDir().replace("\\", "."));
-			copyMap.put("policyVersion", "1");
-			//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));
-			}
-			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());
-		} catch (Exception e) {
-			policyLogger.error(XACMLErrorConstants.ERROR_PROCESS_FLOW + e);
-		}	
-	}
+    private static final Logger policyLogger = FlexLogger.getLogger(CreateBRMSParamController.class);
+
+    private static CommonClassDao commonClassDao;
+
+    public static CommonClassDao getCommonClassDao() {
+        return commonClassDao;
+    }
+
+    public static void setCommonClassDao(CommonClassDao commonClassDao) {
+        CreateBRMSParamController.commonClassDao = commonClassDao;
+    }
+
+    @Autowired
+    private CreateBRMSParamController(CommonClassDao commonClassDao){
+        CreateBRMSParamController.commonClassDao = commonClassDao;
+    }
+
+    public CreateBRMSParamController(){
+        // Empty constructor
+    }
+    protected PolicyRestAdapter policyAdapter = null;
+
+    private HashMap<String, String> dynamicLayoutMap;
+
+    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{
+            dynamicLayoutMap = new HashMap<>();
+            ObjectMapper mapper = new ObjectMapper();
+            mapper.configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);
+            JsonNode root = mapper.readTree(request.getReader());
+            String rule = findRule(root.get(PolicyController.getPolicydata()).toString().replaceAll("^\"|\"$", ""));
+            generateUI(rule);
+            response.setCharacterEncoding(PolicyController.getCharacterencoding());
+            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);
+        }
+    }
+
+    protected 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();
+        }
+        return null;
+    }
+
+    protected void generateUI(String rule) {
+        if (rule==null){
+            return;
+        }
+        try {
+            processRule(rule);
+        } catch (Exception e) {
+            policyLogger.error(XACMLErrorConstants.ERROR_SYSTEM_ERROR + e);
+        }
+
+    }
+
+    private void processRule(String rule) {
+        StringBuilder params = new StringBuilder("");
+        Boolean flag = false;
+        Boolean comment = false;
+        String[] lines = rule.split("\n");
+        for(String line : lines){
+            if (line.isEmpty() || line.startsWith("//")) {
+                continue;
+            }
+            if (line.startsWith("/*")) {
+                comment = true;
+                continue;
+            }
+            if (line.contains("//")) {
+                line = splitSingleLineComment(line);
+            }
+            if (line.contains("/*")) {
+                comment = true;
+                if (line.contains("*/")) {
+                    comment = false;
+                    line = processMultiLineFullComment(line);
+                } else {
+                    line = splitMultiLineStartComment(line);
+                }
+            }
+            if (line.contains("*/")) {
+                comment = false;
+                line = processEndComment(line);
+            }
+            if (comment) {
+                continue;
+            }
+            if (flag) {
+                params.append(line);
+            }
+            if (line.contains("declare Params")) {
+                params.append(line);
+                flag = true;
+            }
+            if (line.contains("end") && flag) {
+                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", "");
+            }
+        }
+    }
+
+    private String splitMultiLineStartComment(String line) {
+        return line.split("\\/\\*")[0];
+    }
+
+    private String splitMultiLineEndComment(String line) {
+        return line.split("\\*\\/")[1].replace("*/", "");
+    }
+
+    private String splitSingleLineComment(String line) {
+        return line.split("\\/\\/")[0];
+    }
+
+    private void createField(String caption, String type) {
+        dynamicLayoutMap.put(caption, type);
+    }
+
+    /*
+     * 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) {
+        dynamicLayoutMap = new HashMap<>();
+        if (policyAdapter.getPolicyData() instanceof PolicyType) {
+            PolicyType policy = (PolicyType) policyAdapter.getPolicyData();
+            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);
+            if (policyLogger.isDebugEnabled()) {
+                policyLogger.debug("Prepopulating form data for BRMS RAW Policy selected:" + policyAdapter.getPolicyName());
+            }
+            policyAdapter.setPolicyName(policyNameValue);
+            String description = "";
+            try{
+                description = policy.getDescription().substring(0, policy.getDescription().indexOf("@CreatedBy:"));
+            }catch(Exception e){
+                policyLogger.info("Just for Logging"+e);
+                description = policy.getDescription();
+            }
+            policyAdapter.setPolicyDescription(description);
+            setDataAdapterFromAdviceExpressions(policy, policyAdapter);
+            paramUIGenerate(policyAdapter, entity);
+            // 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));
+                }
+                policyAdapter.setRuleData(drlRule);
+            }
+            TargetType target = policy.getTarget();
+            if (target != null) {
+                setDataToAdapterFromTarget(target, 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:")){
+                    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);
+                    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("");
+                    }
+                    policyAdapter.setBrmsDependency(dependencies);
+                }else if(attributeAssignment.getAttributeId().startsWith("controller:")){
+                    policyAdapter.setBrmsController(attributeAssignment.getAttributeId().replace("controller:", ""));
+                }
+            }
+            policyAdapter.setAttributes(attributeList);
+        }
+    }
+
+    private void setDataToAdapterFromTarget(TargetType target, PolicyRestAdapter policyAdapter){
+        // Under target we have AnyOFType
+        List<AnyOfType> anyOfList = target.getAnyOf();
+
+        if (anyOfList == 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);
+                }
+            }
+        }
+    }
+
+    private void setDataToAdapterFromMatchList(List<MatchType> matchList, PolicyRestAdapter policyAdapter){
+        Iterator<MatchType> iterMatch = matchList.iterator();
+        while (iterMatch.hasNext()) {
+            MatchType match = iterMatch.next();
+            //
+            // Under the match we have attribute value and
+            // attributeDesignator. So,finally down to the actual attribute.
+            //
+            AttributeValueType attributeValue = match.getAttributeValue();
+            String value = (String) attributeValue.getContent().get(0);
+            AttributeDesignatorType designator = match.getAttributeDesignator();
+            String attributeId = designator.getAttributeId();
+
+            if ("RiskType".equals(attributeId)){
+                policyAdapter.setRiskType(value);
+            }
+            if ("RiskLevel".equals(attributeId)){
+                policyAdapter.setRiskLevel(value);
+            }
+            if ("guard".equals(attributeId)){
+                policyAdapter.setGuard(value);
+            }
+            if ("TTLDate".equals(attributeId) && !value.contains("NA")){
+                PolicyController controller = new PolicyController();
+                String newDate = controller.convertDate(value);
+                policyAdapter.setTtlDate(newDate);
+            }
+        }
+    }
+
+    // This method generates the UI from rule configuration
+    public void paramUIGenerate(PolicyRestAdapter policyAdapter, PolicyEntity entity) {
+        String data = entity.getConfigurationData().getConfigBody();
+        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);
+                    }
+                }
+                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", ""));
+            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);
+
+                }
+            }
+        } 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);
+            line = "";
+        }
+        return line;
+    }
+
+    private String processMultiLineFullComment(String line) {
+        try {
+            line = splitMultiLineStartComment(line)
+                    + splitMultiLineEndComment(line);
+        } catch (Exception e) {
+            policyLogger.info("Just for Logging"+e);
+            line = splitMultiLineStartComment(line);
+        }
+        return line;
+    }
+
+    // set View Rule
+    @SuppressWarnings("unchecked")
+    @RequestMapping(value={"/policyController/ViewBRMSParamPolicyRule.htm"}, method={org.springframework.web.bind.annotation.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())){
+                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");
+
+            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()+":|:");
+                    }
+                }
+                generatedMetadata.append("$%> \n*/ \n");
+            }
+            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());
+            copyMap.put("policyScope", policyData.getDomainDir().replace("\\", "."));
+            copyMap.put("policyVersion", "1");
+            //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));
+            }
+            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());
+        } catch (Exception e) {
+            policyLogger.error(XACMLErrorConstants.ERROR_PROCESS_FLOW + e);
+        }
+    }
 }