summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMartin Trigaux <mat@odoo.com>2021-03-30 14:08:11 +0000
committerMartin Trigaux <mat@odoo.com>2021-04-01 10:07:40 +0000
commitea4dbcd2404663d8542d3f071ceae97ee6dc61ad (patch)
tree06c4c2c14d4088aae150d64f11b13c6229e31b4b
parenta3e10a1ecc0f41196675802d1d6186bd7ffa4b3f (diff)
[FIX] web_editor: use escape for image rendering
To properly escape all tags and not just the first one And move the escape before the loop to apply for everybody This avoid weird preview with selected content rendered (had effect only on preview) closes odoo/odoo#68633 Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>
-rw-r--r--addons/web_editor/static/src/js/wysiwyg/widgets/link_dialog.js4
1 files changed, 2 insertions, 2 deletions
diff --git a/addons/web_editor/static/src/js/wysiwyg/widgets/link_dialog.js b/addons/web_editor/static/src/js/wysiwyg/widgets/link_dialog.js
index cf2fa9dd04a..2a18ba2b436 100644
--- a/addons/web_editor/static/src/js/wysiwyg/widgets/link_dialog.js
+++ b/addons/web_editor/static/src/js/wysiwyg/widgets/link_dialog.js
@@ -261,11 +261,11 @@ var LinkDialog = Dialog.extend({
_getData: function () {
var $url = this.$('input[name="url"]');
var url = $url.val();
- var label = this.$('input[name="label"]').val() || url;
+ var label = _.escape(this.$('input[name="label"]').val() || url);
if (label && this.data.images) {
for (var i = 0; i < this.data.images.length; i++) {
- label = label.replace('<', "&lt;").replace('>', "&gt;").replace(/\[IMG\]/, this.data.images[i].outerHTML);
+ label = label.replace(/\[IMG\]/, this.data.images[i].outerHTML);
}
}