ghost 6.64.0 → 6.65.0

package/core/server/api/endpoints/utils/validators/input/automated_emails.js

@@ -20,54 +20,44 @@ const messages = {
   tokenRequired: 'Token is required',
 };
 
-const validateAutomatedEmail = async function (frame) {
+const validateAutomatedEmail = function (frame, isSlugRequired) {
   if (!frame.data.automated_emails || !frame.data.automated_emails[0]) {
-    return Promise.resolve();
+    return;
   }
 
   const data = frame.data.automated_emails[0];
 
   if (!data.name || !ALLOWED_NAMES.includes(data.name)) {
-    return Promise.reject(
-      new ValidationError({
-        message: tpl(messages.invalidName),
-        property: 'name',
-      }),
-    );
+    throw new ValidationError({
+      message: tpl(messages.invalidName),
+      property: 'name',
+    });
   }
 
   if (data.status && !ALLOWED_STATUSES.includes(data.status)) {
-    return Promise.reject(
-      new ValidationError({
-        message: tpl(messages.invalidStatus),
-        property: 'status',
-      }),
-    );
+    throw new ValidationError({
+      message: tpl(messages.invalidStatus),
+      property: 'status',
+    });
   }
 
-  if (data.slug && !ALLOWED_SLUGS.includes(data.slug)) {
-    return Promise.reject(
-      new ValidationError({
-        message: tpl(messages.invalidSlug),
-        property: 'slug',
-      }),
-    );
+  if ((isSlugRequired && !data.slug) || (data.slug && !ALLOWED_SLUGS.includes(data.slug))) {
+    throw new ValidationError({
+      message: tpl(messages.invalidSlug),
+      property: 'slug',
+    });
   }
 
   if (data.lexical) {
     try {
       JSON.parse(data.lexical);
     } catch (e) {
-      return Promise.reject(
-        new ValidationError({
-          message: tpl(messages.invalidLexical),
-          property: 'lexical',
-        }),
-      );
+      throw new ValidationError({
+        message: tpl(messages.invalidLexical),
+        property: 'lexical',
+      });
     }
   }
-
-  return Promise.resolve();
 };
 
 const validateOptionalStringField = (value, errorMessage) => {
@@ -107,11 +97,11 @@ const validatePreviewData = (frame) => {
 };
 
 module.exports = {
-  async add(apiConfig, frame) {
-    await validateAutomatedEmail(frame);
+  add(apiConfig, frame) {
+    validateAutomatedEmail(frame, true);
   },
-  async edit(apiConfig, frame) {
-    await validateAutomatedEmail(frame);
+  edit(apiConfig, frame) {
+    validateAutomatedEmail(frame, false);
   },
   editSenders(apiConfig, frame) {
     const senderName = frame.data.sender_name;

package/core/server/api/endpoints/utils/validators/input/settings.js

@@ -28,7 +28,7 @@ const iconUrlRegex = new RegExp(/^(\/|__GHOST_URL__\/)/);
 const iconUrlOptions = { require_protocol: true, protocols: ['http', 'https'] };
 
 function parseArraySettingValue(value) {
-  if (_.isArray(value)) {
+  if (Array.isArray(value)) {
     return value;
   }
 
@@ -163,10 +163,10 @@ module.exports = {
         //       settings API use raw unstringified objects (e.g. when adding notifications)
         //       The conditional can be removed once internals are changed to do the calls properly
         //       and the JSON.parse should be left as the only valid way to check the value.
-        if (!_.isArray(setting.value)) {
+        if (!Array.isArray(setting.value)) {
           try {
             const value = JSON.parse(setting.value);
-            if (!_.isArray(value)) {
+            if (!Array.isArray(value)) {
               errors.push(typeError);
             }
           } catch (err) {

package/core/server/data/exporter/exporter.js

@@ -1,4 +1,3 @@
-const _ = require('lodash');
 const db = require('../../data/db');
 const commands = require('../schema').commands;
 const ghostVersion = require('@tryghost/version');
@@ -14,7 +13,7 @@ const { TABLES_ALLOWLIST, SETTING_KEYS_BLOCKLIST } = require('./table-lists');
 const exportTable = function exportTable(tableName, options) {
   if (
     TABLES_ALLOWLIST.includes(tableName) ||
-    (options.include && _.isArray(options.include) && options.include.indexOf(tableName) !== -1)
+    (options.include && Array.isArray(options.include) && options.include.indexOf(tableName) !== -1)
   ) {
     const query = (options.transacting || db.knex)(tableName);
 

package/core/server/data/exporter/table-lists.js

@@ -34,6 +34,7 @@ const BACKUP_TABLES = [
   'members_payment_events',
   'members_login_events',
   'members_email_change_events',
+  'members_metafield_change_events',
   'members_status_events',
   'members_paid_subscription_events',
   'members_subscribe_events',

package/core/server/data/importer/handlers/image.js

@@ -3,9 +3,8 @@ const path = require('path');
 const config = require('../../../../shared/config');
 const urlUtils = require('../../../../shared/url-utils').default;
 const adapterManager = require('../../../services/adapter-manager').default;
-let ImageHandler;
 
-ImageHandler = {
+const ImageHandler = {
   type: 'images',
   extensions: config.get('uploads').images.extensions,
   contentTypes: config.get('uploads').images.contentTypes,

package/core/server/data/importer/handlers/json.js

@@ -9,9 +9,7 @@ const messages = {
   checkImportJsonIsValid: 'check that the import file is valid JSON.',
 };
 
-let JSONHandler;
-
-JSONHandler = {
+const JSONHandler = {
   type: 'data',
   extensions: ['.json'],
   contentTypes: ['application/octet-stream', 'application/json'],

package/core/server/data/importer/handlers/markdown.js

@@ -5,13 +5,9 @@ const featuredImageRegex = /^(!\[]\(([^)]*?)\)\s+)(?=#)/;
 const titleRegex = /^#\s?([\w\W]*?)(?=\n)/;
 const statusRegex = /(published||draft)-/;
 const dateRegex = /(\d{4}-\d{2}-\d{2})-/;
-let processDateTime;
-let processFileName;
-let processMarkdownFile;
-let MarkdownHandler;
 
 // Takes a date from the filename in y-m-d-h-m form, and converts it into a Date ready to import
-processDateTime = function (post, datetime) {
+const processDateTime = function (post, datetime) {
   const format = 'YYYY-MM-DD-HH-mm';
   datetime = moment.utc(datetime, format).valueOf();
 
@@ -24,7 +20,7 @@ processDateTime = function (post, datetime) {
   return post;
 };
 
-processFileName = function (filename) {
+const processFileName = function (filename) {
   let post = {};
   let name = filename.split('.')[0];
   let match;
@@ -50,7 +46,7 @@ processFileName = function (filename) {
   return post;
 };
 
-processMarkdownFile = function (filename, content) {
+const processMarkdownFile = function (filename, content) {
   const post = processFileName(filename);
   let match;
 
@@ -77,7 +73,7 @@ processMarkdownFile = function (filename, content) {
   return post;
 };
 
-MarkdownHandler = {
+const MarkdownHandler = {
   type: 'data',
   extensions: ['.md', '.markdown'],
   contentTypes: ['application/octet-stream', 'text/plain'],

package/core/server/data/importer/import-manager.js

@@ -29,7 +29,7 @@ const messages = {
   zipContainsMultipleDataFormats:
     'Zip file contains multiple data formats. Please split up and import separately.',
 };
-let defaults = {
+const defaults = {
   extensions: ['.zip'],
   contentTypes: ['application/zip', 'application/x-zip-compressed'],
   directories: [],

package/core/server/data/importer/importers/content-file-importer.js

@@ -1,11 +1,7 @@
 const _ = require('lodash');
 const path = require('node:path');
-let replaceImage;
-let preProcessPosts;
-let preProcessTags;
-let preProcessUsers;
 
-replaceImage = function (markdown, image) {
+const replaceImage = function (markdown, image) {
   if (!markdown) {
     return;
   }
@@ -34,7 +30,7 @@ replaceImage = function (markdown, image) {
  * @param {string} contentFile.originalPath
  * @param {string} contentFile.newPath
  */
-preProcessPosts = function (data, contentFile) {
+const preProcessPosts = function (data, contentFile) {
   _.each(data.posts, function (post) {
     post.markdown = replaceImage(post.markdown, contentFile);
     if (post.html) {
@@ -52,7 +48,7 @@ preProcessPosts = function (data, contentFile) {
   });
 };
 
-preProcessTags = function (data, image) {
+const preProcessTags = function (data, image) {
   _.each(data.tags, function (tag) {
     if (tag.feature_image) {
       tag.feature_image = replaceImage(tag.feature_image, image);
@@ -60,7 +56,7 @@ preProcessTags = function (data, image) {
   });
 };
 
-preProcessUsers = function (data, image) {
+const preProcessUsers = function (data, image) {
   _.each(data.users, function (user) {
     if (user.cover_image) {
       user.cover_image = replaceImage(user.cover_image, image);

package/core/server/data/importer/importers/data/base.js

@@ -70,7 +70,7 @@ class Base {
   sanitizeValues() {
     _.each(this.dataToImport, (obj) => {
       _.each(_.pick(obj, ['updated_at', 'created_at', 'published_at']), (value, key) => {
-        let temporaryDate = new Date(value);
+        const temporaryDate = new Date(value);
 
         if (isNaN(temporaryDate)) {
           this.problems.push({
@@ -109,11 +109,11 @@ class Base {
   }
 
   handleError(errs, obj) {
-    let errorsToReject = [];
-    let problems = [];
+    const errorsToReject = [];
+    const problems = [];
 
     // CASE: validation errors, see models/base/events.js onValidate
-    if (!_.isArray(errs)) {
+    if (!Array.isArray(errs)) {
       errs = [errs];
     }
 
@@ -190,7 +190,7 @@ class Base {
       }
     }).id;
 
-    let userReferenceProblems = {};
+    const userReferenceProblems = {};
 
     const handleObject = (obj, key) => {
       if (!Object.prototype.hasOwnProperty.call(obj, key)) {
@@ -214,11 +214,11 @@ class Base {
       }
 
       // CASE: first match the user reference with in the imported file
-      let userFromFile = _.find(this.requiredFromFile.users, { id: obj[key] });
+      const userFromFile = _.find(this.requiredFromFile.users, { id: obj[key] });
 
       if (!userFromFile) {
         // CASE: if user does not exist in file, try to lookup the existing db users
-        let existingUser = _.find(this.requiredExistingData.users, { id: obj[key].toString() });
+        const existingUser = _.find(this.requiredExistingData.users, { id: obj[key].toString() });
 
         // CASE: fallback to owner
         if (!existingUser) {
@@ -240,7 +240,7 @@ class Base {
       // Result: `this.requiredImportedData.users` will be empty.
       // We already generate identifiers for each object in the importer layer. Accessible via `dataToImport`.
       if (this.modelName === 'User' && !this.requiredImportedData.users.length) {
-        let userToImport = _.find(this.dataToImport, { slug: userFromFile.slug });
+        const userToImport = _.find(this.dataToImport, { slug: userFromFile.slug });
 
         if (userToImport) {
           obj[key] = userToImport.id;
@@ -253,7 +253,7 @@ class Base {
 
       // CASE: user exists in the file, let's find his db id
       // NOTE: lookup by email, because slug can change on insert
-      let importedUser = _.find(this.requiredImportedData.users, { email: userFromFile.email });
+      const importedUser = _.find(this.requiredImportedData.users, { email: userFromFile.email });
 
       // CASE: found. let's assign the new ID
       if (importedUser) {
@@ -320,7 +320,7 @@ class Base {
   async doImport(options, importOptions) {
     debug('doImport', this.modelName, this.dataToImport.length);
 
-    let ops = [];
+    const ops = [];
 
     _.each(this.dataToImport, (obj, index) => {
       ops.push(async () => {

package/core/server/data/importer/importers/data/custom-theme-settings-importer.js

@@ -20,7 +20,7 @@ class CustomThemeSettingsImporter extends BaseImporter {
   async doImport(options, importOptions) {
     debug('doImport', this.modelName, this.dataToImport.length);
 
-    let ops = [];
+    const ops = [];
 
     _.each(this.dataToImport, (item) => {
       ops.push(async () => {

package/core/server/data/importer/importers/data/data-importer.js

@@ -17,10 +17,9 @@ const RevueSubscriberImporter = require('./revue-subscriber-importer');
 const RolesImporter = require('./roles-importer');
 const { slugify } = require('@tryghost/string/lib');
 
-let importers = {};
-let DataImporter;
+const importers = {};
 
-DataImporter = {
+const DataImporter = {
   type: 'data',
 
   preProcess: function preProcess(importData) {
@@ -78,7 +77,7 @@ DataImporter = {
     const ops = [];
     let problems = [];
     let errors = [];
-    let importedData = {};
+    const importedData = {};
 
     const modelOptions = {
       importing: true,

package/core/server/data/importer/importers/data/posts-importer.js

@@ -57,9 +57,9 @@ class PostsImporter extends BaseImporter {
    * Sanitizes post metadata, picking data from sepearate table(for >= v3) or post itself(for < v3)
    */
   sanitizePostsMeta(model) {
-    let postsMetaFromFile =
+    const postsMetaFromFile =
       _.find(this.requiredFromFile.posts_meta, { post_id: model.id }) || _.pick(model, metaAttrs);
-    let postsMetaData = Object.assign(
+    const postsMetaData = Object.assign(
       {},
       _.mapValues(postsMetaSchema, (value) => {
         return Reflect.has(value, 'defaultTo') ? value.defaultTo : null;
@@ -103,14 +103,14 @@ class PostsImporter extends BaseImporter {
           return;
         }
 
-        let postToImport = _.find(this.dataToImport, { id: relation.post_id });
+        const postToImport = _.find(this.dataToImport, { id: relation.post_id });
 
         // CASE: we won't import a relation when the target post does not exist
         if (!postToImport) {
           return;
         }
 
-        if (!postToImport[target] || !_.isArray(postToImport[target])) {
+        if (!postToImport[target] || !Array.isArray(postToImport[target])) {
           postToImport[target] = [];
         }
 
@@ -143,13 +143,13 @@ class PostsImporter extends BaseImporter {
         return;
       }
 
-      let indexesToRemove = [];
+      const indexesToRemove = [];
       _.each(postToImport[targetProperty], (object, index) => {
         // this is the original relational object (old id)
-        let objectInFile = _.find(this.requiredFromFile[tableName], { id: object.id });
+        const objectInFile = _.find(this.requiredFromFile[tableName], { id: object.id });
 
         if (!objectInFile) {
-          let existingObject = _.find(this.requiredExistingData[tableName], { id: object.id });
+          const existingObject = _.find(this.requiredExistingData[tableName], { id: object.id });
 
           // CASE: is not in file, is not in db
           if (!existingObject) {
@@ -177,7 +177,7 @@ class PostsImporter extends BaseImporter {
         }
 
         // CASE: search through existing data by unique attribute
-        let existingObject = _.find(this.requiredExistingData[tableName], {
+        const existingObject = _.find(this.requiredExistingData[tableName], {
           slug: objectInFile.slug,
         });
 
@@ -274,7 +274,7 @@ class PostsImporter extends BaseImporter {
         try {
           mobiledoc = JSON.parse(model.mobiledoc);
 
-          if (!mobiledoc.cards || !_.isArray(mobiledoc.cards)) {
+          if (!mobiledoc.cards || !Array.isArray(mobiledoc.cards)) {
             mobiledoc = mobiledocLib.blankDocument;
           }
         } catch (err) {
@@ -317,7 +317,7 @@ class PostsImporter extends BaseImporter {
 
     // NOTE: We only support removing duplicate posts within the file to import.
     // For any further future duplication detection, see https://github.com/TryGhost/Ghost/issues/8717.
-    let slugs = [];
+    const slugs = [];
     this.dataToImport = _.filter(this.dataToImport, (post) => {
       if (!!post.slug && slugs.indexOf(post.slug) !== -1) {
         this.problems.push({

package/core/server/data/importer/importers/data/products-importer.js

@@ -63,7 +63,7 @@ class ProductsImporter extends BaseImporter {
   validateStripePrice() {
     // the stripe price either needs to exist in the current db,
     // or be imported as part of the same import
-    let invalidProducts = [];
+    const invalidProducts = [];
     _.each(['monthly_price_id', 'yearly_price_id'], (field) => {
       _.each(this.dataToImport, (objectInFile) => {
         const importedObject = _.find(this.requiredFromFile.stripe_prices, {
@@ -90,7 +90,7 @@ class ProductsImporter extends BaseImporter {
 
   preventDuplicates() {
     debug('preventDuplicates');
-    let duplicateProducts = [];
+    const duplicateProducts = [];
     _.each(this.dataToImport, (objectInFile) => {
       const existingObject = _.find(this.requiredExistingData.products, {
         name: objectInFile.name,
@@ -114,7 +114,7 @@ class ProductsImporter extends BaseImporter {
   }
 
   preventInvalidFree() {
-    let invalidFreeProducts = [];
+    const invalidFreeProducts = [];
     _.each(this.dataToImport, (product) => {
       // A free product must not have any pricing data (otherwise it wouldn't be free, duh!)
       if (

package/core/server/data/importer/importers/data/settings-importer.js

@@ -145,7 +145,7 @@ class SettingsImporter extends BaseImporter {
 
         if (deprecatedSettingValue) {
           deprecatedSupportedSettingsOneToManyMap[key].forEach(({ from, to }) => {
-            let value = _.isObject(deprecatedSettingValue)
+            const value = _.isObject(deprecatedSettingValue)
               ? _.get(deprecatedSettingValue, from)
               : deprecatedSetting.value;
 
@@ -274,7 +274,7 @@ class SettingsImporter extends BaseImporter {
   async doImport(options) {
     debug('doImport', this.dataToImport.length);
 
-    let ops = [];
+    const ops = [];
 
     _.each(this.dataToImport, (model) => {
       ops.push(async () => {

package/core/server/data/importer/importers/data/stripe-prices-importer.js

@@ -23,7 +23,7 @@ class StripePricesImporter extends BaseImporter {
 
   validateStripeProduct() {
     // ensure we have a valid stripe_product_id in the stripe_products table
-    let invalidPrices = [];
+    const invalidPrices = [];
     _.each(this.dataToImport, (objectInFile) => {
       const importedObject = _.find(this.requiredImportedData.stripe_products, {
         stripe_product_id: objectInFile.stripe_product_id,

package/core/server/data/importer/importers/data/stripe-products-importer.js

@@ -34,7 +34,7 @@ class StripeProductsImporter extends BaseImporter {
     debug('replaceIdentifiers');
 
     // map product_id -> product.id
-    let invalidProducts = [];
+    const invalidProducts = [];
     _.each(this.dataToImport, (objectInFile) => {
       const importedObject = _.find(this.requiredImportedData.products, {
         originalId: objectInFile.product_id,

package/core/server/data/importer/importers/data/tags-importer.js

@@ -36,7 +36,7 @@ class TagsImporter extends BaseImporter {
   async doImport(options, importOptions) {
     debug('doImport', this.modelName, this.dataToImport.length);
 
-    let ops = [];
+    const ops = [];
 
     _.each(this.dataToImport, (obj) => {
       ops.push(async () => {

package/core/server/data/importer/importers/data/users-importer.js

@@ -33,7 +33,7 @@ class UsersImporter extends BaseImporter {
     debug('beforeImport');
 
     let role;
-    let lookup = {};
+    const lookup = {};
 
     // NOTE: sort out duplicated roles based on incremental id
     _.each(this.requiredFromFile.roles_users, (attachedRole) => {
@@ -70,7 +70,7 @@ class UsersImporter extends BaseImporter {
 
       _.each(this.dataToImport, (obj) => {
         if (attachedRole.user_id === obj.id) {
-          if (!_.isArray(obj.roles)) {
+          if (!Array.isArray(obj.roles)) {
             obj.roles = [];
           }
 

package/core/server/data/importer/importers/json-to-html.js

@@ -25,9 +25,9 @@ const getValidURL = (url) => {
 
 // Take the array of items for a specific post and return the converted HTML
 const itemsToHtml = (items) => {
-  let itemHTMLChunks = [];
+  const itemHTMLChunks = [];
   items.forEach((item) => {
-    let type = item.item_type;
+    const type = item.item_type;
 
     if (type === 'header') {
       itemHTMLChunks.push(`<h3>${item.title}</h3>`);
@@ -37,7 +37,7 @@ const itemsToHtml = (items) => {
       // We have 2 values to work with here. `image` is smaller and most suitable, and `original_image_url` is the full-res that would need to be resized
       // - item.image (https://s3.amazonaws.com/revue/items/images/019/005/542/web/anita-austvika-C-JUrfmYqcw-unsplash.jpg?1667924147)
       // - item.original_image_url (https://s3.amazonaws.com/revue/items/images/019/005/542/original/anita-austvika-C-JUrfmYqcw-unsplash.jpg?1667924147)
-      let cardOpts = {
+      const cardOpts = {
         env: { dom: new SimpleDom.Document() },
         payload: {
           src: item.image,
@@ -50,7 +50,7 @@ const itemsToHtml = (items) => {
       // This could be a bookmark, or it could be a paragraph of text with a linked header, there's no way to tell
       // The safest option here is to output an image with text under it
       const itemURL = getValidURL(item.url);
-      let cardOpts = {
+      const cardOpts = {
         env: { dom: new SimpleDom.Document() },
         payload: {
           src: item.image,
@@ -62,7 +62,7 @@ const itemsToHtml = (items) => {
       const linkTitleHTML = itemURL
         ? `<a href="${serializer.escapeAttrValue(itemURL)}">${item.title}</a>`
         : item.title;
-      let linkHTML = `<h4>${linkTitleHTML}</h4>${item.description}`;
+      const linkHTML = `<h4>${linkTitleHTML}</h4>${item.description}`;
       itemHTMLChunks.push(linkHTML);
     } else if (type === 'tweet') {
       // Should this be an oEmbed call? Probably.
@@ -76,19 +76,22 @@ const itemsToHtml = (items) => {
       const isVimeo = /vimeo.com/.test(item.url);
       let videoHTML = '';
       if (isLongYouTube) {
-        let videoID = item.url.replace(
+        const videoID = item.url.replace(
           /https?:\/\/(?:www\.)?youtube\.com\/watch\?v=([a-zA-Z0-9_-]*)/gi,
           '$1',
         );
         videoHTML = `<iframe width="200" height="113" src="https://www.youtube.com/embed/${videoID}?feature=oembed" frameborder="0" allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture" allowfullscreen></iframe>`;
       } else if (isShortYouTube) {
-        let videoID = item.url.replace(/https?:\/\/(?:www\.)?youtu\.be\/([a-zA-Z0-9_-]*)/gi, '$1');
+        const videoID = item.url.replace(
+          /https?:\/\/(?:www\.)?youtu\.be\/([a-zA-Z0-9_-]*)/gi,
+          '$1',
+        );
         videoHTML = `<iframe width="200" height="113" src="https://www.youtube.com/embed/${videoID}?feature=oembed" frameborder="0" allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture" allowfullscreen></iframe>`;
       } else if (isVimeo) {
-        let videoID = item.url.replace(/https?:\/\/(?:www\.)?vimeo\.com\/([0-9]+)/gi, '$1');
+        const videoID = item.url.replace(/https?:\/\/(?:www\.)?vimeo\.com\/([0-9]+)/gi, '$1');
         videoHTML = `<iframe src="https://player.vimeo.com/video/${videoID}" width="200" height="113" frameborder="0" allow="autoplay; fullscreen; picture-in-picture" allowfullscreen></iframe>`;
       }
-      let cardOpts = {
+      const cardOpts = {
         env: { dom: new SimpleDom.Document() },
         payload: {
           html: videoHTML,

package/core/server/data/migrations/versions/5.0/2022-05-11-13-12-rename-settings.js

@@ -27,7 +27,7 @@ module.exports = createTransactionalMigration(
         _.find(settings, { key: renameMapping.to }) &&
         _.find(settings, { key: renameMapping.from })
       ) {
-        let updatedValue = _.find(settings, { key: renameMapping.from }).value;
+        const updatedValue = _.find(settings, { key: renameMapping.from }).value;
         // CASE: default settings were added already, update them with old values & remove old settings
         logging.info(`Updating ${renameMapping.to} with value from ${renameMapping.from}`);
         await knex('settings').where('key', renameMapping.to).update('value', updatedValue);

package/core/server/data/migrations/versions/6.65/2026-09-16-15-35-26-add-members-metafield-change-events.js

@@ -0,0 +1,17 @@
+const { addTable } = require('../../utils');
+
+module.exports = addTable('members_metafield_change_events', {
+  id: { type: 'string', maxlength: 24, nullable: false, primary: true },
+  member_id: {
+    type: 'string',
+    maxlength: 24,
+    nullable: false,
+    references: 'members.id',
+    cascadeDelete: true,
+  },
+  written_by_type: { type: 'string', maxlength: 50, nullable: false },
+  written_by_id: { type: 'string', maxlength: 24, nullable: true },
+  source: { type: 'string', maxlength: 50, nullable: false },
+  metafields: { type: 'text', maxlength: 16777215, fieldtype: 'medium', nullable: false },
+  created_at: { type: 'dateTime', nullable: false },
+});

package/core/server/data/schema/fixtures/fixture-manager.js

@@ -55,11 +55,11 @@ class FixtureManager {
    * @returns {Function} matching function
    */
   static matchFunc(match, key, value) {
-    if (_.isArray(match)) {
+    if (Array.isArray(match)) {
       return function (item) {
         let valueTest = true;
 
-        if (_.isArray(value)) {
+        if (Array.isArray(value)) {
           valueTest = value.indexOf(item.get(match[1])) > -1;
         } else if (value !== 'all') {
           valueTest = item.get(match[1]) === value;
@@ -78,7 +78,7 @@ class FixtureManager {
   static matchObj(match, item) {
     const matchedObj = {};
 
-    if (_.isArray(match)) {
+    if (Array.isArray(match)) {
       _.each(match, (matchProp) => {
         matchedObj[matchProp] = item.get(matchProp);
       });

package/core/server/data/schema/index.js

@@ -2,4 +2,4 @@ module.exports.tables = require('./schema');
 module.exports.views = require('./views');
 module.exports.commands = require('./commands');
 module.exports.defaultSettings = require('./default-settings');
-module.exports.validate = require('./validator');
+module.exports.validate = require('./validator').validateSchema;