@webiny/api-headless-cms 5.20.0-beta.1 → 5.21.0

package/content/plugins/crud/contentModel.crud.js

@@ -23,8 +23,6 @@ var _createFieldModels = require("./contentModel/createFieldModels");
 
 var _validateLayout = require("./contentModel/validateLayout");
 
-var _apiSecurity = require("@webiny/api-security");
-
 var _error = _interopRequireDefault(require("@webiny/error"));
 
 var _CmsModelPlugin = require("../CmsModelPlugin");
@@ -148,7 +146,7 @@ const createModelsCrud = params => {
     return databaseModels.concat(pluginsModels);
   };
 
-  const listOperations = async () => {
+  const listModels = async () => {
     const permission = await checkModelPermissions("r");
     const models = await modelsList();
     return utils.filterAsync(models, async model => {
@@ -160,7 +158,7 @@ const createModelsCrud = params => {
     });
   };
 
-  const get = async modelId => {
+  const getModel = async modelId => {
     const permission = await checkModelPermissions("r");
     const model = await modelsGet(modelId);
     utils.checkOwnership(context, permission, model);
@@ -168,7 +166,9 @@ const createModelsCrud = params => {
     return model;
   };
 
-  const getManager = async modelId => {
+  const getModelManager = async target => {
+    const modelId = typeof target === "string" ? target : target.modelId;
+
     if (managers.has(modelId)) {
       return managers.get(modelId);
     }
@@ -183,72 +183,57 @@ const createModelsCrud = params => {
     return await updateManager(context, model);
   };
 
-  const onBeforeCreate = (0, _pubsub.createTopic)();
-  const onAfterCreate = (0, _pubsub.createTopic)();
-  const onBeforeCreateFrom = (0, _pubsub.createTopic)();
-  const onAfterCreateFrom = (0, _pubsub.createTopic)();
-  const onBeforeUpdate = (0, _pubsub.createTopic)();
-  const onAfterUpdate = (0, _pubsub.createTopic)();
-  const onBeforeDelete = (0, _pubsub.createTopic)();
-  const onAfterDelete = (0, _pubsub.createTopic)();
+  const onBeforeModelCreate = (0, _pubsub.createTopic)();
+  const onAfterModelCreate = (0, _pubsub.createTopic)();
+  const onBeforeModelCreateFrom = (0, _pubsub.createTopic)();
+  const onAfterModelCreateFrom = (0, _pubsub.createTopic)();
+  const onBeforeModelUpdate = (0, _pubsub.createTopic)();
+  const onAfterModelUpdate = (0, _pubsub.createTopic)();
+  const onBeforeModelDelete = (0, _pubsub.createTopic)();
+  const onAfterModelDelete = (0, _pubsub.createTopic)();
   /**
    * We need to assign some default behaviors.
    */
 
   (0, _beforeCreate.assignBeforeModelCreate)({
-    onBeforeCreate,
-    onBeforeCreateFrom,
+    onBeforeModelCreate,
+    onBeforeModelCreateFrom,
     plugins: context.plugins,
     storageOperations
   });
   (0, _afterCreate.assignAfterModelCreate)({
     context,
-    onAfterCreate
+    onAfterModelCreate
   });
   (0, _beforeUpdate.assignBeforeModelUpdate)({
-    onBeforeUpdate,
+    onBeforeModelUpdate,
     plugins: context.plugins,
     storageOperations
   });
   (0, _afterUpdate.assignAfterModelUpdate)({
     context,
-    onAfterUpdate
+    onAfterModelUpdate
   });
   (0, _beforeDelete.assignBeforeModelDelete)({
-    onBeforeDelete,
+    onBeforeModelDelete,
     plugins: context.plugins,
     storageOperations
   });
   (0, _afterDelete.assignAfterModelDelete)({
     context,
-    onAfterDelete
+    onAfterModelDelete
   });
   return {
-    onBeforeModelCreate: onBeforeCreate,
-    onAfterModelCreate: onAfterCreate,
-    onBeforeModelCreateFrom: onBeforeCreateFrom,
-    onAfterModelCreateFrom: onAfterCreateFrom,
-    onBeforeModelUpdate: onBeforeUpdate,
-    onAfterModelUpdate: onAfterUpdate,
-    onBeforeModelDelete: onBeforeDelete,
-    onAfterModelDelete: onAfterDelete,
-    silentAuthModel: () => {
-      return {
-        list: async () => {
-          try {
-            return await listOperations();
-          } catch (ex) {
-            if (ex instanceof _apiSecurity.NotAuthorizedError) {
-              return [];
-            }
-
-            throw ex;
-          }
-        }
-      };
-    },
-    getModel: get,
-    listModels: listOperations,
+    onBeforeModelCreate,
+    onAfterModelCreate,
+    onBeforeModelCreateFrom,
+    onAfterModelCreateFrom,
+    onBeforeModelUpdate,
+    onAfterModelUpdate,
+    onBeforeModelDelete,
+    onAfterModelDelete,
+    getModel,
+    listModels,
 
     async createModel(inputData) {
       await checkModelPermissions("w");
@@ -287,9 +272,9 @@ const createModelsCrud = params => {
         layout: [],
         webinyVersion: context.WEBINY_VERSION
       };
-      await onBeforeCreate.publish({
-        model,
-        input
+      await onBeforeModelCreate.publish({
+        input,
+        model
       });
       const createdModel = await storageOperations.models.create({
         input,
@@ -297,7 +282,7 @@ const createModelsCrud = params => {
       });
       loaders.listModels.clearAll();
       await updateManager(context, model);
-      await onAfterCreate.publish({
+      await onAfterModelCreate.publish({
         input,
         model: createdModel
       });
@@ -320,7 +305,7 @@ const createModelsCrud = params => {
         webinyVersion: context.WEBINY_VERSION
       });
 
-      await onBeforeUpdate.publish({
+      await onBeforeModelUpdate.publish({
         input: {},
         original,
         model
@@ -332,7 +317,7 @@ const createModelsCrud = params => {
       });
       await updateManager(context, resultModel);
       loaders.listModels.clearAll();
-      await onAfterUpdate.publish({
+      await onAfterModelUpdate.publish({
         input: {},
         original,
         model: resultModel
@@ -346,7 +331,7 @@ const createModelsCrud = params => {
        * Get a model record; this will also perform ownership validation.
        */
 
-      const original = await get(modelId);
+      const original = await getModel(modelId);
       const createdData = new _models.CreateContentModelModelFrom().populate({
         name: data.name,
         modelId: data.modelId,
@@ -398,10 +383,10 @@ const createModelsCrud = params => {
         webinyVersion: context.WEBINY_VERSION
       });
 
-      await onBeforeCreateFrom.publish({
+      await onBeforeModelCreateFrom.publish({
+        input,
         model,
-        original,
-        input
+        original
       });
       const createdModel = await storageOperations.models.create({
         input,
@@ -409,7 +394,7 @@ const createModelsCrud = params => {
       });
       loaders.listModels.clearAll();
       await updateManager(context, model);
-      await onAfterCreateFrom.publish({
+      await onAfterModelCreateFrom.publish({
         input,
         original,
         model: createdModel
@@ -420,7 +405,7 @@ const createModelsCrud = params => {
     async updateModel(modelId, inputData) {
       await checkModelPermissions("w"); // Get a model record; this will also perform ownership validation.
 
-      const original = await get(modelId);
+      const original = await getModel(modelId);
       const updatedData = new _models.UpdateContentModelModel().populate(inputData);
       await updatedData.validate();
       const input = await updatedData.toJSON({
@@ -463,7 +448,7 @@ const createModelsCrud = params => {
       });
 
       (0, _validateLayout.validateLayout)(model, modelFields);
-      await onBeforeUpdate.publish({
+      await onBeforeModelUpdate.publish({
         input,
         original,
         model
@@ -474,18 +459,18 @@ const createModelsCrud = params => {
         input
       });
       await updateManager(context, resultModel);
-      await onAfterUpdate.publish({
+      await onAfterModelUpdate.publish({
+        input,
         original,
-        model: resultModel,
-        input
+        model: resultModel
       });
       return resultModel;
     },
 
     async deleteModel(modelId) {
       await checkModelPermissions("d");
-      const model = await get(modelId);
-      await onBeforeDelete.publish({
+      const model = await getModel(modelId);
+      await onBeforeModelDelete.publish({
         model
       });
 
@@ -500,13 +485,13 @@ const createModelsCrud = params => {
         });
       }
 
-      await onAfterDelete.publish({
+      await onAfterModelDelete.publish({
         model
       });
       managers.delete(model.modelId);
     },
 
-    getModelManager: getManager,
+    getModelManager,
     getManagers: () => managers
   };
 };

package/content/plugins/graphqlFields/boolean.js

@@ -5,6 +5,8 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
+var _requiredField = require("./requiredField");
+
 const createListFilters = ({
   field
 }) => {
@@ -57,10 +59,10 @@ const plugin = {
       field
     }) {
       if (field.multipleValues) {
-        return field.fieldId + ": [Boolean]";
+        return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": [Boolean]", field);
       }
 
-      return field.fieldId + ": Boolean";
+      return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": Boolean", field);
     }
 
   }

package/content/plugins/graphqlFields/datetime.js

@@ -4,6 +4,9 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
+
+var _requiredField = require("./requiredField");
+
 const fieldGraphQLTypes = {
   time: "Time",
   dateTimeWithoutTimezone: "DateTime",
@@ -74,7 +77,7 @@ const plugin = {
     createInputField({
       field
     }) {
-      return `${field.fieldId}: ${getFieldGraphQLType(field)}`;
+      return (0, _requiredField.attachRequiredFieldValue)(`${field.fieldId}: ${getFieldGraphQLType(field)}`, field);
     }
 
   }

package/content/plugins/graphqlFields/file.js

@@ -4,6 +4,9 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
+
+var _requiredField = require("./requiredField");
+
 const plugin = {
   name: "cms-model-field-to-graphql-file",
   type: "cms-model-field-to-graphql",
@@ -37,10 +40,10 @@ const plugin = {
       field
     }) {
       if (field.multipleValues) {
-        return field.fieldId + ": [String]";
+        return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": [String]", field);
       }
 
-      return field.fieldId + ": String";
+      return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": String", field);
     }
 
   }

package/content/plugins/graphqlFields/longText.js

@@ -5,6 +5,8 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
+var _requiredField = require("./requiredField");
+
 const createListFilters = ({
   field
 }) => {
@@ -50,10 +52,10 @@ const plugin = {
       field
     }) {
       if (field.multipleValues) {
-        return field.fieldId + ": [String]";
+        return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": [String]", field);
       }
 
-      return field.fieldId + ": String";
+      return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": String", field);
     }
 
   }

package/content/plugins/graphqlFields/number.js

@@ -5,6 +5,8 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
+var _requiredField = require("./requiredField");
+
 const createListFilters = ({
   field
 }) => {
@@ -63,10 +65,10 @@ const plugin = {
       field
     }) {
       if (field.multipleValues) {
-        return field.fieldId + ": [Number]";
+        return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": [Number]", field);
       }
 
-      return field.fieldId + ": Number";
+      return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": Number", field);
     }
 
   }

package/content/plugins/graphqlFields/object.js

@@ -15,6 +15,8 @@ var _renderInputFields = require("../utils/renderInputFields");
 
 var _createTypeName = require("../utils/createTypeName");
 
+var _requiredField = require("./requiredField");
+
 const typeFromField = ({
   typeOfType,
   model,
@@ -182,7 +184,7 @@ const plugin = {
         typeDefs
       } = result;
       return {
-        fields: `${field.fieldId}: ${field.multipleValues ? `[${fieldType}!]` : fieldType}`,
+        fields: (0, _requiredField.attachRequiredFieldValue)(`${field.fieldId}: ${field.multipleValues ? `[${fieldType}!]` : fieldType}`, field),
         typeDefs
       };
     },

package/content/plugins/graphqlFields/ref.js

@@ -13,6 +13,8 @@ var _createTypeName = require("../utils/createTypeName");
 
 var _utils = require("@webiny/utils");
 
+var _requiredField = require("./requiredField");
+
 function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (Object.getOwnPropertySymbols) { var symbols = Object.getOwnPropertySymbols(object); if (enumerableOnly) { symbols = symbols.filter(function (sym) { return Object.getOwnPropertyDescriptor(object, sym).enumerable; }); } keys.push.apply(keys, symbols); } return keys; }
 
 function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = arguments[i] != null ? arguments[i] : {}; if (i % 2) { ownKeys(Object(source), true).forEach(function (key) { (0, _defineProperty2.default)(target, key, source[key]); }); } else if (Object.getOwnPropertyDescriptors) { Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)); } else { ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } } return target; }
@@ -228,10 +230,10 @@ const plugin = {
       field
     }) {
       if (field.multipleValues) {
-        return field.fieldId + ": [RefFieldInput!]";
+        return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": [RefFieldInput!]", field);
       }
 
-      return field.fieldId + ": RefFieldInput";
+      return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": RefFieldInput", field);
     },
 
     createListFilters

package/content/plugins/graphqlFields/requiredField.d.ts

@@ -0,0 +1,2 @@
+import { CmsModelField } from "../../../types";
+export declare const attachRequiredFieldValue: (def: string, field: CmsModelField) => string;

package/content/plugins/graphqlFields/requiredField.js

@@ -0,0 +1,24 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.attachRequiredFieldValue = void 0;
+
+const attachRequiredFieldValue = (def, field) => {
+  if (!field.validation || field.validation.length === 0) {
+    return def;
+  }
+
+  const isRequired = field.validation.some(validation => {
+    return validation.name === "required";
+  });
+
+  if (!isRequired) {
+    return def;
+  }
+
+  return `${def}!`;
+};
+
+exports.attachRequiredFieldValue = attachRequiredFieldValue;

package/content/plugins/graphqlFields/richText.js

@@ -4,6 +4,9 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
+
+var _requiredField = require("./requiredField");
+
 const plugin = {
   name: "cms-model-field-to-graphql-rich-text",
   type: "cms-model-field-to-graphql",
@@ -43,10 +46,10 @@ const plugin = {
       field
     }) {
       if (field.multipleValues) {
-        return field.fieldId + ": [JSON]";
+        return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": [JSON]", field);
       }
 
-      return field.fieldId + ": JSON";
+      return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": JSON", field);
     }
 
   }

package/content/plugins/graphqlFields/text.js

@@ -5,6 +5,8 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
+var _requiredField = require("./requiredField");
+
 const createListFilters = ({
   field
 }) => {
@@ -60,10 +62,10 @@ const plugin = {
       field
     }) {
       if (field.multipleValues) {
-        return field.fieldId + ": [String]";
+        return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": [String]", field);
       }
 
-      return field.fieldId + ": String";
+      return (0, _requiredField.attachRequiredFieldValue)(field.fieldId + ": String", field);
     }
 
   }

package/content/plugins/schema/contentModelGroups.js

@@ -62,11 +62,15 @@ const plugin = context => {
     resolvers = {
       CmsContentModelGroup: {
         contentModels: async (group, _, context) => {
-          const models = await context.cms.silentAuthModel().list();
+          context.security.disableAuthorization();
+          const models = await context.cms.listModels();
+          context.security.enableAuthorization();
           return models.filter(m => m.group.id === group.id);
         },
         totalContentModels: async (group, _, context) => {
-          const models = await context.cms.silentAuthModel().list();
+          context.security.disableAuthorization();
+          const models = await context.cms.listModels();
+          context.security.enableAuthorization();
           return models.filter(m => m.group === group.id).length;
         },
         plugin: async (group, _, context) => {

package/content/plugins/storage/object.js

@@ -29,6 +29,11 @@ const processValue = async ({
 }) => {
   return await (0, _pReduce.default)(fields, async (values, field) => {
     const plugin = getStoragePlugin(field.type);
+
+    if (!plugin) {
+      throw new Error(`Missing storage plugin for field type "${field.type}".`);
+    }
+
     const value = await plugin[operation]({
       plugins,
       model,

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@webiny/api-headless-cms",
-	"version": "5.20.0-beta.1",
+	"version": "5.21.0",
 	"main": "index.js",
 	"keywords": [
 		"cms:base"
@@ -21,23 +21,23 @@
 		"@babel/runtime": "7.16.3",
 		"@commodo/fields": "1.1.2-beta.20",
 		"@graphql-tools/schema": "7.1.5",
-		"@webiny/api-file-manager": "5.20.0-beta.1",
-		"@webiny/api-i18n": "5.20.0-beta.1",
-		"@webiny/api-i18n-content": "5.20.0-beta.1",
-		"@webiny/api-i18n-ddb": "5.20.0-beta.1",
-		"@webiny/api-security": "5.20.0-beta.1",
-		"@webiny/api-tenancy": "5.20.0-beta.1",
-		"@webiny/api-upgrade": "5.20.0-beta.1",
-		"@webiny/error": "5.20.0-beta.1",
-		"@webiny/handler": "5.20.0-beta.1",
-		"@webiny/handler-aws": "5.20.0-beta.1",
-		"@webiny/handler-db": "5.20.0-beta.1",
-		"@webiny/handler-graphql": "5.20.0-beta.1",
-		"@webiny/handler-http": "5.20.0-beta.1",
-		"@webiny/plugins": "5.20.0-beta.1",
-		"@webiny/pubsub": "5.20.0-beta.1",
-		"@webiny/utils": "5.20.0-beta.1",
-		"@webiny/validation": "5.20.0-beta.1",
+		"@webiny/api-file-manager": "5.21.0",
+		"@webiny/api-i18n": "5.21.0",
+		"@webiny/api-i18n-content": "5.21.0",
+		"@webiny/api-i18n-ddb": "5.21.0",
+		"@webiny/api-security": "5.21.0",
+		"@webiny/api-tenancy": "5.21.0",
+		"@webiny/api-upgrade": "5.21.0",
+		"@webiny/error": "5.21.0",
+		"@webiny/handler": "5.21.0",
+		"@webiny/handler-aws": "5.21.0",
+		"@webiny/handler-db": "5.21.0",
+		"@webiny/handler-graphql": "5.21.0",
+		"@webiny/handler-http": "5.21.0",
+		"@webiny/plugins": "5.21.0",
+		"@webiny/pubsub": "5.21.0",
+		"@webiny/utils": "5.21.0",
+		"@webiny/validation": "5.21.0",
 		"boolean": "3.1.4",
 		"commodo-fields-object": "1.0.6",
 		"dataloader": "2.0.0",
@@ -54,10 +54,10 @@
 		"@babel/core": "^7.5.5",
 		"@babel/preset-env": "^7.5.5",
 		"@babel/preset-flow": "^7.0.0",
-		"@webiny/api-security-so-ddb": "^5.20.0-beta.1",
-		"@webiny/api-tenancy-so-ddb": "^5.20.0-beta.1",
-		"@webiny/cli": "^5.20.0-beta.1",
-		"@webiny/project-utils": "^5.20.0-beta.1",
+		"@webiny/api-security-so-ddb": "^5.21.0",
+		"@webiny/api-tenancy-so-ddb": "^5.21.0",
+		"@webiny/cli": "^5.21.0",
+		"@webiny/project-utils": "^5.21.0",
 		"apollo-graphql": "^0.4.1",
 		"get-yarn-workspaces": "^1.0.2",
 		"graphql": "^14.6.0",
@@ -77,5 +77,5 @@
 		"build": "yarn webiny run build",
 		"watch": "yarn webiny run watch"
 	},
-	"gitHead": "40e30fb4c778d0bcc773b3b8231ce8e0861b6d52"
+	"gitHead": "c3d4955bf74e7ffdb9628867e3b23cdfe64ea8dc"
 }

package/plugins/crud/system.crud.js

@@ -120,6 +120,10 @@ const createSystemCrud = params => {
       if (version) {
         return;
       }
+      /**
+       * First trigger before install event.
+       */
+
 
       await onBeforeInstall.publish({
         tenant: getTenant().id
@@ -136,9 +140,6 @@ const createSystemCrud = params => {
         });
       }
 
-      await onAfterInstall.publish({
-        tenant: getTenant().id
-      });
       const system = {
         version: context.WEBINY_VERSION,
         readAPIKey: createReadAPIKey(),
@@ -151,6 +152,13 @@ const createSystemCrud = params => {
       await storageOperations.system.create({
         system
       });
+      /**
+       * And trigger after install event.
+       */
+
+      await onAfterInstall.publish({
+        tenant: getTenant().id
+      });
     },
 
     async upgradeSystem(version) {