Ensure unique fields are localized

Signed-off-by: soupette <cyril.lpz@gmail.com>

Author: soupette

packages/strapi-plugin-content-type-builder/admin/src/components/CheckboxWithDescription/index.js

@@ -3,7 +3,7 @@ import PropTypes from 'prop-types';
 import { Checkbox, Text } from '@buffetjs/core';
 import Wrapper from './Wrapper';
 
-const CheckboxWithDescription = ({ description, label, value, ...rest }) => {
+const CheckboxWithDescription = ({ description, error, label, value, ...rest }) => {
   return (
     <>
       <Wrapper>
@@ -13,19 +13,26 @@ const CheckboxWithDescription = ({ description, label, value, ...rest }) => {
             {description}
           </Text>
         )}
+        {error && (
+          <Text color="lightOrange" lineHeight="18px">
+            {error}
+          </Text>
+        )}
       </Wrapper>
     </>
   );
 };
 
 CheckboxWithDescription.defaultProps = {
   description: null,
+  error: null,
   label: null,
   value: false,
 };
 
 CheckboxWithDescription.propTypes = {
   description: PropTypes.string,
+  error: PropTypes.string,
   label: PropTypes.string,
   value: PropTypes.bool,
 };

packages/strapi-plugin-i18n/admin/src/index.js

@@ -1,5 +1,5 @@
 import React from 'react';
-import { get } from 'lodash';
+import get from 'lodash/get';
 import * as yup from 'yup';
 import pluginPkg from '../../package.json';
 import pluginLogo from './assets/images/logo.svg';
@@ -103,9 +103,26 @@ export default strapi => {
         });
 
         ctbFormsAPI.extendFields(LOCALIZED_FIELDS, {
-          validator: () => ({
+          validator: args => ({
             i18n: yup.object().shape({
-              localized: yup.bool(),
+              localized: yup.bool().test({
+                name: 'ensure-unique-localization',
+                message: getTrad('plugin.schema.i18n.ensure-unique-localization'),
+                test(value) {
+                  if (value === undefined || value) {
+                    return true;
+                  }
+
+                  const unique = get(args, ['3', 'modifiedData', 'unique'], null);
+
+                  // Unique fields must be localized
+                  if (unique && !value) {
+                    return false;
+                  }
+
+                  return true;
+                },
+              }),
             }),
           }),
           form: {

packages/strapi-plugin-i18n/admin/src/translations/en.json

@@ -11,6 +11,7 @@
   "plugin.schema.i18n.localized.description-content-type": "Allow you to have content in different locales",
   "plugin.schema.i18n.localized.label-field": "Enable localization for this field",
   "plugin.schema.i18n.localized.description-field": "The field can have different values in each locale",
+  "plugin.schema.i18n.ensure-unique-localization": "Unique fields must be localized",
   "CheckboxConfirmation.Modal.content": "Disabling localization will engender the deletion of all your content but the one associated to your default locale (if existing).",
   "CheckboxConfirmation.Modal.body": "Do you want to disable it?",
   "CheckboxConfirmation.Modal.button-confirm": "Yes, disable",

packages/strapi-plugin-i18n/admin/src/utils/localizedFields.js

@@ -18,7 +18,6 @@ const LOCALIZED_FIELDS = [
   'string',
   'text',
   'time',
-  'uid',
 ];
 
 export default LOCALIZED_FIELDS;