libre.sh/keycloak-scim
4
0
Fork 0
Code Issues 15 Pull requests Releases 1 Activity Actions

added min length > 0 check to required (#20234)

Browse source 
fixes: #20096
This commit is contained in:
Erik Jan de Wit 2023-05-11 18:17:24 +02:00 committed by GitHub
parent 4326a46dd0
commit a2bdfab9c5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
js/apps/admin-ui/src/user/UserProfileFields.tsx
View file

@ -1,7 +1,4 @@
import type {
UserProfileAttribute,
UserProfileAttributeRequired,
} from "@keycloak/keycloak-admin-client/lib/defs/userProfileConfig";
import type { UserProfileAttribute } from "@keycloak/keycloak-admin-client/lib/defs/userProfileConfig";
import {
Form,
FormGroup,
@ -92,8 +89,9 @@ const FormField = ({ attribute, roles }: FormFieldProps) => {
const isRootAttribute = (attr?: string) =>
attr && ROOT_ATTRIBUTES.includes(attr);
const isRequired = (required: UserProfileAttributeRequired | undefined) =>
Object.keys(required || {}).length !== 0;
const isRequired = (attribute: UserProfileAttribute) =>
Object.keys(attribute.required || {}).length !== 0 ||
((attribute.validations?.length?.min as number) || 0) > 0;
const fieldName = (attribute: UserProfileAttribute) =>
`${isRootAttribute(attribute.name) ? "" : "attributes."}${attribute.name}`;
@ -107,7 +105,7 @@ const FormField = ({ attribute, roles }: FormFieldProps) => {
: attribute.displayName) || attribute.name
}
fieldId={attribute.name}
isRequired={isRequired(attribute.required)}
isRequired={isRequired(attribute)}
validated={errors.username ? "error" : "default"}
helperTextInvalid={t("common:required")}
>