[BREAK] Removed deprecated settings for Legacy Messages, Marked, Snippet Messages, Autolinker and IssueLinks

apps/meteor/app/api/server/lib/messages.ts

@@ -3,7 +3,6 @@ import type { IMessage, IUser } from '@rocket.chat/core-typings';
 import { Rooms, Messages, Users } from '@rocket.chat/models';
 
 import { canAccessRoomAsync } from '../../../authorization/server/functions/canAccessRoom';
-import { getValue } from '../../../settings/server/raw';
 
 export async function findMentionedMessages({
 	uid,
@@ -84,10 +83,6 @@ export async function findStarredMessages({
 }
 
 export async function findSnippetedMessageById({ uid, messageId }: { uid: string; messageId: string }): Promise<IMessage> {
-	if (!(await getValue('Message_AllowSnippeting'))) {
-		throw new Error('error-not-allowed');
-	}
-
 	if (!uid) {
 		throw new Error('invalid-user');
 	}
@@ -125,9 +120,6 @@ export async function findSnippetedMessages({
 	offset: number;
 	total: number;
 }> {
-	if (!(await getValue('Message_AllowSnippeting'))) {
-		throw new Error('error-not-allowed');
-	}
 	const room = await Rooms.findOneById(roomId);
 
 	if (!room || !(await canAccessRoomAsync(room, { _id: uid }))) {

apps/meteor/app/autotranslate/server/autotranslate.ts

@@ -165,7 +165,7 @@ export abstract class AutoTranslate {
 	tokenizeURLs(message: IMessage): IMessage {
 		let count = message.tokens?.length || 0;
 
-		const schemes = settings.get<string>('Markdown_SupportSchemesForLink')?.split(',').join('|');
+		const schemes = 'http,https';
 
 		// Support ![alt text](http://image url) and [text](http://link)
 		message.msg = message.msg.replace(

apps/meteor/app/lib/server/functions/loadMessageHistory.ts

@@ -1,5 +1,4 @@
 /* eslint-disable @typescript-eslint/explicit-function-return-type */
-import { settings } from '../../../settings/server';
 import { Messages, Rooms } from '../../../models/server';
 import { normalizeMessagesForUser } from '../../../utils/server/lib/normalizeMessagesForUser';
 import { getHiddenSystemMessages } from '../lib/getHiddenSystemMessages';
@@ -34,12 +33,6 @@ export function loadMessageHistory({
 		fields: {},
 	};
 
-	if (!settings.get('Message_ShowEditedStatus')) {
-		options.fields = {
-			editedAt: 0,
-		};
-	}
-
 	const records =
 		end != null
 			? Messages.findVisibleByRoomIdBeforeTimestampNotContainingTypes(rid, end, hiddenMessageTypes, options, showThreadMessages).fetch()

apps/meteor/app/lib/server/methods/getChannelHistory.ts

@@ -4,7 +4,6 @@ import _ from 'underscore';
 
 import { canAccessRoom, hasPermission } from '../../../authorization/server';
 import { Subscriptions, Messages, Rooms } from '../../../models/server';
-import { settings } from '../../../settings/server';
 import { normalizeMessagesForUser } from '../../../utils/server/lib/normalizeMessagesForUser';
 import { getHiddenSystemMessages } from '../lib/getHiddenSystemMessages';
 
@@ -59,10 +58,6 @@ Meteor.methods({
 			limit: count,
 		};
 
-		if (!settings.get('Message_ShowEditedStatus')) {
-			options.fields = { editedAt: 0 };
-		}
-
 		const records = _.isUndefined(oldest)
 			? Messages.findVisibleByRoomIdBeforeTimestampNotContainingTypes(
 					rid,

apps/meteor/app/lib/server/startup/settings.ts

@@ -524,13 +524,6 @@ settingsRegistry.addGroup('Accounts', function () {
 			public: true,
 			i18nLabel: 'Notifications_Sound_Volume',
 		});
-
-		this.add('Accounts_Default_User_Preferences_useLegacyMessageTemplate', false, {
-			type: 'boolean',
-			public: true,
-			i18nLabel: 'Use_Legacy_Message_Template',
-			alert: 'This_is_a_deprecated_feature_alert',
-		});
 	});
 
 	this.section('Avatar', function () {
@@ -1135,13 +1128,6 @@ settingsRegistry.addGroup('General', function () {
 
 settingsRegistry.addGroup('Message', function () {
 	this.section('Message_Attachments', function () {
-		this.add('Message_Attachments_GroupAttach', false, {
-			type: 'boolean',
-			public: true,
-			i18nDescription: 'Message_Attachments_GroupAttachDescription',
-			alert: 'This_is_a_deprecated_feature_alert',
-		});
-
 		this.add('Message_Attachments_Thumbnails_Enabled', true, {
 			type: 'boolean',
 			public: true,
@@ -1216,11 +1202,6 @@ settingsRegistry.addGroup('Message', function () {
 	this.add('Message_AlwaysSearchRegExp', false, {
 		type: 'boolean',
 	});
-	this.add('Message_ShowEditedStatus', true, {
-		type: 'boolean',
-		public: true,
-		alert: 'This_is_a_deprecated_feature_alert',
-	});
 	this.add('Message_ShowDeletedStatus', false, {
 		type: 'boolean',
 		public: true,
@@ -1253,14 +1234,6 @@ settingsRegistry.addGroup('Message', function () {
 		type: 'boolean',
 		public: true,
 	});
-	/**
-	 * @deprecated
-	 */
-	this.add('Message_ShowFormattingTips', true, {
-		type: 'boolean',
-		public: true,
-		alert: 'This_is_a_deprecated_feature_alert',
-	});
 	this.add('Message_GroupingPeriod', 300, {
 		type: 'int',
 		public: true,
@@ -1287,22 +1260,6 @@ settingsRegistry.addGroup('Message', function () {
 		actionText: 'clear',
 		i18nLabel: 'clear_cache_now',
 	});
-	// TODO: deprecate this setting in favor of App
-	this.add('API_EmbedDisabledFor', '', {
-		type: 'string',
-		public: true,
-		i18nDescription: 'API_EmbedDisabledFor_Description',
-		alert: 'This_is_a_deprecated_feature_alert',
-	});
-	// TODO: deprecate this setting in favor of App
-	this.add('API_EmbedIgnoredHosts', 'localhost, 127.0.0.1, 10.0.0.0/8, 172.16.0.0/12, 192.168.0.0/16', {
-		type: 'string',
-		i18nDescription: 'API_EmbedIgnoredHosts_Description',
-	});
-	// TODO: deprecate this setting in favor of App
-	this.add('API_EmbedSafePorts', '80, 443', {
-		type: 'string',
-	});
 	this.add('Message_TimeFormat', 'LT', {
 		type: 'string',
 		public: true,
@@ -1334,6 +1291,24 @@ settingsRegistry.addGroup('Message', function () {
 		public: true,
 	});
 
+	// TODO: deprecate this setting in favor of App
+	this.add('API_EmbedDisabledFor', '', {
+		type: 'string',
+		public: true,
+		i18nDescription: 'API_EmbedDisabledFor_Description',
+		alert: 'This_is_a_deprecated_feature_alert',
+	});
+
+	// TODO: deprecate this setting in favor of App
+	this.add('API_EmbedIgnoredHosts', 'localhost, 127.0.0.1, 10.0.0.0/8, 172.16.0.0/12, 192.168.0.0/16', {
+		type: 'string',
+		i18nDescription: 'API_EmbedIgnoredHosts_Description',
+	});
+	// TODO: deprecate this setting in favor of App
+	this.add('API_EmbedSafePorts', '80, 443', {
+		type: 'string',
+	});
+
 	this.add('Message_ErasureType', 'Delete', {
 		type: 'select',
 		public: true,