Skip to content

Commit

Permalink
Fix eslint issue
Browse files Browse the repository at this point in the history
  • Loading branch information
AfraHussaindeen committed Nov 26, 2024
1 parent 468ea6e commit a5bd5d1
Showing 1 changed file with 18 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,13 @@ import {
import { getUsernameConfiguration } from "@wso2is/admin.users.v1/utils/user-management-utils";
import { useValidationConfigData } from "@wso2is/admin.validation.v1/api";
import { IdentityAppsError } from "@wso2is/core/errors";
import { AlertLevels, Claim, ClaimsGetParams, IdentifiableComponentInterface, Property, UniquenessScope } from "@wso2is/core/models";
import {
AlertLevels,
Claim,
ClaimsGetParams,
IdentifiableComponentInterface,
UniquenessScope
} from "@wso2is/core/models";
import { addAlert } from "@wso2is/core/store";
import { Field, Form } from "@wso2is/form";
import { ConfirmationModal, ContentLoader, EmphasizedSegment, Message, PageLayout } from "@wso2is/react-components";
Expand Down Expand Up @@ -341,15 +347,14 @@ const AlternativeLoginIdentifierInterface: FunctionComponent<AlternativeLoginIde
* Does not modify claims that are not selected.
*/
const updateClaimUniquenessScope = (claim: Claim, checkedClaims: string[]) => {
const isSelected = checkedClaims?.includes(claim.claimURI);

// Only update if selected and either uniquenessScope is undefined or not ACROSS_USERSTORES
const needsUpdate = isSelected &&
const isSelected: boolean = checkedClaims?.includes(claim.claimURI);

const needsUpdate: boolean = isSelected &&
(!claim.uniquenessScope || claim.uniquenessScope !== UniquenessScope.ACROSS_USERSTORES);

return {
isClaimUpdate: needsUpdate,
updatedClaim: needsUpdate
updatedClaim: needsUpdate
? {
...claim,
uniquenessScope: UniquenessScope.ACROSS_USERSTORES
Expand Down Expand Up @@ -431,8 +436,9 @@ const AlternativeLoginIdentifierInterface: FunctionComponent<AlternativeLoginIde
* Checks if any claims need uniqueness scope update
*/
const needsUniquenessScopeUpdate = (checkedClaims: string[]): boolean => {
return checkedClaims.some(claimURI => {
const claim = availableClaims.find(c => c.claimURI === claimURI);
return checkedClaims.some((claimURI: string) => {
const claim: Claim = availableClaims.find((c: Claim) => c.claimURI === claimURI);

return !claim.uniquenessScope || claim.uniquenessScope !== UniquenessScope.ACROSS_USERSTORES;
});
};
Expand All @@ -444,14 +450,15 @@ const AlternativeLoginIdentifierInterface: FunctionComponent<AlternativeLoginIde
formValues: AlternativeLoginIdentifierFormInterface,

Check warning on line 450 in features/admin.alternative-login-identifier.v1/pages/alternative-login-identifier-page.tsx

View workflow job for this annotation

GitHub Actions / ⬣ ESLint (STATIC ANALYSIS) (lts/*, 8.7.4)

Trailing spaces not allowed
hasUserConsent: boolean = false
): void => {
const checkedClaims = getProcessedCheckedClaims(formValues);
const updatedConnectorData = getUpdatedConfigurations(checkedClaims);
const requiresUniquenessScopeUpdate = needsUniquenessScopeUpdate(checkedClaims);
const checkedClaims: string[] = getProcessedCheckedClaims(formValues);
const updatedConnectorData: any = getUpdatedConfigurations(checkedClaims);
const requiresUniquenessScopeUpdate: boolean = needsUniquenessScopeUpdate(checkedClaims);

// Show confirmation modal if uniqueness scope update is needed and no consent yet
if (!hasUserConsent && requiresUniquenessScopeUpdate) {
setPendingFormValues(formValues);
setShowConfirmationModal(true);

return;
}

Expand Down

0 comments on commit a5bd5d1

Please sign in to comment.