yujunjun/strapi
1
0
Fork 0
mirror of https://github.com/strapi/strapi.git synced 2025-11-03 03:17:11 +00:00
Code Issues Packages Projects Releases Wiki Activity

Synchronize empty values check between admin UI and server validation

Browse Source
This commit is contained in:
Gustav Hansen 2022-07-07 16:11:37 +02:00
parent 2845ba1ec1
commit 8f0906d6af
2 changed files with 12 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
packages/core/content-type-builder/admin/src/components/FormModal/attributes/types.js
View File

@ -165,7 +165,7 @@ const types = {
.test({
name: 'doesNotHaveEmptyValues',
message: getTrad('error.validation.enum-empty-string'),
test: values => !values.some(val => val === ''),
test: values => !values.map(toRegressedEnumValue).some(val => val === ''),
})
.test({
name: 'doesMatchRegex',

17
packages/core/strapi/lib/core/domain/content-type/validator.js
View File

@ -57,20 +57,25 @@ const contentTypeSchemaValidator = yup.object().shape({
for (const attrName in attributes) {
const attr = attributes[attrName];
if (attr.type === 'enumeration') {
const regressedValues = attr.enum.map(toRegressedEnumValue);
// should match the GraphQL regex
if (
!attr.enum.map(toRegressedEnumValue).every(value => GRAPHQL_ENUM_REGEX.test(value))
) {
if (!regressedValues.every(value => GRAPHQL_ENUM_REGEX.test(value))) {
const message = `Invalid enumervarion value. Values should always have an alphabetical character preceding any number. Update your enumeration '${attrName}'.`;
return this.createError({ message });
}
// should not contain empty values
if (regressedValues.some(value => value === '')) {
return this.createError({
message: `At least one value of the enumeration '${attrName}' appears to be empty.`,
});
}
// should not collide
const duplicates = _.uniq(
attr.enum
.map(toRegressedEnumValue)
.filter((value, index, values) => values.indexOf(value) !== index)
regressedValues.filter((value, index, values) => values.indexOf(value) !== index)
);
if (duplicates.length) {