Address comments

Thisara-Welmilla · Feb 3, 2025 · 4036ef9 · 4036ef9
1 parent 0814e54
commit 4036ef9
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/...va/org/wso2/carbon/identity/application/common/constant/AuthenticatorMgtSQLConstants.java b/...va/org/wso2/carbon/identity/application/common/constant/AuthenticatorMgtSQLConstants.java
@@ -67,7 +67,7 @@ public static class Query {
         public static final String UPDATE_AUTHENTICATOR_SQL = "UPDATE IDP_AUTHENTICATOR SET IS_ENABLED = " +
                 ":IS_ENABLED;, DISPLAY_NAME = :DISPLAY_NAME;, IMAGE_URL = :IMAGE_URL;, DESCRIPTION = :DESCRIPTION; " +
                 "WHERE NAME = :NAME; AND TENANT_ID = :TENANT_ID;";
-        public static final String GET_AUTHENTICATOR_SQL = "SELECT * FROM IDP_AUTHENTICATOR " +
+        public static final String GET_USER_DEFINED_LOCAL_AUTHENTICATOR_SQL = "SELECT * FROM IDP_AUTHENTICATOR " +
                 "WHERE DEFINED_BY = :DEFINED_BY; AND NAME = :NAME; AND TENANT_ID = :TENANT_ID;" +
                 "AND IDP_ID IN (SELECT ID FROM IDP WHERE IDP.NAME = :IDP_NAME; " +
                         "AND IDP.TENANT_ID = :TENANT_ID;)";

diff --git a/.../org/wso2/carbon/identity/application/common/dao/impl/AuthenticatorManagementDAOImpl.java b/.../org/wso2/carbon/identity/application/common/dao/impl/AuthenticatorManagementDAOImpl.java
@@ -200,7 +200,7 @@ private UserDefinedLocalAuthenticatorConfig getUserDefinedLocalAuthenticatorByNa
 
         NamedJdbcTemplate jdbcTemplate = new NamedJdbcTemplate(IdentityDatabaseUtil.getDataSource());
         AuthenticatorConfigDaoModel configDaoModel = jdbcTemplate.withTransaction(template ->
-            template.fetchSingleRecord(Query.GET_AUTHENTICATOR_SQL,
+            template.fetchSingleRecord(Query.GET_USER_DEFINED_LOCAL_AUTHENTICATOR_SQL,
                 (resultSet, rowNumber) -> {
                     UserDefinedLocalAuthenticatorConfig config = getLocalAuthenticatorConfigBasedOnType(
                             resultSet.getString(Column.AUTHENTICATION_TYPE));

diff --git a/...rg.wso2.carbon.idp.mgt/src/main/java/org/wso2/carbon/idp/mgt/dao/IdPManagementFacade.java b/...rg.wso2.carbon.idp.mgt/src/main/java/org/wso2/carbon/idp/mgt/dao/IdPManagementFacade.java
@@ -331,7 +331,6 @@ private IdentityProvider populateEndpointConfig(IdentityProvider identityProvide
         }
         endpointConfigurationManager.resolveEndpointConfig(identityProvider.getFederatedAuthenticatorConfigs()[0],
                 tenantDomain);
-        identityProvider.setDefaultAuthenticatorConfig(identityProvider.getFederatedAuthenticatorConfigs()[0]);
         return identityProvider;
     }