Address comments

wso2 · Dec 16, 2024 · 3969687 · 3969687
1 parent b4c7215
commit 3969687
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/.../org/wso2/carbon/identity/application/common/dao/impl/AuthenticatorManagementDAOImpl.java b/.../org/wso2/carbon/identity/application/common/dao/impl/AuthenticatorManagementDAOImpl.java
@@ -132,7 +132,8 @@ public List<UserDefinedLocalAuthenticatorConfig> getAllUserDefinedLocalAuthentic
                         config.setDisplayName(resultSet.getString(Column.DISPLAY_NAME));
                         config.setEnabled(resultSet.getString(Column.IS_ENABLED).equals(IS_TRUE_VALUE));
                         config.setDefinedByType(DefinedByType.valueOf(resultSet.getString(Column.DEFINED_BY)));
-                        return authenticatorConfigMap.put(resultSet.getInt(Column.ID), config);
+                        authenticatorConfigMap.put(resultSet.getInt(Column.ID), config);
+                        return null;
                     },
                     statement -> {
                         statement.setString(Column.DEFINED_BY, DefinedByType.USER.toString());
@@ -171,7 +172,7 @@ public void deleteUserDefinedLocalAuthenticator(String authenticatorConfigName,
     }
 
     private UserDefinedLocalAuthenticatorConfig getUserDefinedLocalAuthenticatorByName(String authenticatorConfigName,
-            int tenantId) throws AuthenticatorMgtServerException, TransactionException {
+            int tenantId) throws TransactionException {
 
         NamedJdbcTemplate jdbcTemplate = new NamedJdbcTemplate(IdentityDatabaseUtil.getDataSource());
         UserDefinedLocalAuthenticatorConfig retrievedConfigs = null;
@@ -185,7 +186,8 @@ private UserDefinedLocalAuthenticatorConfig getUserDefinedLocalAuthenticatorByNa
                     config.setDisplayName(resultSet.getString(Column.DISPLAY_NAME));
                     config.setEnabled(resultSet.getString(Column.IS_ENABLED).equals(IS_TRUE_VALUE));
                     config.setDefinedByType(DefinedByType.USER);
-                    return authenticatorConfigMap.put(resultSet.getInt(Column.ID), config);
+                    authenticatorConfigMap.put(resultSet.getInt(Column.ID), config);
+                    return null;
                 },
                 statement -> {
                     statement.setString(Column.NAME, authenticatorConfigName);