@webiny/api-headless-cms 0.0.0-unstable.40876133bb → 0.0.0-unstable.60e968001a

package/crud/contentModel.crud.js

@@ -11,16 +11,12 @@ var _objectSpread2 = _interopRequireDefault(require("@babel/runtime/helpers/obje
 
 var _dataloader = _interopRequireDefault(require("dataloader"));
 
+var _error = _interopRequireDefault(require("@webiny/error"));
+
 var _handlerGraphql = require("@webiny/handler-graphql");
 
 var _contentModelManagerFactory = require("./contentModel/contentModelManagerFactory");
 
-var _models = require("./contentModel/models");
-
-var _createFieldModels = require("./contentModel/createFieldModels");
-
-var _error = _interopRequireDefault(require("@webiny/error"));
-
 var _pubsub = require("@webiny/pubsub");
 
 var _beforeCreate = require("./contentModel/beforeCreate");
@@ -49,56 +45,23 @@ var _access = require("../utils/access");
 
 var _validateModelFields = require("./contentModel/validateModelFields");
 
-var _semver = _interopRequireDefault(require("semver"));
+var _validation = require("./contentModel/validation");
+
+var _utils = require("@webiny/utils");
 
 /**
- * TODO: remove for 5.34.0
- * Required because of the 5.33.0 upgrade.
- * Until the upgrade is done, API will break because there is no storageId assigned.
+ * Given a model, return an array of tags ensuring the `type` tag is set.
  */
-const featureVersion = _semver.default.coerce("5.33.0");
-
-const attachStorageIdToFields = fields => {
-  return fields.map(field => {
-    var _field$settings;
-
-    if ((_field$settings = field.settings) !== null && _field$settings !== void 0 && _field$settings.fields) {
-      field.settings.fields = attachStorageIdToFields(field.settings.fields);
-    }
-
-    if (!field.storageId) {
-      field.storageId = field.fieldId;
-    }
-
-    return field;
-  });
-};
-
-const attachStorageIdToModelFields = model => {
-  if (!model.webinyVersion) {
-    return model.fields;
-  }
-
-  const version = _semver.default.coerce(model.webinyVersion);
-
-  if (!version) {
-    return model.fields;
-  }
-  /**
-   * Unfortunately we need to check for beta and next.
-   * TODO remove after 5.33.0
-   */
-
-
-  if (model.webinyVersion.match(/beta|next/)) {
-    return attachStorageIdToFields(model.fields);
-  }
+const ensureTypeTag = model => {
+  // Let's make sure we have a `type` tag assigned.
+  // If `type` tag is not set, set it to a default one (`model`).
+  const tags = model.tags || [];
 
-  if (_semver.default.compare(version, featureVersion) >= 0) {
-    return model.fields;
+  if (!tags.some(tag => tag.startsWith("type:"))) {
+    tags.push("type:model");
   }
 
-  return attachStorageIdToFields(model.fields);
+  return tags;
 };
 
 const createModelsCrud = params => {
@@ -119,7 +82,7 @@ const createModelsCrud = params => {
       });
       return [models.map(model => {
         return (0, _objectSpread2.default)((0, _objectSpread2.default)({}, model), {}, {
-          fields: attachStorageIdToModelFields(model),
+          tags: ensureTypeTag(model),
           tenant: model.tenant || getTenant().id,
           locale: model.locale || getLocale().code
         });
@@ -157,19 +120,20 @@ const createModelsCrud = params => {
      */
     .filter(plugin => {
       const {
-        tenant: t,
-        locale: l
+        tenant: modelTenant,
+        locale: modelLocale
       } = plugin.contentModel;
 
-      if (t && t !== tenant) {
+      if (modelTenant && modelTenant !== tenant) {
         return false;
-      } else if (l && l !== locale) {
+      } else if (modelLocale && modelLocale !== locale) {
         return false;
       }
 
       return true;
     }).map(plugin => {
       return (0, _objectSpread2.default)((0, _objectSpread2.default)({}, plugin.contentModel), {}, {
+        tags: ensureTypeTag(plugin.contentModel),
         tenant,
         locale,
         webinyVersion: context.WEBINY_VERSION
@@ -207,6 +171,7 @@ const createModelsCrud = params => {
     }
 
     return (0, _objectSpread2.default)((0, _objectSpread2.default)({}, model), {}, {
+      tags: ensureTypeTag(model),
       tenant: model.tenant || getTenant().id,
       locale: model.locale || getLocale().code
     });
@@ -253,20 +218,40 @@ const createModelsCrud = params => {
     }
 
     return await updateManager(context, model);
-  }; // create
+  };
+  /**
+   * Create
+   */
 
 
   const onModelBeforeCreate = (0, _pubsub.createTopic)("cms.onModelBeforeCreate");
-  const onModelAfterCreate = (0, _pubsub.createTopic)("cms.onModelAfterCreate"); // create from
+  const onModelAfterCreate = (0, _pubsub.createTopic)("cms.onModelAfterCreate");
+  const onModelCreateError = (0, _pubsub.createTopic)("cms.onModelCreateError");
+  /**
+   * Create from / clone
+   */
 
   const onModelBeforeCreateFrom = (0, _pubsub.createTopic)("cms.onModelBeforeCreateFrom");
-  const onModelAfterCreateFrom = (0, _pubsub.createTopic)("cms.onModelAfterCreateFrom"); // update
+  const onModelAfterCreateFrom = (0, _pubsub.createTopic)("cms.onModelAfterCreateFrom");
+  const onModelCreateFromError = (0, _pubsub.createTopic)("cms.onModelCreateFromError");
+  /**
+   * Update
+   */
 
   const onModelBeforeUpdate = (0, _pubsub.createTopic)("cms.onModelBeforeUpdate");
-  const onModelAfterUpdate = (0, _pubsub.createTopic)("cms.onModelAfterUpdate"); // delete
+  const onModelAfterUpdate = (0, _pubsub.createTopic)("cms.onModelAfterUpdate");
+  const onModelUpdateError = (0, _pubsub.createTopic)("cms.onModelUpdateError");
+  /**
+   * Delete
+   */
 
   const onModelBeforeDelete = (0, _pubsub.createTopic)("cms.onModelBeforeDelete");
   const onModelAfterDelete = (0, _pubsub.createTopic)("cms.onModelAfterDelete");
+  const onModelDeleteError = (0, _pubsub.createTopic)("cms.onModelDeleteError");
+  /**
+   * Initialize
+   */
+
   const onModelInitialize = (0, _pubsub.createTopic)("cms.onModelInitialize");
   /**
    * We need to assign some default behaviors.
@@ -322,36 +307,43 @@ const createModelsCrud = params => {
      */
     onModelBeforeCreate,
     onModelAfterCreate,
+    onModelCreateError,
     onModelBeforeCreateFrom,
     onModelAfterCreateFrom,
+    onModelCreateFromError,
     onModelBeforeUpdate,
     onModelAfterUpdate,
+    onModelUpdateError,
     onModelBeforeDelete,
     onModelAfterDelete,
+    onModelDeleteError,
     onModelInitialize,
     clearModelsCache,
     getModel,
     listModels,
 
-    async createModel(inputData) {
+    async createModel(input) {
       await checkModelPermissions("w");
-      const createdData = new _models.CreateContentModelModel().populate(inputData);
-      await createdData.validate();
-      const input = await createdData.toJSON();
+      const result = await (0, _validation.createModelCreateValidation)().safeParseAsync(input);
+
+      if (!result.success) {
+        throw (0, _utils.createZodError)(result.error);
+      }
+
+      const data = result.data;
       context.security.disableAuthorization();
-      const group = await context.cms.getGroup(input.group);
+      const group = await context.cms.getGroup(data.group);
       context.security.enableAuthorization();
 
       if (!group) {
-        throw new _handlerGraphql.NotFoundError(`There is no group "${input.group}".`);
+        throw new _handlerGraphql.NotFoundError(`There is no group "${data.group}".`);
       }
 
-      const fields = await (0, _createFieldModels.createFieldModels)(input.fields);
       const identity = getIdentity();
       const model = {
-        name: input.name,
-        description: input.description || "",
-        modelId: input.modelId || "",
+        name: data.name,
+        description: data.description || "",
+        modelId: data.modelId || "",
         titleFieldId: "id",
         locale: getLocale().code,
         tenant: getTenant().id,
@@ -366,25 +358,37 @@ const createModelsCrud = params => {
         },
         createdOn: new Date().toISOString(),
         savedOn: new Date().toISOString(),
-        fields,
+        fields: data.fields,
         lockedFields: [],
-        layout: input.layout || [],
+        layout: data.layout || [],
+        tags: [...(data.tags || [])],
         webinyVersion: context.WEBINY_VERSION
       };
-      await onModelBeforeCreate.publish({
-        input,
-        model
-      });
-      const createdModel = await storageOperations.models.create({
-        model
-      });
-      loaders.listModels.clearAll();
-      await updateManager(context, model);
-      await onModelAfterCreate.publish({
-        input,
-        model: createdModel
-      });
-      return createdModel;
+      model.tags = ensureTypeTag(model);
+
+      try {
+        await onModelBeforeCreate.publish({
+          input: data,
+          model
+        });
+        const createdModel = await storageOperations.models.create({
+          model
+        });
+        loaders.listModels.clearAll();
+        await updateManager(context, model);
+        await onModelAfterCreate.publish({
+          input: data,
+          model: createdModel
+        });
+        return createdModel;
+      } catch (ex) {
+        await onModelCreateError.publish({
+          input: data,
+          model,
+          error: ex
+        });
+        throw ex;
+      }
     },
 
     /**
@@ -401,44 +405,59 @@ const createModelsCrud = params => {
         locale: initialModel.locale || getLocale().code,
         webinyVersion: context.WEBINY_VERSION
       });
-      await onModelBeforeUpdate.publish({
-        input: {},
-        original,
-        model
-      });
-      const resultModel = await storageOperations.models.update({
-        model
-      });
-      await updateManager(context, resultModel);
-      loaders.listModels.clearAll();
-      await onModelAfterUpdate.publish({
-        input: {},
-        original,
-        model: resultModel
-      });
-      return resultModel;
+
+      try {
+        await onModelBeforeUpdate.publish({
+          input: {},
+          original,
+          model
+        });
+        const resultModel = await storageOperations.models.update({
+          model
+        });
+        await updateManager(context, resultModel);
+        loaders.listModels.clearAll();
+        await onModelAfterUpdate.publish({
+          input: {},
+          original,
+          model: resultModel
+        });
+        return resultModel;
+      } catch (ex) {
+        await onModelUpdateError.publish({
+          input: {},
+          original,
+          model,
+          error: ex
+        });
+        throw ex;
+      }
     },
 
-    async createModelFrom(modelId, data) {
+    async createModelFrom(modelId, userInput) {
       await checkModelPermissions("w");
       /**
        * Get a model record; this will also perform ownership validation.
        */
 
       const original = await getModel(modelId);
-      const createdData = new _models.CreateContentModelModelFrom().populate({
-        name: data.name,
-        modelId: data.modelId,
-        description: data.description || original.description,
-        group: data.group,
-        locale: data.locale
+      const result = await (0, _validation.createModelCreateFromValidation)().safeParseAsync({
+        name: userInput.name,
+        modelId: userInput.modelId,
+        description: userInput.description || original.description,
+        group: userInput.group,
+        locale: userInput.locale
       });
-      await createdData.validate();
-      const input = await createdData.toJSON();
-      const locale = await context.i18n.getLocale(input.locale || original.locale);
+
+      if (!result.success) {
+        throw (0, _utils.createZodError)(result.error);
+      }
+
+      const data = result.data;
+      const locale = await context.i18n.getLocale(data.locale || original.locale);
 
       if (!locale) {
-        throw new _handlerGraphql.NotFoundError(`There is no locale "${input.locale}".`);
+        throw new _handlerGraphql.NotFoundError(`There is no locale "${data.locale}".`);
       }
       /**
        * Use storage operations directly because we cannot get group from different locale via context methods.
@@ -446,13 +465,13 @@ const createModelsCrud = params => {
 
 
       const group = await context.cms.storageOperations.groups.get({
-        id: input.group,
+        id: data.group,
         tenant: original.tenant,
         locale: locale.code
       });
 
       if (!group) {
-        throw new _handlerGraphql.NotFoundError(`There is no group "${input.group}".`);
+        throw new _handlerGraphql.NotFoundError(`There is no group "${data.group}".`);
       }
 
       const identity = getIdentity();
@@ -462,9 +481,9 @@ const createModelsCrud = params => {
           id: group.id,
           name: group.name
         },
-        name: input.name,
-        modelId: input.modelId || "",
-        description: input.description || "",
+        name: data.name,
+        modelId: data.modelId || "",
+        description: data.description || "",
         createdBy: {
           id: identity.id,
           displayName: identity.displayName,
@@ -475,35 +494,48 @@ const createModelsCrud = params => {
         lockedFields: [],
         webinyVersion: context.WEBINY_VERSION
       });
-      await onModelBeforeCreateFrom.publish({
-        input,
-        model,
-        original
-      });
-      const createdModel = await storageOperations.models.create({
-        model
-      });
-      loaders.listModels.clearAll();
-      await updateManager(context, model);
-      await onModelAfterCreateFrom.publish({
-        input,
-        original,
-        model: createdModel
-      });
-      return createdModel;
+
+      try {
+        await onModelBeforeCreateFrom.publish({
+          input: data,
+          model,
+          original
+        });
+        const createdModel = await storageOperations.models.create({
+          model
+        });
+        loaders.listModels.clearAll();
+        await updateManager(context, model);
+        await onModelAfterCreateFrom.publish({
+          input: data,
+          original,
+          model: createdModel
+        });
+        return createdModel;
+      } catch (ex) {
+        await onModelCreateFromError.publish({
+          input: data,
+          original,
+          model,
+          error: ex
+        });
+        throw ex;
+      }
     },
 
-    async updateModel(modelId, inputData) {
+    async updateModel(modelId, input) {
       await checkModelPermissions("w"); // Get a model record; this will also perform ownership validation.
 
       const original = await getModel(modelId);
-      const updatedData = new _models.UpdateContentModelModel().populate(inputData);
-      await updatedData.validate();
-      const input = await updatedData.toJSON({
-        onlyDirty: true
-      });
+      const result = await (0, _validation.createModelUpdateValidation)().safeParseAsync(input);
 
-      if (Object.keys(input).length === 0) {
+      if (!result.success) {
+        throw (0, _utils.createZodError)(result.error);
+      }
+
+      const data = result.data;
+
+      if (Object.keys(data).length === 0) {
         /**
          * We need to return the original if nothing is to be updated.
          */
@@ -515,13 +547,13 @@ const createModelsCrud = params => {
         name: original.group.name
       };
 
-      if (input.group) {
+      if (data.group) {
         context.security.disableAuthorization();
-        const groupData = await context.cms.getGroup(input.group);
+        const groupData = await context.cms.getGroup(data.group);
         context.security.enableAuthorization();
 
         if (!groupData) {
-          throw new _handlerGraphql.NotFoundError(`There is no group "${input.group}".`);
+          throw new _handlerGraphql.NotFoundError(`There is no group "${data.group}".`);
         }
 
         group = {
@@ -530,57 +562,76 @@ const createModelsCrud = params => {
         };
       }
 
-      const fields = await (0, _createFieldModels.createFieldModels)(inputData.fields);
-      const model = (0, _objectSpread2.default)((0, _objectSpread2.default)((0, _objectSpread2.default)({}, original), input), {}, {
+      const model = (0, _objectSpread2.default)((0, _objectSpread2.default)((0, _objectSpread2.default)({}, original), data), {}, {
         group,
         tenant: original.tenant || getTenant().id,
         locale: original.locale || getLocale().code,
         webinyVersion: context.WEBINY_VERSION,
-        fields,
         savedOn: new Date().toISOString()
       });
-      await onModelBeforeUpdate.publish({
-        input,
-        original,
-        model
-      });
-      const resultModel = await storageOperations.models.update({
-        model
-      });
-      await updateManager(context, resultModel);
-      await onModelAfterUpdate.publish({
-        input,
-        original,
-        model: resultModel
-      });
-      return resultModel;
+      model.tags = ensureTypeTag(model);
+
+      try {
+        await onModelBeforeUpdate.publish({
+          input: data,
+          original,
+          model
+        });
+        const resultModel = await storageOperations.models.update({
+          model
+        });
+        await updateManager(context, resultModel);
+        await onModelAfterUpdate.publish({
+          input: data,
+          original,
+          model: resultModel
+        });
+        return resultModel;
+      } catch (ex) {
+        await onModelUpdateError.publish({
+          input: data,
+          model,
+          original,
+          error: ex
+        });
+        throw ex;
+      }
     },
 
     async deleteModel(modelId) {
       await checkModelPermissions("d");
       const model = await getModel(modelId);
-      await onModelBeforeDelete.publish({
-        model
-      });
 
       try {
-        await storageOperations.models.delete({
+        await onModelBeforeDelete.publish({
+          model
+        });
+
+        try {
+          await storageOperations.models.delete({
+            model
+          });
+        } catch (ex) {
+          throw new _error.default(ex.message || "Could not delete the content model", ex.code || "CONTENT_MODEL_DELETE_ERROR", {
+            error: ex,
+            modelId: model.modelId
+          });
+        }
+
+        await onModelAfterDelete.publish({
           model
         });
+        managers.delete(model.modelId);
       } catch (ex) {
-        throw new _error.default(ex.message || "Could not delete the content model", ex.code || "CONTENT_MODEL_DELETE_ERROR", {
-          error: ex,
-          modelId: model.modelId
+        await onModelDeleteError.publish({
+          model,
+          error: ex
         });
+        throw ex;
       }
-
-      await onModelAfterDelete.publish({
-        model
-      });
-      managers.delete(model.modelId);
     },
 
-    async initializeModel(modelId) {
+    async initializeModel(modelId, data) {
       /**
        * We require that users have write permissions to initialize models.
        * Maybe introduce another permission for it?
@@ -588,7 +639,8 @@ const createModelsCrud = params => {
       await checkModelPermissions("w");
       const model = await getModel(modelId);
       await onModelInitialize.publish({
-        model
+        model,
+        data
       });
       return true;
     },