codeninjasllc/discourse
1
0
Fork 0
mirror of https://github.com/codeninjasllc/discourse.git synced 2024-11-30 10:58:31 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

FIX: hoisting issue with regexp replacement patterns

Browse source
This commit is contained in:
Régis Hanol 2015-03-23 16:33:41 +01:00
parent dce258017f
commit 3954f69514
2 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
app/assets/javascripts/discourse/dialects/dialect.js
View file

@ -278,7 +278,9 @@ Discourse.Dialect = {
var keys = Object.keys(hoisted); var keys = Object.keys(hoisted);
if (keys.length) { if (keys.length) {
keys.forEach(function(key) { keys.forEach(function(key) {
result = result.replace(new RegExp(key, "g"), hoisted[key]); result = result.replace(new RegExp(key, "g"), function() {
return hoisted[key];
});
}); });
} }

4
test/javascripts/lib/markdown-test.js.es6
View file

@ -531,4 +531,8 @@ test("code blocks/spans hoisting", function() {
cooked("```\n\n some code\n```", cooked("```\n\n some code\n```",
"<p><pre><code class=\"lang-auto\"> some code</code></pre></p>", "<p><pre><code class=\"lang-auto\"> some code</code></pre></p>",
"it works when nesting standard markdown code blocks within a fenced code block"); "it works when nesting standard markdown code blocks within a fenced code block");
cooked("`$&`",
"<p><code>$&amp;</code></p>",
"it works even when hoisting special replacement patterns");
}); });