lowered code smells in IdNameList & IdNameLookup 54/107654/1
authorIndrijeet kumar <indriku1@in.ibm.com>
Thu, 14 May 2020 03:21:11 +0000 (08:51 +0530)
committerIndrijeet kumar <indriku1@in.ibm.com>
Thu, 14 May 2020 03:21:18 +0000 (08:51 +0530)
lowered code smells in IdNameList & IdNameLookup

Issue-ID: PORTAL-813
Change-Id: I45cfae48f8e4e1f6e1e1001834dcca29cc9d8796
Signed-off-by: Indrijeet Kumar <indriku1@in.ibm.com>
ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/model/base/IdNameList.java
ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/model/base/IdNameLookup.java

index 8f44e37..154c94b 100644 (file)
@@ -176,20 +176,6 @@ public class IdNameList extends Vector {
        public void loadData(String pageNo, String searchString, String dbInfo) throws RaptorException {}
        private void loadData(int pageNo, String searchString, String dbInfo) throws RaptorException {} 
 
-/*    
-       public void loadData(int pageNo, String dbInfo) throws RaptorException {
-       }
-
-    public void loadUserData(int pageNo, String dbInfo, String userId) throws RaptorException {
-    }
-
-
-
-
-       public void loadData(String pageNo, String searchString) throws RaptorException {
-       }
-
-*/
        protected static String nvl(String s) {
                return (s == null) ? "" : s;
        }
index f2a15f4..590fd1a 100644 (file)
@@ -60,29 +60,6 @@ public class IdNameLookup extends IdNameSql {
                this(dbTableName, dbIdField, dbNameField, null, "", false);
        } // IdNameLookup
 
-/*    public IdNameLookup(String dbTableName, String dbIdField, String dbNameField,
-            String dbSortByField) {
-        super();
-
-        setDbTableName(dbTableName);
-        setDbIdField(dbIdField);
-        setDbNameField(dbNameField);
-        setDbSortByField(dbSortByField);
-        updateParentSQL();
-    } // IdNameLookup
-
-    public IdNameLookup(String dbTableName, String dbIdField, String dbNameField,
-                       String dbSortByField, String defaultSQL) {
-               super();
-
-               setDbTableName(dbTableName);
-               setDbIdField(dbIdField);
-               setDbNameField(dbNameField);
-               setDbSortByField(dbSortByField);
-        setDefaultSQL(defaultSQL);        
-               updateParentSQL();
-       } // IdNameLookup
-*/
     public IdNameLookup(String dbTableName, String dbIdField, String dbNameField,
             String dbSortByField, boolean textField) {
         super();
@@ -153,10 +130,6 @@ public class IdNameLookup extends IdNameSql {
                                                        dbSortByField.indexOf(' ')) : dbSortByField) + " sort";
 
                setSqlNoOrderBy(sql_start + sql_middle + sql_end);
-//        System.out.println("SQL Start " + sql_start);
-//        System.out.println("SQL Middle " + sql_middle);
-//        System.out.println("SQL End " + sql_end);
-//        System.out.println("DbSortByField " + dbSortByField);
         
                setSql(sql_start + sql_middle + sql_end + " ORDER BY " + nvl(dbSortByField, "2"));
        } // updateParentSQL
@@ -173,16 +146,6 @@ public class IdNameLookup extends IdNameSql {
                return "SELECT " + dbIdField + " FROM " + dbTableName;
        } // getBaseSQL
        
-       /*
-       public void loadData(int pageNo) throws RaptorException {
-               loadData(pageNo, "");
-       } // loadData
-
-       public void loadData(String pageNo) throws RaptorException {
-               loadData(pageNo, "");
-       } // loadData
-*/
-       
        public void loadData(String pageNo, String searchString, String dbInfo) throws RaptorException {
                int iPageNo = 0;