mirror of
https://github.com/discourse/discourse.git
synced 2024-11-28 14:48:34 +08:00
c5a70eca6e
We were looking up each mention one by one without any form of caching and that results in a problem somewhat similar to an N+1. When we have to do alot of DB lookups, it also increased the time spent in the V8 context which may eventually lead to a timeout. The change here makes it such that mention lookups only does a single DB query per post that happens outside of the V8 context.
119 lines
2.5 KiB
JavaScript
119 lines
2.5 KiB
JavaScript
import {
|
|
cook as cookIt,
|
|
setup as setupIt
|
|
} from "pretty-text/engines/discourse-markdown-it";
|
|
|
|
export function registerOption() {
|
|
// TODO next major version deprecate this
|
|
// if (window.console) {
|
|
// window.console.log("registerOption is deprecated");
|
|
// }
|
|
}
|
|
|
|
export function buildOptions(state) {
|
|
const {
|
|
siteSettings,
|
|
getURL,
|
|
lookupAvatar,
|
|
lookupPrimaryUserGroup,
|
|
getTopicInfo,
|
|
topicId,
|
|
categoryHashtagLookup,
|
|
userId,
|
|
getCurrentUser,
|
|
currentUser,
|
|
lookupAvatarByPostNumber,
|
|
lookupPrimaryUserGroupByPostNumber,
|
|
formatUsername,
|
|
emojiUnicodeReplacer,
|
|
lookupInlineOnebox,
|
|
lookupImageUrls,
|
|
previewing,
|
|
linkify,
|
|
censoredWords,
|
|
invalidateOneboxes
|
|
} = state;
|
|
|
|
let features = {
|
|
"bold-italics": true,
|
|
"auto-link": true,
|
|
mentions: true,
|
|
bbcode: true,
|
|
quote: true,
|
|
html: true,
|
|
"category-hashtag": true,
|
|
onebox: true,
|
|
linkify: linkify !== false,
|
|
newline: !siteSettings.traditional_markdown_linebreaks
|
|
};
|
|
|
|
if (state.features) {
|
|
features = _.merge(features, state.features);
|
|
}
|
|
|
|
const options = {
|
|
sanitize: true,
|
|
getURL,
|
|
features,
|
|
lookupAvatar,
|
|
lookupPrimaryUserGroup,
|
|
getTopicInfo,
|
|
topicId,
|
|
categoryHashtagLookup,
|
|
userId,
|
|
getCurrentUser,
|
|
currentUser,
|
|
lookupAvatarByPostNumber,
|
|
lookupPrimaryUserGroupByPostNumber,
|
|
formatUsername,
|
|
emojiUnicodeReplacer,
|
|
lookupInlineOnebox,
|
|
lookupImageUrls,
|
|
censoredWords,
|
|
allowedHrefSchemes: siteSettings.allowed_href_schemes
|
|
? siteSettings.allowed_href_schemes.split("|")
|
|
: null,
|
|
allowedIframes: siteSettings.allowed_iframes
|
|
? siteSettings.allowed_iframes.split("|")
|
|
: [],
|
|
markdownIt: true,
|
|
injectLineNumbersToPreview:
|
|
siteSettings.enable_advanced_editor_preview_sync,
|
|
previewing,
|
|
invalidateOneboxes
|
|
};
|
|
|
|
// note, this will mutate options due to the way the API is designed
|
|
// may need a refactor
|
|
setupIt(options, siteSettings, state);
|
|
|
|
return options;
|
|
}
|
|
|
|
export default class {
|
|
constructor(opts) {
|
|
if (!opts) {
|
|
opts = buildOptions({ siteSettings: {} });
|
|
}
|
|
this.opts = opts;
|
|
}
|
|
|
|
disableSanitizer() {
|
|
this.opts.sanitizer = this.opts.discourse.sanitizer = ident => ident;
|
|
}
|
|
|
|
cook(raw) {
|
|
if (!raw || raw.length === 0) {
|
|
return "";
|
|
}
|
|
|
|
let result;
|
|
result = cookIt(raw, this.opts);
|
|
return result ? result : "";
|
|
}
|
|
|
|
sanitize(html) {
|
|
return this.opts.sanitizer(html).trim();
|
|
}
|
|
}
|