From a933d6c71a1cf6ec8d2cb0c6965cc84bcf3aa42e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pierre=20No=C3=ABl?= Date: Thu, 16 Sep 2021 11:52:46 +0200 Subject: [PATCH] remove possibility to set collectionName table for a CT or component through that CTB API --- .../server/services/schema-builder/component-builder.js | 5 ++--- .../services/schema-builder/content-type-builder.js | 9 ++------- 2 files changed, 4 insertions(+), 10 deletions(-) diff --git a/packages/core/content-type-builder/server/services/schema-builder/component-builder.js b/packages/core/content-type-builder/server/services/schema-builder/component-builder.js index 52af3de7dc..422b6406f7 100644 --- a/packages/core/content-type-builder/server/services/schema-builder/component-builder.js +++ b/packages/core/content-type-builder/server/services/schema-builder/component-builder.js @@ -42,13 +42,13 @@ module.exports = function createComponentBuilder() { filename: `${nameToSlug(infos.name)}.json`, }); - const defaultCollectionName = `components_${nameToCollectionName( + const collectionName = `components_${nameToCollectionName( infos.category )}_${nameToCollectionName(pluralize(infos.name))}`; handler .setUID(uid) - .set('collectionName', infos.collectionName || defaultCollectionName) + .set('collectionName', collectionName) .set(['info', 'name'], infos.name) .set(['info', 'icon'], infos.icon) .set(['info', 'description'], infos.description) @@ -98,7 +98,6 @@ module.exports = function createComponentBuilder() { component .setUID(newUID) .setDir(newDir) - .set('collectionName', infos.collectionName) .set(['info', 'name'], infos.name) .set(['info', 'icon'], infos.icon) .set(['info', 'description'], infos.description) diff --git a/packages/core/content-type-builder/server/services/schema-builder/content-type-builder.js b/packages/core/content-type-builder/server/services/schema-builder/content-type-builder.js index 10b335bc4a..af6e36a7e3 100644 --- a/packages/core/content-type-builder/server/services/schema-builder/content-type-builder.js +++ b/packages/core/content-type-builder/server/services/schema-builder/content-type-builder.js @@ -84,8 +84,6 @@ module.exports = function createComponentBuilder() { this.contentTypes.set(uid, contentType); - const defaultCollectionName = nameToCollectionName(infos.pluralName); - // support self referencing content type relation Object.keys(infos.attributes).forEach(key => { const { target } = infos.attributes[key]; @@ -97,16 +95,14 @@ module.exports = function createComponentBuilder() { contentType .setUID(uid) .set('kind', infos.kind || typeKinds.COLLECTION_TYPE) - .set('collectionName', infos.collectionName || defaultCollectionName) + .set('collectionName', nameToCollectionName(infos.pluralName)) .set('info', { singularName: infos.singularName, pluralName: infos.pluralName, displayName: infos.displayName, description: infos.description, }) - .set('options', { - draftAndPublish: infos.draftAndPublish || false, - }) + .set('options', { draftAndPublish: infos.draftAndPublish || false }) .set('pluginOptions', infos.pluginOptions) .setAttributes(this.convertAttributes(infos.attributes)); @@ -213,7 +209,6 @@ module.exports = function createComponentBuilder() { }); contentType - .set('collectionName', infos.collectionName) .set('kind', infos.kind || contentType.schema.kind) .set(['info', 'displayName'], infos.displayName) .set(['info', 'description'], infos.description)