ghost 5.8.3 → 5.9.2

package/core/built/admin/index.html

@@ -8,7 +8,7 @@
     <title>Ghost Admin</title>
 
     
-<meta name="ghost-admin/config/environment" content="%7B%22modulePrefix%22%3A%22ghost-admin%22%2C%22environment%22%3A%22production%22%2C%22rootURL%22%3A%22%22%2C%22locationType%22%3A%22trailing-hash%22%2C%22EmberENV%22%3A%7B%22FEATURES%22%3A%7B%7D%2C%22EXTEND_PROTOTYPES%22%3A%7B%22Date%22%3Afalse%2C%22Array%22%3Atrue%2C%22String%22%3Atrue%2C%22Function%22%3Afalse%7D%2C%22_APPLICATION_TEMPLATE_WRAPPER%22%3Afalse%2C%22_JQUERY_INTEGRATION%22%3Atrue%2C%22_TEMPLATE_ONLY_GLIMMER_COMPONENTS%22%3Atrue%7D%2C%22APP%22%3A%7B%22version%22%3A%225.8%22%2C%22name%22%3A%22ghost-admin%22%7D%2C%22ember-simple-auth%22%3A%7B%7D%2C%22moment%22%3A%7B%22includeTimezone%22%3A%22all%22%7D%2C%22%40sentry%2Fember%22%3A%7B%22disablePerformance%22%3Atrue%2C%22sentry%22%3A%7B%7D%7D%2C%22ember-cli-mirage%22%3A%7B%22usingProxy%22%3Afalse%2C%22useDefaultPassthroughs%22%3Atrue%7D%2C%22exportApplicationGlobal%22%3Afalse%2C%22ember-load%22%3A%7B%22loadingIndicatorClass%22%3A%22ember-load-indicator%22%7D%7D" />
+<meta name="ghost-admin/config/environment" content="%7B%22modulePrefix%22%3A%22ghost-admin%22%2C%22environment%22%3A%22production%22%2C%22rootURL%22%3A%22%22%2C%22locationType%22%3A%22trailing-hash%22%2C%22EmberENV%22%3A%7B%22FEATURES%22%3A%7B%7D%2C%22EXTEND_PROTOTYPES%22%3A%7B%22Date%22%3Afalse%2C%22Array%22%3Atrue%2C%22String%22%3Atrue%2C%22Function%22%3Afalse%7D%2C%22_APPLICATION_TEMPLATE_WRAPPER%22%3Afalse%2C%22_JQUERY_INTEGRATION%22%3Atrue%2C%22_TEMPLATE_ONLY_GLIMMER_COMPONENTS%22%3Atrue%7D%2C%22APP%22%3A%7B%22version%22%3A%225.9%22%2C%22name%22%3A%22ghost-admin%22%7D%2C%22ember-simple-auth%22%3A%7B%7D%2C%22moment%22%3A%7B%22includeTimezone%22%3A%22all%22%7D%2C%22%40sentry%2Fember%22%3A%7B%22disablePerformance%22%3Atrue%2C%22sentry%22%3A%7B%7D%7D%2C%22ember-cli-mirage%22%3A%7B%22usingProxy%22%3Afalse%2C%22useDefaultPassthroughs%22%3Atrue%7D%2C%22exportApplicationGlobal%22%3Afalse%2C%22ember-load%22%3A%7B%22loadingIndicatorClass%22%3A%22ember-load-indicator%22%7D%7D" />
 
     <meta name="HandheldFriendly" content="True" />
     <meta name="MobileOptimized" content="320" />
@@ -37,7 +37,7 @@
     </style>
 
     <link integrity="" rel="stylesheet" href="assets/vendor-bc9d2c9e5c8a33f0c92e81189d48e04c.css">
-    <link integrity="" rel="stylesheet" href="assets/ghost-b469423d0fbe5e40af17b560f7e3cead.css" title="light">
+    <link integrity="" rel="stylesheet" href="assets/ghost-0319628ec5dcb83e6968e99eaa629a75.css" title="light">
 
     
 </head>
@@ -54,8 +54,8 @@
     <div id="ember-basic-dropdown-wormhole"></div>
 
     <script src="assets/vendor-516c9e43b4aeb92079dc1ab92c9ce492.js"></script>
-<script src="assets/chunk.351.cbc224ca65c14ef5322d.js"></script>
-<script src="assets/chunk.143.cf66fd97f99af03c697f.js"></script>
-    <script src="assets/ghost-f1d63ad9698b7d38261df6384513c952.js"></script>
+<script src="assets/chunk.351.73f27952f867334a8228.js"></script>
+<script src="assets/chunk.143.4e4590d80452422483d8.js"></script>
+    <script src="assets/ghost-f144e03b578116a5d0ed41bceaf2bab8.js"></script>
 </body>
 </html>

package/core/frontend/helpers/comment_count.js

@@ -1,8 +1,7 @@
 const {SafeString} = require('../services/handlebars');
-const {labs} = require('../services/proxy');
 const {html} = require('common-tags');
 
-function commentCount(options) {
+module.exports = function commentCount(options) {
     const empty = options.hash.empty === undefined ? '' : options.hash.empty;
     const singular = options.hash.singular === undefined ? 'comment' : options.hash.singular;
     const plural = options.hash.plural === undefined ? 'comments' : options.hash.plural;
@@ -21,17 +20,4 @@ function commentCount(options) {
         >
         </script>
     `);
-}
-
-module.exports = function commentsLabsWrapper() {
-    const self = this;
-    const args = arguments;
-
-    return labs.enabledHelper({
-        flagKey: 'comments',
-        flagName: 'Comments',
-        helperName: 'comment_count'
-    }, () => {
-        return commentCount.apply(self, args);
-    });
 };

package/core/frontend/helpers/comments.js

@@ -1,8 +1,8 @@
 const {SafeString} = require('../services/handlebars');
-const {urlUtils, getFrontendKey, labs, settingsCache} = require('../services/proxy');
+const {urlUtils, getFrontendKey, settingsCache} = require('../services/proxy');
 const {getFrontendAppConfig, getDataAttributes} = require('../utils/frontend-apps');
 
-async function comments(options) {
+module.exports = async function comments(options) {
     // todo: For now check on the comment id to exclude normal pages (we probably have a better way to do this)
 
     const commentId = this.comment_id;
@@ -66,7 +66,8 @@ async function comments(options) {
         'avatar-saturation': avatarSaturation,
         'accent-color': accentColor,
         'app-version': appVersion,
-        'comments-enabled': commentsEnabled
+        'comments-enabled': commentsEnabled,
+        publication: settingsCache.get('title')
     };
 
     const dataAttributes = getDataAttributes(data);
@@ -74,19 +75,6 @@ async function comments(options) {
     return new SafeString(`
         <script defer src="${scriptUrl}" ${dataAttributes} crossorigin="anonymous"></script>
     `);
-}
-
-module.exports = async function commentsLabsWrapper() {
-    const self = this;
-    const args = arguments;
-
-    return labs.enabledHelper({
-        flagKey: 'comments',
-        flagName: 'Comments',
-        helperName: 'comments'
-    }, () => {
-        return comments.apply(self, args);
-    });
 };
 
 module.exports.async = true;

package/core/frontend/helpers/ghost_head.js

@@ -229,7 +229,7 @@ module.exports = async function ghost_head(options) { // eslint-disable-line cam
                 head.push(`<link rel="stylesheet" type="text/css" href="${getAssetUrl('public/cards.min.css')}">`);
             }
 
-            if (labs.isSet('comments') && settingsCache.get('enable_comments') !== 'off') {
+            if (settingsCache.get('enable_comments') !== 'off') {
                 head.push(`<script defer src="${getAssetUrl('public/comment-counts.min.js')}" data-ghost-comments-counts-api="${urlUtils.getSiteUrl(true)}members/api/comments/counts/"></script>`);
             }
 

package/core/frontend/services/routing/controllers/unsubscribe.js

@@ -18,6 +18,9 @@ module.exports = async function unsubscribeController(req, res) {
     if (query.newsletter) {
         redirectUrl.searchParams.append('newsletter', query.newsletter);
     }
+    if (query.comments) {
+        redirectUrl.searchParams.append('comments', query.comments);
+    }
     redirectUrl.searchParams.append('action', 'unsubscribe');
 
     return res.redirect(302, redirectUrl.href);

package/core/server/api/endpoints/comments-members.js

@@ -3,7 +3,7 @@ const tpl = require('@tryghost/tpl');
 const errors = require('@tryghost/errors');
 const models = require('../../models');
 const commentsService = require('../../services/comments');
-const ALLOWED_INCLUDES = ['post', 'member', 'likes', 'replies', 'parent'];
+const ALLOWED_INCLUDES = ['member', 'replies', 'replies.member', 'replies.count.likes', 'replies.liked', 'count.replies', 'count.likes', 'liked', 'post', 'parent'];
 const UNSAFE_ATTRS = ['status'];
 
 const messages = {
@@ -37,6 +37,28 @@ module.exports = {
         }
     },
 
+    replies: {
+        options: [
+            'include',
+            'page',
+            'limit',
+            'fields',
+            'filter',
+            'order',
+            'debug',
+            'id'
+        ],
+        validation: {
+            options: {
+                include: ALLOWED_INCLUDES
+            }
+        },
+        permissions: 'browse',
+        query(frame) {
+            return commentsService.controller.replies(frame);
+        }
+    },
+
     read: {
         options: [
             'include'

package/core/server/api/endpoints/index.js

@@ -1,4 +1,4 @@
-const shared = require('../shared');
+const apiFramework = require('@tryghost/api-framework');
 const localUtils = require('./utils');
 
 // ESLint Override Notice
@@ -9,19 +9,19 @@ const localUtils = require('./utils');
 
 module.exports = {
     get authentication() {
-        return shared.pipeline(require('./authentication'), localUtils);
+        return apiFramework.pipeline(require('./authentication'), localUtils);
     },
 
     get db() {
-        return shared.pipeline(require('./db'), localUtils);
+        return apiFramework.pipeline(require('./db'), localUtils);
     },
 
     get identities() {
-        return shared.pipeline(require('./identities'), localUtils);
+        return apiFramework.pipeline(require('./identities'), localUtils);
     },
 
     get integrations() {
-        return shared.pipeline(require('./integrations'), localUtils);
+        return apiFramework.pipeline(require('./integrations'), localUtils);
     },
 
     // @TODO: transform
@@ -30,147 +30,147 @@ module.exports = {
     },
 
     get schedules() {
-        return shared.pipeline(require('./schedules'), localUtils);
+        return apiFramework.pipeline(require('./schedules'), localUtils);
     },
 
     get pages() {
-        return shared.pipeline(require('./pages'), localUtils);
+        return apiFramework.pipeline(require('./pages'), localUtils);
     },
 
     get redirects() {
-        return shared.pipeline(require('./redirects'), localUtils);
+        return apiFramework.pipeline(require('./redirects'), localUtils);
     },
 
     get roles() {
-        return shared.pipeline(require('./roles'), localUtils);
+        return apiFramework.pipeline(require('./roles'), localUtils);
     },
 
     get slugs() {
-        return shared.pipeline(require('./slugs'), localUtils);
+        return apiFramework.pipeline(require('./slugs'), localUtils);
     },
 
     get webhooks() {
-        return shared.pipeline(require('./webhooks'), localUtils);
+        return apiFramework.pipeline(require('./webhooks'), localUtils);
     },
 
     get posts() {
-        return shared.pipeline(require('./posts'), localUtils);
+        return apiFramework.pipeline(require('./posts'), localUtils);
     },
 
     get invites() {
-        return shared.pipeline(require('./invites'), localUtils);
+        return apiFramework.pipeline(require('./invites'), localUtils);
     },
 
     get mail() {
-        return shared.pipeline(require('./mail'), localUtils);
+        return apiFramework.pipeline(require('./mail'), localUtils);
     },
 
     get notifications() {
-        return shared.pipeline(require('./notifications'), localUtils);
+        return apiFramework.pipeline(require('./notifications'), localUtils);
     },
 
     get settings() {
-        return shared.pipeline(require('./settings'), localUtils);
+        return apiFramework.pipeline(require('./settings'), localUtils);
     },
 
     get membersStripeConnect() {
-        return shared.pipeline(require('./members-stripe-connect'), localUtils);
+        return apiFramework.pipeline(require('./members-stripe-connect'), localUtils);
     },
 
     get members() {
-        return shared.pipeline(require('./members'), localUtils);
+        return apiFramework.pipeline(require('./members'), localUtils);
     },
 
     get offers() {
-        return shared.pipeline(require('./offers'), localUtils);
+        return apiFramework.pipeline(require('./offers'), localUtils);
     },
 
     get tiers() {
-        return shared.pipeline(require('./tiers'), localUtils);
+        return apiFramework.pipeline(require('./tiers'), localUtils);
     },
 
     get memberSigninUrls() {
-        return shared.pipeline(require('./member-signin-urls.js'), localUtils);
+        return apiFramework.pipeline(require('./member-signin-urls.js'), localUtils);
     },
 
     get labels() {
-        return shared.pipeline(require('./labels'), localUtils);
+        return apiFramework.pipeline(require('./labels'), localUtils);
     },
 
     get images() {
-        return shared.pipeline(require('./images'), localUtils);
+        return apiFramework.pipeline(require('./images'), localUtils);
     },
 
     get media() {
-        return shared.pipeline(require('./media'), localUtils);
+        return apiFramework.pipeline(require('./media'), localUtils);
     },
 
     get files() {
-        return shared.pipeline(require('./files'), localUtils);
+        return apiFramework.pipeline(require('./files'), localUtils);
     },
 
     get tags() {
-        return shared.pipeline(require('./tags'), localUtils);
+        return apiFramework.pipeline(require('./tags'), localUtils);
     },
 
     get users() {
-        return shared.pipeline(require('./users'), localUtils);
+        return apiFramework.pipeline(require('./users'), localUtils);
     },
 
     get previews() {
-        return shared.pipeline(require('./previews'), localUtils);
+        return apiFramework.pipeline(require('./previews'), localUtils);
     },
 
     get emailPost() {
-        return shared.pipeline(require('./email-post'), localUtils);
+        return apiFramework.pipeline(require('./email-post'), localUtils);
     },
 
     get oembed() {
-        return shared.pipeline(require('./oembed'), localUtils);
+        return apiFramework.pipeline(require('./oembed'), localUtils);
     },
 
     get slack() {
-        return shared.pipeline(require('./slack'), localUtils);
+        return apiFramework.pipeline(require('./slack'), localUtils);
     },
 
     get config() {
-        return shared.pipeline(require('./config'), localUtils);
+        return apiFramework.pipeline(require('./config'), localUtils);
     },
 
     get explore() {
-        return shared.pipeline(require('./explore'), localUtils);
+        return apiFramework.pipeline(require('./explore'), localUtils);
     },
 
     get themes() {
-        return shared.pipeline(require('./themes'), localUtils);
+        return apiFramework.pipeline(require('./themes'), localUtils);
     },
 
     get actions() {
-        return shared.pipeline(require('./actions'), localUtils);
+        return apiFramework.pipeline(require('./actions'), localUtils);
     },
 
     get email_previews() {
-        return shared.pipeline(require('./email-previews'), localUtils);
+        return apiFramework.pipeline(require('./email-previews'), localUtils);
     },
 
     get emails() {
-        return shared.pipeline(require('./emails'), localUtils);
+        return apiFramework.pipeline(require('./emails'), localUtils);
     },
 
     get site() {
-        return shared.pipeline(require('./site'), localUtils);
+        return apiFramework.pipeline(require('./site'), localUtils);
     },
 
     get snippets() {
-        return shared.pipeline(require('./snippets'), localUtils);
+        return apiFramework.pipeline(require('./snippets'), localUtils);
     },
 
     get stats() {
-        return shared.pipeline(require('./stats'), localUtils);
+        return apiFramework.pipeline(require('./stats'), localUtils);
     },
 
     get customThemeSettings() {
-        return shared.pipeline(require('./custom-theme-settings'), localUtils);
+        return apiFramework.pipeline(require('./custom-theme-settings'), localUtils);
     },
 
     get serializers() {
@@ -178,11 +178,11 @@ module.exports = {
     },
 
     get newsletters() {
-        return shared.pipeline(require('./newsletters'), localUtils);
+        return apiFramework.pipeline(require('./newsletters'), localUtils);
     },
 
     get comments() {
-        return shared.pipeline(require('./comments'), localUtils);
+        return apiFramework.pipeline(require('./comments'), localUtils);
     },
 
     /**
@@ -194,38 +194,38 @@ module.exports = {
      * `api.admin` soon. Need to figure out how serializers & validation works then.
      */
     get pagesPublic() {
-        return shared.pipeline(require('./pages-public'), localUtils, 'content');
+        return apiFramework.pipeline(require('./pages-public'), localUtils, 'content');
     },
 
     get tagsPublic() {
-        return shared.pipeline(require('./tags-public'), localUtils, 'content');
+        return apiFramework.pipeline(require('./tags-public'), localUtils, 'content');
     },
 
     get publicSettings() {
-        return shared.pipeline(require('./settings-public'), localUtils, 'content');
+        return apiFramework.pipeline(require('./settings-public'), localUtils, 'content');
     },
 
     get postsPublic() {
-        return shared.pipeline(require('./posts-public'), localUtils, 'content');
+        return apiFramework.pipeline(require('./posts-public'), localUtils, 'content');
     },
 
     get authorsPublic() {
-        return shared.pipeline(require('./authors-public'), localUtils, 'content');
+        return apiFramework.pipeline(require('./authors-public'), localUtils, 'content');
     },
 
     get tiersPublic() {
-        return shared.pipeline(require('./tiers-public'), localUtils, 'content');
+        return apiFramework.pipeline(require('./tiers-public'), localUtils, 'content');
     },
 
     get newslettersPublic() {
-        return shared.pipeline(require('./newsletters-public'), localUtils, 'content');
+        return apiFramework.pipeline(require('./newsletters-public'), localUtils, 'content');
     },
 
     get offersPublic() {
-        return shared.pipeline(require('./offers-public'), localUtils, 'content');
+        return apiFramework.pipeline(require('./offers-public'), localUtils, 'content');
     },
 
     get commentsMembers() {
-        return shared.pipeline(require('./comments-members'), localUtils, 'members');
+        return apiFramework.pipeline(require('./comments-members'), localUtils, 'members');
     }
 };

package/core/server/api/endpoints/oembed.js

@@ -1,7 +1,7 @@
 const config = require('../../../shared/config');
 const externalRequest = require('../../lib/request-external');
 
-const OEmbed = require('../../services/oembed');
+const OEmbed = require('@tryghost/oembed-service');
 const oembed = new OEmbed({config, externalRequest});
 
 const NFT = require('../../services/nft-oembed');

package/core/server/api/endpoints/utils/serializers/input/comments.js

@@ -0,0 +1,18 @@
+module.exports = {
+    all(_apiConfig, frame) {
+        if (!frame.options.withRelated || frame.options.withRelated.length === 0) {
+            return;
+        }
+
+        // Map the 'liked' relation to 'count.liked'
+        frame.options.withRelated = frame.options.withRelated.map((relation) => {
+            if (relation === 'liked') {
+                return 'count.liked';
+            }
+            if (relation === 'replies.liked') {
+                return 'replies.count.liked';
+            }
+            return relation;
+        });
+    }
+};

package/core/server/api/endpoints/utils/serializers/input/db.js

@@ -1,6 +1,6 @@
 const _ = require('lodash');
 const debug = require('@tryghost/debug')('api:endpoints:utils:serializers:input:db');
-const optionsUtil = require('../../../../shared/utils/options');
+const optionsUtil = require('@tryghost/api-framework').utils.options;
 
 const INTERNAL_OPTIONS = ['transacting', 'forUpdate'];
 

package/core/server/api/endpoints/utils/serializers/input/index.js

@@ -45,5 +45,9 @@ module.exports = {
 
     get webhooks() {
         return require('./webhooks');
+    },
+
+    get comments() {
+        return require('./comments');
     }
 };

package/core/server/api/endpoints/utils/serializers/input/integrations.js

@@ -3,9 +3,9 @@ const debug = require('@tryghost/debug')('api:endpoints:utils:serializers:input:
 
 function setDefaultFilter(frame) {
     if (frame.options.filter) {
-        frame.options.filter = `(${frame.options.filter})+type:[custom,builtin]`;
+        frame.options.filter = `(${frame.options.filter})+type:[custom,builtin,core]`;
     } else {
-        frame.options.filter = 'type:[custom,builtin]';
+        frame.options.filter = 'type:[custom,builtin,core]';
     }
 }
 

package/core/server/api/endpoints/utils/serializers/output/mappers/comments.js

@@ -25,6 +25,11 @@ const postFields = [
     'url'
 ];
 
+const countFields = [
+    'replies',
+    'likes'
+];
+
 const commentMapper = (model, frame) => {
     const jsonModel = model.toJSON ? model.toJSON(frame.options) : model;
 
@@ -36,12 +41,6 @@ const commentMapper = (model, frame) => {
         response.member = null;
     }
 
-    if (jsonModel.likes) {
-        response.likes_count = jsonModel.likes.length;
-    } else {
-        response.likes_count = 0;
-    }
-
     if (jsonModel.replies) {
         response.replies = jsonModel.replies.map(reply => commentMapper(reply, frame));
     }
@@ -56,11 +55,12 @@ const commentMapper = (model, frame) => {
         response.post = _.pick(jsonModel.post, postFields);
     }
 
-    // todo
-    response.liked = false;
-    if (jsonModel.likes && frame.original.context.member && frame.original.context.member.id) {
-        const id = frame.original.context.member.id;
-        response.liked = !!jsonModel.likes.find(l => l.member_id === id);
+    if (jsonModel.count && jsonModel.count.liked !== undefined) {
+        response.liked = jsonModel.count.liked > 0;
+    }
+
+    if (jsonModel.count) {
+        response.count = _.pick(jsonModel.count, countFields);
     }
 
     if (utils.isMembersAPI(frame)) {
@@ -68,7 +68,7 @@ const commentMapper = (model, frame) => {
             response.html = null;
         }
     }
-
+    
     return response;
 };
 

package/core/server/api/endpoints/utils/serializers/output/tiers.js

@@ -3,7 +3,7 @@ const debug = require('@tryghost/debug')('api:endpoints:utils:serializers:output
 
 const allowedIncludes = ['monthly_price', 'yearly_price'];
 const localUtils = require('../../index');
-const utils = require('../../../../shared/utils');
+const {utils} = require('@tryghost/api-framework');
 const labs = require('../../../../../../shared/labs');
 
 module.exports = {

package/core/server/api/index.js

@@ -1,3 +1 @@
 module.exports.endpoints = require('./endpoints');
-
-module.exports.shared = require('./shared');

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

@@ -5,8 +5,8 @@ const _ = require('lodash');
 const BaseImporter = require('./base');
 const models = require('../../../../models');
 const defaultSettings = require('../../../schema').defaultSettings;
-const keyGroupMapper = require('../../../../api/shared/serializers/input/utils/settings-key-group-mapper');
-const keyTypeMapper = require('../../../../api/shared/serializers/input/utils/settings-key-type-mapper');
+const keyGroupMapper = require('../../../../api/endpoints/utils/serializers/input/utils/settings-key-group-mapper');
+const keyTypeMapper = require('../../../../api/endpoints/utils/serializers/input/utils/settings-key-type-mapper');
 const {WRITABLE_KEYS_ALLOWLIST} = require('../../../../../shared/labs');
 
 const labsDefaults = JSON.parse(defaultSettings.labs.labs.defaultValue);

package/core/server/data/migrations/versions/5.9/2022-08-09-08-32-added-new-integration-type.js

@@ -0,0 +1,24 @@
+const logging = require('@tryghost/logging');
+const {createTransactionalMigration} = require('../../utils');
+
+module.exports = createTransactionalMigration(
+    async function up(knex) {
+        logging.info('Changing Ghost Explore Integration to type "core"');
+        await knex('integrations')
+            .where({
+                name: 'Ghost Explore',
+                slug: 'ghost-explore'
+            })
+            .update('type', 'core');
+    },
+    async function down(knex) {
+        logging.info('Changing Ghost Explore Integration to type "builtin"');
+
+        await knex('integrations')
+            .where({
+                name: 'Ghost Explore',
+                slug: 'ghost-explore'
+            })
+            .update('type', 'builtin');
+    }
+);

package/core/server/data/schema/clients/mysql.js

@@ -27,22 +27,7 @@ const getColumns = function getColumns(table, transaction) {
     });
 };
 
-// This function changes the type of posts.html and posts.markdown columns to mediumtext. Due to
-// a wrong datatype in schema.js some installations using mysql could have been created using the
-// data type text instead of mediumtext.
-// For details see: https://github.com/TryGhost/Ghost/issues/1947
-const checkPostTable = function checkPostTable(transaction = db.knex) {
-    return transaction.raw('SHOW FIELDS FROM posts where Field ="html" OR Field = "markdown"').then(function (response) {
-        return _.flatten(_.map(response[0], function (entry) {
-            if (entry.Type.toLowerCase() !== 'mediumtext') {
-                return (transaction || db.knex).raw('ALTER TABLE posts MODIFY ' + entry.Field + ' MEDIUMTEXT');
-            }
-        }));
-    });
-};
-
 module.exports = {
-    checkPostTable: checkPostTable,
     getTables: getTables,
     getIndexes: getIndexes,
     getColumns: getColumns

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

@@ -411,14 +411,6 @@ function getColumns(table, transaction = db.knex) {
     return Promise.reject(tpl(messages.noSupportForDatabase, {client: client}));
 }
 
-function checkTables(transaction = db.knex) {
-    const client = transaction.client.config.client;
-
-    if (DatabaseInfo.isMySQL(transaction)) {
-        return clients[client].checkPostTable();
-    }
-}
-
 function createColumnMigration(...migrations) {
     async function runColumnMigration(conn, migration) {
         const {
@@ -449,7 +441,6 @@ function createColumnMigration(...migrations) {
 }
 
 module.exports = {
-    checkTables: checkTables,
     createTable: createTable,
     deleteTable: deleteTable,
     getTables: getTables,